Skip to content

scripts: authenticate git push for release#8352

Merged
pbrezina merged 1 commit into
masterfrom
release
Jan 14, 2026
Merged

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

Conversation

@pbrezina

Copy link
Copy Markdown
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
Copy Markdown
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.

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 <[email protected]>
@sssd-bot

Copy link
Copy Markdown
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