Skip to content

Feature/v4 certify#293

Open
steven-schattenberg-itential wants to merge 13 commits intoitential:feature/v4from
steven-schattenberg-itential:feature/v4-certify
Open

Feature/v4 certify#293
steven-schattenberg-itential wants to merge 13 commits intoitential:feature/v4from
steven-schattenberg-itential:feature/v4-certify

Conversation

@steven-schattenberg-itential
Copy link
Contributor

@steven-schattenberg-itential steven-schattenberg-itential commented Jan 29, 2026

This change adds:

  • A playbook to verify that the servers are suitable for installation.
  • A playbook to certify an installation that has occurred.
    • An associated set of tasks to perform the certification, one per component
    • A template to aggregate the certification results

This change also replaces the old preflight role and removes those files.

Copy link
Contributor

@kvelarde-itential kvelarde-itential left a comment

Choose a reason for hiding this comment

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

I think there are still some playbook/role structure issues. Let's discuss on a call.

Copy link
Contributor

Choose a reason for hiding this comment

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

As part of the integration with the new redis role, I think we will want to split this file into a certify-redis.yml and certify-sentinel.yml since redis and sentinel may run on different hosts.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed. I'd like to get this first iteration merged and then circle back and break out sentinel into its own certify and verify. I still have to merge the v4 redis changes anyway.

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.

2 participants