Skip to content

Run zenko tests in codespace + some cleanup of unused files#2324

Open
SylvainSenechal wants to merge 2 commits intoimprovement/ZENKO-5180from
improvement/ZENKO-5181
Open

Run zenko tests in codespace + some cleanup of unused files#2324
SylvainSenechal wants to merge 2 commits intoimprovement/ZENKO-5180from
improvement/ZENKO-5181

Conversation

@SylvainSenechal
Copy link
Contributor

Issue: ZENKO-5181

@SylvainSenechal SylvainSenechal requested review from a team, delthas and maeldonn February 16, 2026 17:48
@SylvainSenechal SylvainSenechal marked this pull request as draft February 16, 2026 17:49
"ghcr.io/devcontainers/features/docker-in-docker": {},
"ghcr.io/devcontainers/features/github-cli:1": {},
"ghcr.io/devcontainers/features/node:1": {},
"ghcr.io/devcontainers/features/python:1": {},
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python is already available in the codespace, but some yarn install command because something "gyp" related to python failed to install. This should fix it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated to running zenko tests in Codespace, but I did some decluttering in this pr and removed 4 unused files (removed docker entry point, and the associated files that werent used by anything else)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This structure and usage is very similar to what I did for the Cucumber tests.
Tbh, I was also able to run the test without a pod, but it required port forward, and using different values for some variables, which later will probably become annoying because we don't want to have to deal with 2 values for the same variables depending on whether the test is running in github ci or codespace.
So maybe later we'll come back to this if we decide to run all tests outside of cluster, but don't really wanna deal with it for now

TEST_GREP="${1:-should list objects in V2 format}"
TEST_PATH="${2:-cloudserver}"

ZENKO_ACCESS_KEY=$(kubectl get secret end2end-management-account-creds.v1 -o jsonpath='{.data.accessKey}' | base64 -d)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These env variables will eventually be removed (calculated automatically from within the test)

@SylvainSenechal SylvainSenechal force-pushed the improvement/ZENKO-5180 branch 12 times, most recently from 59e3b03 to 90d69c5 Compare February 17, 2026 14:23
@SylvainSenechal SylvainSenechal marked this pull request as ready for review February 17, 2026 16:36
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

Successfully merging this pull request may close these issues.

1 participant

Comments