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

qa: krbd_exclusive_option.sh: fixup for json.tool ordering change #32358

Merged
merged 1 commit into from Jan 6, 2020

Conversation

idryomov
Copy link
Contributor

In Python 3.5 json.tool was changed to produce unsorted output and
--sort-keys option was added to compensate. This wasn't caught by
4fe245c ("qa: update krbd tests for python3") because it raced
with 50933b8 ("qa: krbd_exclusive_option.sh: update for recent
kernel changes").

Signed-off-by: Ilya Dryomov idryomov@gmail.com

Copy link

@dillaman dillaman left a comment

Choose a reason for hiding this comment

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

lgtm

In Python 3.5 json.tool was changed to produce unsorted output and
--sort-keys option was added to compensate.  This wasn't caught by
4fe245c ("qa: update krbd tests for python3") because it raced
with 50933b8 ("qa: krbd_exclusive_option.sh: update for recent
kernel changes").

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Copy link

@dillaman dillaman left a comment

Choose a reason for hiding this comment

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

lgtm

@idryomov
Copy link
Contributor Author

idryomov commented Jan 6, 2020

jenkins test make check

1 similar comment
@idryomov
Copy link
Contributor Author

idryomov commented Jan 6, 2020

jenkins test make check

@idryomov
Copy link
Contributor Author

idryomov commented Jan 6, 2020

@idryomov idryomov merged commit 53066d0 into ceph:master Jan 6, 2020
@idryomov idryomov deleted the wip-krbd-py3-fixup branch January 6, 2020 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants