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

Remove deprecated function from module_utils/common.py #726

Conversation

abikouo
Copy link
Contributor

@abikouo abikouo commented May 23, 2024

SUMMARY

Remove deprecated functions and class from module_utils/common.py in order to prepare release 4.0.0

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

module_utils/common.py

@abikouo abikouo force-pushed the remove_features_05232024_a branch from 99375d4 to a74c4ee Compare May 23, 2024 15:12
@abikouo abikouo added the test-all-the-targets In order to tell to the integration test splitter to test all targets label May 23, 2024
Copy link

Copy link

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/f0cfbd6dd93f4be8aed35626bd4d9b84

✔️ ansible-galaxy-importer SUCCESS in 4m 14s
✔️ build-ansible-collection SUCCESS in 8m 25s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 072a080 into ansible-collections:main May 24, 2024
109 of 110 checks passed
sp3nx0r referenced this pull request in sp3nx0r/homelab Jun 22, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[kubernetes.core](https://togithub.com/ansible-collections/kubernetes.core)
| galaxy-collection | major | `3.0.1` -> `5.0.0` |

---

### Release Notes

<details>
<summary>ansible-collections/kubernetes.core (kubernetes.core)</summary>

###
[`v5.0.0`](https://togithub.com/ansible-collections/kubernetes.core/blob/HEAD/CHANGELOG.rst#v500)

[Compare
Source](https://togithub.com/ansible-collections/kubernetes.core/compare/4.0.0...5.0.0)

\======

## Release Summary

This major release drops support for `ansible-core<2.15`.

## Minor Changes

- inventory/k8s.py - Defer removal of k8s inventory plugin to version
6.0.0
([https://github.com/ansible-collections/kubernetes.core/pull/734](https://togithub.com/ansible-collections/kubernetes.core/pull/734)).

## Breaking Changes / Porting Guide

- Remove support for `ansible-core<2.15`
([https://github.com/ansible-collections/kubernetes.core/pull/737](https://togithub.com/ansible-collections/kubernetes.core/pull/737)).

###
[`v4.0.0`](https://togithub.com/ansible-collections/kubernetes.core/blob/HEAD/CHANGELOG.rst#v400)

[Compare
Source](https://togithub.com/ansible-collections/kubernetes.core/compare/3.2.0...4.0.0)

\======

## Release Summary

This major release brings several bug fixes. We have also removed
support for `ansible-core<2.15` and deprecated functions and class from
`module_utils/common.py`.

## Minor Changes

- inventory/k8s.py - Defer removal of k8s inventory plugin to version
5.0
([https://github.com/ansible-collections/kubernetes.core/pull/723](https://togithub.com/ansible-collections/kubernetes.core/pull/723)).
- k8s - The module and K8sService were changed so warnings returned by
the K8S API are now displayed to the user.

## Removed Features (previously deprecated)

- k8s - Support for `merge_type=json` has been removed in version 4.0.0.
Please use `kubernetes.core.k8s_json_patch` instead
([https://github.com/ansible-collections/kubernetes.core/pull/722](https://togithub.com/ansible-collections/kubernetes.core/pull/722)).
- k8s_exec - the previously deprecated `result.return_code` return value
has been removed, consider using `result.rc` instead
([https://github.com/ansible-collections/kubernetes.core/pull/726](https://togithub.com/ansible-collections/kubernetes.core/pull/726)).
- module_utils/common.py - the previously deprecated `K8sAnsibleMixin`
class has been removed
([https://github.com/ansible-collections/kubernetes.core/pull/726](https://togithub.com/ansible-collections/kubernetes.core/pull/726)).
- module_utils/common.py - the previously deprecated
`configuration_digest()` function has been removed
([https://github.com/ansible-collections/kubernetes.core/pull/726](https://togithub.com/ansible-collections/kubernetes.core/pull/726)).
- module_utils/common.py - the previously deprecated `get_api_client()`
function has been removed
([https://github.com/ansible-collections/kubernetes.core/pull/726](https://togithub.com/ansible-collections/kubernetes.core/pull/726)).
- module_utils/common.py - the previously deprecated `unique_string()`
function has been removed
([https://github.com/ansible-collections/kubernetes.core/pull/726](https://togithub.com/ansible-collections/kubernetes.core/pull/726)).

## Bugfixes

- Resolve Collections util resource discovery fails when complex
subresources present
([https://github.com/ansible-collections/kubernetes.core/pull/676](https://togithub.com/ansible-collections/kubernetes.core/pull/676)).
- align `helmdiff_check()` function commandline rendering with the
`deploy()` function
([https://github.com/ansible-collections/kubernetes.core/pull/670](https://togithub.com/ansible-collections/kubernetes.core/pull/670)).
- avoid unsafe conditions in integration tests
([https://github.com/ansible-collections/kubernetes.core/pull/665](https://togithub.com/ansible-collections/kubernetes.core/pull/665)).
- helm - use `reuse-values` when running `helm diff` command
([https://github.com/ansible-collections/kubernetes.core/issues/680](https://togithub.com/ansible-collections/kubernetes.core/issues/680)).
- integrations test helm_kubeconfig - set helm version to v3.10.3 to
avoid incompatability with new bitnami charts
([https://github.com/ansible-collections/kubernetes.core/pull/670](https://togithub.com/ansible-collections/kubernetes.core/pull/670)).

###
[`v3.2.0`](https://togithub.com/ansible-collections/kubernetes.core/compare/3.1.0...3.2.0)

[Compare
Source](https://togithub.com/ansible-collections/kubernetes.core/compare/3.1.0...3.2.0)

###
[`v3.1.0`](https://togithub.com/ansible-collections/kubernetes.core/blob/HEAD/CHANGELOG.rst#v310)

[Compare
Source](https://togithub.com/ansible-collections/kubernetes.core/compare/3.0.1...3.1.0)

\======

## Release Summary

This release comes with some bugfixes and documentation updates. It also
adds new features to the kubectl connection plugin and the kustomize
lookup plugin.

## Minor Changes

- kubectl - added support of local enviroment variable that will be used
for kubectl and may be requried for establishing connections ifself
([https://github.com/ansible-collections/kubernetes.core/pull/702](https://togithub.com/ansible-collections/kubernetes.core/pull/702))
- kustomize - new parameter added to --enable-helm
([https://github.com/ansible-collections/kubernetes.core/issues/568](https://togithub.com/ansible-collections/kubernetes.core/issues/568))

## Bugfixes

- helm - expand kubeconfig path with user's home directory for
consistency with k8s
- k8s_json_patch - rename action symlink to ensure k8s action plugin is
used
([https://github.com/ansible-collections/kubernetes.core/pull/652](https://togithub.com/ansible-collections/kubernetes.core/pull/652)).

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on saturday" (UTC), Automerge - At
any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/sp3nx0r/homelab).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zOTMuMCIsInVwZGF0ZWRJblZlciI6IjM3LjM5My4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9hbnNpYmxlIiwidHlwZS9tYWpvciJdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit test-all-the-targets In order to tell to the integration test splitter to test all targets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants