Skip to content

Explicitly set -mod flag to mod in OTE build: release-4.14#293

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:release-4.14from
shiftstack:set-mod-flag-release-4.14
Jul 16, 2026
Merged

Explicitly set -mod flag to mod in OTE build: release-4.14#293
openshift-merge-bot[bot] merged 1 commit into
openshift:release-4.14from
shiftstack:set-mod-flag-release-4.14

Conversation

@danchild

@danchild danchild commented Jul 15, 2026

Copy link
Copy Markdown
Member

This is a continuation of work completed in #275 and #282.

A separate go module was added alongside openstack-test in
order to deal with ginkgo version conflicts. The separate
OTE module is purposely not vendored, because doing so
would introduce duplicate files and bloat the size of the
git objects.

rhel-9-release-golang-1.23-openshift-4.16 forces vendoring
through the GOFLAGS env var. So, to accomplish building the
OTE binary without vendoring, an explicity command line option
is required.

This is a continuation of work completed in openshift#275 and openshift#282.

A separate go module was added alongside `openstack-test` in
order to deal with ginkgo version conflicts. The separate
OTE module is purposely not vendored, because doing so
would introduce duplicate files and bloat the size of the
git objects.

`rhel-9-release-golang-1.23-openshift-4.16` forces vendoring
through the GOFLAGS env var. So, to accomplish building the
OTE binary without vendoring, an explicity command line option
is required.

Signed-off-by: Dan Childers <dchilder@redhat.com>
@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 15, 2026
@openshift-ci

openshift-ci Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@danchild danchild marked this pull request as ready for review July 15, 2026 13:28
@openshift-ci openshift-ci Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 15, 2026
@openshift-ci openshift-ci Bot requested review from gryf and luis5tb July 15, 2026 13:30
@danchild

Copy link
Copy Markdown
Member Author

/assign @dlaw4608

@danchild

Copy link
Copy Markdown
Member Author

/verified by @danchild

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jul 15, 2026
@openshift-ci-robot

Copy link
Copy Markdown

@danchild: This PR has been marked as verified by @danchild.

Details

In response to this:

/verified by @danchild

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci

openshift-ci Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

@danchild: all tests passed!

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

@dlaw4608

Copy link
Copy Markdown

/lgtm
/approve

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 15, 2026
@danchild

Copy link
Copy Markdown
Member Author

/assign @gryf

@gryf

gryf commented Jul 16, 2026

Copy link
Copy Markdown
Member

/lgtm
/approve

@openshift-ci

openshift-ci Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dlaw4608, gryf

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 16, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit e6ce37b into openshift:release-4.14 Jul 16, 2026
5 checks passed
@openshift-merge-bot openshift-merge-bot Bot deleted the set-mod-flag-release-4.14 branch July 16, 2026 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants