Skip to content
This repository was archived by the owner on Jan 16, 2024. It is now read-only.

Conversation

@dkelleher
Copy link
Contributor

Adding arch support for one-context scripts. Let me know what you think. I edited the init.d script and made it into a simple executable so that I could use a systemd unit to call it. I edited the 00-network file to work with systemd-networkd. I also moved the init.d script since arch doesn't use it.

Dan Kelleher added 3 commits August 18, 2015 09:44
Addded Arch linux support for one-context.  Moved the
init.d directory to distro specific directories in
order at add systemd compatability.  00-Network was
also reworked to use systemd-networkd.
consider the Unit to be active if the start exits successfully.
@jfontan
Copy link
Contributor

jfontan commented Aug 19, 2015

I like the changes and the idea of moving startup script from SysV script to another file. If the functionality is the same maybe we can put this file in /usr/sbin/vmcontext and call it from systemd and SysV scripts.

Also it would be good to add a PKGBUILD file and some documentation on how to install it. Do you think it is easy to add this to AUR?

Thanks!

@dkelleher
Copy link
Contributor Author

The functionality should be the same and I'll move it to sbin. I'm working on making a PKGBUILD which should be very simple and I did intend to add it to the AUR. I want to work on the systemd service a bit, it appears to be running to early in the startup sometimes so I have to figure out what other targets I need to add so that it's more robust.

Dan Kelleher added 4 commits August 19, 2015 13:01
One-context should really have been in sbin all along.  Updated
one-context.service to run before systemd-networkd but after udev
or kdbus have run.  This should eliminate the requirement for a
systemd-networkd restart.
also updated 00-network and 05-hostname

One-context.service should be linked to systemd-networkd.service,
tying to do this using the Wants flag.  Additionally since
one-context.service should now run before systemd-networkd.service
it's superfluous to restart the unit, this was edited out of
00-network.  05-hostname was edited to use the hostnameclt, the
echo style can probably be removed.
@dkelleher
Copy link
Contributor Author

Added a bunch more. I'm going to write some systemd scripts for the RPM and Deb packages. Also some testing with libcloud next week.

Dan Kelleher added 5 commits August 27, 2015 09:59
I was unable to make systemd scripts for Debian 7 and CentOS 6
so for backwards compatability made the SysV script call
/usr/sbin/one-contextd.  This should be revisited in the future.
One-contextd was missing a call to the start function.
The init.d scripts were calling one-context not
one-contextd
@dkelleher
Copy link
Contributor Author

@jfontan This should be all set for your review. I intend to push the PKGBUILD info to the AUR once I can point it at the OpenNebula github instead of FASRC

@dkelleher dkelleher changed the title [WIP] Arch one-context support Arch one-context support Sep 2, 2015
@jfontan
Copy link
Contributor

jfontan commented Sep 3, 2015

Great, thank you! I'll ping you as soon as I have reviewed the changes.

@jfontan jfontan added this to the v4.14.2 milestone Sep 3, 2015
@jfontan
Copy link
Contributor

jfontan commented Sep 3, 2015

Why is package cloud-utils-bzr required? It is giving me errors with keys and adds a lot of dependencies.

If only growpart is used I got it installed with package mkinitcpio-growrootfs.

@dkelleher
Copy link
Contributor Author

Great find I'll update that I had only found growpart in cloud-utils

@jfontan
Copy link
Contributor

jfontan commented Sep 3, 2015

Tested both rpm and deb packages. Everything seems to be working ok. Merging to master.

This will be released as 4.14.2. Thank you!

@jfontan jfontan closed this Sep 3, 2015
@jfontan jfontan reopened this Sep 3, 2015
jfontan added a commit that referenced this pull request Sep 3, 2015
gh-14: Arch one-context support
@jfontan jfontan merged commit 0116273 into OpenNebula:master Sep 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants