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

fix #24780 change default isolation method in dol_clone #29391

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Al-thi
Copy link

@Al-thi Al-thi commented Apr 18, 2024

No description provided.

@eldy eldy added the PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) label Apr 18, 2024
@Al-thi Al-thi changed the base branch from 17.0 to develop April 18, 2024 13:22
@Al-thi Al-thi force-pushed the fix/24780 branch 2 times, most recently from 9621378 to dd50db3 Compare April 18, 2024 13:53
@Al-thi
Copy link
Author

Al-thi commented Apr 18, 2024

I am having Travis CI issues, I see a lot of failure from ppa.launchpad.net for multiple develop PRs.

Shouldn't we implement apt retry mechanism in travis configuration ?

cf. travis-ci/travis-ci#9112 (comment)

@Al-thi Al-thi marked this pull request as ready for review April 18, 2024 14:45
@Al-thi Al-thi force-pushed the fix/24780 branch 2 times, most recently from 31eeeb2 to e484a58 Compare April 19, 2024 07:46
@Al-thi Al-thi requested a review from eldy April 19, 2024 12:43
This fixes serialization error when attempting to update a custom field value.
This change is made globally because the previous native = 0 default isolation
method is labeled as `deprecated` in the code and therefore shouldn't be used
anymore.

Signed-off-by: Alexis Thietard <alexis.thietard@rtone.fr>
@eldy eldy added PR postponed PR is postponed (will be processed later). Ie: feature pushed during a beta or need transition steps and removed PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) labels Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR postponed PR is postponed (will be processed later). Ie: feature pushed during a beta or need transition steps
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants