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

web-apps: Add support for all-namespaces in the Volumes and TensorBoards web apps #6778

Merged

Conversation

kimwnasptd
Copy link
Member

This should be the last work item for supporting all-namespaces for the web apps https://github.com/kubeflow/kubeflow/blob/master/components/proposals/20220621-all-namespaces.md

This PR:

  • Adds support to the Volumes web app for all namespaces
  • Adds support to the TensorBoards web app for all namespaces
  • Updates the CentralDashboard to have the All namespaces option for the JWA, VWA and TWA
  • Adds some e2e helpers (attribute selectors) and tests

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
@kimwnasptd kimwnasptd force-pushed the feature-kimwnasptd-vtwa-all-namespaces branch from 9fc0bf1 to 09b80f1 Compare November 28, 2022 18:34
@kimwnasptd
Copy link
Member Author

/assign @tasos-ale

@tasos-ale
Copy link
Contributor

/lgtm

@google-oss-prow google-oss-prow bot added the lgtm label Nov 29, 2022
@elikatsis
Copy link
Member

/approve

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: elikatsis

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot merged commit 858df90 into kubeflow:master Nov 29, 2022
@kimwnasptd kimwnasptd deleted the feature-kimwnasptd-vtwa-all-namespaces branch November 29, 2022 11:01
maroroman pushed a commit to maroroman/kubeflow that referenced this pull request Feb 7, 2023
…rds web apps (kubeflow#6778)

* vwa(front): Support all-namespaces

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Support all-namespaces

The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add cypress attributes

Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add e2e tests for all-namespaces

Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* dashboard: Allow all-namespaces for vwa/twa

The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 22, 2024
…rds web apps (kubeflow/kubeflow#6778)

* vwa(front): Support all-namespaces

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Support all-namespaces

The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add cypress attributes

Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add e2e tests for all-namespaces

Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* dashboard: Allow all-namespaces for vwa/twa

The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 22, 2024
…rds web apps (kubeflow/kubeflow#6778)

* vwa(front): Support all-namespaces

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Support all-namespaces

The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add cypress attributes

Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add e2e tests for all-namespaces

Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* dashboard: Allow all-namespaces for vwa/twa

The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 22, 2024
…rds web apps (kubeflow/kubeflow#6778)

* vwa(front): Support all-namespaces

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Support all-namespaces

The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add cypress attributes

Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add e2e tests for all-namespaces

Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* dashboard: Allow all-namespaces for vwa/twa

The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 23, 2024
…rds web apps (kubeflow/kubeflow#6778)

* vwa(front): Support all-namespaces

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Support all-namespaces

The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add cypress attributes

Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add e2e tests for all-namespaces

Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* dashboard: Allow all-namespaces for vwa/twa

The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 23, 2024
…rds web apps (kubeflow/kubeflow#6778)

* vwa(front): Support all-namespaces

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Support all-namespaces

The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add cypress attributes

Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add e2e tests for all-namespaces

Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* dashboard: Allow all-namespaces for vwa/twa

The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
milinddethe15 pushed a commit to milinddethe15/dashboard that referenced this pull request Feb 27, 2025
…rds web apps (kubeflow/kubeflow#6778)

* vwa(front): Support all-namespaces

The VWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Support all-namespaces

The TWA will now be able to work with the "All namespaces" option of
the CentralDashboard.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add cypress attributes

Add more attributes for elements to be selectable when running e2e
tests.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add e2e tests for all-namespaces

Add a new e2e test that checks if the Namespaces column is shown
when selecting all-namespaces.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* dashboard: Allow all-namespaces for vwa/twa

The dashboard will be allowing the VWA and TWA to work with
all-namespaces, now that we added this functionality.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants