Skip to content

Trigger integration tests in python deps update#3463

Draft
derrickaw wants to merge 2 commits intoGoogleCloudPlatform:mainfrom
derrickaw:20260309_updatePythonWorkflow
Draft

Trigger integration tests in python deps update#3463
derrickaw wants to merge 2 commits intoGoogleCloudPlatform:mainfrom
derrickaw:20260309_updatePythonWorkflow

Conversation

@derrickaw
Copy link
Contributor

@derrickaw derrickaw commented Mar 9, 2026

Edit: Currently with generated python deps workflow, the workflow does not kick off the IT tests associated with these changes and we manually have to close and reopen the PR to kick this process off. Was trying to force the kick off process through these proposed changes.

@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@derrickaw derrickaw added the improvement Making existing code better label Mar 9, 2026
@derrickaw derrickaw marked this pull request as ready for review March 9, 2026 19:04
@derrickaw derrickaw changed the title [wip] - Trigger integration tests in python deps update Trigger integration tests in python deps update Mar 9, 2026
@derrickaw derrickaw requested a review from Abacn March 9, 2026 19:28
env:
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: |
gh workflow run bigtable-pr.yml --ref chore/update-python-deps
Copy link
Contributor

Choose a reason for hiding this comment

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

IIUC it triggered the test through workflow_dispatch trigger. It's status won't show in PR page then, not convenient for reviewers.

Current list of PR triggered tests looks sufficient: #3459

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, should have explained better. The tests do not get kicked off. We have to manually close and reopen the PR to get the tests to kick off. Was trying to automate this away.

Copy link
Contributor

Choose a reason for hiding this comment

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

I see, sounds a limitation of GitHub for bot generated PRs.

Note: If this PR does not trigger the template integration tests, please close the PR and reopen it. This should trigger all the tests.

though it sounds not convenient to manually check all workflow-dispatch triggered. After that I doubt we still need to close/reopen PRs get a green signal for PR itself

@pull-request-size pull-request-size bot added size/XS and removed size/S labels Mar 9, 2026
@derrickaw derrickaw marked this pull request as draft March 10, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Making existing code better size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants