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

tests(helpers): separate miscellaneous and shell functions #13542

Merged
merged 13 commits into from
Sep 4, 2024

Conversation

chronolaw
Copy link
Contributor

@chronolaw chronolaw commented Aug 22, 2024

Summary

It is one of the serial refactors of helpers.lua
KAG-5236

The function kong_exec() needs a upvalue conf, so it must be reloaded each time.

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

@github-actions github-actions bot added the cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee label Aug 22, 2024
@chronolaw chronolaw marked this pull request as ready for review August 22, 2024 05:27
@chronolaw chronolaw force-pushed the refactor/clean_helpers_no2 branch from 95d257c to 1228ed8 Compare August 22, 2024 09:47
@chronolaw chronolaw changed the title refactor(helpers): separate miscellaneous functions tests(helpers): separate miscellaneous functions Aug 22, 2024
@chronolaw chronolaw changed the title tests(helpers): separate miscellaneous functions tests(helpers): separate miscellaneous and shell functions Aug 22, 2024
@chronolaw chronolaw force-pushed the refactor/clean_helpers_no2 branch from 7bda185 to 904d5f5 Compare September 4, 2024 01:39
@ADD-SP ADD-SP merged commit 9bce2d2 into master Sep 4, 2024
25 checks passed
@ADD-SP ADD-SP deleted the refactor/clean_helpers_no2 branch September 4, 2024 08:07
@team-gateway-bot
Copy link
Collaborator

Cherry-pick failed for master, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git remote add upstream https://github.com/kong/kong-ee
git fetch upstream master
git worktree add -d .worktree/cherry-pick-13542-to-master-to-upstream upstream/master
cd .worktree/cherry-pick-13542-to-master-to-upstream
git checkout -b cherry-pick-13542-to-master-to-upstream
ancref=$(git merge-base 4abf3ba028b87535c200dc5d00e97b58616116dc 904d5f5fb661d872f24bbd390afe99cb41149bc3)
git cherry-pick -x $ancref..904d5f5fb661d872f24bbd390afe99cb41149bc3

@github-actions github-actions bot added the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Sep 4, 2024
andrewgkew pushed a commit to andrewgkew/kong that referenced this pull request Sep 5, 2024
It is one of the serial refactors of helpers.lua
KAG-5236

The function `kong_exec()` needs an upvalue `conf`, which must be reloaded each time.
@kikito kikito removed the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Sep 10, 2024
curiositycasualty pushed a commit that referenced this pull request Oct 15, 2024
It is one of the serial refactors of helpers.lua
KAG-5236

The function `kong_exec()` needs an upvalue `conf`, which must be reloaded each time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee size/XL skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants