Set helm_install_namespace in all example cnf-testsuite.yml - Githubissues

cnti-testcatalog / testsuite

📞📱☎️📡🌐 Cloud Native Telecom Initiative (CNTI) Test Catalog is a tool to check for and provide feedback on the use of K8s + cloud native best practices in networking applications and platforms

https://wiki.lfnetworking.org/display/LN/Test+Catalog

Apache License 2.0

173 stars 71 forks source link

Set helm_install_namespace in all example cnf-testsuite.yml #1953

Closed collivier closed 5 months ago

collivier commented 5 months ago

Description

example-cnfs/coredns fails passing default_namespace by default and fails in gates too [1].

It forces now deployment coredns-coredns to be in cnfspace. Please note it sets helm_install_namespace in all example cnf-testsuite.yml when missing.

[1] https://github.com/cnti-testcatalog/testsuite/actions/runs/8564278589/job/23493678803#step:11:1170

Issues:

Refs: #1952

How has this been tested:

[X] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Verified all A/C passes
- [ ] develop
- [ ] master
- [ ] tag/other branch
[ ] Test environment
- [ ] Shared Packet K8s cluster
- [ ] New Packet K8s cluster
- [ ] Kind cluster
[ ] Have not tested

Types of changes:

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

martin-mat commented 5 months ago

lgtm