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 support for taking over library prefix of deprecated components #734

Merged
merged 7 commits into from
Feb 8, 2023

Conversation

simu
Copy link
Member

@simu simu commented Feb 7, 2023

Commodore will verify that either the component who's prefix we're trying to take over isn't present in the cluster configuration (in this case the alias is just a regular alias) or that the component is deprecated, and has nominated us as their replacement component. Library prefixes of non-deprecated components, or deprecated components which don't specify a replacement can't be taken over.

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Update the documentation.
  • Update tests.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency, internal
    as they show up in the changelog

@simu simu changed the title Implement support for taking over library prefix of deprecated compon… Implement support for taking over library prefix of deprecated components Feb 7, 2023
@simu simu force-pushed the feat/steal-libalias-prefix branch from 08b7115 to ff85b41 Compare February 7, 2023 16:16
@simu simu added the enhancement New feature or request label Feb 7, 2023
@simu simu force-pushed the feat/steal-libalias-prefix branch 3 times, most recently from 6d31d0a to 163edb2 Compare February 8, 2023 10:09
@simu simu marked this pull request as ready for review February 8, 2023 10:16
@simu simu requested a review from a team as a code owner February 8, 2023 10:16
simu added 6 commits February 8, 2023 13:04
…ents

Commodore will verify that either the component who's prefix we're
trying to take over isn't present in the cluster configuration (in this
case the alias is just a regular alias) or that the component is
deprecated, and has nominated us as their replacement component. Library
prefixes of non-deprecated components, or deprecated components which
don't specify a replacement can't be taken over.
Allow specifying full `_metadata` through test parametrization. This
allows us to reuse the test to specify test cases for taking over
library prefixes.
@simu simu force-pushed the feat/steal-libalias-prefix branch from 163edb2 to 6c1872b Compare February 8, 2023 12:04
@simu simu requested a review from a team February 8, 2023 12:54
Copy link
Contributor

@bastjan bastjan left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@simu simu changed the title Implement support for taking over library prefix of deprecated components Add support for taking over library prefix of deprecated components Feb 8, 2023
@simu simu merged commit 2914eb2 into master Feb 8, 2023
@simu simu deleted the feat/steal-libalias-prefix branch February 8, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants