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

Add component backstage #1237

Merged
merged 2 commits into from
Apr 25, 2023
Merged

Add component backstage #1237

merged 2 commits into from
Apr 25, 2023

Conversation

mrodm
Copy link
Contributor

@mrodm mrodm commented Apr 25, 2023

This PR creates the Component resource for elastic-package to be used by Backstage

@mrodm mrodm requested review from leo-ri, sharbuz and a team April 25, 2023 08:19
@mrodm mrodm self-assigned this Apr 25, 2023
@mrodm mrodm force-pushed the add_component_backstage branch from ef81708 to e78efde Compare April 25, 2023 08:21
@mrodm mrodm marked this pull request as ready for review April 25, 2023 08:24
spec:
type: tool
owner: group:ingest-fp # Find your group in https://backstage.elastic.dev/catalog?filters%5Bkind%5D=group
system: platform-ingest # Find your group in https://backstage.elastic.dev/catalog?filters%5Bkind%5D=system
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not yet shown in the list of services in backstage, otherwise LGTM

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This platform-ingest already appears in the list 👍

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @mrodm

@alexsapran
Copy link
Contributor

I think we can now merge, given that the domain/system is sorted in backstage

@mrodm mrodm merged commit 6d6579d into elastic:main Apr 25, 2023
@mrodm mrodm deleted the add_component_backstage branch April 25, 2023 14:18
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.

3 participants