Skip to content

rgw: add reshard test#22939

Closed
oritwas wants to merge 7 commits intoceph:masterfrom
oritwas:wip-rgw-add-reshard-test
Closed

rgw: add reshard test#22939
oritwas wants to merge 7 commits intoceph:masterfrom
oritwas:wip-rgw-add-reshard-test

Conversation

@oritwas
Copy link
Member

@oritwas oritwas commented Jul 8, 2018

No description provided.

@oritwas
Copy link
Member Author

oritwas commented Jul 23, 2018

jenkins test this please

@oritwas oritwas force-pushed the wip-rgw-add-reshard-test branch 2 times, most recently from 6733153 to 65e6a3e Compare July 23, 2018 06:46
@oritwas
Copy link
Member Author

oritwas commented Jul 24, 2018

jenkins test this please

Copy link
Contributor

@cbodley cbodley left a comment

Choose a reason for hiding this comment

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

looks good if it passes!

connection = boto.s3.connection.S3Connection(
aws_access_key_id=access_key,
aws_secret_access_key=secret_key,
is_secure=False,
Copy link
Contributor

Choose a reason for hiding this comment

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

if we ever run this in the verify suite (or enable ssl in other suites) we'll need to enable this. in my wip branch for radosgw_admin_rest task, i'm using this line:

        is_secure=True if endpoint.cert else False,

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 , do you think it should run in verify suite?

Copy link
Contributor

Choose a reason for hiding this comment

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

not really, no - at least not in the same job as s3/swift tests because of the ceph.conf overrides

oritwas added 7 commits July 25, 2018 11:13
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
@oritwas oritwas force-pushed the wip-rgw-add-reshard-test branch 2 times, most recently from 6f19a02 to 98bc9f9 Compare July 25, 2018 08:15
@stale
Copy link

stale bot commented Oct 17, 2018

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you are a maintainer or core committer, please follow-up on this issue to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label Oct 17, 2018
@mattbenjamin
Copy link
Contributor

@cbodley anything blocking this?

@stale stale bot removed the stale label Oct 17, 2018
@cbodley
Copy link
Contributor

cbodley commented Oct 19, 2018

@cbodley anything blocking this?

@mattbenjamin nope, it looks like a valuable test to catch regressions in stats. i hope it still passes

@stale
Copy link

stale bot commented Dec 18, 2018

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you are a maintainer or core committer, please follow-up on this issue to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label Dec 18, 2018
@cbodley
Copy link
Contributor

cbodley commented Dec 18, 2018

please don't kill this, friendly robot

@stale stale bot removed the stale label Dec 18, 2018
@stale
Copy link

stale bot commented Feb 16, 2019

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you are a maintainer or core committer, please follow-up on this issue to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label Feb 16, 2019
@stale
Copy link

stale bot commented May 17, 2019

This pull request has been automatically closed because there has been no activity for 90 days. Please feel free to reopen this pull request (or open a new one) if the proposed change is still appropriate. Thank you for your contribution!

@stale stale bot closed this May 17, 2019
@cbodley cbodley reopened this May 21, 2019
@stale stale bot removed the stale label May 21, 2019

# check bucket shards num
(err, out) = rgwadmin(ctx, client, ['reshard', 'list'], check_status=True)
assert len(out) == 1
Copy link
Contributor

Choose a reason for hiding this comment

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

a test run is showing a failure here:

http://qa-proxy.ceph.com/teuthology/cbodley-2019-05-23_20:37:05-rgw-wip-kefu-testing-2019-05-23-0009-distro-basic-smithi/3972928/teuthology.log

it may be due to:

2019-05-23T20:52:24.095 INFO:teuthology.orchestra.run.smithi084.stderr:parse error setting 'rgw_reshard_thread_interval' to '30' (Value '30' is below minimum 600)

@stale
Copy link

stale bot commented Jul 22, 2019

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label Jul 22, 2019
@cbodley cbodley removed the stale label Aug 1, 2019
@stale
Copy link

stale bot commented Sep 20, 2019

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label Sep 20, 2019
@stale stale bot removed the stale label Dec 11, 2019
@yuriw
Copy link
Contributor

yuriw commented Dec 11, 2019

@stale
Copy link

stale bot commented Feb 11, 2020

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale
Copy link

stale bot commented May 12, 2020

This pull request has been automatically closed because there has been no activity for 90 days. Please feel free to reopen this pull request (or open a new one) if the proposed change is still appropriate. Thank you for your contribution!

@stale stale bot closed this May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants