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

qa/tests: Wip ceph deploy upgrade #17651

Merged
merged 3 commits into from Dec 13, 2017
Merged

Conversation

vasukulkarni
Copy link
Contributor

for the master branch upgrade from jewel(stable) -> luminous(stable) -> master

Copy link

@amitkumar50 amitkumar50 left a comment

Choose a reason for hiding this comment

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

looks good

@liewegas
Copy link
Member

@vasukulkarni what's teh status of this?

@liewegas liewegas added the tests label Nov 17, 2017
@vasukulkarni
Copy link
Contributor Author

@liewegas will retest and update.

@vasukulkarni vasukulkarni changed the title [DNM] qa/tests: Wip ceph deploy upgrade qa/tests: Wip ceph deploy upgrade Dec 4, 2017
@vasukulkarni
Copy link
Contributor Author

Seeing 2 genuine issues :

a) CentOS transaction error http://tracker.ceph.com/issues/22304
b) kraken -> luminous mgr upgrade issue: http://tracker.ceph.com/issues/21122

…s->master

Signed-off-by: Vasu Kulkarni <vasu@redhat.com>
…sequent calls.

Signed-off-by: Vasu Kulkarni <vasu@redhat.com>
…upgrade

Signed-off-by: Vasu Kulkarni <vasu@redhat.com>
@yuriw
Copy link
Contributor

yuriw commented Dec 13, 2017

@vasukulkarni did you run tests ?

Copy link
Contributor

@yuriw yuriw left a comment

Choose a reason for hiding this comment

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

👍

@vasukulkarni
Copy link
Contributor Author

Final run which has known trackers linked above: http://pulpito.ceph.com/vasu-2017-12-08_16:35:01-upgrade-master-distro-basic-vps/

@vasukulkarni vasukulkarni merged commit 36aa1b8 into master Dec 13, 2017
@vasukulkarni vasukulkarni deleted the wip-ceph-deploy-upgrade branch December 13, 2017 00:10
@tchaikov
Copy link
Contributor

@vasukulkarni could you add a "Reviewed-by" line when merging approved PRs next time? thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants