Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

yq here is v3 only, and breaks when main kata repo is trying to move forward #5820

Open
beraldoleal opened this issue May 31, 2024 · 0 comments
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.

Comments

@beraldoleal
Copy link
Member

We are trying to move forward with yq v4 on main kata-containers repo.

This repo is still installing yq 3 and parsing using v3 syntax against all PRs on main repo. However if that PR's is changing the syntax, it will break the tests.

@beraldoleal beraldoleal added bug Incorrect behaviour needs-review Needs to be assessed by the team. labels May 31, 2024
beraldoleal added a commit that referenced this issue May 31, 2024
Some tests uses yq installed by this repository but other tests might
use yq installed by the kata-containers repository.

We are pushing yq to v4 within the main kata repository and this shim
will handle only basic syntax of yq v3 and v4 during the transition
period.

Fixes: #5820

Signed-off-by: Beraldo Leal <[email protected]>
beraldoleal added a commit to beraldoleal/kata-tests that referenced this issue May 31, 2024
Some tests uses yq installed by this repository but other tests might
use yq installed by the kata-containers repository.

We are pushing yq to v4 within the main kata repository and this shim
will handle only basic syntax of yq v3 and v4 during the transition
period.

Fixes: kata-containers#5820

Signed-off-by: Beraldo Leal <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.
Projects
None yet
Development

No branches or pull requests

1 participant