Bug 1872251 - [aws-ebs-csi-driver] Verify job in CI doesn't check for vendor dir sanity
Summary: [aws-ebs-csi-driver] Verify job in CI doesn't check for vendor dir sanity
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Storage
Version: 4.6
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: 4.7.0
Assignee: Christian Huffman
QA Contact: Qin Ping
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-08-25 09:38 UTC by Fabio Bertinatto
Modified: 2021-02-24 15:17 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-02-24 15:16:22 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift aws-ebs-csi-driver pull 166 0 None closed Bug 1872251: Included vendor verification 2021-01-05 20:25:35 UTC
Github openshift aws-ebs-csi-driver pull 172 0 None closed Bug 1872251: Upstream: 555: Vendor deps 2021-01-05 20:25:38 UTC
Github openshift release pull 11615 0 None closed Include test for verifying dependencies. 2021-01-05 20:25:38 UTC
Red Hat Product Errata RHSA-2020:5633 0 None None None 2021-02-24 15:17:05 UTC

Description Fabio Bertinatto 2020-08-25 09:38:42 UTC
Description of problem:

Verify job doesn't check if vendor directory was updated correctly. This is a problem because upstream doesn't have a vendor directory, and usually after every rebase the go.mod file changes. This breaks our CI and we don't notice it.

How reproducible:

Always.

Steps to Reproduce:
1. Update go.mod without running "go mod vendor"
2. Submit the changes to openshift/aws-ebs-csi-driver
2. Verify job will pass

Actual results:
Verify job in CI will pass.

Expected results:
Verify job detects there are inconsistencies in vendor dir and fail.

Comment 2 Christian Huffman 2020-09-03 18:53:26 UTC
The following PRs were submitted:

- https://github.com/openshift/aws-ebs-csi-driver/pull/166 - introduces the changes in the aws-ebs-csi-driver repo.
- https://github.com/openshift/release/pull/11615 - introduces the change in the release repo.

Comment 18 errata-xmlrpc 2021-02-24 15:16:22 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Container Platform 4.7.0 security, bug fix, and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:5633


Note You need to log in before you can comment on or make changes to this bug.