kong icon indicating copy to clipboard operation
kong copied to clipboard

tests(deck): add deck integration tests

Open samugi opened this issue 1 year ago • 2 comments

Summary

These tests are meant to catch changes in Kong that result in incompatibilities with decK.

The approach used here is to configure all the bundled plugins in Kong and a few other entities and confirm that deck dump + sync executes without errors.

This new workflow "continues-on-error" instead of producing failures on CI: changes in gateway schemas might have to be merged despite making decK fail, the purpose of this is to notify about such incompatibilities. When those failures are detected, the workflow posts a review in the PR requesting changes, and a link to the failing run.

~The review is then dismissed if new changes are pushed to the PR that make the workflow pass.~ <-- not anymore: branch protection rules prevent this

Checklist

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

Issue reference

KAG-4236

samugi avatar May 06 '24 07:05 samugi

decK integration tests

:white_check_mark: passed successfully.

github-actions[bot] avatar May 06 '24 10:05 github-actions[bot]

re-drafting to explore the option of using a review (by the bot) instead of a comment, to make the failures more visible when they occur

Edit: done

samugi avatar May 10 '24 15:05 samugi

Successfully created cherry-pick PR for master:

  • https://github.com/kong/kong-ee/pull/9429

team-gateway-bot avatar Jun 11 '24 17:06 team-gateway-bot