Having python files with exec bit and shebang defined in
/usr/lib/python-*/site-package/ is not fine in a RPM package.
Instead of carrying a patch in nodepool RPM packaging better
to fix this directly upstream.
Change-Id: I5a01e21243f175d28c67376941149e357cdacd26
Now that we publish dev releases to pypi, we should also allow those
versions to be displayed with --version flag.
Change-Id: I045c9d5382a1035cd7678f9882e32d371f108555
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This duplicates the logic in zuul, and makes us consistent with
current nodepool documentation that says we already support this.
Change-Id: Ib92272b302a5225726a830ee50571fb7ad96e457
This patch makes the nodepool process avoid starting up as a daemon in
the Docker images, as it's not meant to become a background process
within a container. In order to have consistent logging like in the
daemonized mode we need to add a new foreground option that runs in
foreground but without debug logging.
Change-Id: I77e9e6e4f94cf726336419a2b22916cc1e974e62
Co-Authored-By: Tobias Henkel <tobias.henkel@bmw.de>
Argparse can output the default values of the runtime
arguments. Enable it so we have them documented.
Change-Id: I4fd7f21546615bd249485707521d8222fab10962
After daemonization, relative paths may be different, so resolve
all of the command line pathname arguments before then.
Change-Id: I2092c3f8e4595d8f874c769f41dacdf47554d681
Add a default logging configuration based on what was recently
added to Zuul. This will use sane defaults for system logging,
though it does not yet address image build logging.
Change-Id: Ie86fc6d6839e2eb199c27515346b57b2ebede703
When execute command "nodepool" without any argument,
an internal error TypeError and traceback info
will be raised out now. Output help message to instead.
Change-Id: If96761fd2342c76a2c69a5bb16bd526279502ac0
The newer dictConfig logging configuration allows a much more flexible
configuration and should be used in preference to the file config. For
backwards compatibility if the logging configuration has a yaml
extension then load it, otherwise fall back to the file config.
Change-Id: Ia29e294144fa057ded7b72e4475265766f855494
There is a lot of commonality between the nodepool apps that is
replicated between each of the files. This will mean adding more again
when new apps are added.
Separate a base NodepoolApp and NodepoolDaemonApp that handle the common
parts of setting up a nodepool app and use them in the existing apps.
First Proposed as: I098b0a1d749e9dc400d45355f9295ca07058768b
Then reverted: I7d8f1cd866b768326157f4503c1f729ebb703c0c
Change-Id: I81e80af8e2c18e56db8b6dca59ceb279dc9d591c
Logging is currently broken due to this change. Nothing is logged.
This reverts commit 03c41ccc35.
Change-Id: I7d8f1cd866b768326157f4503c1f729ebb703c0c
There is a lot of commonality between the nodepool apps that is
replicated between each of the files. This will mean adding more again
when new apps are added.
Separate a base NodepoolApp and NodepoolDaemonApp that handle the common
parts of setting up a nodepool app and use them in the existing apps.
Change-Id: I098b0a1d749e9dc400d45355f9295ca07058768b
This works well for zuul, so it should work for nodepool.
Change-Id: I536694494d58c7cb4bb9dd1b10d24ddbdaf127ae
Signed-off-by: Paul Belanger <pabelanger@redhat.com>