Skip to content

Commit

Permalink
Merge pull request #192 from peopledoc/hvac
Browse files Browse the repository at this point in the history
  • Loading branch information
Joachim Jablon committed Jul 5, 2021
2 parents 3904eb1 + 33e0770 commit 8505388
Show file tree
Hide file tree
Showing 7 changed files with 26 additions and 51 deletions.
15 changes: 12 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: CI
on:
pull_request:
push:
branches:
branches:
- 'master'
tags:
- '*'
Expand Down Expand Up @@ -40,12 +40,21 @@ jobs:
include:
- python-version: 3.6
tox-env: py36-tests
auth-method: token
- python-version: 3.7
tox-env: py37-tests
auth-method: token
- python-version: 3.8
tox-env: py38-tests
auth-method: token
- python-version: 3.8
tox-env: py38-tests
auth-method: userpass
- python-version: 3.8
tox-env: py38-tests
auth-method: cert

name: Test (python ${{ matrix.python-version }})
name: Test (python ${{ matrix.python-version }} / ${{ matrix.auth-method }})
runs-on: ubuntu-latest

services:
Expand All @@ -66,7 +75,7 @@ jobs:
set -xeu
pwd
python -VV
./dev-env auth token
./dev-env auth ${{ matrix.auth-method }}
pip install tox
- name: Run tox targets for ${{ matrix.python-version }}
run: tox -e ${{ matrix.tox-env }}
Expand Down
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,6 @@ dist/
docs/_build
htmlcov/
vault.yml
server-chain.crt
client.crt
client.key
5 changes: 5 additions & 0 deletions dev-env
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@

case ${1-default} in
auth)
curl -o server-chain.crt https://raw.githubusercontent.com/ewjoachim/vault-cli-dev/master/conf/certs/server-chain.crt
if [ "${2}" = "cert" ]; then
curl -o client.crt https://raw.githubusercontent.com/ewjoachim/vault-cli-dev/master/conf/certs/client.crt
curl -o client.key https://raw.githubusercontent.com/ewjoachim/vault-cli-dev/master/conf/certs/client.key
fi
ln -fs vault.${2}.yml vault.yml
;;
*)
Expand Down
42 changes: 0 additions & 42 deletions server-chain.crt

This file was deleted.

2 changes: 1 addition & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ install_requires =
Click>=7.0
pyyaml>=5.3.1
jinja2
hvac
hvac<0.10.12 # Temporary fix for #191

[options.entry_points]
console_scripts =
Expand Down
6 changes: 3 additions & 3 deletions vault.cert.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
url: https://localhost:8443
base_path: secretkvv1/
ca_bundle: dev_docker_config/certs/server-chain.crt
login_cert: dev_docker_config/certs/client.crt
login_cert_key: dev_docker_config/certs/client.key
ca_bundle: server-chain.crt
login_cert: client.crt
login_cert_key: client.key
4 changes: 2 additions & 2 deletions vault.userpass.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
base_path: secretkvv1/
ca_bundle: dev_docker_config/certs/server-chain.crt
url: https://localhost:8443
base_path: secretkvv1/
ca_bundle: server-chain.crt
username: userpass-login
password: userpass-pass

0 comments on commit 8505388

Please sign in to comment.