Skip to content

Revert "ci: fix PR not triggering workflows by switching back to pytestbot access token"#10354

Merged
bluetech merged 1 commit intomainfrom
revert-10346-ci-pat
Oct 8, 2022
Merged

Revert "ci: fix PR not triggering workflows by switching back to pytestbot access token"#10354
bluetech merged 1 commit intomainfrom
revert-10346-ci-pat

Conversation

@bluetech
Copy link
Member

@bluetech bluetech commented Oct 7, 2022

Reverts #10346

Unfortunately the CHATOPS credentials no longer work, or don't have the necessary permissions. Was worth a try.

For now let's revert to get things working again, but can try again if:

  • Someone who has access to pytestbot can refresh the CHATOPS secret (with maybe a more appropriate name...)
  • We create a new machine user

@bluetech
Copy link
Member Author

bluetech commented Oct 8, 2022

Merging to unblock backports (cc @nicoddemus)

@bluetech bluetech merged commit 81113c1 into main Oct 8, 2022
@bluetech bluetech deleted the revert-10346-ci-pat branch October 8, 2022 13:31
@nicoddemus
Copy link
Member

Working again, thanks!

webknjaz added a commit to webknjaz/pytest that referenced this pull request Jun 19, 2024
Previously, this workflow was being used to cherry-pick PRs made
against `main` into older stable branches. Now that pytest-dev#12475 integrated
the Patchback GitHub App, it's no longer needed as it's making
duplicate pull requests which don't even trigger CI runs automatically
due to pytest-dev#10354. So this patch removes said workflow to address the
problem.
Glyphack pushed a commit to Glyphack/pytest that referenced this pull request Jun 20, 2024
Previously, this workflow was being used to cherry-pick PRs made
against `main` into older stable branches. Now that pytest-dev#12475 integrated
the Patchback GitHub App, it's no longer needed as it's making
duplicate pull requests which don't even trigger CI runs automatically
due to pytest-dev#10354. So this patch removes said workflow to address the
problem.
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