airship/airshipctl
Code Issues Proposed changes
1,977 Commits 3 Branches 0 Tags
5dcc769243
Commit Graph

6 Commits

Author SHA1 Message Date
Ruslan Aliev
876c6043ee Replace apt/yum ansible modules, use package instead 
apt/yum use is not convenient, package module automatically uses
the underlying OS package manager. Also, some ansible roles currently
use only apt module without yum, therefore patch fixes this bug too.

Change-Id: I5dd49d513d1a791ab51ca6ce6eb1c079542c5624
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
 2020-06-29 20:28:49 +00:00
Ruslan Aliev
76d8716968 Replace become: true notation 
Ansible documentation recommends to use yes/no as boolean value,
the last mention of possible using true/yes related to version 2.4 [1].
Starting from version 2.5 only become: yes specified in docs [2][3].
Since we have both variations, this patch aligns become statement to
use only yes value.

[1] https://docs.ansible.com/ansible/2.4/become.html
[2] https://docs.ansible.com/ansible/2.5/user_guide/become.html
[3] https://docs.ansible.com/ansible/latest/user_guide/become.html

Change-Id: I706e6e1b9d706d6bfcc38c1a3d0d7304575946ce
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
 2020-05-16 01:48:42 -05:00
Yasin, Siraj (SY495P)
c25d223c7b Add copyright for missing files 
* added license templates for go, bash & yaml in tools dir
* added a script that will add license information for all
    missing files. Type:  go, yaml, yml, sh
* skip adding license for all files within testdata
* Syntax:
   > ./tools/add_license.sh

* Skip license for manifests folder
* Added one extra line after licene for yaml files
* Added License after Hashbang for bash.
* Add an extra line after hashbang and before license
* Updated the go template to use multiline comments

New Files:
  1. tools/add_license.sh
  2. tools/license_go.txt
  3. tools/license_yaml.txt
  4. tools/license_bash.txt

Change-Id: Ia4da5b261e7cd518d446896b72c810421877472a
Realtes-To:#147
 2020-04-09 08:35:59 -05:00
Drew Walters
b8101051b1 [#62] Rename missing vars file 
A change recently changed .yml file extensions to .yaml. This change
updates an ansible vars file that was missed during the refactor in
order to restore CI functionality.

Change-Id: Id832f71d2073ad1b3a552ad67a0a1617e80ea32a
Signed-off-by: Drew Walters <andrew.walters@att.com>
 2020-02-24 13:32:30 -06:00
Alexander Hughes
d5e28362f2 [#62] Standardize yaml file extensions 
Currently we have mixed extensions of .yml and .yaml, this patch is
an effort to standardize all yaml documents as .yaml

Change-Id: I7eb815584de5856bbdac9a828ccad8518e8c1729
 2020-02-24 14:50:32 +00:00
Kostiantyn Kalynovskyi
d7df2fb571 Moving roles from zuul-airship-roles. 
Changes is needed because we can not make dependencies on PRs from
other repositories such as zuul-airship-roles, this would allow more
robust development in the stage we currently are in. When there will
be less activity on gating roles will be moved back to separate repo.

Change-Id: I85c9bdd47b5aaba90df5458b20c90ff5c912c05f
 2020-02-12 22:52:17 +00:00