Skip to content

Commit

Permalink
Merge pull request #59 from nicholasjackson/dev
Browse files Browse the repository at this point in the history
Post Deployment Tests
  • Loading branch information
nicholasjackson authored Apr 25, 2022
2 parents e0fe0f7 + 7ec0e55 commit 37c3582
Show file tree
Hide file tree
Showing 87 changed files with 22,529 additions and 283 deletions.
12 changes: 7 additions & 5 deletions .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
- name: Set up Go
uses: actions/setup-go@v2
with:
go-version: 1.17
go-version: 1.18

- name: Build
run: go build -v ./...
Expand All @@ -30,7 +30,7 @@ jobs:
- name: Set up Go
uses: actions/setup-go@v2
with:
go-version: 1.17
go-version: 1.18

- name: Setup Kubebuilder
run: |
Expand Down Expand Up @@ -58,13 +58,15 @@ jobs:
- "@k8s_canary_existing"
- "@k8s_canary_none"
- "@k8s_canary_rollback"
- "@k8s_canary_with_post_deployment_test"
- "@k8s_canary_with_post_deployment_test_fail"
steps:
- uses: actions/checkout@v2

- name: Set up Go
uses: actions/setup-go@v2
with:
go-version: 1.17
go-version: 1.18

- name: Setup Functional Tests
run: |
Expand Down Expand Up @@ -185,10 +187,10 @@ jobs:
if: ${{ needs.check_labels.outputs.pr_labels_set == 'true' }}

steps:
- name: Set up Go 1.17
- name: Set up Go 1.18
uses: actions/setup-go@v2
with:
go-version: 1.17
go-version: 1.18

- name: Set up Node 14
uses: actions/setup-node@v2
Expand Down
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
*.tgz
node_modules
docs/.docusaurus
docs/build
7 changes: 7 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,10 @@ generate_helm:
# First generate the Helm specific kustomize config that creates the RBAC and CRDs
kustomize build ./kubernetes/controller/config/helm -o ./deploy/kubernetes/charts/consul-release-controller/templates

# Move the crds to the crds folder for helm
mv ./deploy/kubernetes/charts/consul-release-controller/templates/apiextensions.k8s.io_v1_customresourcedefinition_releases.consul-release-controller.nicholasjackson.io.yaml \
./deploy/kubernetes/charts/consul-release-controller/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_releases.consul-release-controller.nicholasjackson.io.yaml

# Set the version in the chart
cp ./deploy/kubernetes/charts/consul-release-controller/Chart.tpl ./deploy/kubernetes/charts/consul-release-controller/Chart.yaml
sedi=(-i) && [ "$(UNAME)" == "Darwin" ] && sedi=(-i '') ; \
Expand All @@ -98,6 +102,9 @@ generate_helm:
sedi=(-i) && [ "$(UNAME)" == "Darwin" ] && sedi=(-i '') ; \
sed "$${sedi[@]}" -e 's/##VERSION##/${VERSION}/' ./deploy/kubernetes/charts/consul-release-controller/values.yaml

# Fetch the chart deps
helm dep up ./deploy/kubernetes/charts/consul-release-controller

# Now package the Helm chart into a tarball
helm package ./deploy/kubernetes/charts/consul-release-controller

Expand Down
23 changes: 23 additions & 0 deletions changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,29 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Changed
- Helm chart Webhook config failure policy now defaults to `Ignore`
- Configuration for the server moved to global `config` package

### Added
- Added features to run manual tests for candidate services before initial traffic is sent.
Post deployment tests can be configured to automatically call the defined endpoint for the consul
service under test. All traffic is routed over consul service mesh ensuring no requirement to have
the candidate service exposed outside of the mesh.

```yaml
postDeploymentTest:
pluginName: "http"
config:
path: "/"
method: "GET"
requiredTestPasses: 3
interval: "10s"
timeout: "120s"
```
- Added sidecar to controller deployment to allow communication with consul services for post deployment tests
## [0.0.14 - 2022-03-14
### Fixed
- Ensure a release reconfigures the plugins on update
Expand Down
Loading

0 comments on commit 37c3582

Please sign in to comment.