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

centos-release-atomic patch #239

Closed
jasonbrooks opened this issue Jan 25, 2017 · 5 comments
Closed

centos-release-atomic patch #239

jasonbrooks opened this issue Jan 25, 2017 · 5 comments

Comments

@jasonbrooks
Copy link
Collaborator

The current, downstream-7.20170117, release of centos atomic is subject to this issue with rpm-ostree package layering: https://bugzilla.redhat.com/show_bug.cgi?id=1399770

The upcoming downstream release includes rpm-ostree-client-2016.13-1.atomic.el7.x86_64, and doesn't have this issue, but @cgwalters has suggested that we adopt a patch similar to https://bugzilla.redhat.com/show_bug.cgi?id=1399770#c2 for centos-release-atomic.

Colin's patch puts os-release into /usr/lib/ and symlinks that to /etc/. One potential issue here is that centos-release-atomic depends on centos-release, and that's where our os-release comes from. https://git.centos.org/blob/rpms!centos-release-atomic/b132798c53c3ebe5544790053b59af19529307c1/SPECS!centos-release-atomic.spec

@kbsingh Should we adopt a release pkg approach similar to rhel, I don't recall all the reasons for doing this the way we have?

@cgwalters
Copy link
Member

It's likely simpler to backport the fix: rpm-software-management/libdnf#215

Or bump to the latest rpm-ostree.

@jlebon
Copy link
Contributor

jlebon commented Feb 13, 2017

Any news on this? A lot of people are hitting this issue.
@jasonbrooks How hard would it be to backport the libdnf patch?

@jasonbrooks
Copy link
Collaborator Author

Just waiting on @kbsingh to rsync the updated tree into place.

@jasonbrooks
Copy link
Collaborator Author

It looks like the new tree is in place now:

centos-atomic-host:centos-atomic-host/7/x86_64/standard
       Version: 7.20170209 (2017-02-10 00:54:47)
        Commit: d433342b09673c9c4d75ff6eef50a447e73a7541491e5197e1dde14147b164b8

I'll send out an announcement today

@cgwalters
Copy link
Member

Ok, let's mark as done. Thanks for all your work on this @jasonbrooks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants