Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

host flavor: fix copying for Debian - #1811

Merged
alban merged 2 commits into
rkt:masterfrom
kinvolk:alban/debian-host-flavor
Nov 30, 2015
Merged

alban merged 2 commits into
rkt:masterfrom
kinvolk:alban/debian-host-flavor

Conversation

@alban

@alban alban commented Nov 30, 2015

Copy link
Copy Markdown
Member

Fixes #1809


This makes the host flavor work on Debian for me. Tested with systemd-228-2.

/cc @onlyjob

@krnowak

krnowak commented Nov 30, 2015

Copy link
Copy Markdown
Collaborator

Looks ok, but seems it does not fix the issue, right?

@alban

alban commented Nov 30, 2015

Copy link
Copy Markdown
Member Author

@krnowak It worked for me on Debian (tested in Vagrant). And make clean also worked for me. But it was not working for @onlyjob (see #1809 (comment)). And it breaks something on Arch, @iaguis is looking at this.

iaguis and others added 2 commits November 30, 2015 17:23
This makes the host flavor work on Debian for me. Tested with
systemd-228-2.

Fixes rkt#1809
@iaguis
iaguis force-pushed the alban/debian-host-flavor branch from 5af87c1 to ec44009 Compare November 30, 2015 16:24
@iaguis

iaguis commented Nov 30, 2015

Copy link
Copy Markdown
Member

It works for both @alban and me now. LGTM.

@alban

alban commented Nov 30, 2015

Copy link
Copy Markdown
Member Author

Updated to ec44009 with fixes for Arch.

  • works for me in Debian (tested in Vagrant with systemd-228).
  • works for Iago in Arch

@onlyjob: how did you test the patch? Do you cherry pick the commits on top of the 0.12.0 release, or do you take the branch as is? In the first case, you might need to delete the old stage1 with rkt image rm sha512-xxxxxx. I just filed #1813 for that.

alban added a commit that referenced this pull request Nov 30, 2015
@alban
alban merged commit e45e880 into rkt:master Nov 30, 2015
@onlyjob

onlyjob commented Nov 30, 2015

Copy link
Copy Markdown

I applied patch on top of 0.12.0 and indeed the problem was due to already imported stage1 image that rkt did not update automatically. As advised in #1813 I've built with

--with-stage1-default-flavor=host
--with-stage1-default-name=host
--with-stage1-default-version=$(VER)

which fixed the problem. Thank you.

@krnowak

krnowak commented Dec 1, 2015

Copy link
Copy Markdown
Collaborator

@onlyjob: And that worked? That should not be possible. --with-stage1-default-flavor is conflicting with --with-stage1-default-name and --with-stage1-default-version, so the configure should bail out in this case.

@onlyjob

onlyjob commented Dec 1, 2015

Copy link
Copy Markdown

Yes, as far as I can tell it did work. Why would flavour (i.e. type of image) conflict with its name?

@krnowak

krnowak commented Dec 1, 2015

Copy link
Copy Markdown
Collaborator

@onlyjob: --with-stage1-default-flavor=host is basically a convenience shorthand of --with-stage1-default-name=coreos.com/rkt/stage1-host --with-stage1-default-version=<version from AC_INIT>. And these flags only tell rkt to search for this name and version of the stage1 image in the store. They do not affect how/which stage1 is actually built.

@onlyjob

onlyjob commented Dec 1, 2015

Copy link
Copy Markdown

I see... Thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants