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

Dynamic roles #3054

Open
wants to merge 19 commits into
base: development
Choose a base branch
from
Open

Dynamic roles #3054

wants to merge 19 commits into from

Conversation

fridim
Copy link
Collaborator

@fridim fridim commented Oct 15, 2020

This change, if applied, adds hooks in all stages to dynamically load ansible roles on differents hosts.

  • always localhost for pre-infra, infra, destroy
  • groups of hosts for post-infra, pre-software, software and post-software steps

ex:

agnosticd_pre_infra_roles:
- name: infra-ec2-ssh-key
- name: infra-aws-capacity-reservation
  when: a custom condition

agnosticd_pre_software_roles:
- name: set-repositories
  group: all
- name: install_common
  group: all
- name: bastion-lite
  group: bastions
- name: bastion-student-user
  group: bastions
- name: host_virtualenv
  group: bastions

Can be used coupled with agnosticd_dynamic role:

agnosticd_dynamic_role_galaxy_sources:
  - name: docker
    src: geerlingguy.docker
    version: 0.2.0

agnosticd_post_software_roles:
- name: docker
  group: bastion
SUMMARY
ISSUE TYPE
  • Feature Pull Request
  • New config Pull Request
  • New role Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

@fridim fridim force-pushed the dynamic-workloads branch 2 times, most recently from 108432f to b876a6c Compare June 21, 2021 11:00
@fridim fridim force-pushed the dynamic-workloads branch from 2791f53 to 1c8674b Compare July 12, 2021 07:35
@fridim fridim changed the title Allow dynamic workloads in agnosticd (WIP) Dynamic workloads in agnosticd Jul 12, 2021
@fridim fridim changed the title Dynamic workloads in agnosticd Dynamic roles Jul 12, 2021
@fridim fridim marked this pull request as ready for review December 6, 2021 09:27
@fridim fridim requested a review from a team as a code owner December 6, 2021 09:27
@fridim fridim requested a review from jkupferer December 6, 2021 09:36
Copy link
Collaborator

@wilson-walrus wilson-walrus left a comment

Choose a reason for hiding this comment

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

Thumbs up

- update test lookup to support booleans
@fridim fridim requested a review from tonykay March 15, 2022 05:50
@9strands
Copy link
Collaborator

@fridim @tonykay - do we still need this PR? I'll close in Jan if I haven't heard back so we can clean out old stuff.

@9strands 9strands closed this Dec 22, 2022
@fridim fridim reopened this Mar 14, 2023
@bbethell-1 bbethell-1 closed this Mar 18, 2024
@bbethell-1
Copy link
Contributor

Closing this as its now been 2 years Jan 2024

@fridim fridim reopened this Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants