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.

Sign up for GitHub

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

Jump to bottom

Wip 832 hammer #1089

Merged
5 commits merged into from Jul 25, 2016
Merged

Wip 832 hammer #1089

5 commits merged into from Jul 25, 2016

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm commented Jul 21, 2016 •
edited

liewegas and others added 3 commits July 21, 2016 22:01
@liewegas @smithfarm
tasks/ceph_manager: dump pgs if other peering timeouts expire
ce98eed 
We were doing this for one of the recovery timeouts but not all of them.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 6deba7c)
@liewegas @smithfarm
tasks/ceph_manager: dump pgs when recover times out
e80b687 
It is really hard to map a stuck recovery back to the pgs that
are stuck.  This will make it easy.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit a328e3e)
@athanatos @smithfarm
ceph_manager: use time before mon command for timeout
3943c45 
Slow mon commands can cause a false failure.

Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit 8cf2561)
@smithfarm smithfarm assigned ghost Jul 21, 2016
@ghost
Copy link

ghost commented Jul 21, 2016 

Reviewed-by: Loic Dachary <ldachary@redhat.com>

after a run of a relavant rados job confirms is works as expected

@smithfarm
Copy link
Contributor Author

@dachary This run hopefully qualifies: http://pulpito.ceph.com/smithfarm-2016-07-21_14:08:16-rados-hammer-backports---basic-smithi/

@smithfarm smithfarm added this to the hammer milestone Jul 21, 2016
@smithfarm
Copy link
Contributor Author

smithfarm commented Jul 21, 2016 •
edited

Ran into trouble. The job I want to run is rados/singleton/{all/divergent_priors.yaml fs/btrfs.yaml msgr-failures/few.yaml}

The command I'm trying is ./virtualenv/bin/teuthology-suite --filter="rados/singleton/{all/divergent_priors.yaml fs/btrfs.yaml msgr-failures/few.yaml}" --priority 101 --suite rados --suite-branch wip-832-hammer --email ncutler@suse.cz --ceph hammer-backports --machine-type smithi

The result is: 3237/3237 jobs were filtered out.

-Any idea what I'm doing wrong?- Now I see that this combination of YAML facets is rendered impossible by e55a52f in this patchset.

smithfarm
smithfarm reviewed Jul 23, 2016
View reviewed changes
suites/rados/singleton/all/ec-lost-unfound-upgrade.yaml
- - osd.3
tasks:
- install:
branch: infernalis
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dachary This commit introduces tests that install infernalis and then upgrade, so I guess the commit should be dropped from this PR. Agree?

@smithfarm
Copy link
Contributor Author

These tests appear to be broken by 139c100 - @dachary shall we drop it?

@athanatos @smithfarm
rados/singleton: use xfs instead
bfe3a89 
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit 7e0fb5d)
@athanatos @smithfarm
divergent_priors: add sleep after reviving divergent
c96f04b 
Basically same thing as
c0b0ec2
client is getting ENXIO when trying to do osd tell because
the osd isn't quite up yet.

Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit af3af30)
@smithfarm
Copy link
Contributor Author

smithfarm commented Jul 24, 2016 •
edited

Changelog:

  • dropped backport of ef75c89 from the patchset (because it references infernalis)
  • dropped backport of b2a08ef from the patchset (because it may break tests in hammer)

@smithfarm
Copy link
Contributor Author

@dachary Here's a rados run with the latest changes:

http://158.69.66.4:8081/ubuntu-2016-07-24_20:00:22-rados-hammer-backports---basic-openstack/

@ghost ghost merged commit 32014ad into ceph:hammer Jul 25, 2016
@smithfarm smithfarm deleted the wip-832-hammer branch July 25, 2016 08:43
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
hammer
3 participants
@smithfarm @liewegas @athanatos