Skip to content

test: Add Calico Inbound and Outbound policies to LKE nodes for E2E - #418

Merged
ykim-akamai merged 3 commits into
linode:devfrom
ykim-akamai:test/lke_calico_rules_e2e
Jun 17, 2024
Merged

test: Add Calico Inbound and Outbound policies to LKE nodes for E2E#418
ykim-akamai merged 3 commits into
linode:devfrom
ykim-akamai:test/lke_calico_rules_e2e

Conversation

@ykim-akamai

@ykim-akamai ykim-akamai commented Jun 12, 2024

Copy link
Copy Markdown
Contributor

📝 Description

A new script, lke_calico_rules_e2e.sh, has been added and is called in the E2E workflow file after the Run the integration test suite step in e2e-suite.yml

Key Points:

  • kubectl and calicoctl binaries are required as pre-req which are downloed in the e2e worflow file
  • The script retrieves all LKE clusters in the test account and applies Calico rules to the corresponding nodes. If no clusters exist, no action is taken. In most cases, clusters are cleaned and deleted properly after test execution
  • Due to the delay in the availability of LKE cluster configurations and the provisioning times of node instances, it is challenging to apply Calico rules directly within the test framework. Therefore, the script is separated and executed independently in the workflow file`

✔️ How to Test

TBD: Will post forked execution and steps

📷 Preview

If applicable, include a screenshot or code snippet of this change. Otherwise, please remove this section.

@ykim-akamai
ykim-akamai requested a review from a team as a code owner June 12, 2024 16:33
@ykim-akamai
ykim-akamai requested review from jriddle-linode and lgarber-akamai and removed request for a team June 12, 2024 16:33

@lgarber-akamai lgarber-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, script was tested in linode/linode-cli#619

@ykim-akamai ykim-akamai added the testing for updates to the testing suite in the changelog. label Jun 13, 2024
@ykim-akamai
ykim-akamai merged commit 54e4dfa into linode:dev Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants