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

e2e: Easy to locate e2e namespaces #1611

Closed
6 tasks
Tracked by #1717
nirs opened this issue Oct 28, 2024 · 0 comments · Fixed by #1707
Closed
6 tasks
Tracked by #1717

e2e: Easy to locate e2e namespaces #1611

nirs opened this issue Oct 28, 2024 · 0 comments · Fixed by #1707
Assignees
@nirs
Labels
enhancement New feature or request test Testing related issue

Comments

@nirs
Copy link
Member

nirs commented Oct 28, 2024 •
edited
Loading

Currently we have inconsistent namespaces for e2e workloads:

  • appset-deploy-rbd-busybox
  • appset-deploy-cephfs-busybox
  • subscr-deploy-rbd-busybox
  • subscr-deploy-cephfs-busybox
  • disapp-deploy-rbd-busybox

When listing the clusters, the deployments/namespaces are scattered everywhere and hard to find.

We want to have a common prefix to make the workloads easy to find, and have shorter names:

  • e2e-appset-block
  • e2e-appset-file
  • e2e-subscr-block
  • e2e-subscr-file
  • e2e-disapp-block

Tasks:

  • Add "e2e-" prefix
  • Log the namespace when creating/deleting since now it is not the same as the test name
  • Log the namespace in other operations?

We can consider other changes later:

  • Remove the "-busybox" suffix
  • Remove the "deploy-" part
  • Replace rbd and cephfs with block and file
@nirs nirs added enhancement New feature or request test Testing related issue labels Oct 28, 2024
@nirs nirs self-assigned this Dec 5, 2024
nirs added a commit to nirs/ramen that referenced this issue Dec 9, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
246c33c 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    XXX

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
nirs added a commit to nirs/ramen that referenced this issue Dec 9, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
6c16eed 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    XXX

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
nirs added a commit to nirs/ramen that referenced this issue Dec 10, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
c233c52 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    XXX

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
nirs added a commit to nirs/ramen that referenced this issue Dec 10, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
e410ad0 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    % kubectl get ns --context hub | grep e2e-
    e2e-subscr-deploy-cephfs-busybox     Active   6m2s
    e2e-subscr-deploy-rbd-busybox        Active   6m2s

    % kubectl get ns --context dr1 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   4m49s
    e2e-subscr-deploy-cephfs-busybox      Active   4m14s
    e2e-subscr-deploy-rbd-busybox         Active   4m48s

    % kubectl get ns --context dr2 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   5m10s
    e2e-subscr-deploy-cephfs-busybox      Active   5m8s
    e2e-subscr-deploy-rbd-busybox         Active   4m35s

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
nirs added a commit to nirs/ramen that referenced this issue Dec 10, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
0b407d4 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    % kubectl get ns --context hub | grep e2e-
    e2e-subscr-deploy-cephfs-busybox     Active   6m2s
    e2e-subscr-deploy-rbd-busybox        Active   6m2s

    % kubectl get ns --context dr1 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   4m49s
    e2e-subscr-deploy-cephfs-busybox      Active   4m14s
    e2e-subscr-deploy-rbd-busybox         Active   4m48s

    % kubectl get ns --context dr2 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   5m10s
    e2e-subscr-deploy-cephfs-busybox      Active   5m8s
    e2e-subscr-deploy-rbd-busybox         Active   4m35s

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
nirs added a commit to nirs/ramen that referenced this issue Dec 10, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
48bed2e 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    % kubectl get ns --context hub | grep e2e-
    e2e-subscr-deploy-cephfs-busybox     Active   6m2s
    e2e-subscr-deploy-rbd-busybox        Active   6m2s

    % kubectl get ns --context dr1 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   4m49s
    e2e-subscr-deploy-cephfs-busybox      Active   4m14s
    e2e-subscr-deploy-rbd-busybox         Active   4m48s

    % kubectl get ns --context dr2 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   5m10s
    e2e-subscr-deploy-cephfs-busybox      Active   5m8s
    e2e-subscr-deploy-rbd-busybox         Active   4m35s

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
nirs added a commit to nirs/ramen that referenced this issue Dec 10, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
f01b873 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    % kubectl get ns --context hub | grep e2e-
    e2e-subscr-deploy-cephfs-busybox     Active   6m2s
    e2e-subscr-deploy-rbd-busybox        Active   6m2s

    % kubectl get ns --context dr1 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   4m49s
    e2e-subscr-deploy-cephfs-busybox      Active   4m14s
    e2e-subscr-deploy-rbd-busybox         Active   4m48s

    % kubectl get ns --context dr2 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   5m10s
    e2e-subscr-deploy-cephfs-busybox      Active   5m8s
    e2e-subscr-deploy-rbd-busybox         Active   4m35s

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
@nirs nirs mentioned this issue Dec 11, 2024
Open
19 tasks
nirs added a commit to nirs/ramen that referenced this issue Dec 11, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
f4df064 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    % kubectl get ns --context hub | grep e2e-
    e2e-subscr-deploy-cephfs-busybox     Active   6m2s
    e2e-subscr-deploy-rbd-busybox        Active   6m2s

    % kubectl get ns --context dr1 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   4m49s
    e2e-subscr-deploy-cephfs-busybox      Active   4m14s
    e2e-subscr-deploy-rbd-busybox         Active   4m48s

    % kubectl get ns --context dr2 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   5m10s
    e2e-subscr-deploy-cephfs-busybox      Active   5m8s
    e2e-subscr-deploy-rbd-busybox         Active   4m35s

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
@nirs nirs mentioned this issue Dec 11, 2024
Merged
nirs added a commit to nirs/ramen that referenced this issue Dec 11, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
40d0a78 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    % kubectl get ns --context hub | grep e2e-
    e2e-subscr-deploy-cephfs-busybox     Active   6m2s
    e2e-subscr-deploy-rbd-busybox        Active   6m2s

    % kubectl get ns --context dr1 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   4m49s
    e2e-subscr-deploy-cephfs-busybox      Active   4m14s
    e2e-subscr-deploy-rbd-busybox         Active   4m48s

    % kubectl get ns --context dr2 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   5m10s
    e2e-subscr-deploy-cephfs-busybox      Active   5m8s
    e2e-subscr-deploy-rbd-busybox         Active   4m35s

Fixes: RamenDR#1611
Signed-off-by: Nir Soffer <[email protected]>
nirs added a commit to nirs/ramen that referenced this issue Dec 11, 2024
@nirs
e2e: Add e2e- prefix to test namespaces
48a3fb4 
It it hard to manage the namespaces created for every test when they
don't have a common prefix. Add "e2e-" prefix so all of them sort
together when listing namespaces.

Example namespaces during test:

    % kubectl get ns --context hub | grep e2e-
    e2e-subscr-deploy-cephfs-busybox     Active   6m2s
    e2e-subscr-deploy-rbd-busybox        Active   6m2s

    % kubectl get ns --context dr1 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   4m49s
    e2e-subscr-deploy-cephfs-busybox      Active   4m14s
    e2e-subscr-deploy-rbd-busybox         Active   4m48s

    % kubectl get ns --context dr2 | grep e2e-
    e2e-disapp-deploy-rbd-busybox         Active   5m10s
    e2e-subscr-deploy-cephfs-busybox      Active   5m8s
    e2e-subscr-deploy-rbd-busybox         Active   4m35s

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

@nirs nirs

Labels
enhancement New feature or request test Testing related issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

e2e: Easier to manage namespaces nirs/ramen
1 participant
@nirs