search

canonical / kubeflow-dashboard-operator

Operator for Kubeflow Dashboard
Apache License 2.0
1 stars 1 forks source link

The `kubeflow-dashboard` charm integration tests do not test that the dashboard is accessible #189

Closed NohaIhab closed 3 weeks ago

NohaIhab commented 1 month ago

Bug Description

The current integration tests do not cover testing that the dashboard is accessible. This means that the integration tests can pass when the dashboard is in fact non functional. This was exploited when the integration tests failed for dex-auth in https://github.com/canonical/dex-auth-operator/pull/187, where test_login failed with timeout to reach the dashboard. We should have a test to verify this in this repo since this can potentially break the bundle.

For example, the dashboard is now broken on juju 3.5 due to #188, and yet the integration tests passed for #182 and #183

To Reproduce

  1. Run the charm's integration tests withtox -e integration in main on Juju 3.5
  2. The tests pass
  3. Try to access the dashboard -> fail

Environment

juju 3.5/stable microk8s 1.25-strict/stable

syncronize-issues-to-jira[bot] commented 1 month ago

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/KF-5655.

This message was autogenerated