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

jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart #16293

Merged
merged 2 commits into from Sep 7, 2017

Conversation

Projects
None yet
2 participants
@smithfarm
Contributor

smithfarm commented Jul 12, 2017

@smithfarm smithfarm self-assigned this Jul 12, 2017

@smithfarm smithfarm added this to the jewel milestone Jul 12, 2017

@smithfarm smithfarm changed the title from jewel: test_envlibrados_for_rocksdb.yaml fails on crypto restart to jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart Jul 12, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Aug 1, 2017

Contributor

test this please

Contributor

smithfarm commented Aug 1, 2017

test this please

@smithfarm smithfarm changed the title from jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart to [DNM] jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart Aug 3, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Aug 3, 2017

Contributor

Marking DNM until we can get a clean "make check" and figure out why wip-jewel-backports only builds for Xenial.

Contributor

smithfarm commented Aug 3, 2017

Marking DNM until we can get a clean "make check" and figure out why wip-jewel-backports only builds for Xenial.

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Aug 29, 2017

Contributor

Jenkins re-test this please

Contributor

smithfarm commented Aug 29, 2017

Jenkins re-test this please

@smithfarm smithfarm changed the title from [DNM] jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart to jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart Aug 29, 2017

@smithfarm smithfarm changed the title from jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart to [DNM] jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart Sep 1, 2017

@tchaikov tchaikov self-assigned this Sep 1, 2017

@tchaikov

This comment has been minimized.

Show comment
Hide comment
@tchaikov

tchaikov Sep 1, 2017

Contributor

@smithfarm will take a look next Monday.

Contributor

tchaikov commented Sep 1, 2017

@smithfarm will take a look next Monday.

@tchaikov

This comment has been minimized.

Show comment
Hide comment
@tchaikov

tchaikov Sep 5, 2017

Contributor

i am trying to reproduce it locally.

Contributor

tchaikov commented Sep 5, 2017

i am trying to reproduce it locally.

Show outdated Hide outdated src/osdc/Objecter.h

@tchaikov tchaikov removed their assignment Sep 6, 2017

tchaikov added some commits Jun 21, 2017

crypto: allow PK11 module to load even if it's already initialized
there is chance that other pieces of application loads PK11 module
already and does not finalize it before calling common_init_finish().

also, upon fork, PK11 module resets its entire status including `nsc_init`,
by which PK11 module tell if it is initialized or not. so the behavior
of NSS_InitContext() could be different before and after fork. that's
another reason to ignore CKR_CRYPTOKI_ALREADY_INITIALIZED error (see
NSS_GetError()).

Fixes: http://tracker.ceph.com/issues/19741
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit fcc3eff)
osdc/Objecter: release message if it is not handled
Fixes: http://tracker.ceph.com/issues/19741
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 5c27742)

Conflicts:
   src/osdc/Objecter.h (in master, ms_fast_dispatch function declaration has an
                       "override" specifier; in jewel, this specifier is missing)

@smithfarm smithfarm changed the title from [DNM] jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart to jewel: core: test_envlibrados_for_rocksdb.yaml fails on crypto restart Sep 6, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 6, 2017

Contributor

jenkins build docs

Contributor

smithfarm commented Sep 6, 2017

jenkins build docs

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 6, 2017

Contributor

jenkins test docs

Contributor

smithfarm commented Sep 6, 2017

jenkins test docs

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 7, 2017

Contributor

Passed a rados suite at http://tracker.ceph.com/issues/20613#note-48 with some failures that are, I believe, unrelated.

Contributor

smithfarm commented Sep 7, 2017

Passed a rados suite at http://tracker.ceph.com/issues/20613#note-48 with some failures that are, I believe, unrelated.

@smithfarm smithfarm merged commit 26db2e7 into ceph:jewel Sep 7, 2017

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm deleted the smithfarm:wip-20460-jewel branch Sep 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment