Skip to content

Bug 2017471: ensure nss-altfiles comes from AppStream#663

Merged
openshift-merge-robot merged 1 commit intoopenshift:release-4.8from
miabbott:bz2017471
Oct 27, 2021
Merged

Bug 2017471: ensure nss-altfiles comes from AppStream#663
openshift-merge-robot merged 1 commit intoopenshift:release-4.8from
miabbott:bz2017471

Conversation

@miabbott
Copy link
Copy Markdown
Member

In #637, it was observed that nss-altfiles could come from the RHAOS
repos, which is unnecessary since the same version exists in the RHEL8
AppStream repos. The package has since been untagged from the RHAOS
repo, but to ensure that we always get the package from AppStream we'll
pin to the specific version. (The version hasn't changed since Apr 2019)

In openshift#637, it was observed that `nss-altfiles` could come from the RHAOS
repos, which is unnecessary since the same version exists in the RHEL8
AppStream repos.  The package has since been untagged from the RHAOS
repo, but to ensure that we always get the package from AppStream we'll
pin to the specific version.  (The version hasn't changed since Apr 2019)
@openshift-ci openshift-ci Bot added bugzilla/severity-low Referenced Bugzilla bug's severity is low for the branch this PR is targeting. bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Oct 27, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 27, 2021

@miabbott: This pull request references Bugzilla bug 2017471, which is invalid:

  • expected dependent Bugzilla bug 2017470 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but it is ASSIGNED instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

Bug 2017471: ensure nss-altfiles comes from AppStream

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.

@openshift-ci openshift-ci Bot requested review from jlebon and zonggen October 27, 2021 14:53
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 27, 2021
@travier
Copy link
Copy Markdown
Member

travier commented Oct 27, 2021

Looks like we should prioritize adding lockfiles support to RHCOS.

@miabbott
Copy link
Copy Markdown
Member Author

Looks like we should prioritize adding lockfiles support to RHCOS.

Agreed! I would have liked to use the repo locking that we use in 4.9/4.10 but we didn't backport that to 4.8

@miabbott
Copy link
Copy Markdown
Member Author

/bugzilla refresh

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 27, 2021

@miabbott: An error was encountered querying GitHub for users with public email (mnguyen@redhat.com) for bug 2017471 on the Bugzilla server at https://bugzilla.redhat.com. No known errors were detected, please see the full error message for details.

Full error message. non-200 OK status code: 403 Forbidden body: "{\n \"documentation_url\": \"https://docs.github.com/en/free-pro-team@latest/rest/overview/resources-in-the-rest-api#secondary-rate-limits\",\n \"message\": \"You have exceeded a secondary rate limit. Please wait a few minutes before you try again.\"\n}\n"

Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

Details

In response to this:

/bugzilla refresh

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.

@miabbott
Copy link
Copy Markdown
Member Author

/bugzilla refresh

@openshift-ci openshift-ci Bot added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. and removed bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Oct 27, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 27, 2021

@miabbott: This pull request references Bugzilla bug 2017471, which is valid.

6 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target release (4.8.z) matches configured target release for branch (4.8.z)
  • bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
  • dependent bug Bugzilla bug 2017470 is in the state CLOSED (CURRENTRELEASE), which is one of the valid states (VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE))
  • dependent Bugzilla bug 2017470 targets the "4.9.z" release, which is one of the valid target releases: 4.9.0, 4.9.z
  • bug has dependents

Requesting review from QA contact:
/cc @mike-nguyen

Details

In response to this:

/bugzilla refresh

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.

@openshift-ci openshift-ci Bot requested a review from mike-nguyen October 27, 2021 17:08
Copy link
Copy Markdown
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Oct 27, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 27, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jlebon, miabbott

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-merge-robot openshift-merge-robot merged commit 0187f19 into openshift:release-4.8 Oct 27, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 27, 2021

@miabbott: All pull requests linked via external trackers have merged:

Bugzilla bug 2017471 has been moved to the MODIFIED state.

Details

In response to this:

Bug 2017471: ensure nss-altfiles comes from AppStream

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.

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. bugzilla/severity-low Referenced Bugzilla bug's severity is low for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants