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

dex-auth-operator fails tests track/2.31 #136

Closed
i-chvets opened this issue Jul 14, 2023 · 4 comments
Closed

dex-auth-operator fails tests track/2.31 #136

i-chvets opened this issue Jul 14, 2023 · 4 comments

Comments

@i-chvets
Copy link
Contributor

i-chvets commented Jul 14, 2023

Description

Discovered during running on pull request workflows when not related image retrieval script was checked in.
Integration tests are not starting after many attempts.Fails on setup:

 /usr/bin/sg microk8s -c juju bootstrap --debug --verbose microk8s github-pr-08a8c --model-default test-mode=true --model-default automatically-retry-hooks=false --model-default logging-config="<root>=DEBUG" --agent-version=2.9.34 --bootstrap-constraints=""
  ERROR cannot load ssh client keys: mkdir /home/runner/.local: permission denied
  Error: The process '/usr/bin/sg' failed with exit code 2

This PR has the logs/traces for failing tests #135

Two problems were observed:

Need to review and address failing tests.

Solution

Solutions were proposed in canonical/bundle-kubeflow#648
SDI schema update solved the issue with integration tests.
canonical/serialized-data-interface#51
canonical/serialized-data-interface#51

@i-chvets i-chvets changed the title dex-auth-operator fails to start tests dex-auth-operator fails to start tests track/2.31 Jul 14, 2023
@i-chvets
Copy link
Contributor Author

i-chvets commented Jul 25, 2023

Invalid solution:

In order for Dex inb track/2.31 to pass CI, requirements must be updated. Updating requirements requires to bump SDI. This in turn requires code change to dex in track/2.31. This is the commit that has needed change.
Code change required:

This was referenced Jul 25, 2023
@i-chvets
Copy link
Contributor Author

After debugging session it was concluded that solutions proposed in canonical/bundle-kubeflow#648 cannot be used in Dex's case. New possible approach is summarized in the same issue to keep information in one place.

@i-chvets i-chvets changed the title dex-auth-operator fails to start tests track/2.31 dex-auth-operator fails tests track/2.31 Jul 25, 2023
i-chvets pushed a commit that referenced this issue Jul 26, 2023
#136

Summary of changes:
- Pinned SDDI to 0.3.6 that has a fix. Details are in canonical/bundle-kubeflow#648
@i-chvets
Copy link
Contributor Author

PR is ready for review #143

i-chvets added a commit that referenced this issue Jul 28, 2023
* fix: use new sdi 0.3.6
#136

Summary of changes:
- Pinned SDDI to 0.3.6 that has a fix. Details are in canonical/bundle-kubeflow#648
-  update integrate
- Pinned juju channel.
- Updated integration requirements
- Updated integation workflow
- Pinned ops
- Added comment
@i-chvets
Copy link
Contributor Author

Fix is merged. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant