Skip to content

CASSSIDECAR-420 Add manual approval step to CircleCI builds#331

Open
nvharikrishna wants to merge 2 commits intoapache:trunkfrom
nvharikrishna:420-circleci-manual-approval-trunk
Open

CASSSIDECAR-420 Add manual approval step to CircleCI builds#331
nvharikrishna wants to merge 2 commits intoapache:trunkfrom
nvharikrishna:420-circleci-manual-approval-trunk

Conversation

@nvharikrishna
Copy link
Copy Markdown
Contributor

@nvharikrishna nvharikrishna commented Mar 29, 2026

CASSSIDECAR-420: Added manual approval step to CircleCI pipeline so that they can be started when needed.

After this change, the user needs to give one-time approval to start the CI pipelines.

image

@frankgh
Copy link
Copy Markdown
Contributor

frankgh commented Apr 4, 2026

@nvharikrishna I don't see the GH actions active anymore in this project. Is this something that we have available somewhere else?

@nvharikrishna
Copy link
Copy Markdown
Contributor Author

I don't see the GH actions active anymore in this project. Is this something that we have available somewhere else?

@frankgh From the Actions tab I see below failure for the PRs. I am not sure if it is available anywhere else.

image

Copy link
Copy Markdown
Contributor

@frankgh frankgh left a comment

Choose a reason for hiding this comment

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

No harm in adding this, but I don't think the project longer supports GH actions.

@nvharikrishna nvharikrishna force-pushed the 420-circleci-manual-approval-trunk branch from 0635925 to d0ee6bd Compare April 4, 2026 18:26
@nvharikrishna
Copy link
Copy Markdown
Contributor Author

@frankgh @yifan-c Forgot to update the CHANGES.txt earlier, updated it now. Thanks for the review!

Copy link
Copy Markdown
Contributor

@bbotella bbotella left a comment

Choose a reason for hiding this comment

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

+1!

Thanks!

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