Skip to content

scripts: authenticate git push for release#8352

Merged
pbrezina merged 1 commit intomasterfrom
release
Jan 14, 2026
Merged

scripts: authenticate git push for release#8352
pbrezina merged 1 commit intomasterfrom
release

Conversation

@pbrezina
Copy link
Member

actions/checkout@v6 changed behavior and the credentials can be reused
only if it is run from the same directory. But we copy the repository to
a temporary location to prepare the release, so we need to
re-authenticate in the tmp folder.

@pbrezina
Copy link
Member Author

I will just push this under one-liner rule, no review is required. It can be only tested from Github Actions environment, I will unpush it if it won't work.

@pbrezina
Copy link
Member Author

actions/checkout@v6 changed behavior and the credentials can be reused
only if it is run from the same directory. But we copy the repository to
a temporary location to prepare the release, so we need to
re-authenticate in the tmp folder.

Reviewed-by: Justin Stephenson <jstephen@redhat.com>
@sssd-bot
Copy link
Contributor

The pull request was accepted by @pbrezina with the following PR CI status:


🟢 CodeFactor (success)
🟢 CodeQL (success)
🟡 osh-diff-scan:fedora-rawhide-x86_64:upstream (in_progress)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-42-x86_64:upstream (success)
🟢 rpm-build:fedora-43-x86_64:upstream (success)
🟢 rpm-build:fedora-rawhide-x86_64:upstream (success)
🟢 Analyze (target) / cppcheck (success)
🟢 Build / freebsd (success)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (success)
🟢 Build / make-distcheck (success)
🟢 ci / intgcheck (centos-10) (success)
🔴 ci / intgcheck (fedora-42) (failure)
🟢 ci / intgcheck (fedora-43) (success)
🟢 ci / intgcheck (fedora-44) (success)
🟢 ci / prepare (success)
🟡 ci / system (centos-10) (in_progress)
🟡 ci / system (fedora-42) (in_progress)
🟡 ci / system (fedora-43) (in_progress)
🟡 ci / system (fedora-44) (in_progress)
➖ Coverity scan / coverity (skipped)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@pbrezina pbrezina merged commit 4757527 into master Jan 14, 2026
13 of 17 checks passed
@pbrezina pbrezina deleted the release branch January 14, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants