Commit Graph

8 Commits

Author SHA1 Message Date
Ian Wienand e7822cf89a nodepool-zuul-functional: remove
As described in the dependent change, the testing done here is better
done by the quickstart jobs these days.  The dependent change has
removed the tox environment this calls in Zuul.  This removes the job
definiton and related files from nodepool.

Change-Id: I17e1002012e9ac6abc434454af989f1da1c379b7
Depends-On: https://review.opendev.org/c/zuul/zuul/+/826772
2022-01-28 08:20:08 +11:00
Tobias Henkel d642e14cc6
Upgrade pip before installing zuul
The new cryptography release needs either a recent pip to be able to
use abi3 wheels or a rust compiler on the system to make the source
installation work. Thus upgrade pip to use the wheels.

Change-Id: I4024a89e54de8bbb4e90cbe152a1504e31515fa8
Depends-On: https://review.opendev.org/c/zuul/zuul-jobs/+/775511
2021-02-14 09:21:02 +01:00
Ian Wienand 7c6b862b5e nodepool-zuul-functional: switch to editable install
For some reason, something changed and fake-image-create no longer
gets installed with execute permissions.  Something to do with the
wheel generation and installation ...

Anyway, switch to an editable install which should run directly out of
the checkout.  The code already uses relative paths to find
fake-image-create, this should work without further modifications.

Change-Id: Iefbd40ef8f2a2dee84d232de355e369788f0ab2d
2020-03-25 09:50:17 +11:00
Ian Wienand 9367cf8ed8 Add a dib-cmd option for diskimages
This change allows you to specify a dib-cmd parameter for disk images,
which overrides the default call to "disk-image-create".  This allows
you to essentially decide the disk-image-create binary to be called
for each disk image configured.

It is inspired by a couple of things:

The "--fake" argument to nodepool-builder has always been a bit of a
wart; a case of testing-only functionality leaking across into the
production code.  It would be clearer if the tests used exposed
methods to configure themselves to use the fake builder.

Because disk-image-create is called from the $PATH, it makes it more
difficult to use nodepool from a virtualenv.  You can not just run
"nodepool-builder"; you have to ". activate" the virtualenv before
running the daemon so that the path is set to find the virtualenv
disk-image-create.

In addressing activation issues by automatically choosing the
in-virtualenv binary in Ie0e24fa67b948a294aa46f8164b077c8670b4025, it
was pointed out that others are already using wrappers in various ways
where preferring the co-installed virtualenv version would break.

With this, such users can ensure they call the "disk-image-create"
binary they want.  We can then make a change to prefer the
co-installed version without fear of breaking.

In theory, there's no reason why a totally separate
"/custom/venv/bin/disk-image-create" would not be valid if you
required a customised dib for some reason for just one image.  This is
not currently possible, even modulo PATH hacks, etc., all images will
use the same binary to build.  It is for this flexibility I think this
is best at the diskimage level, rather than as, say a global setting
for the whole builder instance.

Thus add a dib-cmd option for diskimages.  In the testing case, this
points to the fake-image-create script, and the --fake command-line
option and related bits are removed.

It should have no backwards compatibility effects; documentation and a
release note is added.

Change-Id: I6677e11823df72f8c69973c83039a987b67eb2af
2019-08-22 10:09:00 +10:00
OpenDev Sysadmins f8ac79661a OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:25:31 +00:00
Tristan Cacqueray b620bc7810 Remove nodepool-k8s-functional and install-nodepool roles
Those roles aren't actually used.

Change-Id: I778df2318d43cd3c8b7ed78ba7a1a9808d610fca
2018-11-14 10:13:02 +00:00
Tristan Cacqueray 4295ff6870 Implement a Kubernetes driver
This changes implements a Kubernetes resource provider.
The driver supports namespace request and pod request to enable both
containers as machine and native containers workflow.

Depends-On: https://review.openstack.org/605823
Change-Id: I81b5dc5abe92b71cc98b0d71c8a2863cddff6027
2018-10-25 10:24:45 +00:00
Andreas Jaeger 1d1c88a89e Refactor playbooks/nodepool-zuul-functional/pre.yaml
Instead of using tasks and roles together, create a new role and use it.
This makes ordering easier to understand.

This is followup for
https://review.openstack.org/#/c/545158/14/playbooks/nodepool-zuul-functional/pre.yaml

Change-Id: I5242e9333dd8761272731d1e5e8af2fd5ca5ee8f
2018-02-20 19:56:11 +01:00