-
Notifications
You must be signed in to change notification settings - Fork 41
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
Teal #119
Conversation
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Argh local iso works, boots, install fine, but the packer builds on qemu when it runs has no network.. @Itxaka does that ring any bell?! |
What packer version? I see 1.7.4 in https://github.com/rancher-sandbox/os2/blob/master/ros-image-build#L32 but I only tested properly up to 1.7.3...weird that it worked before... |
Does this works locally? I mean, its only failing on CI? currently cloning.... |
weird, also fails locally with packer 1.7.3 and 1.7.4 ...... Also if you vnc into the machine and manually add the address ( I would have a look at the slirp package which provides the user networking for qemu ( |
yeap indeed it's weird, locally running with qemu or vbox just works, it is only with packer/qemu (networking setting I guess?) that it fails.. it reminded me your issue when it was not getting the IP (as it looks likes the same here) gotta check, for sure it has to do with the base image, as master it's green and tests passed just today.. so the image somehow isn't playing well with packer settings |
:O |
libslirp for 15.4 received a dhcp fix patch a few days ago: https://build.opensuse.org/package/show/openSUSE:Leap:15.4/libslirp The same lib doesnt have any patches on 15.3 :/ I found out that if you downgrade libslirp to
|
Im talking about |
Thanks for the inputs! blargh. that's so... weird that doesn't fail with an openSUSE based image.. it's all pinned to leap 15.3 on master.. maybe that's the reason, old packages.. |
yep yep ISO boots fine, everything seems to work.. it's just packer which is failing on setting up network properly |
very weird becuase I can see it installing the proper version of libslirp on the ci build on master: https://github.com/rancher-sandbox/os2/runs/6405828614?check_suite_focus=true#step:4:2854 So on this patch the base image has changed somehow. In my tests the base image which run packer was based of opensuse 15.4, while on master seems to be 15.3? Is this because now we use the elemental image as TOOLS and that may be based on opensuse 15.4? EDIT: yeah, elemental image is based of 15.4 so it may well as be that libslirp is broken on 15.4 https://github.com/rancher-sandbox/elemental/blob/main/Dockerfile#L3 |
yup most likely so, I'm using the elemental image here as tools to keep it tight and simply bump it if we ever need to change tooling (at the end, this makes sense just for CI runs and local development, so..). If we had a latest tag I'd even have attached to it 😅 |
mmm packer still time outs here, but I've bumped it.. maybe I was too optimistic... |
yep, even with a fixed libslirp, the qemu plugin on 1.8.0 seems to be broken for user networking. Also I just noticed..the |
I'll just stick with leap 15.3 now, at the end this could be done in so many ways... 🤷♂️ |
Basing on leap 15.3 seems to work, thanks @Itxaka ! |
That should be raised via bugzilla.opensuse.org |
Looks good here, good for review @mjura @Itxaka @davidcassany |
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
SLE Micro for Rancher has selinux supports, but we don't have profiles for it yet. Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Shouldn't it be enough to consume the latest luet packages in the Dockerfile instead? Is all this to avoid luet install in the Dockerfile so it can be built in OBS? |
There was a problem hiding this 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 even I have doubts the framework data exposed this as is here is sustainable.
I'd like to understand under which constraints this static bunch of files is required and discuss possible alternatives. In any case I believe this is disconnected form the teal flavor, so to start moving on with teal this is good.
yes exactly, note this is not strictly required here, we could have used OBS services or other means like rancher/elemental-toolkit#1305 . I actually wanted to do a separate repo with the static files as I'm a fan of git submodules, but I do understand it might be a stopper so I preferred to vendor them here and treat them as a static asset to keep it KISS for now. I wanted to make this repository usable in a way that https://github.com/rancher-sandbox/rancher-node-image can be replaced directly with this repository, avoid maintaining both. Note there is a mechanism to sync back to cOS so there is no need to do anything on our side besides merging PRs afterwards Also... for local iteration we could have a switch that fetches the assets from cOS in the Dockerfile, but I'd prefer to do that in a follow up |
See #125 for an example of sync-up PR |
Signed-off-by: David Cassany <dcassany@suse.com>
selinux
as SLE Micro for Rancher has supports for it, but as we don't have profiles for it, fails bootingIt is better to browse it directly: https://github.com/rancher-sandbox/os2/tree/teal as most of things got simplified and dropped
Draft as gotta test this locally still and trying to wire up the CI
Supersedes #115
Part of #94