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

[Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. #136

Merged
merged 3 commits into from
Jan 10, 2024

Conversation

calebevans
Copy link
Collaborator

The DPTP team is moving all artifacts to the test-platform-results GCS bucket. This is happening today, this PR should be merged following the completion of the task.

DPTP comms about this change: https://redhat-internal.slack.com/archives/CFUGK0K9R/p1704920416570079

@calebevans calebevans self-assigned this Jan 10, 2024
@redhat-qe-bot
Copy link
Collaborator

Report bugs in Issues

The following are automatically added:

  • Add reviewers from OWNER file (in the root of the repository) under reviewers section.
  • Set PR size label.
  • New issue is created for the PR. (Closed when PR is merged/closed)
  • Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

  • To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
  • To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
  • To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.
    • Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
    • Cherry-pick will be started when PR is merged
  • To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
  • To add a label by comment use /<label name>, to remove, use /<label name> cancel
Supported /retest check runs
Supported labels
  • hold
  • verified
  • wip
  • lgtm

@calebevans calebevans changed the title [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. WIP: [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. Jan 10, 2024
@calebevans calebevans changed the title WIP: [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. Jan 10, 2024
@calebevans calebevans changed the title [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. WIP: [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. Jan 10, 2024
@calebevans calebevans changed the title WIP: [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. [Urgent Change] Update the default gcs bucket to test-platform-results per DPTPs change. Jan 10, 2024
@calebevans
Copy link
Collaborator Author

/verified

@calebevans
Copy link
Collaborator Author

calebevans commented Jan 10, 2024

Tested this change in stage and it is working.

Stage ticket created using the new bucket: https://issues.stage.redhat.com/browse/LPTOCPCI-886

@calebevans calebevans enabled auto-merge (squash) January 10, 2024 21:29
@amp-rh
Copy link
Collaborator

amp-rh commented Jan 10, 2024

/lgtm

@calebevans
Copy link
Collaborator Author

/verified

@btjd
Copy link

btjd commented Jan 10, 2024

/lgtm

@calebevans
Copy link
Collaborator Author

@amp-rh my bad, I found a couple of extra instances of this old bucket in the links to logs in tickets. Can you check this again?

@calebevans
Copy link
Collaborator Author

/verified

@calebevans calebevans merged commit 3cf4d29 into RedHatQE:main Jan 10, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants