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

rgw: changes to support ragweed #13644

Merged
merged 10 commits into from Feb 22, 2018
Merged

rgw: changes to support ragweed #13644

merged 10 commits into from Feb 22, 2018

Conversation

yehudasa
Copy link
Member

@yehudasa yehudasa commented Feb 25, 2017

  • teuthology ragweed task
  • test suite yaml config for ragweed tests (upgrade, plain)

@yehudasa yehudasa changed the title rgw: changes to support ragweed [DNM] rgw: changes to support ragweed Feb 25, 2017
@yehudasa yehudasa mentioned this pull request Feb 25, 2017
@yehudasa yehudasa force-pushed the wip-rgw-ragweed branch 3 times, most recently from 446b44a to 0192cca Compare February 25, 2017 00:58
@yehudasa
Copy link
Member Author

@cbodley this has dnm, as it needs #13645 to be merged first. It currently points at the wip-rgw-support-ragweed branch for the base version, but that needs to be changed once the other PR gets merged. Potentially we'll want to have the other PR also backported to kraken, and use kraken as the base version.

client.0:
force-branch: ceph-master
rgw_server: client.0
stages: check
Copy link
Contributor

Choose a reason for hiding this comment

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

can this not just use a single ragweed task with stages: check, prepare? (same with multifs/tasks/rgw_ragweed.yaml)

Copy link
Member Author

Choose a reason for hiding this comment

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

@cbodley we can, but this lays out the template for multi-stage runs which are more interesting.

- desc: install ceph/<base> latest
tasks:
- install:
branch: wip-rgw-support-ragweed
Copy link
Contributor

Choose a reason for hiding this comment

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

TODO, as you pointed out

@liewegas
Copy link
Member

@yehudasa ping

@yehudasa yehudasa changed the title [DNM] rgw: changes to support ragweed rgw: changes to support ragweed Jan 2, 2018
@yehudasa
Copy link
Member Author

yehudasa commented Jan 2, 2018

@cbodley can we merge this?

@cbodley
Copy link
Contributor

cbodley commented Jan 2, 2018

as long as it's passing, sure 👍

@yehudasa
Copy link
Member Author

yehudasa commented Jan 3, 2018

@cbodley passing? who said anything about passing?

@yuriw
Copy link
Contributor

yuriw commented Feb 14, 2018

assert isinstance(config, dict)
testdir = teuthology.get_testdir(ctx)
attrs = ["!fails_on_rgw"]
if not ctx.rgw.use_fastcgi:
Copy link
Contributor

Choose a reason for hiding this comment

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

AttributeError: 'Namespace' object has no attribute 'use_fastcgi'

fcgi stuff was removed from the rgw task a while ago

@cbodley
Copy link
Contributor

cbodley commented Feb 15, 2018

quite a few test failures in http://pulpito.front.sepia.ceph.com/yuriw-2018-02-15_01:25:35-rgw-wip-yuri-testing-2018-02-14-2132-distro-basic-smithi/

mainly bootstrap failures from cd /home/ubuntu/cephtest/ragweed && ./bootstrap

@yehudasa
Copy link
Member Author

bootstraps failing on centos for some reason

@yehudasa
Copy link
Member Author

@cbodley @yuriw I pushed a new ragweed version that fixes the bootstrap issue, and updated this branch to avoid the second issue that casey was pointing at (fastcgi use)

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
and only remove users when doing check.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
and also whitelist 'wrongly marked me down'

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Change directory structure to reflect base version of upgrade test.
Also set base version to kraken.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
@cbodley
Copy link
Contributor

cbodley commented Feb 22, 2018

@cbodley cbodley merged commit fbb6262 into ceph:master Feb 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants