Skip to content
This repository was archived by the owner on Feb 6, 2025. It is now read-only.

Update sonobuoy version #1164

Merged
merged 2 commits into from
Jun 16, 2020
Merged

Conversation

pablochacin
Copy link
Contributor

@pablochacin pablochacin commented Jun 15, 2020

Why is this PR needed?

Conformance tests are failing with error: The maximum supported Kubernetes version is 1.17.99, but the server version is v1.18.2. Sonobuoy will continue but unexpected results may occur.

Fixes https://github.com/SUSE/avant-garde/issues/1708

What does this PR do?

Pass sonobuoy versio as a paramter to conformace tests.

Update sonobuoy version for master branch to one that supports k8s 1.18

Removes unnecessary checkout logic in the pipeline that prevents testing against the job agains a pr branch.

Note to reviewers

The PR was tested running the conformance test job against the pr branch

Screenshot from 2020-06-16 11-41-49

Merge restrictions

(Please do not edit this)

We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:

What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

@pablochacin pablochacin marked this pull request as draft June 15, 2020 15:43
@pablochacin pablochacin force-pushed the update-sonobuoy-version branch 2 times, most recently from 84973df to ead9233 Compare June 15, 2020 16:10
@pablochacin pablochacin removed the wip label Jun 15, 2020
Pass sonobuoy versio as a paramter to conformace tests.

Update sonobuoy version for master branch to one that supports k8s 1.18

Signed-off-by: Pablo Chacin <pchacin@suse.com>
@pablochacin pablochacin force-pushed the update-sonobuoy-version branch from ead9233 to 9760deb Compare June 15, 2020 16:20
Signed-off-by: Pablo Chacin <pchacin@suse.com>
@jordimassaguerpla
Copy link
Member

Do we have to update the sonobuoy package in Devel:CaaSP:5 ?

@dirkmueller
Copy link
Member

no, 1.18.2 is already in Devel:CaaSP:5. ideally somebody would review ibs rq 220338 though and then this could refer to the latest version instead.

@pablochacin pablochacin marked this pull request as ready for review June 16, 2020 10:16
@JanZerebecki JanZerebecki merged commit c162422 into SUSE:master Jun 16, 2020
@pablochacin pablochacin deleted the update-sonobuoy-version branch June 16, 2020 11:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants