Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: a pytest-specific action #2

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

bukzor
Copy link
Collaborator

@bukzor bukzor commented May 10, 2024

No description provided.

@bukzor bukzor force-pushed the bukzor/pytest-specific-action branch from 2fd1b08 to 34d077b Compare May 10, 2024 18:35
@bukzor bukzor force-pushed the bukzor/pytest-specific-action branch from 34d077b to f9c846f Compare May 10, 2024 18:37
@bukzor bukzor force-pushed the bukzor/pytest-specific-action branch from 6fa6df9 to 232e268 Compare May 10, 2024 18:42
@bukzor bukzor force-pushed the bukzor/pytest-specific-action branch 2 times, most recently from ad2f539 to 84e0bf2 Compare May 10, 2024 18:45
run:
shell: bash -euxo pipefail {0}
env:
PS4: "\e[36m$\e[m "
Copy link
Member

Choose a reason for hiding this comment

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

What is this used for?

uses: ./pytest
with:
_deploy_environ: dev
_gcp_project: di-dev-mtr--buck-5
Copy link
Member

Choose a reason for hiding this comment

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

Should we have a dedicated gcp project for self-test? Just so we don't accidentally clobber it.

workload_identity_provider: ${{ secrets.SENTRY_GCP_DEV_WORKLOAD_IDENTITY_POOL }}
service_account_email: ${{ secrets.SUDO_GCP_SERVICE_ACCOUNT }}
_deploy_environ: dev
_gcp_project: di-dev-mtr--buck-5
Copy link
Member

Choose a reason for hiding this comment

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

same comment as above

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