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.

Sign up for GitHub

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

Jump to bottom

Add Helm over Operator job to full E2E #3

Draft
wants to merge 1 commit into
base: devel
Choose a base branch
from
Draft

Add Helm over Operator job to full E2E #3

wants to merge 1 commit into from

Conversation

dfarrell07
Copy link
Owner

As we move to better support Helm over Operator deployments, we need to
start adding tests to other repos. For now only add a basic,
default-only canary job.

Signed-off-by: Daniel Farrell dfarrell@redhat.com

@dfarrell07 dfarrell07 force-pushed the add_canary_helm_job branch from db07694 to 655e7c6 Compare June 2, 2021 15:34
@dfarrell07 dfarrell07 mentioned this pull request Jun 2, 2021
Closed
33 tasks
@dfarrell07 dfarrell07 force-pushed the add_canary_helm_job branch 2 times, most recently from 31ecdef to 68125af Compare June 8, 2021 17:20
@dfarrell07
Copy link
Owner Author

There seems to be some sort of bug that breaks default k8s_version values when one isn't provided here:

2021-06-02T15:36:41.1765851Z /opt/shipyard/scripts/clusters.sh: line 27: kind_k8s_versions: bad array subscript
2021-06-02T15:36:41.1768333Z Running with: k8s_version=, olm_version=0.14.1, olm=false, globalnet=false, registry_inmemory=true, cluster_settings=/go/src/github.com/submariner-io/submariner-operator/scripts/kind-e2e/cluster_settings, timeout=5m

Which breaks the install:

2021-06-02T15:38:55.4520743Z �[36m[submariner-operator]$ [cluster1] command kubectl --context=cluster1 apply -f https://cloud.weave.works/k8s/net?k8s-version=v&env.IPALLOC_RANGE=10.1.0.0/16�[0m
2021-06-02T15:38:55.4523241Z [cluster1] error: unable to read URL "https://cloud.weave.works/k8s/net?k8s-version=v&env.IPALLOC_RANGE=10.1.0.0/16", server reported 400 Bad Request, status code=400

@dfarrell07 dfarrell07 force-pushed the add_canary_helm_job branch 10 times, most recently from d2cbe79 to eeff185 Compare June 9, 2021 14:45
@dfarrell07
Add Helm over Operator job to full E2E
c77160c 
As we move to better support Helm over Operator deployments, we need to
start adding tests to other repos. For now only add a basic,
default-only canary job.

Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
@dfarrell07 dfarrell07 force-pushed the add_canary_helm_job branch from eeff185 to c77160c Compare June 9, 2021 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ready-to-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dfarrell07