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/tasks/ceph_deploy: use dev option instead of dev-commit #12514

Merged
merged 1 commit into from Dec 16, 2016

Conversation

vasukulkarni
Copy link
Contributor

Signed-off-by: Vasu Kulkarni vasu@redhat.com

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

@vasukulkarni What is this for? I seem to remember the dev-commit being needed for a reason (upgrade tests maybe?) but I can't remember.

@liewegas liewegas added the tests label Dec 16, 2016
@vasukulkarni
Copy link
Contributor Author

@liewegas sorry i should have added tracker, http://tracker.ceph.com/issues/18253 , This is for ceph-deploy suite to work with shaman and will not affect upgrade tests,

Assigning to @alfredodeza

@vasukulkarni
Copy link
Contributor Author

@liewegas liewegas changed the title use dev option instead of dev-commit qa/tasks/ceph_deploy: use dev option instead of dev-commit Dec 16, 2016
@liewegas liewegas merged commit de31862 into ceph:master Dec 16, 2016
@alfredodeza
Copy link
Contributor

note that before these options where mutually exclusive. Now they aren't, because with the new system, when you specify a SHA1, you must point to the branch too because they are built tied to branches.

It isn't possible to specify just a SHA1 (which this PR addresses)

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