This PS implements the following changes:
- switches freeze approach to requirements-direct.txt and
requirements-frozen.txt files
- adjusts code tabulation style according to yapf recommendations
- replaces deprecated usage of responce.body attribute with
responce.text
- fixes integration tests in controlled by Makefile + tox
- uplifts Helm to v3.9.4
Change-Id: I751db72eb8f670825382f11a36657112faeb169a
The existing max_workers setting for the ThreadPoolExecutor caps the
number of nodes that can be deployed concurrently to 16. This change
allows more threads to be run in parallel, allowing the operator to
deploy more nodes at once.
Change-Id: I6d1dfa7ad8f5e03a8328311e6b59ee116a250462
During drydock node deployments, sometimes MaaS node deployment for
some nodes fails when the node tries to pull the node bootaction
files, using drydock api.
Drydock api call fails with `500 Internal Server Error`, when
drydock tries to create the booaction files for the node. The logs,
however do no provide any additional clues on what caused drydock to
fail. This issue does not happen always, and subsequent site updates
will most of the deploy the failed nodes.
The additional checks and logs are added to help pinpoint the root cause
of the 500 return code, if/when this issue heppens again.
This ps also, uplifted `MarkupSafe` pip library from 1.0 to 1.1.1 to
address the issue with MarkupSafe and latest version of setuptools
described here: https://github.com/pallets/markupsafe/issues/116
Change-Id: I08a088d9690d8d9dd1f771dc5e84d1eb02fbd39f
Flake8 version recently updated to include new PEP8 rules. Some of
the codebase is not compliant with the new rules.
Change-Id: I0f5b3d41ee54ff0d9ffa05f733f98c7e34f0f258
Signed-off-by: Alexander Hughes <Alexander.Hughes@pm.me>
- Currently several failure paths won't log any messages
when doing a site validation. Add these messages
- Also, for validation steps that are dependent on external resources
make the resource inaccessibility a warning rather than a failure.
Change-Id: I431ed188e2f6cd3fc3fa41ae2729f3a099fdfbf5
- Instead of forcing a user to provide the full URL for the MAAS API,
instead use the endpoints pattern and render the URI via HTK templates.
- Add secret name to chart to support HTK ingress
- Install libyaml to take advantage of faster parsing by pyyaml
- Add exception logging when node compiling fails.
- Add caching of parsed design to gain efficiency
- Add TLS certificate secret for use by the ingress document
Change-Id: I5a2dbc415483c336d38d67edcebdfc5812f7bb0c
- Add unit tests for node filtering based on rack name,
node tag and node label.
- Code updates to get all unit tests passing.
Change-Id: I79e7a8b35c5635364c37e22153b26687a684bfed
- When compiling a list of packages to be installed on deploying nodes
the function call to render all bootactions was missing a parameter
introduced in [0] to secure the bootaction.
- Add unit testing for this flow
Change-Id: I9f59819d6b79290a9f72db2a5a4c3422f2c79739
Adding the baclient code to Drydock requires a refactor
of the build automation to support multiple languages
and multiple artifacts included in a single Docker image
NOTE: the go source here is a placeholder 'hello world' sample
Change-Id: I1b4883f018b33b3d4fcd7cbcb6cba660fcdc93de