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.

Sign up for GitHub

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

Jump to bottom

pulp_file after being merged still reports as python-package = "pulp-file" #4728

Closed
mdellweg opened this issue Nov 16, 2023 · 1 comment · Fixed by #4729
Closed

pulp_file after being merged still reports as python-package = "pulp-file" #4728

mdellweg opened this issue Nov 16, 2023 · 1 comment · Fixed by #4729
Labels
Issue

Comments

@mdellweg
Copy link
Member

Instead, i think it should be "pulpcore".
Complication: To my knowledge the current way is needed to allow tools to call bindings generation correctly. We may need to add a python_module="pulp_file" to the status response.
@mdellweg
Copy link
Member Author

OTOH python_module should probably never be exposed to the user.
The sad part is, we do when querying the api for a certain installed plugin.

mdellweg added a commit to mdellweg/pulpcore that referenced this issue Nov 16, 2023
@mdellweg
Correctly report file plugin provided by pulpcore
82f9ec8 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
@mdellweg mdellweg mentioned this issue Nov 16, 2023
Merged
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Nov 16, 2023
@mdellweg
Correctly report file plugin provided by pulpcore
1fc49fd 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Nov 24, 2023
@mdellweg
Correctly report file plugin provided by pulpcore
9d171ef 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Nov 27, 2023
@mdellweg
Correctly report file plugin provided by pulpcore
2de73c2 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Nov 27, 2023
@mdellweg
Correctly report file plugin provided by pulpcore
d11bf8e 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Nov 28, 2023
@mdellweg
Correctly report file plugin provided by pulpcore
ebcabb4 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Jan 9, 2024
@mdellweg
Correctly report file plugin provided by pulpcore
0f7e4f7 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Jan 10, 2024
@mdellweg
Correctly report file plugin provided by pulpcore
1e1e971 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit to mdellweg/pulpcore that referenced this issue Jan 15, 2024
@mdellweg
Correctly report file plugin provided by pulpcore
18e8c87 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes pulp#4728
mdellweg added a commit that referenced this issue Jan 15, 2024
@mdellweg
Correctly report file plugin provided by pulpcore
4d5c5ff 
Since being merged, the python package for the pulp app labeled 'file'
is supposed to be "pulpcore". Due to some inconsistencies around the use
of app label vs. python package vs. python module, we need some
straightening all around.

fixes #4728
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Issue
Projects
RH Pulp Kanban board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Correctly report file plugin provided by pulpcore mdellweg/pulpcore
2 participants
@dkliban @mdellweg