Skip to content

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

Merged
ykim-akamai merged 4 commits intolinode:mainfrom
ykim-akamai:test/lke_calico_rules_e2e
Jun 17, 2024
Merged

test: Add Calico Inbound and Outbound policies to LKE nodes for E2E#525
ykim-akamai merged 4 commits intolinode:mainfrom
ykim-akamai:test/lke_calico_rules_e2e

Conversation

@ykim-akamai
Copy link
Contributor

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

📝 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

Forked run - https://github.com/ykim-1/linodego/actions/runs/9491201493/job/26156141578

Forked run on PR - https://github.com/ykim-1/linodego/actions/runs/9491223844/job/26156204794
Note test failed but it was able to apply the calico rules

📷 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 23:20
@ykim-akamai ykim-akamai requested review from lgarber-akamai and zliang-akamai and removed request for a team June 12, 2024 23:20
Copy link
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

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

Script tested in linode/linode-cli#619

@ykim-akamai ykim-akamai merged commit 0e39dca into linode:main Jun 17, 2024
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