cloud-init icon indicating copy to clipboard operation
cloud-init copied to clipboard

Fix some non-Linux test failures

Open TheRealFalcon opened this issue 3 months ago • 11 comments

Proposed Commit Message

Fix some non-Linux test failures

At least partially fixes GH-5336

Additional Context

With these changes (and changes to the default TMPDIR), tests pass for me on a Mac. This PR contains multiple commits and each commit should be independently reviewable. I'll leave it up to whoever merges to decide if they want to squash or not.

Test Steps

Merge type

  • [x] Squash merge using "Proposed Commit Message"
  • [ ] Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

TheRealFalcon avatar Sep 14 '25 01:09 TheRealFalcon

Bwahaha...CLA bites me too. I'll get that taken care of

TheRealFalcon avatar Sep 14 '25 01:09 TheRealFalcon

Update: we're looking into doing the company-wide CLA, so it may be a while

TheRealFalcon avatar Sep 18 '25 20:09 TheRealFalcon

Update: we're looking into doing the company-wide CLA, so it may be a while

Thanks for the update. Keep us posted!

holmanb avatar Sep 18 '25 20:09 holmanb

Update: still waiting on CLA. Mostly just commenting to stop the stale checker from flagging this again 😄

TheRealFalcon avatar Oct 17 '25 19:10 TheRealFalcon

CLA has been signed on our end, so now just waiting for whatever magic needs to happen on the Canonical side for the CLA checker to be updated.

TheRealFalcon avatar Oct 29 '25 18:10 TheRealFalcon

CLA has been signed on our end, so now just waiting for whatever magic needs to happen on the Canonical side for the CLA checker to be updated.

Woo hoo! Welcome back!

blackboxsw avatar Oct 29 '25 22:10 blackboxsw

@TheRealFalcon in prepartion for landing, a couple of hacktoberfest commits landed dropping tests/helpers.py::TestCase which resulted in a merge conflict. I'll make sure I touch base with Canonical CLA folks to ensure the CLA signing is permitted now for your company so we can clear the CI workflow.

blackboxsw avatar Oct 29 '25 22:10 blackboxsw

Noted, I'll do the rebase once the CLA checker says I'm good. I've just been manually curling it to see.

TheRealFalcon avatar Oct 30 '25 15:10 TheRealFalcon

Thanks!

the CLA job hasn't passed yet

Still waiting on this unfortunately. I believe we submitted it a few weeks ago. Is there something stuck on the Canonical side?

TheRealFalcon avatar Nov 20 '25 22:11 TheRealFalcon

Thanks!

the CLA job hasn't passed yet

Still waiting on this unfortunately. I believe we submitted it a few weeks ago. Is there something stuck on the Canonical side?

I've copied internal folks here saying we're blocked with "external" contributors having signed the CLA. My visibility here is that legal has been asked about this directly and they CLA-folks are awaiting a response. I'll bump this topic again so we can unblock.

blackboxsw avatar Nov 20 '25 23:11 blackboxsw

sigh looks like there's still more to sort on our end. Commenting again to keep the auto-closer quiet

TheRealFalcon avatar Dec 03 '25 21:12 TheRealFalcon