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

kata: support large ConfigMaps #1023

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

kata: support large ConfigMaps #1023

wants to merge 2 commits into from

Conversation

burgerdev
Copy link
Contributor

@burgerdev burgerdev commented Nov 21, 2024

Small ConfigMaps are mounted watchable, but large ones are not. We do not really care enough about the exact mount source, since it's provided by the host anyway, so we allow mounting from watchable and non-watchable spaces.

@burgerdev burgerdev added the no changelog PRs not listed in the release notes label Nov 21, 2024
@burgerdev
Copy link
Contributor Author

Phase 1 done: test failed successfully: https://github.com/edgelesssys/contrast/actions/runs/11958600883/job/33338483638?pr=1023.

Now we need a fix.

@burgerdev burgerdev changed the title e2e: test ConfigMap mount kata: support large ConfigMaps Nov 22, 2024
@burgerdev burgerdev requested a review from 3u13r November 22, 2024 07:35
@burgerdev
Copy link
Contributor Author

Phase 2 done: test only fails for Redis.

@burgerdev burgerdev marked this pull request as ready for review November 22, 2024 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant