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

Per Yehuda's comment in #12154, set s3tests branch to appropriate assignment #471

Closed
wants to merge 1 commit into from

Conversation

yuriw
Copy link
Contributor

@yuriw yuriw commented Jun 26, 2015

@yehudasa Pls take a look and it OK I will cherry pink on hammer

Signed-off-by: Yuri Weinstein yweinste@redhat.com

@ceph-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-qa-suite-pull-requests/426/
Tests passed for this pull request.

@yehudasa
Copy link
Member

@yuriw I don't think this is right. If upgrading from hammer to hammer it should be using the hammer branch. Beforehand it was supposed to use the firefly-original branch, but it was actually using the master branch. That was wrong. Something is broken so that it ignores that branch completely.

@ceph-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-qa-suite-pull-requests/427/
Tests passed for this pull request.

@yuriw
Copy link
Contributor Author

yuriw commented Jun 26, 2015

@yehudasa see changes, pls

@yehudasa
Copy link
Member

@yuriw I don't know if it's going to work or not. You'll have to run it to verify that it's actually doing the correct thing.

@yuriw
Copy link
Contributor Author

yuriw commented Jun 26, 2015

@ceph-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-qa-suite-pull-requests/428/
Tests passed for this pull request.

@yuriw yuriw changed the title Per Yehuda's comment in #12154, changed s3test branch to master Per Yehuda's comment in #12154, set s3tests branch to appropriate assignment Jun 26, 2015
@ceph-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-qa-suite-pull-requests/434/
Tests passed for this pull request.

we installed giant then run s3tests in parallel with upgrade to hammer, s3tests should run off giant branch
Added force_branch to assure correct pull for s3tests

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
@ceph-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-qa-suite-pull-requests/435/
Tests passed for this pull request.

@yuriw yuriw closed this Oct 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants