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

With a combination of failed and successful syncs, it is possible to have modulemd content with no artifact associated #2735

Closed
dralley opened this issue Aug 24, 2022 · 1 comment · Fixed by #2813

Comments

@dralley
Copy link
Contributor

dralley commented Aug 24, 2022

Version
3.14, 3.17, possibly main branch

Describe the bug

With a combination of multiple failed and successful sync attempts on a repo like "Appstream 8", It may happen that, One or more artifacts related to modulemd metdata is physically missing from the filesystem. But if we check the pulpcore database, then the same artifact is having an artifact_id associated with it which means pulp thinks the file is alredy created\downloaded on the filesystem.

note from dalley: this means the artifact associated with modulemd content which stores the snippets

@pavelpicka This might cause problems for #2707

To Reproduce

See bugzilla for details

Expected behavior
This should never happen

Additional context
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2107701

@dralley
Copy link
Contributor Author

dralley commented Aug 25, 2022

Action item: determine whether this is something that can happen in normal usage, or if you need to do something weird, like cloning the database without copying the file storage.

If it is the latter, then this issue can be closed as not applicable.

@pavelpicka pavelpicka self-assigned this Sep 9, 2022
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 12, 2022
When artifact with modulemd snippet missing re-sync will fix it.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 12, 2022
When artifact with modulemd snippet missing re-sync will fix it.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 23, 2022
Recreate modular metadata from DB.
Logic is backportable to earlier versions which using artifacts to save
snipepts.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 23, 2022
Recreate modular metadata from DB.
Logic is backportable to earlier versions which using artifacts to save
snipepts.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 23, 2022
Recreate modular metadata from DB.
Logic is backportable to earlier versions which using artifacts to save
snipepts.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 26, 2022
Recreate modular metadata from DB.
Logic is backportable to earlier versions which using artifacts to save
snipepts.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 30, 2022
extend RpmQueryExisting stage to fix missing modular snippets.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 30, 2022
extend RpmQueryExisting stage to fix missing modular snippets.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 30, 2022
Extend RpmQueryExisting stage to fix missing modular snippets.
Minor fix of modulemd parsing. Get types right as they should be saved in DB.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Sep 30, 2022
Extend RpmQueryExisting stage to fix missing modular snippets.
Minor fix of modulemd parsing. Get types right as they should be saved in DB.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Oct 3, 2022
Extend RpmQueryExisting stage to fix missing modular snippets.
Minor fix of modulemd parsing. Get types right as they should be saved in DB.

closes: pulp#2735
pulp#2735
pavelpicka added a commit to pavelpicka/pulp_rpm that referenced this issue Oct 3, 2022
Extend RpmQueryExisting stage to fix missing modular snippets.
Minor fix of modulemd parsing. Get types right as they should be saved in DB.

closes: pulp#2735
pulp#2735
@dralley dralley closed this as completed in 74ebcba Oct 3, 2022
patchback bot pushed a commit that referenced this issue Oct 3, 2022
Extend RpmQueryExisting stage to fix missing modular snippets.
Minor fix of modulemd parsing. Get types right as they should be saved in DB.

closes: #2735
#2735
(cherry picked from commit 74ebcba)
dralley pushed a commit that referenced this issue Oct 3, 2022
Extend RpmQueryExisting stage to fix missing modular snippets.
Minor fix of modulemd parsing. Get types right as they should be saved in DB.

closes: #2735
#2735
(cherry picked from commit 74ebcba)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
2 participants