Skip to content
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

chore(helm): move test value file for memory chart to common directory #486

Conversation

fty4
Copy link
Member

@fty4 fty4 commented Jun 15, 2023

WHAT

Move and rename the tractusx-connector-memory testing values file to the same location as other test value files.

WHY

Currently the values file for all tests are located inside edc-tests/deployment/src/main/resources/helm with the exception of the tractusx-connector-memory values file.

This should be unified to the same path.

FURTHER NOTES

also see different locations in the workflow deployment-test:

values_file: edc-tests/deployment/src/main/resources/helm/tractusx-connector-test.yaml

values_file: charts/tractusx-connector-memory/example.yaml

values_file: edc-tests/deployment/src/main/resources/helm/tractusx-connector-azure-vault-test.yaml


Marco Lecheler [email protected] Mercedes-Benz Tech Innovation GmbH (ProviderInformation)

Move and rename the tractusx-connector-memory test values.yaml to the same location as other test value files
@fty4 fty4 added the helmcharts Anything that has to do with helm charts label Jun 15, 2023
@paullatzelsperger
Copy link
Contributor

@fty4 if you could rebase and solve conflicts i'll merge

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@fty4
Copy link
Member Author

fty4 commented Jun 15, 2023

@paullatzelsperger solved conflicts and tests passed

@paullatzelsperger paullatzelsperger merged commit 79fa387 into eclipse-tractusx:main Jun 15, 2023
@fty4 fty4 deleted the chore/helm-test-resources-values branch June 15, 2023 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helmcharts Anything that has to do with helm charts
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

2 participants