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

Test KKP Upgrade - Test Release 2.25 #13042

Closed
8 tasks done
Tracked by #13016
embik opened this issue Feb 5, 2024 · 9 comments
Closed
8 tasks done
Tracked by #13016

Test KKP Upgrade - Test Release 2.25 #13042

embik opened this issue Feb 5, 2024 · 9 comments
Assignees
Milestone

Comments

@embik
Copy link
Member

embik commented Feb 5, 2024

As usual (see #12799 for previous ticket), we need to run a couple of upgrade tests to make sure we don't run into problems with KKP 2.24 being upgraded to KKP 2.25.

Please add your things to test in comments below, so I can add them to the things to be validated during and after a KKP upgrade.

To Be Validated

Found Issues

  1. kind/bug sig/app-management
    Skarlso akash-gautam
  2. kind/bug sig/app-management
    SimonTheLeg
  3. kind/bug sig/cluster-management
    xrstf
  4. kind/bug sig/app-management
    SimonTheLeg archups
  5. kind/bug sig/cluster-management
    xrstf
@SimonTheLeg
Copy link
Member

SimonTheLeg commented Feb 8, 2024

Update: Since we are going with a non-breaking version, this no longer needs to be verified
To be validated: With the change of the values and defaultValues to a string in #13053, we need to verify that this does not brick the upgrade process

@archups
Copy link
Contributor

archups commented Feb 8, 2024

To be validated: With the change of the helm chart version for default applications in #13058, we need to verify that this does not brick the upgrade process.

@embik embik added this to the KKP 2.25 milestone Feb 12, 2024
@wurbanski
Copy link
Contributor

to be validated: MLA upgrades:

@csengerszabo
Copy link
Contributor

/assign @mohamed-rafraf

@embik
Copy link
Member Author

embik commented Feb 29, 2024

/unassign @mohamed-rafraf
/assign

@embik
Copy link
Member Author

embik commented Mar 1, 2024

Results so far:

@embik
Copy link
Member Author

embik commented Mar 1, 2024

Notes on the upgrade:

  • The upgrade itself was pretty painless, I did not need to adjust any configuration.
  • Upgrade guide should still feature all the MLA values changes. Since I had a minimal configuration it didn't affect me, but some users are probably customizing their MLA setup further.

@embik
Copy link
Member Author

embik commented Mar 15, 2024

Upgrading from v2.24.4 to v2.25.0-rc.0:

  • spec.cloud.aws.roleARN got migrated automatically to the fully qualified way. No CCM crashes observed.
  • apps did not vanish from the user clusters and have their upgraded versions available from the version drop-down.
  • logos are in the application catalog.
  • no issue during MLA upgrade.

@embik
Copy link
Member Author

embik commented Mar 15, 2024

Seems we are all good!

@embik embik closed this as completed Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants