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

rpmostreepayload: Substitute ${basearch} in ostreesetup ref #1186

Merged
merged 1 commit into from
Sep 18, 2017

Conversation

cgwalters
Copy link
Contributor

See: https://pagure.io/atomic-wg/issue/299

Basically for multi-arch systems like Fedora it's convenient
to be able to reuse the same kickstart across multiple architectures.

This requires a new rpmostree API: coreos/rpm-ostree#877
(So be careful about backporting this; if we do that I may try
to have it check for the API availability)

See: https://pagure.io/atomic-wg/issue/299

Basically for multi-arch systems like Fedora it's convenient
to be able to reuse the same kickstart across multiple architectures.

This requires a new rpmostree API: coreos/rpm-ostree#877
(So be careful about backporting this; if we do that I may try
 to have it check for the API availability)
@cgwalters
Copy link
Contributor Author

cgwalters commented Sep 13, 2017

@cgwalters
Copy link
Contributor Author

And I changed my local kickstart test to do:

ostreesetup --nogpg --osname=fedora-atomic --remote=fedora-atomic --url=http://192.168.122.1:8000/repomirror/repo --ref=fedora/26/${basearch}/atomic-host

@dustymabe
Copy link
Contributor

LGTM

Copy link
Contributor

@rvykydal rvykydal left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@rvykydal rvykydal changed the base branch from master to f27-devel September 14, 2017 10:02
Copy link
Member

@jkonecny12 jkonecny12 left a comment

Choose a reason for hiding this comment

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

Looks good to me too.

@jkonecny12
Copy link
Member

Jenkins, it's ok to test.

@cgwalters
Copy link
Contributor Author

http://jenkins.install.bos.redhat.com/job/anaconda-f27-mock-PR/179/ ➡️ http://jenkins.install.bos.redhat.com/job/anaconda-f27-mock-PR/179/artifact/result/tests/nosetests.log ➡️

AssertionError: <pyanaconda.kickstart.RepoData object at 0x7fe29d565e80> is not an instance of <class 'pykickstart.commands.repo.F27_RepoData'>

which looks unrelated to this?

@rvykydal
Copy link
Contributor

Yes it is unrelated, it is pykickstart and anaconda builds not being in sync.

@dustymabe
Copy link
Contributor

Yes it is unrelated, it is pykickstart and anaconda builds not being in sync.

so can we merge? or do we need to wait for that to be fixed?

@rvykydal
Copy link
Contributor

jenkins, test this please

@rvykydal rvykydal added the f27 label Sep 18, 2017
@rvykydal
Copy link
Contributor

@dustymabe
Can this go to F27 (is rpm-ostree related change there)? Or was it intended just for master.

@dustymabe
Copy link
Contributor

Can this go to F27 (is rpm-ostree related change there)? Or was it intended just for master.

Definitely would like to see this in F27 as well.

@jkonecny12 jkonecny12 merged commit 5289ee3 into rhinstaller:f27-devel Sep 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 participants