Skip to content
This repository has been archived by the owner on Nov 29, 2023. It is now read-only.

feat: added a flag to enable/disable gvnic on a node pool #147

Merged
merged 4 commits into from Sep 23, 2021

Conversation

gcf-owl-bot[bot]
Copy link
Contributor

@gcf-owl-bot gcf-owl-bot bot commented Sep 15, 2021

Closes #140

  • Regenerate this pull request now.

PiperOrigin-RevId: 396762326

Source-Link: googleapis/googleapis@1f1d09d

Source-Link: https://github.com/googleapis/googleapis-gen/commit/4bed2d46a5e5a6d537ebc15115cfe4cce5fc0405
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNGJlZDJkNDZhNWU1YTZkNTM3ZWJjMTUxMTVjZmU0Y2NlNWZjMDQwNSJ9

feat: added node pool level network config
feat: added update support for node pool labels, taints and network tags
feat: added configuration for workload certificates and identity service component
feat: added configuration for node pool defaults, autopilot, logging and monitoring
feat: added the option to specify L4 load balancer configuration and IP v6 configuration
feat: added the option to list supported windows versions
fix: deprecated KALM addon config option
fix: deprecated cluster status condition code
docs: clarified SetNodePoolSize API behavior
fix(deps): require proto-plus 1.15.0

… node pool level network config feat: added update support for node pool labels, taints and network tags feat: added configuration for workload certificates and identity service component feat: added configuration for node pool defaults, autopilot, logging and monitoring feat: added the option to specify L4 load balancer configuration and IP v6 configuration feat: added the option to list supported windows versions fix: deprecated KALM addon config option fix: deprecated cluster status condition code docs: clarified SetNodePoolSize API behavior

PiperOrigin-RevId: 396762326

Source-Link: googleapis/googleapis@1f1d09d

Source-Link: googleapis/googleapis-gen@4bed2d4
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNGJlZDJkNDZhNWU1YTZkNTM3ZWJjMTUxMTVjZmU0Y2NlNWZjMDQwNSJ9
@gcf-owl-bot gcf-owl-bot bot requested review from a team as code owners September 15, 2021 06:47
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Sep 15, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 15, 2021
@product-auto-label product-auto-label bot added the api: container Issues related to the googleapis/python-container API. label Sep 15, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 15, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 15, 2021
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Sep 15, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 15, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 15, 2021
@busunkim96
Copy link
Contributor

Unit tests failing with:

==================================== ERRORS ====================================
_ ERROR collecting tests/unit/gapic/container_v1beta1/test_cluster_manager.py __
tests/unit/gapic/container_v1beta1/test_cluster_manager.py:34: in <module>
    from google.cloud.container_v1beta1.services.cluster_manager import (
google/cloud/container_v1beta1/__init__.py:17: in <module>
    from .services.cluster_manager import ClusterManagerClient
google/cloud/container_v1beta1/services/cluster_manager/__init__.py:16: in <module>
    from .client import ClusterManagerClient
google/cloud/container_v1beta1/services/cluster_manager/client.py:34: in <module>
    from google.cloud.container_v1beta1.services.cluster_manager import pagers
google/cloud/container_v1beta1/services/cluster_manager/pagers.py:27: in <module>
    from google.cloud.container_v1beta1.types import cluster_service
google/cloud/container_v1beta1/types/__init__.py:16: in <module>
    from .cluster_service import (
google/cloud/container_v1beta1/types/cluster_service.py:4043: in <module>
    class StatusCondition(proto.Message):
google/cloud/container_v1beta1/types/cluster_service.py:4058: in StatusCondition
    class Code(proto.Enum):
.nox/unit-3-6/lib/python3.6/site-packages/proto/enums.py:34: in __new__
    cls = super().__new__(mcls, name, bases, attrs)
/usr/local/lib/python3.6/enum.py:201: in __new__
    enum_member = __new__(enum_class, *args)
E   TypeError: int() argument must be a string, a bytes-like object or a number, not 'dict'
- generated xml file: /tmpfs/src/github/python-container/unit_3.6_sponge_log.xml -

This looks similar to the error @atulep saw in the generator.

@bourgeoisor
Copy link
Contributor

Hi @busunkim96! Do you have some more details on that previous generate error? Are the tests failing here (on #147) due to the generator not generating properly and/or is this something I could look into to help push this PR?

@bourgeoisor bourgeoisor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 17, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 17, 2021
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Sep 21, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 21, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 21, 2021
@parthea parthea self-assigned this Sep 23, 2021
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Sep 23, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 23, 2021
@google-cla
Copy link

google-cla bot commented Sep 23, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no This human has *not* signed the Contributor License Agreement. and removed cla: yes This human has signed the Contributor License Agreement. labels Sep 23, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 23, 2021
@parthea
Copy link
Contributor

parthea commented Sep 23, 2021

@googlebot I consent.

@google-cla google-cla bot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Sep 23, 2021
@parthea parthea changed the title feat: added a flag to enable/disable gvnic on a node pool feat: added node pool level network config feat: added update support for node pool labels, taints and network tags feat: added configuration for workload certificates and identity service compo... feat: added a flag to enable/disable gvnic on a node pool Sep 23, 2021
@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Sep 23, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit 616b21a into main Sep 23, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the owl-bot-d7bf71c0-3f84-491e-8ace-fac81678e6d9 branch September 23, 2021 09:48
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 23, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Sep 24, 2021
🤖 I have created a release \*beep\* \*boop\*
---
## [2.8.0](https://www.github.com/googleapis/python-container/compare/v2.7.1...v2.8.0) (2021-09-23)


### Features

* added a flag to enable/disable gvnic on a node pool ([#147](https://www.github.com/googleapis/python-container/issues/147)) ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* added configuration for node pool defaults, autopilot, logging and monitoring ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* added configuration for workload certificates and identity service component ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* added node pool level network config ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* added the option to list supported windows versions ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* added the option to specify L4 load balancer configuration and IP v6 configuration ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* added update support for node pool labels, taints and network tags ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))


### Bug Fixes

* add 'dict' annotation type to 'request' ([c912605](https://www.github.com/googleapis/python-container/commit/c9126057cde7fc28094785cceab9cf43e42ca8e0))
* deprecated cluster status condition code ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* deprecated KALM addon config option ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
* **deps:** require proto-plus 1.15.0 ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))


### Documentation

* clarified SetNodePoolSize API behavior ([616b21a](https://www.github.com/googleapis/python-container/commit/616b21a6abe2b0c4dd647cf56d544c2aff7312f7))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: container Issues related to the googleapis/python-container API. cla: yes This human has signed the Contributor License Agreement. owl-bot-copy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing ability to set taints, tags and labels of nodepool
4 participants