Skip to content

PS-10483 - Fixes needed for bumping molecule version - #1199

Open
tplavcic wants to merge 11 commits into
masterfrom
ps-10483-bump-molecule
Open

PS-10483 - Fixes needed for bumping molecule version#1199
tplavcic wants to merge 11 commits into
masterfrom
ps-10483-bump-molecule

Conversation

@tplavcic

@tplavcic tplavcic commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

These small fixes are needed for bumping Molecule version to 26.6.0.
Corresponding PR in jenkins-pipelines repo: Percona-Lab/jenkins-pipelines#4354

These changes are required to run the scenarios under Molecule 26.x. Molecule 26.x validates molecule.yml against a strict schema (scenario has additionalProperties: false), so the long-standing clenup_sequence misspelling — silently ignored by Molecule 3.3.0 — now fails validation and must be corrected to cleanup_sequence.

Separately, the Molecule v6 rewrite dropped the implicit roles path it used to inject, so scenarios whose converge playbook loads a role by name can no longer find it; adding ANSIBLE_ROLES_PATH: "${MOLECULE_PROJECT_DIRECTORY}/.." to the provisioner env restores that resolution.

Copilot AI 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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@tplavcic
tplavcic requested a review from panchal-yash August 19, 2026 12:25
@tplavcic
tplavcic requested a lite review from Copilot August 20, 2026 06:35

Copilot AI 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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@tplavcic
tplavcic marked this pull request as ready for review August 21, 2026 08:31

@kaushikpuneet07 kaushikpuneet07 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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.

4 participants