-
Notifications
You must be signed in to change notification settings - Fork 45
[Konflux] add konflux workspace #1860
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
base: main
Are you sure you want to change the base?
Conversation
Add konflux workspace.
|
| - cicd | ||
| spec: | ||
| packageName: "@red-hat-developer-hub/backstage-plugin-konflux-backend" | ||
| dynamicArtifact: "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not so sure about this field, is it going to be populated by some workflow in this repo? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once the PR is merged, automatic action will publish its package, which you need to use in dynamicArtifact:
| dynamicArtifact: "" | |
| dynamicArtifact: "ghcr.io/redhat-developer/rhdh-plugin-export-overlays/red-hat-developer-hub-backstage-plugin-konflux-backend:bs_1.45.3__0.1.1!red-hat-developer-hub-backstage-plugin-konflux-backend" |
| - cicd | ||
| spec: | ||
| packageName: "@red-hat-developer-hub/backstage-plugin-konflux-backend" | ||
| dynamicArtifact: "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once the PR is merged, automatic action will publish its package, which you need to use in dynamicArtifact:
| dynamicArtifact: "" | |
| dynamicArtifact: "ghcr.io/redhat-developer/rhdh-plugin-export-overlays/red-hat-developer-hub-backstage-plugin-konflux-backend:bs_1.45.3__0.1.1!red-hat-developer-hub-backstage-plugin-konflux-backend" |
| authProvider: impersonationHeaders | ||
| clusters: | ||
| konflux: | ||
| apiUrl: ${KONFLUX_CLUSTER_API_URL} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add test file with these env variables as dummy values so test workflow can successfully test that plugin loads?
For example like here
| - cicd | ||
| spec: | ||
| packageName: "@red-hat-developer-hub/backstage-plugin-konflux" | ||
| dynamicArtifact: "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| dynamicArtifact: "" | |
| dynamicArtifact: "ghcr.io/redhat-developer/rhdh-plugin-export-overlays/red-hat-developer-hub-backstage-plugin-konflux:bs_1.45.3__0.1.1!red-hat-developer-hub-backstage-plugin-konflux" |
| support: dev-preview | ||
| lifecycle: active | ||
| partOf: | ||
| - konflux |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please also create konflux plugin here?
It will need to have name: konflux and will need to specify plugin packages using:
packages:
- red-hat-developer-hub-backstage-plugin-konflux-backend
- red-hat-developer-hub-backstage-plugin-konflux
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@polasudo is there already any system for shortcutting of names in place? E.g name: rhdh-backstage-plugin-konflux-backend? Anyway, these names are under 64 chars so we are good.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/publish |
|
Publish workflow has completed with success.
Backstage Compatibility Check✅ All workspaces are compatible with the target Backstage version ( No action required. No E2E tests available for this workspace. |
|
❌ Error logs from container |



Description
hey there! I'm adding the new Konflux workspace to this repository.
The automated bot didn't see it yet probably because it was just published to NPM today, so I am setting it up manually to speed things up.
I'm able to see the packages when I search them in npm:
The build that finished earlier publishing to npm: https://github.com/redhat-developer/rhdh-plugins/actions/runs/21434209291/job/61721047494
note on Draft status:
I am keeping this as a Draft for now to see if the automated discovery workflow creates a PR by tomorrow morning. If the automation doesn't trigger, I will finish setting this up manually and mark it as "Ready for review" ☕