Skip to content

Conversation

@gionn
Copy link
Member

@gionn gionn commented Nov 26, 2024

@gionn gionn self-assigned this Nov 26, 2024
@gionn gionn force-pushed the OPSEXP-2880-audit-storage branch from eed77e9 to 6258361 Compare November 26, 2024 11:58
@gionn gionn requested review from alxgomz and pmacius November 26, 2024 13:38
@gionn gionn force-pushed the OPSEXP-2880-audit-storage branch from 6ed42e6 to 587c44c Compare November 26, 2024 15:12
@gionn gionn force-pushed the OPSEXP-2880-audit-storage branch from 587c44c to de5e01a Compare November 26, 2024 15:32
@gionn gionn removed request for alxgomz and pmacius November 26, 2024 16:51
@gionn gionn changed the title OPSEXP-2880 Add audit-storage and systemd-service roles OPSEXP-2880 Add audit-storage role Nov 27, 2024
@gionn gionn requested review from alxgomz and pmacius November 27, 2024 13:42
@gionn gionn marked this pull request as ready for review November 27, 2024 13:42
pmacius
pmacius previously approved these changes Nov 28, 2024
Copy link
Contributor

@pmacius pmacius left a comment

Choose a reason for hiding this comment

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

Are we disabling this role for older version?

@gionn
Copy link
Member Author

gionn commented Nov 28, 2024

Are we disabling this role for older version?

good catch, let me add an explicit validation for that

@gionn gionn requested a review from pmacius November 29, 2024 09:34
@gionn gionn added the ec2-test Triggers ec2 integrations tests label Nov 29, 2024
Copy link
Contributor

@pmacius pmacius left a comment

Choose a reason for hiding this comment

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

We can bump dtas version to v1.6.0 and configure tests for audit-storage

@gionn gionn force-pushed the OPSEXP-2880-audit-storage branch from fbc964c to 4df8a0a Compare November 29, 2024 14:07
@gionn gionn requested a review from pmacius December 2, 2024 11:32
pmacius
pmacius previously approved these changes Dec 2, 2024
Copy link
Contributor

@pmacius pmacius left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@alxgomz alxgomz left a comment

Choose a reason for hiding this comment

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

Just a few comments, but maybe fix the password defaults in the module if there's not reason for them?

@gionn gionn merged commit 8e80050 into master Dec 3, 2024
62 checks passed
@gionn gionn deleted the OPSEXP-2880-audit-storage branch December 3, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ec2-test Triggers ec2 integrations tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants