Skip to content

dehydrated: place each domain on a separate line - #475

Merged
aagbsn merged 1 commit into
mainfrom
fix_474_dehydrated_domains
Aug 5, 2026
Merged

dehydrated: place each domain on a separate line#475
aagbsn merged 1 commit into
mainfrom
fix_474_dehydrated_domains

Conversation

@aagbsn

@aagbsn aagbsn commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

this will create a certificate for each domain instead of a single certificate with altnames. fixes #474

this will create a certificate for each domain instead of a single
certificate with altnames
@aagbsn

aagbsn commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

Ansible Run Output 🤖

Ansible Playbook Recap 🔍



Ansible playbook output 📖success

Show Execution

$ ansible-playbook playbook.yml --check --diff -i ../tf/modules/ansible_inventory/inventories/inventory-dev.ini
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
[ERROR]: The role 'geerlingguy.docker' was not found in: /home/runner/work/devops/devops/ansible/roles:/home/runner/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/home/runner/work/devops/devops/ansible
Origin: /home/runner/work/devops/devops/ansible/deploy-testlists.yml:16:7

14         node_exporter_host: "0.0.0.0"
15         node_exporter_options: ""
16     - role: geerlingguy.docker
         ^ column 7

Pusher @aagbsn
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Wed, 05 Aug 2026 09:15:14 GMT

@aagbsn aagbsn mentioned this pull request Aug 5, 2026

@LDiazN LDiazN left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Worked fine for the detector panel, approved!

@aagbsn
aagbsn merged commit 550866b into main Aug 5, 2026
3 checks passed
@aagbsn
aagbsn deleted the fix_474_dehydrated_domains branch August 5, 2026 17:01
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.

dehydrated role puts all domains on 1 line in domains.txt

2 participants