-
Notifications
You must be signed in to change notification settings - Fork 194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Install Enterprise via repo, updated Vault to v1.18.0 #345
base: master
Are you sure you want to change the base?
Changes from all commits
f4b1f1e
5711e19
d3aa939
3dd12eb
54bd2f8
52f5a0e
e8414cc
452e6dc
d61f2a1
4e38fc0
e0e8cf5
86f4e96
ad437c5
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,3 +9,6 @@ examples/hosts | |
files/vault | ||
files/vault*_SHA256SUMS | ||
meta/.galaxy_install_info | ||
.envrc | ||
.venv | ||
.direnv |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. unrelated |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. unrelated |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
{ | ||
description = "A Nix-flake-based ansible development environment"; | ||
|
||
inputs.nixpkgs.url = "https://flakehub.com/f/NixOS/nixpkgs/0.2405.*.tar.gz"; | ||
|
||
outputs = { self, nixpkgs }: | ||
let | ||
supportedSystems = [ "x86_64-linux" "aarch64-linux" "x86_64-darwin" "aarch64-darwin" ]; | ||
forEachSupportedSystem = f: nixpkgs.lib.genAttrs supportedSystems (system: f { | ||
pkgs = import nixpkgs { inherit system; }; | ||
}); | ||
in | ||
{ | ||
devShells = forEachSupportedSystem ({ pkgs }: { | ||
default = pkgs.mkShell { | ||
venvDir = ".venv"; | ||
packages = with pkgs; [ | ||
ansible | ||
ansible-lint | ||
python312 | ||
] ++ | ||
(with pkgs.python312Packages; [ | ||
pip | ||
venvShellHook | ||
]); | ||
}; | ||
}); | ||
}; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -65,18 +65,24 @@ | |
name: "{{ vault_rhsm_repo_id }}" | ||
state: enabled | ||
|
||
- name: Ensure Enterprise package is not present when OSS is desired and vice versa | ||
package: | ||
name: "{{ 'vault' ~ ('-enterprise' if not (vault_enterprise | bool) else '') }}" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. "if not" -> "if"? |
||
state: absent | ||
become: true | ||
|
||
- name: Install Vault package | ||
package: | ||
name: "{{ _vault_repo_pkg }}" | ||
state: present | ||
become: true | ||
vars: | ||
_vault_repo_pkg: "{% if (ansible_pkg_mgr in ['yum', 'dnf']) %}\ | ||
vault-{{ vault_version }}\ | ||
vault-{{ 'enterprise-' if (vault_enterprise | bool) else '' }}{ vault_version }}\ | ||
{% elif (ansible_pkg_mgr == 'apt') %}\ | ||
vault={{ vault_version }}\ | ||
vault{{ '-enterprise' if (vault_enterprise | bool) else '' }}={{ vault_version }}\ | ||
{% else %}\ | ||
vault={{ vault_version }}\ | ||
vault{{ '-enterprise' if (vault_enterprise | bool) else '' }}={{ vault_version }}\ | ||
{% endif %}" | ||
|
||
- name: Mask default Vault config from package | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -58,7 +58,7 @@ | |
when: | ||
- vault_enterprise | bool | ||
- not vault_install_remotely | bool | ||
- not vault_install_remote_repo | bool | ||
- not vault_install_hashi_repo | bool | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. vault_install_remote_repo does not seem to be used elsewhere. But I'm not sure if we want to remove it in this PR. |
||
- installation_required | bool | ||
|
||
- name: Install OS packages and Vault via control host | ||
|
@@ -72,7 +72,6 @@ | |
- name: Install Vault via HashiCorp repository | ||
include_tasks: install_hashi_repo.yml | ||
when: | ||
- not vault_enterprise | bool | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we really want to remove it here? |
||
- not vault_install_remotely | bool | ||
- vault_install_hashi_repo | bool | ||
- installation_required | bool | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
v2.5.2 | ||
v2.5.9 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated