Skip to content

chore: add version check to crds workflow#2218

Open
vrutkovs wants to merge 2 commits into
masterfrom
skip-crd-workflow-for-lts
Open

chore: add version check to crds workflow#2218
vrutkovs wants to merge 2 commits into
masterfrom
skip-crd-workflow-for-lts

Conversation

@vrutkovs
Copy link
Copy Markdown
Collaborator

Make sure that LTS releases don't create PRs for helm-chart to update the CRDs

Make sure that LTS releases don't create PRs for helm-chart to update the CRDs
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 2 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread .github/workflows/crds.yaml Outdated
@vrutkovs vrutkovs force-pushed the skip-crd-workflow-for-lts branch from aa07f9b to e334158 Compare May 27, 2026 10:29
fi

set +e
python3 ../__vm-operator-repo/hack/compare_versions.py "$chart_app_version" "$RELEASE_VERSION"
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.

can this script just produce stdout value and store it to >> $GITHUB_OUTPUT?
then it chould be used in subsequent steps if section

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

That was my first option but then I thought that some stdout may also leak there?

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.

only data explicitly written to GITHUB_OUTPUT is a step output, it doesn't capture whole stdout.

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