-
Notifications
You must be signed in to change notification settings - Fork 168
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
Mdr upgrade scenario #8699
base: master
Are you sure you want to change the base?
Mdr upgrade scenario #8699
Conversation
b40218f
to
834b2c9
Compare
Rebased on latest master |
e06f779
to
5f90af0
Compare
Rebased on latest master |
85a9521
to
3965bef
Compare
Rebased on latest master |
tests/conftest.py
Outdated
(MultiClusterUpgradeParametrize.MULTICLUSTER_UPGRADE_MARKERS) | ||
) | ||
) | ||
and ocsci_config.multicluster_scenario |
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.
change this to ocsci_config.multicluster
ocs_ci/ocs/dr_upgrade.py
Outdated
self.pre_upgrade_images = self.get_pre_upgrade_image(self.csv_name_pre_upgrade) | ||
self.load_version_config_file(self.upgrade_version) | ||
|
||
with CephHealthMonitor(self.ceph_cluster): |
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.
Need to instantiate self.ceph_cluster
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.
Suggest dropping out the ceph health monitor for now, as this will require more adjustment in the ceph cluster object and also in the health monitor context manager to enforce it to run on specific cluster and propagate specific kubeconfig for all of underlaying calls.
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Introduce markers for mco, drhub and acm upgrades Add tests for mco and drhub upgrades Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
1.Introduce abstract methods 2. Fix typos Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
… tests Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
…se of zstream upgrade Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
…lass Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
1. Catsrc name patching issue 2. Format icsp yaml 3. patch channel name Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
rebased on the latest master |
This PR addresses the following