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

chore(orchestrator): use upstream AuditorService #422

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mareklibra
Copy link
Contributor

@mareklibra mareklibra commented Feb 17, 2025

This requires #418

We are switching from the Janus-developed audit logger to its new upstream counterpart which will be part of the Backstage 1.36.0, so far available in the https://github.com/backstage/backstage/releases/tag/v1.36.0-next.3 .


Update: This requires Backstage 1.36 which will be in RHDH 1.6, not sooner. Let's postpone merging this until we branch-out the orchestrator-1.5.

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Feb 17, 2025

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-orchestrator-backend workspaces/orchestrator/plugins/orchestrator-backend patch v5.2.0

@mareklibra mareklibra force-pushed the FLPATH-2096.auditService branch 3 times, most recently from 3512005 to 53774a2 Compare February 17, 2025 08:53
@mareklibra mareklibra changed the title chore(orchestrator): use upstream AudotrService chore(orchestrator): use upstream AuditorService Feb 17, 2025
@mareklibra mareklibra force-pushed the FLPATH-2096.auditService branch from 53774a2 to e0700ea Compare February 17, 2025 09:50
@mareklibra mareklibra force-pushed the FLPATH-2096.auditService branch from e0700ea to 02108b2 Compare February 17, 2025 09:51
@mareklibra
Copy link
Contributor Author

Converting to Draft until we clarify when the AuditorServic will be available in the RHDH

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

Successfully merging this pull request may close these issues.

1 participant