Skip to content
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

CRAYSAT-1849: Update Python k8s version to the latest one supported in CSM 1.6 #270

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

annapoorna-s-alt
Copy link
Contributor

@annapoorna-s-alt annapoorna-s-alt commented Oct 3, 2024

Summary and Scope

Update the version of the Python kubernetes client package which compatible to the version of Kubernetes running in CSM 1.6
Also update the version of csm-api-client, which has been updated along with the kubernetes version

Issues and Related PRs

List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Testing

List the environments in which these changes were tested.

Tested on:

starlord

Test description:

Built the container image, downloaded to odin, and ran the
CoreV1Api.list_node method in a Python interpreter in the sat bash
shell.

Risks and Mitigations

This should be low risk. It is aligning the version of the kubernetes Python package with the version of the Kubernetes cluster in CSM 1.6.

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable
  • HPC Product Announcement prepared, if applicable

@shivaprasad-metimath
Copy link
Contributor

Validation output post csm-api-client unstable build version removal.

ncn-m001:/mnt/shiva # sat bash
(a28471077c8a) sat-container:/sat/share # pip show kubernetes
Name: kubernetes
Version: 24.2.0
Summary: Kubernetes python client
Home-page: https://github.com/kubernetes-client/python
Author: Kubernetes
Author-email:
License: Apache License Version 2.0
Location: /sat/venv/lib/python3.9/site-packages
Requires: certifi, google-auth, python-dateutil, pyyaml, requests, requests-oauthlib, setuptools, six, urllib3, websocket-client
Required-by: cray-product-catalog, csm-api-client, sat
(a28471077c8a) sat-container:/sat/share # python
Python 3.9.18 (main, Aug 26 2023, 11:50:23)
[GCC 10.3.1 20211027] on linux
Type "help", "copyright", "credits" or "license" for more information.

from csm_api_client.k8s import load_kube_api
k = load_kube_api()
n = k.list_node()
len(n.items)
7

Copy link
Contributor

@haasken-hpe haasken-hpe left a comment

Choose a reason for hiding this comment

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

Looks good. Please squash before merging. You also have a typo in the first commit message that you could fix when you do that.

@shivaprasad-metimath shivaprasad-metimath merged commit a2f2381 into main Oct 8, 2024
3 checks passed
@shivaprasad-metimath shivaprasad-metimath deleted the craysat-1849-k8s-update branch October 8, 2024 06:36
shivaprasad-metimath added a commit that referenced this pull request Oct 9, 2024
…n CSM 1.6 (#270) (#271)

* CRAYSAT-1849: Update2 Python k8s version to the latest one supported in CSM 1.6

IM: CRAYSAT-1849
Reviewer: Ryan

Co-authored-by: ANNAPOORNA S <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants