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

Fix rpminspect specname inspection failure #6241

Conversation

rm3l
Copy link
Member

@rm3l rm3l commented Oct 20, 2022

What type of PR is this:
/kind bug
/area release-eng

What does this PR do / why we need it:
To anticipate the sunsetting of rpmdiff (replaced by rpminspect), this
makes sure that the odo RPM package passes rpminspect's inspections.
The only error reported when testing it was about an invalid spec
filename.

specname:
---------
1) Spec filename does not exactly match the primary name odo; got
'openshift-odo.spec'

Result: BAD
Waiver Authorization: Not Waivable

Suggested Remedy: The spec file name does not match the expected
NAME.spec
format.  Rename the spec file to conform to this policy.

See
https://docs.google.com/document/d/1PolwQHoZzf0oeOUN1IK7T_NDRAay-nurwL0xYj2JuOo/edit#
for more details about this.

Note
I am also about to submit a Merge Request to the internal release repo, as I saw some occurrences of this spec file there.
EDIT: See MR 71 in the internal release repository.

Which issue(s) this PR fixes:
Fixes #6213

PR acceptance criteria:

  • Unit test

  • Integration test

  • Documentation

How to test changes / Special notes to the reviewer:

It would be ideal to test this in CI, to prevent further regressions on this or any other inspections, but this is not possible at this time.
So we need to check manually:

# Build a local RPM
./scripts/rpm-prepare.sh && ./scripts/rpm-local-build.sh

# Install rpminspect tool and a data package (like rpminspect-redhat)
# More instructions on
# https://docs.google.com/document/d/1PolwQHoZzf0oeOUN1IK7T_NDRAay-nurwL0xYj2JuOo/edit#heading=h.kr9uelhdllk

# Run rpminspect locally
rpminspect-redhat -v -T specname ./dist/rpmbuild/SRPMS/odo-3.1.0-1.fc36.src.rpm

# You might optionally submit a Scratch build to make sure everything works correctly
See https://docs.google.com/document/d/1PolwQHoZzf0oeOUN1IK7T_NDRAay-nurwL0xYj2JuOo/edit#heading=h.9ihjiiivyk7z

@rm3l rm3l requested review from kadel and anandrkskd October 20, 2022 14:43
@netlify
Copy link

netlify bot commented Oct 20, 2022

Deploy Preview for odo-docusaurus-preview canceled.

Name Link
🔨 Latest commit 95ecaa9
🔍 Latest deploy log https://app.netlify.com/sites/odo-docusaurus-preview/deploys/63621f2dc24cec0008eeaab2

@openshift-ci openshift-ci bot added kind/bug Categorizes issue or PR as related to a bug. area/release-eng Issues or PRs related to the Release Engineering labels Oct 20, 2022
@openshift-ci openshift-ci bot requested a review from feloy October 20, 2022 14:43
@odo-robot
Copy link

odo-robot bot commented Oct 20, 2022

Unit Tests on commit 7e36503 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Oct 20, 2022

Validate Tests on commit 7e36503 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Oct 20, 2022

Windows Tests (OCP) on commit 7e36503 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Oct 20, 2022

OpenShift Tests on commit 7e36503 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Oct 20, 2022

Kubernetes Tests on commit 7e36503 finished successfully.
View logs: TXT HTML

@rm3l rm3l force-pushed the 6213-investigate-impact-of-rpmdiff-sunsetting-on-31-oct-2022-on-our-release-process branch from 172f011 to 0a7ac68 Compare October 20, 2022 15:06
@dharmit
Copy link
Member

dharmit commented Nov 1, 2022

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Nov 1, 2022
@openshift-ci
Copy link

openshift-ci bot commented Nov 1, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dharmit

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:

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. Required by Prow. label Nov 1, 2022
To anticipate the sunsetting of rpmdiff (replaced by rpminspect), this
makes sure that the odo RPM package passes rpminspect's inspections.
The only error reported at this time was about an invalid spec
filename.

```
specname:
---------
1) Spec filename does not exactly match the primary name odo; got
'openshift-odo.spec'

Result: BAD
Waiver Authorization: Not Waivable

Suggested Remedy: The spec file name does not match the expected
NAME.spec
format.  Rename the spec file to conform to this policy.
```

See
https://docs.google.com/document/d/1PolwQHoZzf0oeOUN1IK7T_NDRAay-nurwL0xYj2JuOo/edit#
for more details.
@rm3l rm3l force-pushed the 6213-investigate-impact-of-rpmdiff-sunsetting-on-31-oct-2022-on-our-release-process branch from 0a7ac68 to 95ecaa9 Compare November 2, 2022 07:41
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Nov 2, 2022
@rm3l
Copy link
Member Author

rm3l commented Nov 2, 2022

Rebased and force-pushed to get fixes for the tests.

@odo-robot
Copy link

odo-robot bot commented Nov 2, 2022

NoCluster Tests on commit 7e36503 finished successfully.
View logs: TXT HTML

@rm3l rm3l closed this Nov 2, 2022
@rm3l rm3l reopened this Nov 2, 2022
@rm3l
Copy link
Member Author

rm3l commented Nov 2, 2022

level=fatal msg=failed to fetch Terraform Variables: failed to fetch dependency of "Terraform Variables": failed to generate asset "Platform Provisioning Check": baseDomain: Invalid value: "origin-ci-int-aws.dev.rhcloud.com": the zone already has record sets for the domain of the cluster: [api.ci-op-gzchqyvb-c362a.origin-ci-int-aws.dev.rhcloud.com. (A), \052.apps.ci-op-gzchqyvb-c362a.origin-ci-int-aws.dev.rhcloud.com. (A)]

/test v4.11-integration-e2e

@rm3l rm3l closed this Nov 2, 2022
@rm3l rm3l reopened this Nov 2, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 2, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@dharmit
Copy link
Member

dharmit commented Nov 2, 2022

Seeing infra error again

/test v4.11-integration-e2e
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Nov 2, 2022
@openshift-merge-robot openshift-merge-robot merged commit 8c2c633 into redhat-developer:main Nov 2, 2022
@rm3l rm3l deleted the 6213-investigate-impact-of-rpmdiff-sunsetting-on-31-oct-2022-on-our-release-process branch November 2, 2022 14:41
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. Required by Prow. area/release-eng Issues or PRs related to the Release Engineering kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. Required by Prow.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate impact of RPMDiff sunset on our release process
3 participants