Navigation Menu

Skip to content
This repository has been archived by the owner on Sep 14, 2020. It is now read-only.

Switch to py38 as the primary CI version, keep py37 tested #314

Merged
merged 3 commits into from Feb 20, 2020

Conversation

nolar
Copy link
Contributor

@nolar nolar commented Feb 20, 2020

What do these changes do?

Switch from Python 3.7 to 3.8 as the primary testing environment in Travis, and resolve few issues/warnings with 3.8. Python 3.7 compatibility is still tested and ensured.

Description

The testing is now organised this way:

  • The latest Python — with all supported Kubernetes versions.
  • The latest Kubernetes — with all supported Python versions (3.7 & 3.8 & optionally 3.9-dev).

We assume that if something breaks, it either breaks in the language (and one K8s version is enough to detect it), or in Kubernetes API (and one Python version is enough to detect it) — there is no need to test a real matrix "all Pythons vs. all K8ses".

Travis CI has a limit of 5 builds running in parallel (for us), so more than 5 or 10 builds delays the build results. Currently, we have 10 — and so it will be with the new grid (for a while).

Issues/PRs

Issues: #13

Type of changes

  • Mostly CI/CD automation, contribution experience

Checklist

  • The code addresses only the mentioned problem, and this problem only
  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt

@nolar nolar added the automation CI/CD: testing, linting, releasing automatically label Feb 20, 2020
@zincr
Copy link

zincr bot commented Feb 20, 2020

🤖 zincr found 0 problems , 0 warnings

✅ Large Commits
✅ Approvals
✅ Specification
✅ Dependency Licensing

@zincr
Copy link

zincr bot commented Feb 20, 2020

🤖 zincr found 1 problem , 0 warnings

❌ Approvals
✅ Large Commits
✅ Specification
✅ Dependency Licensing

Details on how to resolve are provided below


Approvals

All proposed changes must be reviewed by project maintainers before they can be merged

Not enough people have approved this pull request - please ensure that 1 additional user, who have not contributed to this pull request approve the changes.

  • ✅ Approved by PR author @nolar
  • ❌ 1 additional approval needed
     

@nolar nolar merged commit a45fe91 into zalando-incubator:master Feb 20, 2020
@nolar nolar deleted the py38 branch February 20, 2020 16:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automation CI/CD: testing, linting, releasing automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants