Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

[LP#2033082] Use yaml.safe_load over yaml.load #11

Conversation

addyess
Copy link
Contributor

@addyess addyess commented Aug 25, 2023

PYyaml 6.0 finally drops yaml.load without a Loader. Switching to safe_load instead

Extra:
Modernize the integration testing

  1. Works with juju 3.1
  2. Uses vsphere-cloud-provider charm to create storage classes
  3. Using updated pytest asyncio mode auto

evilnick and others added 26 commits April 22, 2020 12:47
…port

enable jammy support for this charm
@Cynerva
Copy link

Cynerva commented Aug 25, 2023

Looks like this PR includes a bunch of commits that are in charmed-kubernetes/charm-vsphere-integrator but never made it to juju-solutions/charm-vsphere-integrator. Any chance you could separate that stuff out from this PR? It's a bit hard to review as-is

@addyess addyess closed this Aug 25, 2023
@addyess addyess deleted the bug/lp2033082/yaml-update branch August 25, 2023 19:40
@addyess addyess restored the bug/lp2033082/yaml-update branch August 25, 2023 19:40
@addyess addyess deleted the bug/lp2033082/yaml-update branch August 25, 2023 19:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants