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

github: Build from the pushed commit #25

Merged
merged 1 commit into from Jul 19, 2022

Conversation

didib
Copy link
Member

@didib didib commented Jul 7, 2022

Do not merge/rebase, so that the git hash in the generated RPM's names
matches the hash of the commit we build from. By default, the checkout
action merges with HEAD, generating an ad-hoc hash that's discarded
after the build and can't be found anywhere, making it very hard to find
out which commit was used to build a particular RPM.

See also:

https://github.com/marketplace/actions/checkout?version=v2.4.2#checkout-pull-request-head-commit-instead-of-merge-commit

https://lists.ovirt.org/archives/list/devel@ovirt.org/thread/7SEFKOASOATTMO2NK2SBWMOV4CV6LZOS/

Change-Id: Idf4dad88c028f1664b55314a34a9f798c9b1a23d
Signed-off-by: Yedidyah Bar David didi@redhat.com

Do not merge/rebase, so that the git hash in the generated RPM's names
matches the hash of the commit we build from. By default, the checkout
action merges with HEAD, generating an ad-hoc hash that's discarded
after the build and can't be found anywhere, making it very hard to find
out which commit was used to build a particular RPM.

See also:

https://github.com/marketplace/actions/checkout?version=v2.4.2#checkout-pull-request-head-commit-instead-of-merge-commit

https://lists.ovirt.org/archives/list/devel@ovirt.org/thread/7SEFKOASOATTMO2NK2SBWMOV4CV6LZOS/

Change-Id: Idf4dad88c028f1664b55314a34a9f798c9b1a23d
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
@didib
Copy link
Member Author

didib commented Jul 7, 2022

This is an alternative to #23 and #24, and is my own preference.

Copy link
Member

@sandrobonazzola sandrobonazzola left a comment

Choose a reason for hiding this comment

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

LGTM

@didib didib merged commit 6dcc157 into oVirt:master Jul 19, 2022
@didib didib deleted the github-do-not-merge-on-check-patch branch July 19, 2022 09:38
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.

None yet

2 participants