Having an executor within applier package creates
potential import cycling. This patch moves it to a separate
package which can be used to conveniently store all the executors
at one place.
Change-Id: I68d909489b691e4fb7129446ef9a3fb085f8683c
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #432
Having an executor within isogen package creates
potential import cycling. This patch moves it to a separate
package which can be used to conveniently store all the executors
at one place.
Change-Id: I2154c0af6c28f47abb2448863332766260106759
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #432
Having an executor within clusterctl.client package creates
potential import cycling. This patch moves it to a separate
package which can be used to conveniently store all the executors
at one place.
Change-Id: Ib0a6072a393e68885d9ef911aa2894a0de055668
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #374
We should follow black-box pattern in unit test, so they should be
performed in a separate package. This patch fixes it for isogen
package.
Change-Id: I60a22213e943eaa85ce75979071897c42308e1ae
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #432
Currently MockDocument implementation located under pkg/bootstrap/isogen
package, which is not relevant for it and makes it impossible to import
in other test packages. Also _test.go filename is not proper one for
this source file since it doesn't contain any unit tests inside.
Change-Id: I9b3f28df653637701f5d6fca4376da5a22f39658
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #432
Currently mockContainer implementation is located under
pkg/bootstrap/isogen and it's not possible to import it from
other packages. This patch moves it to appropriate new package
under testutil.
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To : #432
Change-Id: I36510e8e065434de597e7316dfbe51da0825d343
This commit fixes the minor comment message on pkg/cluster/clustermap/map.go
file and not related to any issue.
Change-Id: Ie92a7c09ada8b0983631cbcd0cd82622c068fc5b
* Phase executors accept run options which
have progress boolean field.
`airshipctl phase run` must be able to set
this options from the command line flag.
Change-Id: I0645680ae68bf8690800aac5f3837ca78909ade0
This reverts commit 813bdd3614d13cc6bacbf0846c139d8f6da027af.
Reason for revert: This commit was done to avoid duplication of certmanager across all clusterctl components. However with the current notion of maintaining the upstream version as is, this change should be reverted. Also there is WIP to implement cert-manager as a standalone component even before clusterctl tries to deploy its embedded certmanager. So this commit will avoid manipulating the override in the standalone certmanager component
Change-Id: I16b0b002da990eb3f070f69d025cb904f7ef9e27
This PS updates ironic deployment with IPA init container
which contains python agent agent packages which are ubuntu based
packages. The docker image for this can found here [0]
[0]
https://opendev.org/airship/images/src/branch/master/ipa-downloader-image
Change-Id: Idaaafb2ddb562cf22a62df36100dd1e6c76211fc
The existing `image_repo` value was leading to invalid image tag name
errors e.g. "https://quay.io/api/v1//airshipit/airshipctl:latest". This
updates the value to match the default DOCKER_REGISTRY value in the Makefile.
Change-Id: Ifa9112c8b91475593300ff04fd069dff703762af
Closes: #437
This patch set includes the methods supporting the bootstrap
container feature, which will be used by the BootConfiguration
executor, as part of "phase run" implementation.
The methods under command.go file are responsible for creating the
bootstrap container, monitor its state, etc.
Change-Id: I3f6b1a0bb7c7be7ac29487d09ae2ccd5e9f4bd87
This commit adds documentation related to clusterAPIRef.
The PS (https://review.opendev.org/c/airship/airshipctl/+/759514)
related to this already got merged.
Change-Id: Icd372908f371b9578933d7751189af485c81070f
Signed-off-by: bijayasharma <vetbijaya@gmail.com>
Relates-To: #376
Added support for following phases in CAPG provider
clusterctl-init-ephemeral
controlplane-ephemeral
initinfra-target
clusterctl-init-target
clusterctl-move
workers-target
Added support for deploying calico CNI for CAPG
through initinfra-target phase
Relates-To: #425
Change-Id: I91b29e561444d82d33802157eba5a1d94b25ba04
Some of sites defined in document model have no metadata.yaml,
which is essential to work properly with that sites.
* az-test-site and gcp-test-site metadata.yaml added
* metadata.yaml for test-site moved to appropriate directory
for consistency
Change-Id: Ibec109e41a5e3fd36794dc6a879888cfc610c9cc
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #19
The implementation is inspired by pkg/remote module, and evolves
it to be built on top of document bundles as well as makes it
more command line oriented by adding RunOperation method.
Change-Id: If95a007986eb61c977c1ddbe5a94cfbeefeac225
Relates-To: #397
Relates-To: #362
Relates-To: #359
* Updated the clustered objects and namespaced objects to
use a custom list when defined, otherwise fallback to
the existing default list.
* This will allow other providers to trim down or change the
resource names for log collection
Change-Id: I4e19e558470186943f66b6e1f69221b85bab739d
These methods will be used instead of calling their direct
analogs in the code in order to have more consistency.
Change-Id: Ie2eaf03b1fbd049ed5788f1a769a332ef863494b
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #415
In order to widely use filesystem interface in airshipctl
there is need to move it to separate package to avoid
importing unnecessary dependencies from document
package and, as a consequence, possible cyclic dependencies.
* filesystem moved from document/fs to pkg/fs
Change-Id: I3b6298462f03db43594a9fa26bf23ab7687c5589
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #415
Azure does not currently suport Calico networking.
As a workaround, this patch set includes CAPZ Calico manifests that uses
VXLAN, instead.
The CAPZ Calico manifest are located under
manifest/function/cni/calico-capz.
Change-Id: Iadb2d5e10131e6a2df8cef49e2ec189ab948eeb9
Reject commits to the airshipctl repo that contain a
manually edited zz_generated.deepcopy.go file (i.e. one that does not
match the auto-generated version).
This will also prevent code from merging which fails to generate a
deepcopy file.
Change-Id: I4f06cd852237f7b3d8643d859775c16c3cd47530
