-
Notifications
You must be signed in to change notification settings - Fork 108
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
Helm acceptance tests #2002
Helm acceptance tests #2002
Changes from 36 commits
f458dad
11b1bb8
ebfbf87
6a1a753
2fd45c5
ff07562
70d6b56
eb5e521
183f2e4
3bbe810
b6a93d4
7ed0efd
e6ff621
ccdd510
1df680a
acfccd3
60b92cd
fdbc2a4
9be6d0e
7c7c3e1
4616eb7
dbb1d46
69e3541
a63a8a9
b04ae23
162878d
4071ea3
1e634a4
ac97392
6305a3e
ce4bc20
5126fef
0315456
05f9e27
1a34249
8ae6a81
5ed0dd3
668e54c
19175e3
34c0149
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 |
---|---|---|
@@ -0,0 +1,35 @@ | ||
{{- if .Values.test.enabled -}} | ||
apiVersion: v1 | ||
kind: Pod | ||
metadata: | ||
labels: {{- include "hedera-mirror.labels" . | nindent 4 }} | ||
name: {{ include "hedera-mirror.fullname" . }}-acceptance-test | ||
namespace: {{ include "hedera-mirror.namespace" . }} | ||
annotations: | ||
helm.sh/hook: test | ||
helm.sh/hook-delete-policy: hook-failed,hook-succeeded | ||
spec: | ||
containers: | ||
- name: acceptance-tests | ||
image: "{{ .Values.test.image.repository }}:{{ .Values.test.image.tag | default .Chart.AppVersion }}" | ||
imagePullPolicy: {{ .Values.test.image.pullPolicy }} | ||
command: | ||
- 'sh' | ||
- '-c' | ||
- | | ||
Nana-EC marked this conversation as resolved.
Show resolved
Hide resolved
|
||
cp /etc/secrets/* /usr/etc/hedera-mirror-node/hedera-mirror-test/src/test/resources/ | ||
find /usr/etc/hedera-mirror-node/hedera-mirror-test/src/test/resources/ | ||
cat /usr/etc/hedera-mirror-node/hedera-mirror-test/src/test/resources/*.yml | ||
./mvnw integration-test --projects hedera-mirror-test/ -P=acceptance-tests -Dcucumber.filter.tags={{ .Values.test.cucumberTags }} | ||
volumeMounts: | ||
- name: acceptance-tests | ||
mountPath: /etc/secrets | ||
readOnly: true | ||
restartPolicy: Never | ||
terminationGracePeriodSeconds: 1 | ||
volumes: | ||
- name: acceptance-tests | ||
secret: | ||
defaultMode: 420 | ||
secretName: {{ include "hedera-mirror.fullname" . }}-acceptance-test | ||
{{- end -}} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
{{- if .Values.test.enabled -}} | ||
apiVersion: v1 | ||
kind: Secret | ||
metadata: | ||
labels: {{ include "hedera-mirror.labels" . | nindent 4 }} | ||
name: {{ include "hedera-mirror.fullname" . }}-acceptance-test | ||
namespace: {{ include "hedera-mirror.namespace" . }} | ||
annotations: | ||
helm.sh/hook: test | ||
helm.sh/hook-delete-policy: hook-failed,hook-succeeded | ||
|
||
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. nit: extra line 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. Removed. |
||
type: Opaque | ||
stringData: | ||
application-default.yml: |- | ||
{{- tpl (toYaml .Values.test.config) $ | nindent 4 }} | ||
{{- end -}} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -259,6 +259,27 @@ rest: | |
name: mirror-passwords | ||
key: HEDERA_MIRROR_IMPORTER_DB_RESTUSERNAME | ||
|
||
test: | ||
config: | ||
hedera: | ||
mirror: | ||
test: | ||
acceptance: | ||
existingTopicNum: | ||
# grpc endpoint | ||
mirrorNodeAddress: "{{ .Release.Name }}-grpc:5600" | ||
network: | ||
# Do not use use 0.0.2 or 0.0.50 for operator to ensure crypto transfers are not waived | ||
operatorId: | ||
operatorKey: | ||
Nana-EC marked this conversation as resolved.
Show resolved
Hide resolved
|
||
restBaseUrl: "http://{{ .Release.Name }}-rest/api/v1" | ||
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. This should be 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. Good catch, I just got lucky that the default was testnet. I tested with this corrected and it works, also tested with a bad endpoint and it failed so I know it's using it now. |
||
cucumberTags: "@acceptance" | ||
enabled: false | ||
image: | ||
pullPolicy: IfNotPresent | ||
repository: gcr.io/mirrornode/hedera-mirror-test | ||
tag: "" # Defaults to the chart's app version | ||
|
||
timescaledb: | ||
enabled: false | ||
image: | ||
|
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.
This should also use the
global.image.tag
. You'll need to add aglobal.image: {}
to the values.yaml.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.
Sure, added that.