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

WIP Proxy pass template #3741

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

WIP Proxy pass template #3741

wants to merge 2 commits into from

Conversation

acozine
Copy link
Contributor

@acozine acozine commented Mar 7, 2023

Supersedes #2776. I opened that PR off of a clone of the repo that no longer exists.

I've rebased the branch today so it will run on our new CI infrastructure. I'll close the other PR but please look there for earlier conversations on this topic!

The template we've been using for proxy pass configuration is more of a file than a template.

This PR pulls the data for proxy passes into its own (alphabetized) vars file, makes the template a template, and adds a task to fill out the template. Separating the data out doesn't reduce maintenance, but it does make the entries more readable, so folks can more easily add or edit entries for sites that need proxy passes.

@acozine acozine mentioned this pull request Mar 7, 2023
@acozine
Copy link
Contributor Author

acozine commented Mar 7, 2023

Current CI failure is in the libwww role:

  TASK [libwww : libwww | ensure MaxRequestWorkers is set for system use] ********
  An exception occurred during task execution. To see the full traceback, use -vvv. The error was: NoneType: None
  fatal: [instance]: FAILED! => {"changed": false, "msg": "src and dest are required"}

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.

1 participant