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

Adds test for re-import after broken import-attempt. #969

Merged
merged 1 commit into from Aug 3, 2023

Conversation

ggainey
Copy link
Contributor

@ggainey ggainey commented Jun 13, 2023

Tests the support added to fix pulp/pulpcore#3737

[noissue]

@ggainey
Copy link
Contributor Author

ggainey commented Jun 14, 2023

lowerbounds will infinite-loop until pulp/pulpcore#3924 is available - and after that, it will just fail because the fix that's being tested here, isn't in core/3.23, which is the current lowerbound for pulp_file/main.

Setting this to DRAFT - will work to get a "skip if server.status.core in (3.23/24/25)" (or something like that) in place.

@ggainey ggainey marked this pull request as draft June 14, 2023 20:17
@lubosmj
Copy link
Member

lubosmj commented Jul 19, 2023

@ggainey ggainey closed this Aug 3, 2023
@ggainey ggainey reopened this Aug 3, 2023
@ggainey ggainey marked this pull request as ready for review August 3, 2023 16:22
@ggainey
Copy link
Contributor Author

ggainey commented Aug 3, 2023

All required changes made in core (including backporting the change this is testing, to core/3.23 - so we no longer need a requires-plugin-version test). Tests all pass now, merging.

@ggainey ggainey merged commit 7d1e048 into pulp:main Aug 3, 2023
28 of 29 checks passed
@ggainey ggainey deleted the 3737_test_reimport branch February 2, 2024 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants