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

fix(k8s): don't store full version object with test+task results #884

Merged
merged 2 commits into from
Jun 26, 2019

Conversation

edvald
Copy link
Collaborator

@edvald edvald commented Jun 26, 2019

This addresses issues where the list of files, contained in the module
version object, would push the size of the result ConfigMap over 1MB.

@edvald edvald force-pushed the fix-run-result-storage-2 branch from 6e1dbd7 to 05cbb10 Compare June 26, 2019 14:18
@edvald edvald requested a review from 10ko June 26, 2019 14:27
@10ko
Copy link
Member

10ko commented Jun 26, 2019

Tested and fixes my issues, even when running tests with node_modules installed outside the container and without a .gardenignore.
Aside the question above, LGTM. Thanks!

edvald added 2 commits June 26, 2019 18:32
This required some mechanism for plugins to expose commands. We should
later iterate on that, but it's a start.

The cleanup flow and command itself is described in the
In-Cluster Building guide.
This addresses issues where the list of files, contained in the module
version object, would push the size of the result ConfigMap over 1MB.
@edvald edvald force-pushed the fix-run-result-storage-2 branch from 05cbb10 to c4e4059 Compare June 26, 2019 16:37
@edvald
Copy link
Collaborator Author

edvald commented Jun 26, 2019

@10ko fixed that line. Can you give a stamp of approval? :)

@thsig thsig merged commit d486dac into master Jun 26, 2019
@thsig thsig deleted the fix-run-result-storage-2 branch June 26, 2019 18:01
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.

3 participants