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

Automatic backport of pull request #3055 #3521

Merged
merged 3 commits into from Jan 24, 2019

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Jan 22, 2019

This is an automated backport of pull request #3055 done by Mergify.io

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1645379

This condition is useless and it's also creating issues we don't see in
our CI. ceph_release is set by either ceph-common or ceph-docker-common
so let's keep it this way.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1645379

(cherry picked from commit e9188cd)
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
This part of the code is not needed since ceph-ansible@master is
intended to deploy ceph@master only.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 1bbdde2)
@guits guits force-pushed the mergify/bp/stable-3.2/pr-3055 branch from ac67378 to d43d817 Compare January 22, 2019 09:37
when `ceph_origin` is set to `'repository'` and `ceph_repository` to
`'community'` we need to ensure `ceph_release` reflect
`ceph_stable_release`.

4a3f180 simply removed the override
while it should just have to be run only when the condition mentioned
above is satisfied.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 0bfefdd)
@guits
Copy link
Collaborator

guits commented Jan 22, 2019

jenkins test luminous-centos7_cluster

@guits
Copy link
Collaborator

guits commented Jan 24, 2019

jenkins test pipeline

@mergify mergify bot merged commit af17e0d into stable-3.2 Jan 24, 2019
@mergify mergify bot deleted the mergify/bp/stable-3.2/pr-3055 branch January 24, 2019 14:18
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

Successfully merging this pull request may close these issues.

None yet

1 participant