Skip to content

Conversation

wtripp180901
Copy link
Contributor

@wtripp180901 wtripp180901 commented Aug 18, 2025

  • Host to deploy Pulp on now defined by pulp_server group rather than extravars for the deploy-pulp.yml adhoc
  • Pulp now managed by a systemd service
  • Group to trigger Pulp sync during fatimage has been renamed from pulp to pulp_site
  • Format of dnf_repo_timestamps.yml has been changed, with entries now containing all information required for both dnf_repos and pulp_site roles
  • dnf_repos file/package definitions and pulp repo path definitions now templated from structure in dnf_repo_timestamps.yml (no longer need to be defined manually in roles)

Breaking Change
The new Pulp repository definitions may conflict with repos already synced on your Pulp server. The easiest way to remediate this is to delete your existing Pulp state and redeploying. This can be done by

  1. Stopping the Pulp Podman container on the Pulp host
  2. Deleting the state directory at /home/rocky/pulp
  3. Re-run the ansible/adhoc/deploy-pulp.yml playbook

@wtripp180901 wtripp180901 force-pushed the refactor/regularise-pulp branch from 27e86de to 530ecf7 Compare August 18, 2025 14:54
@wtripp180901
Copy link
Contributor Author

@wtripp180901 wtripp180901 marked this pull request as ready for review August 18, 2025 14:56
@wtripp180901 wtripp180901 requested a review from a team as a code owner August 18, 2025 14:56
@wtripp180901
Copy link
Contributor Author

Got gpg check errors from build
Failed to validate GPG signature for epel-release-8-18.el8.noarch: Public key for epel-release-8-18.el8.noarch.rpm is not installed
so we do need the checks disabled for now, but maybe we need a follow-on PR to download the upstream keys

@wtripp180901
Copy link
Contributor Author

@wtripp180901
Copy link
Contributor Author

@wtripp180901
Copy link
Contributor Author

Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see comment/linked PR for ditching the top-level keys in the repo datastructure.

wtripp180901 and others added 3 commits September 1, 2025 11:08
remove extra level of key indentation
Co-authored-by: Steve Brasier <[email protected]>
@wtripp180901 wtripp180901 linked an issue Sep 1, 2025 that may be closed by this pull request
@wtripp180901 wtripp180901 changed the title Refactor Pulp repo definitions Refactor Pulp repo definitions and add more Pulp documentation Sep 1, 2025
@wtripp180901
Copy link
Contributor Author

Added the suggested changes from here #557 while I was updating the docs

@wtripp180901 wtripp180901 requested a review from sjpb September 1, 2025 14:45
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I just have grammar change and suggestion re. timestamps.yml naming

@wtripp180901 wtripp180901 requested a review from sjpb September 4, 2025 11:58
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sjpb sjpb merged commit 275da83 into main Sep 4, 2025
0 of 6 checks passed
@sjpb sjpb deleted the refactor/regularise-pulp branch September 4, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Question about IPA installation and repos more generally
2 participants