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 new backend system support to AWS catalog provider #1708

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

fjudith
Copy link
Contributor

@fjudith fjudith commented Nov 1, 2024

Added support for the new backend system

  • Updated installation instructions
  • Aligned configuration options with @backstage/catalog-backend-module-aws and @backstage/integration-aws-node
  • Removed roleArn reference (deprecated)
  • Added pre-account scheduling support
  • Updated tests to leverage Mockservices
  • Remove DynamicAccountConfig as it is already supported by @backstage/integration-aws-node

✔️ Checklist

  • Added tests for new functionality and regression tests for bug fixes
  • Added changeset (run yarn changeset in the root)
  • Screenshots of before and after attached (for UI changes)
  • Added or updated documentation (if applicable)

Florian JUDITH added 4 commits November 1, 2024 01:33
- Updated installation instructions
- Aligned configuration options with `@backstage/catalog-backend-module-aws`
- Removed deprecated roleArn
- Added pre account scheduling support
- Updated tests to leverage Mockservices
@fjudith fjudith requested a review from a team as a code owner November 1, 2024 05:44
@@ -0,0 +1,5 @@
---
'@roadiehq/catalog-backend-module-aws': minor
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for taking this on! I haven't looked at the rest of modifications yet but this looks like a breaking change so let's make it major.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hello @Xantier
I'm working to manage backward compatiblity right now, better to turn this PR as draft.

@fjudith fjudith marked this pull request as draft November 1, 2024 16:06
Copy link
Contributor

github-actions bot commented Nov 8, 2024

This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution!

@github-actions github-actions bot added the stale label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants