Skip to content

mgr: serialize python objects sent between subinterpreters via remote#66240

Closed
athanatos wants to merge 2 commits intoceph:mainfrom
athanatos:wip-sjust-for-review-mgr-serialize
Closed

mgr: serialize python objects sent between subinterpreters via remote#66240
athanatos wants to merge 2 commits intoceph:mainfrom
athanatos:wip-sjust-for-review-mgr-serialize

Conversation

@athanatos
Copy link
Contributor

Replacement for #66071 on a branch without '/' in the name.

Show available Jenkins commands

You must only issue one Jenkins command per-comment. Jenkins does not understand
comments with more than one command.

@batrick
Copy link
Member

batrick commented Nov 17, 2025

This PR is under test in https://tracker.ceph.com/issues/73883.

@batrick
Copy link
Member

batrick commented Nov 17, 2025

This PR is under test in https://tracker.ceph.com/issues/73884.

@batrick
Copy link
Member

batrick commented Nov 17, 2025

This PR is under test in https://tracker.ceph.com/issues/73885.

@batrick
Copy link
Member

batrick commented Nov 17, 2025

This PR is under test in https://tracker.ceph.com/issues/73888.

@batrick
Copy link
Member

batrick commented Nov 17, 2025

This PR is under test in https://tracker.ceph.com/issues/73889.

@batrick
Copy link
Member

batrick commented Dec 9, 2025

This PR is under test in https://tracker.ceph.com/issues/74161.

Signed-off-by: Samuel Just <sjust@redhat.com>
…to allow unpickle to work correctly

Signed-off-by: Samuel Just <sjust@redhat.com>
@athanatos athanatos force-pushed the wip-sjust-for-review-mgr-serialize branch from b845600 to d4a1f2b Compare December 17, 2025 17:39
@Matan-B
Copy link
Contributor

Matan-B commented Dec 18, 2025

This PR is under test in https://tracker.ceph.com/issues/74161.

@batrick, I've noticed you are using debug builds for the test runs. fyi, I'm planning to merge ceph/ceph-build#2497 that should help with having the two build types by default (and used by crontab).

@batrick
Copy link
Member

batrick commented Dec 18, 2025

This PR is under test in https://tracker.ceph.com/issues/74161.

@batrick, I've noticed you are using debug builds for the test runs. fyi, I'm planning to merge ceph/ceph-build#2497 that should help with having the two build types by default (and used by crontab).

Thanks for the nudge. Please see my review.

@batrick
Copy link
Member

batrick commented Feb 7, 2026

This PR is under test in https://tracker.ceph.com/issues/74811.

@batrick
Copy link
Member

batrick commented Mar 3, 2026

retest this please

@NitzanMordhai
Copy link
Contributor

Closing since we have #66244

@batrick
Copy link
Member

batrick commented Mar 4, 2026

Closing since we have #66244

What's the harm in applying this? We should be moving in a direction where modules are not passing PyObject* to each other.

@batrick batrick reopened this Mar 4, 2026
@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@batrick
Copy link
Member

batrick commented Mar 5, 2026

Can one of the admins verify this patch?

@djgalloway @dmick how to do that? Why is that even triggering for a PR from a contributor in the Ceph org?

@batrick
Copy link
Member

batrick commented Mar 6, 2026

This PR is under test in https://tracker.ceph.com/issues/75377.

@NitzanMordhai
Copy link
Contributor

Closing, we have those commit as part of #66244

@djgalloway
Copy link
Contributor

Can one of the admins verify this patch?

@djgalloway @dmick how to do that? Why is that even triggering for a PR from a contributor in the Ceph org?

That has to have been a glitch.

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.

8 participants