forked from pulp/pulp_file
-
Notifications
You must be signed in to change notification settings - Fork 0
Update plugin template #7
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
Open
mikedep333
wants to merge
68
commits into
dkliban:main
Choose a base branch
from
mikedep333:update_plugin_template
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[noissue]
[noissue]
fixes: #724 Required PR: pulp/pulpcore#2749
[noissue]
[noissue]
[noissue]
[noissue]
[noissue]
fixes: #729 Required PR: pulp/pulpcore#2797
Required PR: pulp/pulpcore#2803 [noissue]
The test fails because it needs to actually pip uninstall during the test. By not doing so, migrations are leftover and the restart script hangs waiting for them to apply. The suggestion on Matrix was to skip for now. [noissue]
This is a pulpcore 3.20 compatability requirement. [noissue]
[noissue] Required PR: pulp/pulpcore#2839
Runs functional tests in container running Pulp. Removes ruby bindings test job. Removes the custom script for running functional tests. [noissue]
[noissue] Required PR: pulp/pulpcore#2795
[noissue]
[noissue]
[noissue]
[noissue]
This test requires a proper uninstallation of the plugin (due to a CI script which checks for unapplied migrations) and then installing the plugin back from the commit in the PR. The latter is not straightforward and is considered not worth the time and effort, because the plugin removal is not a common operation and in most use cases needed for the pulp-2to3-migration plugin only. [noissue]
This patch also removes the test_download_content module of tests. [noissue]
This functionality is already tested as part of the test_auto_publish test module. [noissue]
[noissue]
[noissue]
[noissue]
Comma is a valid filename character in both Linux and Windows
file systems, but the previous way of generating manifest file
did not allow them. This commit fixes this issue, and introduces
a test for this very specific edge-case. The test is split into
all three possible sub-cases: comma in filename, comma in folder
name, comma(s) in both.
closes #630
[noissue]
This patch removes the from_pulpcore.test_crd_publications module. The test_publications module has been augmented with tests that were in from_pulpcore.test_crd_publications module. This patch makes a small adjustment to the CRUD tests for repositories. [noissue]
[noissue]
[noissue] Required PR: pulp/pulpcore#3017
[noissue]
[noissue]
[noissue]
Required PR: pulp/pulpcore#3059 [noissue]
[noissue]
[noissue]
[noissue]
Required PR: pulp/pulpcore#2992 [noissue]
[noissue]
[noissue]
[noissue]
[noissue] Required PR: pulp/pulpcore#3150
Required PR: pulp/pulpcore#3093 [noissue]
A new feature has been added to pulpcore, which allows users to list all master model remotes at /pulp/api/v3/remotes/. A test case covering this feature has been added. [noissue] Required-PR: pulp/pulpcore#3150
[noissue]
Attemting to upload a FileContent unit already present inside a
different repository no longer results in an exception being raised,
href of the existing unit is returned instead.
This commit implements support for the plugin-specific feature
implemented inside pulpcore for pulp_file.
The test case prohibiting duplicate filecontent is now outdated
and has been replaced by two new test cases covering the behavior
with both unspecified/specified target repository when uploading
existing filecontent.
closes #774
[noissue]
[noissue] Required PR: pulp/pulpcore#3179
Includes increasing pulp-workers from 2 to 4 [noissue]
39efcd5 to
7b1c3c1
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.