Skip to content

Conversation

@jlebon
Copy link
Member

@jlebon jlebon commented Apr 27, 2020

By default, cosa fetch ignores overrides in overrides/rpm since
those are meant to be temporary and might be e.g. locally built
packages. (Otherwise, for example, removing a cosa override might cause
cosa build to fail since the no longer overridden package was never
actually downloaded).

However, it can be useful sometimes to not ignore it if we know that
it's not a "temporary" package and that it won't be found otherwise from
the regular repos. This is the case in the CI of fedora-coreos-config
where we use overrides/rpm to test composes with overrides without
having to tag them into the pool first.

Related: coreos/fedora-coreos-config#365

Closes: #1117

By default, `cosa fetch` ignores overrides in `overrides/rpm` since
those are meant to be temporary and might be e.g. locally built
packages. (Otherwise, for example, removing a cosa override might cause
`cosa build` to fail since the no longer overridden package was never
actually downloaded).

However, it can be useful sometimes to not ignore it if we know that
it's not a "temporary" package and that it won't be found otherwise from
the regular repos. This is the case in the CI of `fedora-coreos-config`
where we use `overrides/rpm` to test composes with overrides without
having to tag them into the pool first.

Related: coreos/fedora-coreos-config#365

Closes: coreos#1117
@dustymabe
Copy link
Member

nit: --with-cosa-overrides -> --with-overrides

Or was the hope to disambiguate between rpm files in overrides/rpm/ and manifest-lock.overrides.x86_64.yaml ?

@jlebon
Copy link
Member Author

jlebon commented Apr 27, 2020

Or was the hope to disambiguate between rpm files in overrides/rpm/ and manifest-lock.overrides.x86_64.yaml ?

Right, exactly. (I actually got confused myself when reading back my own comment at #1117 (comment) because "overrides" is already an overloaded word.)

Copy link
Member

@dustymabe dustymabe 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-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

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

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 e81b867 into coreos:master Apr 27, 2020
dustymabe added a commit to dustymabe/fedora-coreos-config that referenced this pull request Apr 27, 2020
dustymabe added a commit to dustymabe/fedora-coreos-config that referenced this pull request Apr 27, 2020
dustymabe added a commit to coreos/fedora-coreos-config that referenced this pull request Apr 27, 2020
@jlebon jlebon deleted the pr/fetch-cosa-overrides branch July 6, 2020 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

corner case for cosa fetch not handling overrides/rpm/

4 participants