James Falcon

User/Group keys with dashes not included in schema

2

# Bug report Due to some [unfortunate old code](https://github.com/canonical/cloud-init/blob/9bcb3dce9d7abfbb7273e313d5056eda0424108f/cloudinit/distros/ug_util.py#L114), every users/groups key in our user data allows for both underscores and dashes. For example, if we define the key as...

Remove DataSourceNone warning in cc_final_message

1

# Bug report In the past, `DataSourceNone` was used purely as a fallback for when a primary datasource failed. However, `DataSourceNone` can be a datasource in an of itself and...

Traceback when running `cloud-init collect-logs` as non-root

2

While it is correct behavior to disallow access to root read-only files, we should emit a helpful error message rather than an ugly traceback. ``` nonroot@me:/tmp$ cloud-init collect-logs version: /usr/bin/cloud-init...

test: Add v2 test coverage to test_net.py

1

## Proposed Commit Message ``` test: Add v2 test coverage to test_net.py Ensure there is a corresponding v2 test for every v1 test in `network_configs.py` where possible/reasonable. Many tests are...

Draft as currently untested. ## Proposed Commit Message ``` fix(ec2): Ensure metadata exists before configuring PBR Fixes GH-5283 ``` ## Additional Context https://github.com/canonical/cloud-init/issues/5283 ## Test Steps Currently untested ## Merge...

## Proposed Commit Message ``` See individual commits ``` ## Additional Context `tests/unittests/cmd/devel/test_logs.py` would fail if you ran the file on its own due to subp happening on import. The...

Add trace logging and refactor logs.py

5

## Proposed Commit Message ``` See individual commits ``` ## Additional Context I have functional changes coming, but since the non-functional stuff grew a bit bigger than I was expecting,...

test: pytestify and cleanup test_cc_mounts.py

2

## Proposed Commit Message ``` test: pytestify and cleanup test_cc_mounts.py Remove `unittest` constructs and remove base classes ``` ## Additional Context I did a bit more refactoring on the first...

In a v6-only environment on EC2, we attempt to reach IMDS via v4 but fail. This is expected. The v6 connection also seems to take longer to work. Both of...

The [code to list GPG keys](https://github.com/canonical/cloud-init/blob/19cb8cf926a31bef1d60a643a7f9d945f9a16591/cloudinit/gpg.py#L80) expects to receive a keyring. It calls ``` gpg --no-options --with-fingerprint --no-default-keyring --list-keys --keyring ``` That's fine when using the deprecated `/etc/apt/trusted.gpg` as `trusted.gpg`...