Skip to content
This repository has been archived by the owner on Jul 24, 2018. It is now read-only.

Armada executes tests with any value not "false" for the test: key in a chart definition #190

Closed
wilkers-steve opened this issue Jan 25, 2018 · 1 comment

Comments

@wilkers-steve
Copy link
Contributor

Is this a bug report or feature request? (choose one): bug

Python Version (output of python --version):

Development or Deployment Environment?:

Release Tag or Master: Master

Expected Behavior: Armada should only execute tests for a chart with the test: key set to true

What Actually Happened: Armada executes tests with any not "false" value set for the test: key

How to Reproduce the Issue (as minimally as possible): Set the test: key to foo and apply your manifest

Any Additional Comments:

Chart definition:


schema: armada/Chart/v1
metadata:
schema: metadata/Document/v1
name: elasticsearch
data:
chart_name: elasticsearch
release: elasticsearch
namespace: osh-infra
timeout: 3200
test: foobar
install:
no_hooks: false
upgrade:
no_hooks: false
values:
labels:
node_selector_key: openstack-control-plane
node_selector_value: enabled
storage:
filesystem_repository:
enabled: false
elasticsearch:
enabled: false
conf:
elasticsearch:
repository:
enabled: false
manifests:
pvc_snapshots: false
job_snapshot_repository: false
source:
type: local
location: "../openstack-helm-infra/"
subpath: elasticsearch
reference: master
dependencies:
- osh-infra-helm-toolkit

Output:

2018-01-25 13:41:50.966 31574 INFO armada.handlers.chartbuilder [-] Building dependency chart osh-infra-helm-toolkit for release elasticsearch.
2018-01-25 13:41:51.014 31574 INFO armada.handlers.armada [-] Installing release elasticsearch
2018-01-25 13:41:51.017 31574 INFO armada.handlers.tiller [-] Wait: True, Timeout: 3200
2018-01-25 13:42:15.583 31574 INFO armada.handlers.armada [-] Testing: osh-elasticsearch
2018-01-25 13:42:16.643 31574 INFO armada.handlers.armada [-] Test INFO: started_at {
seconds: 1516909335
nanos: 610444478
}
completed_at {
seconds: 1516909335
nanos: 721291089
}
results {
name: "osh-elasticsearch-test"
status: FAILURE
info: "pods "osh-elasticsearch-test" already exists"
started_at {
seconds: 1516909335
nanos: 610893412
}
completed_at {
seconds: 1516909335
nanos: 721290086
}
}

2018-01-25 13:42:16.656 31574 INFO armada.handlers.armada [-] PASSED: osh-elasticsearch

jenkins-attcomdev pushed a commit that referenced this issue Jan 25, 2018
Currently armada will execute tests and implement a wait condition
on any chart with a non-false value for the `test:` key. This
makes the check for the value only execute tests for a chart when
the value is set to true

For issue, see: #190

Change-Id: I1591f7f4c652e253c0d97f2a78c4733848687944
@wilkers-steve
Copy link
Contributor Author

Addressed with: https://review.gerrithub.io/#/c/396477/

Closing this issue out

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant