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
CI Workflow: Add all Azure supported k8s versions #26356
Conversation
406e292
to
14e2b13
Compare
935d908
to
002a9d8
Compare
|
||
conformance-schema-validation: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/setup-python@bd6b4b6205c4dbad673328db7b31b7fab9e241c0 # v4.6.1 | ||
with: | ||
python-version: '3.10' | ||
- run: pip install yamale | ||
- name: Checkout code | ||
uses: actions/checkout@8e5e7e5ab8b370d6c329ec480221332ada57f0ab # v3.5.2 | ||
with: | ||
persist-credentials: false | ||
# hard-code the path instead of using ${{ github.repository }} to make sure it works for forked repo as well | ||
path: src/github.com/cilium/cilium | ||
|
||
- name: Validate schema of aws, azure and gke action files | ||
shell: bash | ||
run: | | ||
for dir in aws azure gke;do | ||
for name in k8s-versions test-config; do | ||
file_base="src/github.com/cilium/cilium/.github/actions/${dir}/${name}" | ||
if [ -f ${file_base}.yaml ];then | ||
yamale -s ${file_base}-schema.yaml ${file_base}.yaml; | ||
fi | ||
done | ||
done | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was good that this part was highlighted because tests did not cover some of the files.
Fixed it, and separated it as a commit.
The current aks conformance tests run against one version of k8s either a specific version or the default version that is provided by the cloud provider. This commit adds all the supported k8s versions by Azure in a matrix strategy. Signed-off-by: Birol Bilgin <birol@cilium.io>
This commit adds schema validation for conformance tests action files Signed-off-by: Birol Bilgin <birol@cilium.io>
002a9d8
to
236187d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't feel particularly qualified, see comment, but what is owned by contributing LGTM.
@@ -0,0 +1,6 @@ | |||
include: list(include('includeItem')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
somewhat meta, but should .github/actions/
(maybe even all of .github
) be CODEOWNED by by the cilium/ci-structure
team instead of contributing? This feels far in the GHA action territory.
The current aks conformance tests run against one version of k8s either a specific version or the default version that is provided by the cloud provider.
This PR adds all the supported k8s versions by Azure in a matrix strategy.
Other cloud providers will be added by subsequent PRs.
Normal run examples:
✅ conformance-aks-v1.12.yaml
✅ conformance-aks-v1.13.yaml
✅ conformance-aks.yaml
Schedule run simulated:
✅ conformance-aks-v1.12.yaml
❌ conformance-aks-v1.13.yaml , failures are not related cluster installation
✅ conformance-aks.yaml