Skip to content

Conversation

jovial
Copy link
Contributor

@jovial jovial commented Feb 4, 2022

Pulp requires you to set the proxy for each remote.

Pulp requires you to the proxy for each remote.
@jovial
Copy link
Contributor Author

jovial commented Feb 4, 2022

This is untested when the pulp_proxy_url is unmodified, but I think you can use omit like this:

---

- hosts: localhost
  vars:
    test: "{{ omit }}"
  tasks:
    - debug:
        msg: "{{ test }}"
[will@juno ~]$ ansible-playbook /tmp/test.yml 
[WARNING]: No inventory was parsed, only implicit localhost is available
[WARNING]: provided hosts list is empty, only localhost is available. Note that the
implicit localhost does not match 'all'

PLAY [localhost] *********************************************************************

TASK [Gathering Facts] ***************************************************************
ok: [localhost]

TASK [debug] *************************************************************************
ok: [localhost] => {
    "msg": "Hello world!"
}

PLAY RECAP ***************************************************************************
localhost                  : ok=2    changed=0    unreachable=0    failed=0    skipped=0    rescued=0    ignored=0 

@jovial
Copy link
Contributor Author

jovial commented Jun 28, 2022

I've cherry-picked the proxy support to wallaby here: stackhpc/kayobe#18

markgoddard
markgoddard previously approved these changes Jun 28, 2022
priteau
priteau previously approved these changes Jun 28, 2022
Copy link
Contributor

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

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

Could you add the same for the Apt repos?

The default did not work well when using a proxy deployed on the seed.
Leave it up the user to explictly enable.
@jovial jovial dismissed stale reviews from priteau and markgoddard via eb759c4 November 15, 2022 12:23
@jovial jovial requested a review from a team as a code owner November 15, 2022 12:23
@jovial
Copy link
Contributor Author

jovial commented Nov 15, 2022

Could you add the same for the Apt repos?

Done

@jovial jovial closed this Nov 15, 2022
@jovial jovial reopened this Nov 15, 2022
@jovial jovial closed this Nov 16, 2022
@jovial jovial reopened this Nov 16, 2022
@jovial jovial merged commit a95805e into stackhpc/wallaby Nov 16, 2022
@jovial jovial deleted the feature/pulp-proxy branch November 16, 2022 14:29
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.

3 participants