369 Commits

Author SHA1 Message Date
Zuul
dc8c241c31 Merge "Show the context name and current context using get-context command" 2021-05-24 21:31:52 +00:00
chumkid
02d2fc7e4a Show the context name and current context using get-context command
The command "airshipctl config get-context" will display the name of the contexts as well as the current context name.

Relates-To: #475
Change-Id: I8234d74f6b98869e27e4974a54b68230d8be8565
2021-05-19 13:42:22 +05:30
Sirisha Gopigiri
ed0fee91b1 Generate user guide command links
This PS adds docs.airshipit.org link to long description
of the airshipctl root command output. The implementation
is done in-line with kubectl command.
https://github.com/kubernetes/kubectl/blob/master/pkg/cmd/cmd.go#L479

Closes: #281
Change-Id: Id65c53a72a840221c47677bacf9651f9430da276
2021-05-14 17:35:21 +05:30
Zuul
bcb6f959ab Merge "Cluster- updating cmd files for documentation" 2021-05-11 09:38:12 +00:00
Zuul
7280d5a7bd Merge "Baremetal- updating cmd files for documentation" 2021-05-11 09:38:08 +00:00
Zuul
ab98b63bf3 Merge "Refactor config get-manifest command" 2021-05-11 09:10:36 +00:00
Sirisha Gopigiri
d13d15f453 Cluster- updating cmd files for documentation
The description and examples are updated for the airshipctl
commands, which will be inturn used for generating documentation.
Please ignore the .md file changes in this PS. They are added for zuul
gates to pass. Here is the PS with generated documention
files https://review.opendev.org/c/airship/airshipctl/+/789250

Relates-To: #280
Change-Id: I7c088528842ff859f502d4484ff9a3847ebb1177
2021-05-07 19:23:55 +05:30
Sirisha Gopigiri
ec3840a866 Baremetal- updating cmd files for documentation
The description and examples are updated for the airshipctl
commands, which will be inturn used for generating documentation.
Please ignore the .md file changes in this PS. They are added for zuul
gates to pass. Here is the PS with generated documention
files https://review.opendev.org/c/airship/airshipctl/+/789250

Relates-To: #280
Change-Id: I323fa145b512e58cb878b117656c51bc39e76360
2021-05-07 19:23:55 +05:30
Vladimir Kozhukalov
639e53f9a0 Change default manifest target path
Now the target path for the default manifest is
  $HOME/.airship/default. This is reasonable default
  location.

  The target path is a mandatory field in the
  definition of a manifest. If a user wants
  to use her current working directory as a location
  the manifests, then it must be properly configured
  in the $HOME/.airship/config.

Relates-To: #416
Relates-To: #417
Change-Id: I96601803939df5c7369d1580842dfabeb19aa017
2021-05-03 12:08:52 +00:00
Niharika Bhavaraju
5085f22c56 [#394] plan list -o yaml changes
Added yaml output flag to show plan list in yaml format

Relates-To: #394
Co-authored By: Niharika Bhavaraju<niha.twinkle@gmail.com>
Change-Id: I8bed077573eb79783cfc77e4ebf82c819f41d125
2021-04-29 18:10:57 +05:30
Zuul
7aa3b92397 Merge "Fix airshipctl config get-context maximum args" 2021-04-29 04:08:27 +00:00
bijayasharma
e9179ff018 Fix typos in various files
Signed-off-by: bijayasharma <vetbijaya@gmail.com>
Change-Id: Ibe0066e74497c3caecc50e1187abd0bde7955b04
2021-04-21 17:32:31 +00:00
Sirisha Gopigiri
f70e93762f Fix airshipctl config get-context maximum args
Limit the number of arguments that can be passed to
get-context command to 1 for now

Closes: #526
Change-Id: Ib140fc836e01c284c05719dfb7e3f29d0b191538
2021-04-16 19:03:54 +05:30
Ruslan Aliev
864184f053 Refactor config get-manifest command
This commit addresses the bug that user is unable to see retrieved
manifest name or names, also simplifies the internal logic and
reorganizes unit tests.

Change-Id: I89bac0af811f744c209be2aee921033a5e8a4fca
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Closes: #505
2021-04-03 22:27:43 -05:00
Zuul
f1cab5b579 Merge "Change logic of config set-management-config command" 2021-03-31 12:02:34 +00:00
Zuul
6a62e11fea Merge "[#323] Cluster list command to output in table/clusternames." 2021-03-31 03:00:29 +00:00
Zuul
bba25054b0 Merge "Change logic of config set-context cmd" 2021-03-31 02:11:41 +00:00
Zuul
5737f3bf26 Merge "Remove NameInKubeconf field from Context struct" 2021-03-31 02:11:36 +00:00
Zuul
ba5e56d3c9 Merge "Introduce plan validate command" 2021-03-31 02:02:55 +00:00
Ruslan Aliev
1684ad45cf Change logic of config set-management-config command
* command internal logic was revised and optimized
 * added ability to delete/reset management config fields
 * unit tests were reorganized, new cases were added

Change-Id: I911df180e6986cb1f4dab6aea01216c8a4e15353
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #348
2021-03-30 12:16:13 -05:00
Niharika Bhavaraju
1f90036413 [#323] Cluster list command to output in table/clusternames.
* airshipctl cluster list
* airshipctl cluster list -o table
* airshipctl cluster list -o name(lists cluster names)

Relates-To: #323
Change-Id: Ic4609b8e05984c14acf8f20053123423d5e0088a
2021-03-30 17:03:13 +05:30
Zuul
310bc282f5 Merge "[#358] Introduce Phase List command to output phase list(cmd module)" 2021-03-30 01:02:39 +00:00
Ruslan Aliev
6076891f31 Introduce plan validate command
This patch implements CLI command to invoke plan validation.
Appropriate functionality was already added to pkg previously.

Change-Id: I038b49c8de0af65f38b62950b27b31d95a6301a9
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #19
Relates-To: #497
Closes: #497
2021-03-27 12:59:30 -05:00
Ruslan Aliev
3dd66f7685 Change logic of config set-context cmd
* command internal logic was revised and optimized
 * added ability to delete/reset context fields
 * unit tests were reorganized, new cases were added

Change-Id: Ie35d11405e88fea21abf33cb75f44b03bb4644fd
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
Relates-To: #348
2021-03-26 19:22:52 -05:00
Ruslan Aliev
b8e0978d6d Remove NameInKubeconf field from Context struct
Since this field is no longer actually used, there is need to
remove it completely.

Change-Id: I9f7002f8def31d1660919bd0128707e43a57ad71
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
2021-03-26 19:22:52 -05:00
Zuul
df0e4cec2d Merge "Modify airshipctl cluster status cmd" 2021-03-26 09:28:12 +00:00
Niharika Bhavaraju
0cc7c9cafd [#358] Introduce Phase List command to output phase list(cmd module)
* Phase list command lists phases of current documentset/plan.
 airshipctl phase list
 airshipctl phase list --plan planName
 airshipctl phase list --plan planName -o table(default output)
 airshipctl phase list --plan planName -o yaml

Co-Authored-By: Niharika Bhavaraju <niha.twinkle@gmail.com>

Relates-To: #358

Change-Id: Iab3a6cdf5a25583ba3d01341da350841c9cc362d
2021-03-25 16:11:10 +00:00
bijayasharma
60488d4d7a Add phase status command
Signed-off-by: bijayasharma <vetbijaya@gmail.com>
Change-Id: I09fc1e84f195982a43c97f80feb82fb2e489771d
Relates-To: #411
2021-03-25 12:36:01 +00:00
bijayasharma
ddf1641168 Modify airshipctl cluster status cmd
* This commits modifies airshipctl cluster status cmd to
  return not implemented error

Change-Id: I7a9fea382217f4eb3d9903b6575f9df5d331255d
Signed-off-by: bijayasharma <vetbijaya@gmail.com>
Relates-To: #370
Closes: #370
2021-03-25 12:35:15 +00:00
Ruslan Aliev
3ccafd9d0f Remove deprecated options for phase/plan run cmd
There are some debries left after removing --kubeconfig flag
from support, this patch cleans them up.

Change-Id: Iabdd6bcbc59d169bf162c9cddb3cb264fb88a3f7
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
2021-03-24 13:39:13 -05:00
Zuul
853b16997a Merge "Remove defaults from Config object at runtime" 2021-03-22 16:40:09 +00:00
Matthew Fuller
585af5a516 Remove defaults from Config object at runtime
This change fixes a bug where the default values that are
populated during an `airshipctl config init` were getting
mixed in with user specified values at runtime. For example,
a user with two contexts defined in their airship config
file would actually see a third phantom context with default
values in the output of `airshipctl config get-context`.

Also fixes a number of the config cmd and pkg unit tests that
were affected by this bug.

Change-Id: Iac591d4c2a616090028eb730576478edefb82544
Closes: #447
2021-03-19 17:31:02 +00:00
Zuul
86bd2c9984 Merge "Get-kubeconfig return all cluster contexts" 2021-03-19 01:20:48 +00:00
Kostiantyn Kalynovskyi
f24bf00d17 Get-kubeconfig return all cluster contexts
Now if we run `airshipctl cluster get-kubeconfig` without args
it will return kubeconfig for the entire site, which will have
contexts for every cluster defined in cluster map.

Relates-To: #460
Closes: #460

Change-Id: Icf1f09724a5c60ac520b1dbcbda69c3280ac4c7e
2021-03-17 19:27:14 +00:00
Zuul
7277a8492a Merge "Remove clusterctl command related functionality" 2021-03-17 17:48:49 +00:00
Zuul
98c50e1464 Merge "Revert "Adding commands to get and set encryption configs"" 2021-03-17 00:22:09 +00:00
Zuul
2d9de5f8b6 Merge "Revert "Adding encrypt and decrypt subcommands to secret command"" 2021-03-16 17:51:26 +00:00
Kostiantyn Kalynovskyi
6cae709dba Remove clusterctl command related functionality
It is not used anywhere anymore as commands themselves were
deprecated and removed in favor of clusterctl move and init
executors.

Change-Id: I526f75d95b70754034f0846edfc9d3e468648ced
2021-03-16 15:05:14 +00:00
Zuul
442f9965fb Merge "Remove deprecated cluster init/move commands" 2021-03-16 04:48:32 +00:00
Kostiantyn Kalynovskyi
6207e2c24d Add multiple kubeconifg sources to ClusterMap
Change ClusterMap API object to support multiple kubeconfig sources
for a cluster. If one kubeconfig source fails, kubeconfig builder
will not fail and move on to the next one. This behaviour will allow
to support cases when ephemeral cluster is not accesible anymore or
when target cluster is not yet accessible.

For more information please read issue #460 in airshipctl github

Relates-To: #460
Related-To: #460

Change-Id: I7cd32f78cd7c4ad8814eac357424c24216f40d76
2021-03-10 18:22:30 +00:00
Ruslan Aliev
159cddb269 Revert "Adding encrypt and decrypt subcommands to secret command"
This reverts commit 8fccf09753663fd83e07c01de38d3ca67d9a08d1.

Reason for revert: This change is no longer needed and never used, specified commands aren't implemented. All the encryption logic was implemented in this [1] PS chain.

[1] https://review.opendev.org/q/topic:%22generator_sops_encrypter%22+(status:open%20OR%20status:merged)
 

Change-Id: I94737137707cea63c05573147ba3bde808bf6f00
2021-03-10 11:33:49 -06:00
Ruslan Aliev
e4436ca36d Revert "Adding commands to get and set encryption configs"
This reverts commit 694067492cfedd5e7b369b396bfce9923c0ca3a1.

Reason for revert: Encryption configs inside airship config are
no longer required. Encrypt feature was implemented different way.

Change-Id: I1c8feec75000402e314e815e4832ce740f0e1254
2021-03-10 11:30:32 -06:00
Zuul
2351051ffd Merge "Remove image build command" 2021-03-10 08:23:00 +00:00
Scott Williams
fb997d49e1 Error messages checked expected value
Modified the test framework so that error messages are actualy checked
for an expected value (instead of previously only checking if there was
an error returned)

Closes-Bug: #451
Change-Id: I9c4b172a705a7648467afe41346c389c503e764b
2021-03-03 12:00:36 -06:00
Zuul
12d4b63a3d Merge "Phase Validation: cmd changes to support Phase Validation sub command" 2021-03-03 16:52:23 +00:00
Zuul
046e8fb05a Merge "Don't allow to run baremetal actions without flags" 2021-03-03 05:43:29 +00:00
muly
b0eabe544b Phase Validation: cmd changes to support Phase Validation sub command
Relates-To: #330

Change-Id: I8e45696723823d568e2f2eac8de65b38d30c09c3
2021-03-02 13:19:45 +00:00
Vladimir Kozhukalov
ab5ff1076f Remove image build command
We use phases for building the ephemeral ISO image, so
 a user can use `phase run` command instead. It is better
 to remove `image build` subcommand to avoid support of
 an unnecessary piece of code.

Change-Id: Ife9b5fdebc3cdfd5dac3ecccc8fef55811741ce8
Closes: #473
2021-03-02 13:08:15 +03:00
Ruslan Aliev
68e76a9059 Extend cluster get-kubeconfig cmd functionality
This PS will allow us to retrieve kubeconfig from different
kind of sources.

Change-Id: I31381cf466c58373efda40d06587e34bef411c68
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
2021-03-02 00:22:45 -06:00
Kostiantyn Kalynovskyi
7dfb0eb98c Don't allow to run baremetal actions without flags
With this change we will be sure that no commands against bmh
hosts will be applied without selector flags specified, now if you
want to target all hosts in inventory you must specify --all flag

Relates-To: #364
Related-To: #364

Closes: #364
Change-Id: I0d615ea226528162e07acc96fa61b26e031145aa
2021-02-24 20:01:15 +00:00