Skip to content

Conversation

@v1v
Copy link
Member

@v1v v1v commented Oct 1, 2025

No description provided.

@v1v v1v requested review from a team October 1, 2025 15:26
@v1v v1v self-assigned this Oct 1, 2025
@v1v v1v enabled auto-merge October 1, 2025 15:26
fr4nc1sc0-r4m0n
fr4nc1sc0-r4m0n previously approved these changes Oct 1, 2025
Copy link

@fr4nc1sc0-r4m0n fr4nc1sc0-r4m0n left a comment

Choose a reason for hiding this comment

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

LGTM

@v1v v1v added this pull request to the merge queue Oct 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 1, 2025
@v1v v1v added this pull request to the merge queue Oct 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 1, 2025
@v1v v1v added this pull request to the merge queue Oct 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 1, 2025
@v1v v1v enabled auto-merge October 1, 2025 17:17
@v1v v1v disabled auto-merge October 7, 2025 12:13
@v1v v1v requested a review from fr4nc1sc0-r4m0n October 7, 2025 12:24
# Only run e2e tests from the main branch as we need some credentials
# that we don't want to risk leaking from pullrequest opened by random contributors
if: github.ref == 'refs/heads/main' || ${{ inputs.run-e2e-tests }}
if: github.ref == 'refs/heads/main' || ( github.event_name == 'workflow_dispatch' && inputs.run-e2e-tests == true )
Copy link
Member Author

Choose a reason for hiding this comment

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

regression, so I had to fix it otherwise it's always true

Tested manually https://github.com/elastic/oblt-updatecli-policies/actions/runs/18312529977/job/52144350325

Copy link

@fr4nc1sc0-r4m0n fr4nc1sc0-r4m0n left a comment

Choose a reason for hiding this comment

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

LGTM

@v1v v1v added this pull request to the merge queue Oct 7, 2025
Merged via the queue into main with commit efac3e6 Oct 7, 2025
2 of 3 checks passed
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.

3 participants