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

NPT-684 DM Upgrade Integration #107

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

NPT-684 DM Upgrade Integration #107

wants to merge 51 commits into from

Conversation

PavitraS
Copy link
Contributor

@PavitraS PavitraS commented Jan 3, 2023

No description provided.

rafal-slominskii and others added 30 commits December 1, 2022 13:47
check for existing resources
move logic to internal func
compare func as parameter
add some behaviour tests
update usecase where we have crs in outdated model
fix tests
Nexus node spec updates made by users may result in backward incompatibility. User should be able to upgrade datamodel based on the force flag and observe the incompatible changes

This PR calls common-library to see if the old and new crd have undergone any modifications, and if there have been any incompatible changes
- Compiler should iterate through all created datamodel objects to ensure backward compatibility
- If there are incompatible modifications and the force flag is enabled, the message notice will be logged in the console
- If the force flag is deactivated and there are incompatible changes, the operation should fail with incompatible errors
@PavitraS PavitraS force-pushed the dm-upgrade-integration branch 2 times, most recently from 03ae036 to b9952fa Compare January 6, 2023 10:06
@PavitraS PavitraS force-pushed the dm-upgrade-integration branch 2 times, most recently from 3b4eeb5 to 18acc82 Compare January 10, 2023 08:55
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.

5 participants