vyos-cloud-init.git/cloudinit/config/cc_rh_subscription.py, branch circinus-public-unmaintained

Adopt Black and isort (SC-700) (#1157)

2021-12-16T02:16:38+00:00

Applied Black and isort, fixed any linting issues, updated tox.ini and CI.

Use proper spelling for Red Hat (#778)

2021-01-20T19:50:10+00:00

The company name has two distinct words. Signed-off-by: Dan Kenigsberg

Move subp into its own module. (#416)

2020-06-08T16:49:12+00:00

This was painful, but it finishes a TODO from cloudinit/subp.py. It moves the following from util to subp: ProcessExecutionError subp which target_path I moved subp_blob_in_tempfile into cc_chef, which is its only caller. That saved us from having to deal with it using write_file and temp_utils from subp (which does not import any cloudinit things now). It is arguable that 'target_path' could be moved to a 'path_utils' or something, but in order to use it from subp and also from utils, we had to get it out of utils.

flake8: use ==/!= to compare str, bytes, and int literals

2019-01-28T15:54:47+00:00

pylint: Fix pylint warnings reported in pylint 2.0.0.

2018-07-21T15:30:58+00:00

Pylint 2.0.0 was recently released and complains more about logging-not-lazy than it used to. I've fixed those warnings, here. The changes in rh_subscription are more extensive. pylint may be complaining incorrectly there, but the tests were not correctly un-doing all of their mock/patching. This cleans those up and makes pylint happy.

pylint: pay attention to unused variable warnings.

2018-04-20T03:30:08+00:00

This enables warnings produced by pylint for unused variables (W0612), and fixes the existing errors.

Fix potential cases of uninitialized variables.

2018-01-26T20:36:30+00:00

While addressing undeclared variable in 'cloud-init status', I also fixed the errors raised by automated code reviews against cloud-init master at https://lgtm.com/projects/g/cloud-init/cloud-init/alerts The following items are addressed: * Fix 'cloud-init status': * Only report 'running' state when any stage in /run/cloud-init/status.json has a start time but no finished time. Default start time to 0 if null. * undeclared variable 'reason' now reports 'Cloud-init enabled by systemd cloud-init-generator' when systemd enables cloud-init * cc_rh_subscription.py util.subp return values aren't set during if an exception is raised, use ProcessExecution as e instead. * distros/freebsd.py: * Drop repetitive looping over ipv4 and ipv6 nic lists. * Initialize bsddev to 'NOTFOUND' in the event that no devs are discovered * declare nics_with_addresses = set() in broader scope outside check_downable conditional * cloudinit/util.py: Raise TypeError if mtype parameter isn't string, iterable or None. LP: #1744796

rh_subscription: Perform null checks for enabled and disabled repos.

2017-11-10T21:26:05+00:00

The rh_subscription module doesn't perform null checks when attempting to iterate on the enabled and disable repos arrays. When only one is specified, cloud-init fails to run.

LICENSE: Allow dual licensing GPL-3 or Apache 2.0

2016-12-22T22:04:28+00:00

This has been a recurring ask and we had initially just made the change to the cloud-init 2.0 codebase. As the current thinking is we'll just continue to enhance the current codebase, its desirable to relicense to match what we'd intended as part of the 2.0 plan here. - put a brief description of license in LICENSE file - put full license versions in LICENSE-GPLv3 and LICENSE-Apache2.0 - simplify the per-file header to reference LICENSE - tox: ignore H102 (Apache License Header check) Add license header to files that ship. Reformat headers, make sure everything has vi: at end of file. Non-shipping files do not need the copyright header, but at the moment tests/ have it.

cloudinit/config/cc_rh_subscription.py: Remove repos before adding

2016-11-22T18:22:39+00:00

A user has pointed out that upon set up of a machine, users typically remove repos (sometimes all of them) and then add repos in. This does make sense for a typical user.