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

Fix local_e2e_tests CI job #7889

Merged
merged 12 commits into from
Jul 15, 2024
Merged

Conversation

beni0888
Copy link
Collaborator

@beni0888 beni0888 commented Jul 10, 2024

Description of the change

CI is broken at local_e2e_tests in the kubeapps-general workflow for the following reasons:

  • It's failing at the Export cluster variables step because it's unable to get the DEX IP. The error happens because docker network inspect kind| jq '.[0].IPAM.Config returns where the item we are looking for has been moved from the first to the second position ([0] -> [1]).
  • It's failing for the flux test group because it's unable to install Flux due to a 404 response from the server.

Benefits

The CI works again.

Possible drawbacks

This is not the ideal solution because is fragile. If the order of the items in the array change again the future, the CI will be broken again.

Applicable issues

  • fixes #

Additional information

Sample failing job: https://github.com/vmware-tanzu/kubeapps/actions/runs/9850222972/job/27223662707

DEFAULT_DEX_IP=172.18.0.2
DEX_IP=null...1
ADDITIONAL_CLUSTER_IP=null...2
Context "kind-kubeapps-ci" modified.
Default IP does not match with current IP used in Kind
Error: Process completed with exit code 1.

This is the output of docker network inspect kind| jq '.[0].IPAM':

runner@fv-az573-34:~/work/kubeapps/kubeapps$ docker network inspect kind| jq '.[0].IPAM'
{
  "Driver": "default",
  "Options": {},
  "Config": [
    {
      "Subnet": "fc00:f853:ccd:e793::/64"
    },
    {
      "Subnet": "172.18.0.0/16",
      "Gateway": "172.18.0.1"
    }
  ]
}
<!-- If there's anything else that's important and relevant to your pull
request, mention that information here.-->

Copy link

netlify bot commented Jul 10, 2024

Deploy Preview for kubeapps-dev canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit daa44ef
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/6694c135e99aaa0008dee3dc

@beni0888 beni0888 changed the title Add upterm step to local_e2e_tests CI job Fix local_e2e_tests CI job Jul 10, 2024
Jesús Benito Calzada added 6 commits July 10, 2024 16:23
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
Jesús Benito Calzada added 5 commits July 12, 2024 17:19
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
Signed-off-by: Jesús Benito Calzada <[email protected]>
@beni0888 beni0888 merged commit 46b0e21 into vmware-tanzu:main Jul 15, 2024
39 checks passed
@beni0888 beni0888 deleted the fix-local-e2e-tests branch July 15, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants