Skip to content

Conversation

@gionn
Copy link
Member

@gionn gionn commented Mar 17, 2025

@gionn gionn self-assigned this Mar 17, 2025
@gionn gionn force-pushed the OPSEXP-3107-updatecli-roles branch from 3e13eb1 to d0423ed Compare March 17, 2025 14:53
@gionn gionn force-pushed the OPSEXP-3107-updatecli-roles branch from d0423ed to 890df51 Compare March 17, 2025 14:54
@gionn gionn requested a review from alxgomz March 17, 2025 15:04
@gionn gionn marked this pull request as ready for review March 17, 2025 15:04
Copy link
Contributor

Choose a reason for hiding this comment

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

Can,'t we have a single var for the target file? Roles should always have these keys in the default vars file

@gionn gionn force-pushed the OPSEXP-3107-updatecli-roles branch from 80f3ff3 to 5eb3a55 Compare March 18, 2025 10:05
@gionn gionn force-pushed the OPSEXP-3107-updatecli-roles branch 2 times, most recently from 6d2507a to feeaaac Compare March 18, 2025 10:19
@gionn gionn force-pushed the OPSEXP-3107-updatecli-roles branch from b4b8dcd to e791fad Compare March 18, 2025 10:23
@gionn gionn requested a review from alxgomz March 18, 2025 10:51
@gionn gionn merged commit 1bcb95d into master Mar 18, 2025
64 checks passed
@gionn gionn deleted the OPSEXP-3107-updatecli-roles branch March 18, 2025 11:45
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