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

Enable configmap deletion protection for e2e #7593

Closed
wants to merge 4 commits into from
Closed

Conversation

linki
Copy link
Member

@linki linki commented May 29, 2024

Let's see which tests actually fail here and check whether we can make them pass.

@linki linki added experiment do-not-merge minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call. labels May 29, 2024
@@ -31,6 +31,9 @@ pipeline:
echo "Running go mod tidy modified go.mod and/or go.sum"
exit 1
fi
for patch in patches/*.patch; do
cd e2e_modules/kubernetes && git apply "../../$patch" && cd -
Copy link
Contributor

@mikkeloscar mikkeloscar Jul 2, 2024

Choose a reason for hiding this comment

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

The usage of e2e_modules was dropped in #7704

@linki
Copy link
Member Author

linki commented Jul 15, 2024

Closing this for now.

@linki linki closed this Jul 15, 2024
@linki linki deleted the linki-patch-1 branch July 15, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge experiment minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants