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.

Sign up for GitHub

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

Jump to bottom

[BUG] clean all cnf-testsuite's namespaces when asked #2213

Open
collivier opened this issue Jan 27, 2025 · 1 comment
Open

[BUG] clean all cnf-testsuite's namespaces when asked #2213

collivier opened this issue Jan 27, 2025 · 1 comment
Assignees
@svteb
Labels
bug Something isn't working

Comments

@collivier
Copy link
Collaborator

Describe the bug

cnf-default (or the one written in config) must be deleted when cnf_uninstall is called as well as cnf-testsuite when uninstall_all
(note: svcaccounts-4016 must be double checked)

# kubectl get namespaces
NAME                 STATUS   AGE
cnf-default          Active   9m54s
cnf-testsuite        Active   10m
default              Active   82m
kube-node-lease      Active   82m
kube-public          Active   82m
kube-system          Active   82m
local-path-storage   Active   82m
svcaccounts-4016     Active   10m
@collivier collivier added the bug Something isn't working label Jan 27, 2025
opnfv-github pushed a commit to opnfv/functest-kubernetes that referenced this issue Jan 27, 2025
@collivier
Upgrade cnf_testsuite to latest change
d912f56 
uninstall_all is needed as well as pss support.
note: we must keep deleting the namespaces until [1] is fixed

[1] cnti-testcatalog/testsuite#2213

Change-Id: I65cd11bf02d8df049f964809eee1d51715e75473
Signed-off-by: Cédric Ollivier <[email protected]>
opnfv-github pushed a commit to opnfv/functest-kubernetes that referenced this issue Jan 27, 2025
@collivier
Upgrade cnf_testsuite to latest change
3250499 
uninstall_all is needed as well as pss support.
note: we must keep deleting the namespaces until [1] is fixed

[1] cnti-testcatalog/testsuite#2213

Change-Id: I65cd11bf02d8df049f964809eee1d51715e75473
Signed-off-by: Cédric Ollivier <[email protected]>
(cherry picked from commit d912f56)
opnfv-github pushed a commit to opnfv/functest-kubernetes that referenced this issue Jan 27, 2025
@collivier
Upgrade cnf_testsuite to latest change
b7ef1ac 
uninstall_all is needed as well as pss support.
note: we must keep deleting the namespaces until [1] is fixed

[1] cnti-testcatalog/testsuite#2213

Change-Id: I65cd11bf02d8df049f964809eee1d51715e75473
Signed-off-by: Cédric Ollivier <[email protected]>
(cherry picked from commit d912f56)
opnfv-github pushed a commit to opnfv/functest-kubernetes that referenced this issue Jan 27, 2025
@collivier
Upgrade cnf_testsuite to latest change
d5f8283 
uninstall_all is needed as well as pss support.
note: we must keep deleting the namespaces until [1] is fixed

[1] cnti-testcatalog/testsuite#2213

Change-Id: I65cd11bf02d8df049f964809eee1d51715e75473
Signed-off-by: Cédric Ollivier <[email protected]>
(cherry picked from commit d912f56)
opnfv-github pushed a commit to opnfv/functest-kubernetes that referenced this issue Jan 27, 2025
@collivier
Upgrade cnf_testsuite to latest change
8e626be 
uninstall_all is needed as well as pss support.
note: we must keep deleting the namespaces until [1] is fixed

[1] cnti-testcatalog/testsuite#2213

Change-Id: I65cd11bf02d8df049f964809eee1d51715e75473
Signed-off-by: Cédric Ollivier <[email protected]>
(cherry picked from commit d912f56)
@svteb
Copy link
Collaborator

svteb commented Jan 28, 2025

To do this, we will likely need to resolve #2194 first.

opnfv-github pushed a commit to opnfv/functest-kubernetes that referenced this issue Jan 30, 2025
@collivier
Upgrade cnf_testsuite to latest change
b5d4f83 
uninstall_all is needed as well as pss support.
note: we must keep deleting the namespaces until [1] is fixed

[1] cnti-testcatalog/testsuite#2213

Change-Id: Ife81a87d5cc0e7ae432c7397c1125adf595d2fa4
Signed-off-by: Cédric Ollivier <[email protected]>
(cherry picked from commit d912f56)
@martin-mat martin-mat moved this to To do in CNF Test Suite Feb 5, 2025
svteb added a commit that referenced this issue Feb 27, 2025
@svteb
Feat: Migrate tar shard
e4d4660 
Refs: #2213
- Migrates tar shard to main testsuite project.
- Additionally migrates/removes the find shard, the Find module only
  has one method and it is only used in tar spec tests. I've thus
  decided to move it into tar_spec.cr, this change can be reversed in
  the future if necessary.
- Tar spec tests are intentionally not runnable, as there is currently
  a conflict between the lib and src/modules (duplicate declarations).

Signed-off-by: svteb <[email protected]>
@svteb svteb mentioned this issue Feb 27, 2025
Closed
20 tasks
svteb added a commit that referenced this issue Feb 28, 2025
@svteb
Feat: Migrate git shard
4e5d408 
Refs: #2213
- Migrates git shard to main testsuite project.
- Flattens out the directory structure.
- Some of the deleted utils files might be necessary, `require`
  statements will be resolved in later commits, if that is the case.

Signed-off-by: svteb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@svteb svteb

Labels
bug Something isn't working
Projects
CNF Test Suite
Status: To do
Milestone
No milestone
Development

No branches or pull requests
2 participants
@collivier @svteb