FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: change the default node image to cos_containerd by Shabirmean · Pull Request #706 · googleapis/java-container · GitHub

This repository was archived by the owner on Sep 16, 2023. It is now read-only.
/ java-container Public archive

fix: change the default node image to cos_containerd - #706

Merged
Shabirmean merged 1 commit into
mainfrom
fix-imagemismatch
Mar 23, 2022
ConversationCommits (1)ChecksFiles changed
Merged

fix: change the default node image to cos_containerd#706
Shabirmean merged 1 commit into
mainfrom
fix-imagemismatch

Conversation

Shabirmean commented Mar 23, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

Fixes #705

Description

  • Integration tests were failing on the nightly builds for the repo
  • The issue was that docker was phased out as the default container runtime for GKE in 1.19.
  • Thus, it was recommended to use the node image (cos_containerd) which is specific to containerd runtime and to move away from the current node image cos
  • In the most recent release of GKE the default node image used is cos_containerd
  • Because of this the integration test that checks for this was failing.
  • See release notes for information.

Changes

  • Fixed the above issue by changing the check to COS_CONTAINERD (the new node image) from COS

Shabirmean requested review from a team, Neenu1995 and NimJay March 23, 2022 15:54
Shabirmean added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 23, 2022
Shabirmean added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Mar 23, 2022
NimJay approved these changes Mar 23, 2022
View reviewed changes

NimJay left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Looks good to me.
Thanks for taking care if this so fast, Shabir!

product-auto-label Bot added the api: container Issues related to the googleapis/java-container API. label Mar 23, 2022
yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 23, 2022
Shabirmean closed this Mar 23, 2022
Shabirmean reopened this Mar 23, 2022
Shabirmean merged commit 9ffa70d into main Mar 23, 2022
Shabirmean deleted the fix-imagemismatch branch March 23, 2022 19:21
release-please Bot mentioned this pull request Mar 23, 2022
Merged
gcf-merge-on-green Bot pushed a commit that referenced this pull request Mar 30, 2022
chore(main): release 2.3.6 (#707)
aa24209 
🤖 I have created a release *beep* *boop*
---


### [2.3.6](v2.3.5...v2.3.6) (2022-03-29)


### Bug Fixes

* change the default node image to cos_containerd ([#706](#706)) ([9ffa70d](9ffa70d))


### Dependencies

* update dependency com.google.cloud:google-cloud-shared-dependencies to v2.9.0 ([#709](#709)) ([bb64318](bb64318))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Neenu1995 Awaiting requested review from Neenu1995
2 more reviewers
NimJay NimJay approved these changes
xtineskim xtineskim approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: container Issues related to the googleapis/java-container API.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

container.v1.it.ITSystemTest: getServerConfigTest failed

4 participants

Back | FazBrowse Home | New Git URL