Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create catalog-info file #438

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

elastic-backstage-prod[bot]
Copy link

If you already have a ./catalog-info.yaml file defined in your repository, you probably want to close this PR.

If not - this PR provides you with the scaffolding for one. The catalog-info.yaml file is where you can define your infrastructure needs (such as Buildkite pipelines) in the form of Real Resource Entities.


If you already have a Buildkite pipeline defined in the elastic/ci repository, you can copy its definition in this catalog-info.yaml file (see instructions).

Once the pipeline definition is ingested into Backstage, Terrazzo will emit a warning about a duplicate definition with a warning similar to:
🙈 Ignoring remote manifest in favour of local one: buildkite.elastic.dev/v1=>Pipeline=> your-pipeline

At this point, it is safe for you to delete the Pipeline definition from the elastic/ci repository.

@elastic-backstage-prod elastic-backstage-prod bot requested a review from a team as a code owner April 10, 2023 15:20
@cla-checker-service
Copy link

❌ Author of the following commits did not sign a Contributor Agreement:
15a8267

Please, read and sign the above mentioned agreement if you want to contribute to this project

@coolblknerd
Copy link

Hey, is there a way to override the failed CLA check? This PR was opened through Backstage automation and I'm trying to figure out how to work my way around this since this is becoming a consistent barrier.

@gigerdo
Copy link
Member

gigerdo commented Apr 11, 2023

@coolblknerd The problem is caused by the bot because it doesn't commit using an @elastic.co email. But it is possible to bypass branch protections and merge it anyway.

Alternatively if you recreate the commits under your own name it should pass the validation (However in that case you need to also set-up commit signing since that will be the next check that will fail).


spec:
type: service
owner: group:control-plane-stateful-foundations
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
owner: group:control-plane-stateful-foundations
owner: group:control-plane-stateful-applications

annotations:
backstage.io/source-location: url:https://github.com/elastic/cloud-sdk-go
github.com/project-slug: elastic/cloud-sdk-go
github.com/team-slug: elastic/control-plane-stateful-foundations
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
github.com/team-slug: elastic/control-plane-stateful-foundations
github.com/team-slug: elastic/control-plane-stateful-applications

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants