build-go.sh: use SOURCE_GIT_COMMIT if available

[sosiouxme]

This will be supplied in downstream builds to relay the commit from the source git repo.

Fixes: bz 1709365

- How to verify it
Check that the resulting OCP image built binary reports a commit from this github repo.

- Description for the changelog
build-go.sh: use SOURCE_GIT_COMMIT if available

[sosiouxme]

@runcom @smarterclayton OS_GIT_COMMIT was breaking origin builds so I went ahead and changed it to SOURCE_GIT_COMMIT at least to get us unblocked. It's not that hard to change later.

[cgwalters]

Oh whee I should have read later GH notifications...

[cgwalters]

Why does #744 exist too?

/approve

[cgwalters]

/lgtm

[sosiouxme]

master vs release-4.1
4.1 is now building from release-4.1 so this one is more urgent

[openshift-bot]

/retest

Please review the full test history for this PR and help us cut down flakes.

[sosiouxme]

/retest

[cgwalters]

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, sosiouxme

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [cgwalters]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[sosiouxme]

/retest

[openshift-bot]

/retest

Please review the full test history for this PR and help us cut down flakes.

[openshift-bot]

/retest

Please review the full test history for this PR and help us cut down flakes.

[runcom]

wait, has the scaleup job been enabled by default? @vrutkovs

[vrutkovs]

@runcom not sure why this happened, openshift/release#3781 would fix that

[runcom]

@runcom not sure why this happened, openshift/release#3781 would fix that

maybe because of the branching - that's the direction we're heading towards anyway though, but not there yet so yeah, let's lift that for now

[vrutkovs]

Feel free to override if it fails

[openshift-ci-robot]

@sosiouxme: The following test failed, say /retest to rerun them all:

Test name	Commit	Details	Rerun command
ci/prow/e2e-rhel-scaleup	4bae977	link	/test e2e-rhel-scaleup

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[cgwalters]

/override e2e-rhel-scaleup

[openshift-ci-robot]

@cgwalters: cgwalters unauthorized: /override is restricted to repo administrators

In response to this:

/override e2e-rhel-scaleup

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[imcleod]

/override e2e-rhel-scaleup

[openshift-ci-robot]

@imcleod: /override requires a failed status context to operate on.
The following unknown contexts were given:

• e2e-rhel-scaleup

Only the following contexts were expected:

• ci/prow/e2e-aws
• ci/prow/e2e-aws-op
• ci/prow/e2e-aws-upgrade
• ci/prow/e2e-rhel-scaleup
• ci/prow/images
• ci/prow/unit
• ci/prow/verify
• tide

In response to this:

/override e2e-rhel-scaleup

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[imcleod]

/override ci/prow/e2e-rhel-scaleup

[openshift-ci-robot]

@imcleod: Overrode contexts on behalf of imcleod: ci/prow/e2e-rhel-scaleup

In response to this:

/override ci/prow/e2e-rhel-scaleup

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.