-
Notifications
You must be signed in to change notification settings - Fork 30
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
feat: Remove kyma.spec.modules[].version from api #1837
Merged
kyma-bot
merged 6 commits into
kyma-project:feat/module-catalogue-improvements
from
Tomasz-Smelcerz-SAP:feat/remove-version-from-api
Sep 10, 2024
Merged
feat: Remove kyma.spec.modules[].version from api #1837
kyma-bot
merged 6 commits into
kyma-project:feat/module-catalogue-improvements
from
Tomasz-Smelcerz-SAP:feat/remove-version-from-api
Sep 10, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kyma-bot
added
the
size/S
Denotes a PR that changes 10-29 lines, ignoring generated files.
label
Sep 6, 2024
Tomasz-Smelcerz-SAP
changed the title
feat: Remove kyma.spec.modules[n].version from api
feat: Remove kyma.spec.modules[].version from api
Sep 6, 2024
1 task
c-pius
reviewed
Sep 9, 2024
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.
Great solution! Just one suggestion to add a comment to the "disabled" field.
Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org>
c-pius
approved these changes
Sep 10, 2024
kyma-bot
merged commit Sep 10, 2024
341b24f
into
kyma-project:feat/module-catalogue-improvements
38 checks passed
Tomasz-Smelcerz-SAP
added a commit
to Tomasz-Smelcerz-SAP/lifecycle-manager
that referenced
this pull request
Sep 17, 2024
* remove kyma.spec.modules[n].version from api * Skip test * disable test * skip test * Skip test * Update api/v1beta2/kyma_types.go Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org>
Tomasz-Smelcerz-SAP
added a commit
to Tomasz-Smelcerz-SAP/lifecycle-manager
that referenced
this pull request
Sep 25, 2024
* Add explicit version attribute * Add test * Add test * Fix lint * Fix test * review fix * Fix compilation errors * Update api/v1beta2/moduletemplate_types.go Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> * ignore new fields in version comparison --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> chore: Sync main to feature branch (kyma-project#1718) * chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) * remove parameter for NewCachedDescriptorProvider * fix dead link * adjust unit test coverage * fix flaky test * docs: Update KLM Local Test Setup Guide (kyma-project#1680) fix errors in local test setup documentation add version info * feat: Drop multiple ways to reference modules in Kyma CR (kyma-project#1672) * remove module reference by namespace/name * remove module reference by objectmeta name * remove module reference by FQDN * add initial test structure * add test cases for different module reference scenarios * fix tests * update documentation * address review comments * address more review comments * fix linting issues * rearrange imports * adjust documentation * chore: Configure different requeue intervals for Manifest reconciliation (kyma-project#1690) * Add different requeue intervals for Manifest reconciliation * Empty-Commit * code review comments * chore: Bump k8s deps (kyma-project#1703) * chore: Bump k8s deps * retrigger jobs * bump api folder as well --------- Co-authored-by: Nesma Badr <Nesma.badr@sap.com> * fix: Manifest CR should update by moduletemplate generation changes (kyma-project#1702) * when moduletemplate generation updated, then manifest CR should also updated. * refactor regular_test.go --------- Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> * fix bug due to modified ModuleName --------- Co-authored-by: Amritanshu Sikdar <amritanshu.sikdar@sap.com> Co-authored-by: Amritanshu Sikdar <amritanshu.sikdar@gmail.com> Co-authored-by: Nesma Badr <Nesma.badr@sap.com> Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> feat: Add Version to Kyma.Spec.Modules (kyma-project#1694) * add version to kyma.spec.modules list * add missing manifest update base on moduletemplate generation change. * remove parameter for NewCachedDescriptorProvider * fix dead link * adjust unit test coverage * fix flaky test * chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) * remove parameter for NewCachedDescriptorProvider * fix dead link * adjust unit test coverage * fix flaky test * refactor FilterTemplate * add integration test * Update tests/integration/controller/kyma/kyma_module_channel_test.go Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com> * Update tests/integration/controller/kyma/kyma_module_version_test.go Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com> * fix existing test --------- Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com> chore: Update branch to main (kyma-project#1753) * update base branch to main * update base branch to main feat: Module catalog improvements implementation (kyma-project#1748) * Implement changes in module catalog handling * review fix * review fix * review fix * remove test * fix lint * review fix feat: Remove kyma.spec.modules[].version from api (kyma-project#1837) * remove kyma.spec.modules[n].version from api * Skip test * disable test * skip test * Skip test * Update api/v1beta2/kyma_types.go Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> wip wip wip review fix
Tomasz-Smelcerz-SAP
added a commit
to Tomasz-Smelcerz-SAP/lifecycle-manager
that referenced
this pull request
Sep 25, 2024
* Add explicit version attribute * Add test * Add test * Fix lint * Fix test * review fix * Fix compilation errors * Update api/v1beta2/moduletemplate_types.go Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> * ignore new fields in version comparison --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> chore: Sync main to feature branch (kyma-project#1718) * chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) * remove parameter for NewCachedDescriptorProvider * fix dead link * adjust unit test coverage * fix flaky test * docs: Update KLM Local Test Setup Guide (kyma-project#1680) fix errors in local test setup documentation add version info * feat: Drop multiple ways to reference modules in Kyma CR (kyma-project#1672) * remove module reference by namespace/name * remove module reference by objectmeta name * remove module reference by FQDN * add initial test structure * add test cases for different module reference scenarios * fix tests * update documentation * address review comments * address more review comments * fix linting issues * rearrange imports * adjust documentation * chore: Configure different requeue intervals for Manifest reconciliation (kyma-project#1690) * Add different requeue intervals for Manifest reconciliation * Empty-Commit * code review comments * chore: Bump k8s deps (kyma-project#1703) * chore: Bump k8s deps * retrigger jobs * bump api folder as well --------- Co-authored-by: Nesma Badr <Nesma.badr@sap.com> * fix: Manifest CR should update by moduletemplate generation changes (kyma-project#1702) * when moduletemplate generation updated, then manifest CR should also updated. * refactor regular_test.go --------- Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> * fix bug due to modified ModuleName --------- Co-authored-by: Amritanshu Sikdar <amritanshu.sikdar@sap.com> Co-authored-by: Amritanshu Sikdar <amritanshu.sikdar@gmail.com> Co-authored-by: Nesma Badr <Nesma.badr@sap.com> Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> feat: Add Version to Kyma.Spec.Modules (kyma-project#1694) * add version to kyma.spec.modules list * add missing manifest update base on moduletemplate generation change. * remove parameter for NewCachedDescriptorProvider * fix dead link * adjust unit test coverage * fix flaky test * chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) * remove parameter for NewCachedDescriptorProvider * fix dead link * adjust unit test coverage * fix flaky test * refactor FilterTemplate * add integration test * Update tests/integration/controller/kyma/kyma_module_channel_test.go Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com> * Update tests/integration/controller/kyma/kyma_module_version_test.go Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com> * fix existing test --------- Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com> chore: Update branch to main (kyma-project#1753) * update base branch to main * update base branch to main feat: Module catalog improvements implementation (kyma-project#1748) * Implement changes in module catalog handling * review fix * review fix * review fix * remove test * fix lint * review fix feat: Remove kyma.spec.modules[].version from api (kyma-project#1837) * remove kyma.spec.modules[n].version from api * Skip test * disable test * skip test * Skip test * Update api/v1beta2/kyma_types.go Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> wip wip wip review fix
Tomasz-Smelcerz-SAP
added a commit
to Tomasz-Smelcerz-SAP/lifecycle-manager
that referenced
this pull request
Sep 25, 2024
chore: Sync main to feature branch (kyma-project#1718) chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) docs: Update KLM Local Test Setup Guide (kyma-project#1680) chore: Configure different requeue intervals for Manifest reconciliation (kyma-project#1690) feat: Drop multiple ways to reference modules in Kyma CR (kyma-project#1672) chore: Bump k8s deps (kyma-project#1703) fix: Manifest CR should update by moduletemplate generation changes (kyma-project#1702) feat: Add Version to Kyma.Spec.Modules (kyma-project#1694) chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) chore: Update branch to main (kyma-project#1753) feat: Module catalog improvements implementation (kyma-project#1748) feat: Remove kyma.spec.modules[].version from api (kyma-project#1837) --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> Co-authored-by: Nesma Badr <Nesma.badr@sap.com> Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com>
Tomasz-Smelcerz-SAP
added a commit
to Tomasz-Smelcerz-SAP/lifecycle-manager
that referenced
this pull request
Sep 26, 2024
chore: Sync main to feature branch (kyma-project#1718) chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) docs: Update KLM Local Test Setup Guide (kyma-project#1680) chore: Configure different requeue intervals for Manifest reconciliation (kyma-project#1690) feat: Drop multiple ways to reference modules in Kyma CR (kyma-project#1672) chore: Bump k8s deps (kyma-project#1703) fix: Manifest CR should update by moduletemplate generation changes (kyma-project#1702) feat: Add Version to Kyma.Spec.Modules (kyma-project#1694) chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) chore: Update branch to main (kyma-project#1753) feat: Module catalog improvements implementation (kyma-project#1748) feat: Remove kyma.spec.modules[].version from api (kyma-project#1837) --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> Co-authored-by: Nesma Badr <Nesma.badr@sap.com> Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com>
Tomasz-Smelcerz-SAP
added a commit
to Tomasz-Smelcerz-SAP/lifecycle-manager
that referenced
this pull request
Sep 27, 2024
chore: Sync main to feature branch (kyma-project#1718) chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) docs: Update KLM Local Test Setup Guide (kyma-project#1680) chore: Configure different requeue intervals for Manifest reconciliation (kyma-project#1690) feat: Drop multiple ways to reference modules in Kyma CR (kyma-project#1672) chore: Bump k8s deps (kyma-project#1703) fix: Manifest CR should update by moduletemplate generation changes (kyma-project#1702) feat: Add Version to Kyma.Spec.Modules (kyma-project#1694) chore: Refactor NewCachedDescriptorProvider (kyma-project#1695) chore: Update branch to main (kyma-project#1753) feat: Module catalog improvements implementation (kyma-project#1748) feat: Remove kyma.spec.modules[].version from api (kyma-project#1837) --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> Co-authored-by: Nesma Badr <Nesma.badr@sap.com> Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com>
ruanxin
pushed a commit
that referenced
this pull request
Sep 27, 2024
* feat: Explicit version for module template (#1699) chore: Sync main to feature branch (#1718) chore: Refactor NewCachedDescriptorProvider (#1695) docs: Update KLM Local Test Setup Guide (#1680) chore: Configure different requeue intervals for Manifest reconciliation (#1690) feat: Drop multiple ways to reference modules in Kyma CR (#1672) chore: Bump k8s deps (#1703) fix: Manifest CR should update by moduletemplate generation changes (#1702) feat: Add Version to Kyma.Spec.Modules (#1694) chore: Refactor NewCachedDescriptorProvider (#1695) chore: Update branch to main (#1753) feat: Module catalog improvements implementation (#1748) feat: Remove kyma.spec.modules[].version from api (#1837) --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> Co-authored-by: Nesma Badr <Nesma.badr@sap.com> Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com> Co-authored-by: Tomasz Smelcerz <tomasz.smelcerz@sap.com> * review fix --------- Co-authored-by: Christoph Schwägerl <acc.pius@mailbox.org> Co-authored-by: Nesma Badr <Nesma.badr@sap.com> Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Remove the
kyma.spec.modules[*].version
attribute from the APIWhen this is merged, if the user wants to enable the module using the specific version (see here), the API server responds with:
Despite disabling the
version
attribute on the API level, the code for handling the attribute stays in the project so it's ready to be re-enabled anytime.Changes proposed in this pull request:
kyma.spec.modules[*].version
attribute from the k8s APIRelated issue(s)
#1817