=== djhankb6 is now known as djhankb
=== esv__ is now known as esv
[13:51] <quintasan> blackboxsw: holy cow, thanks, I wouldn't have figured it out myself
[21:52] <faa> we need commiter help write_file bug in FreeBSD https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265177
[21:52] <ubottu> bugs.freebsd.org bug 265177 in Ports & Packages "net/cloud-init - fails when attempting to use write_files" [Affects Only Me, New]
[22:01] <holmanb> faa: thanks for reporting. Is this a bug in python'
[22:02] <holmanb> s grp module on freebsd?
[22:04] <holmanb> or  perhaps in os.chown()?
[22:04] <faa> unfortunately I don’t know, I just checked, if necessary I can give a more detailed log
[22:08] <holmanb> More detail would indeed be helpful. I don't think that just ignoring the group gid is likely the right solution for non-bsds, though this may work on freebsd. Cloud-init delegates distro/os-specific behavior to distro modules, so assuming this is the right solution for cloud-init in freebsd we could move implementation of this utility into distro-specific files and then update callsites accordingly.
[22:10] <faa> https://pastebin.com/wm7i9anS log 21.2 and 22.2.2
[22:12] <holmanb> faa: freebsd has no root user? what do you call it, wheel?
[22:13] <faa> no, file created ls -lah test-file.sh  -rwxrw-r--  1 root  wheel    18B Jul 12 21:30 test-file.sh
[22:14] <faa> this standart cloud-init.log part, service startet root user
[22:15] <faa> may be cloud init set group roo by default?
[22:15] <faa> root
[22:16] <holmanb> yeah, looks like the root user is assumed to be in group "root" on default. So on freebsd that should be "root:wheel", correct?
[22:16] <holmanb> s/on default/by default/
[22:18] <holmanb> faa: sidenote - for what it's worth, our test coverage for the bsds is very limited, if anybody on the freebsd side wants to make contributions in that area I bet that would be welcome.
[22:19] <holmanb> faa: this is the root group assumption I mentioned: https://github.com/canonical/cloud-init/blob/e0416cd67b423b9ae28e5edcd0db7aecdde4ecbe/cloudinit/config/cc_write_files.py#L18
[22:21] <holmanb> a quick test would be to set   owner: "root:wheel" in the config
[22:21] <faa> yes if set owner: root:wheel error message not logged
[22:22] <holmanb> nice, glad there's a workaround
[22:24] <faa> yes, thank for help
[22:27] <holmanb> faa: np, I can toss a PR up on github while it's fresh in my brain, but I'll need somebody who actually uses freebsd to review/test
[22:28] <faa> I try to help
[22:28] <holmanb> faa: also, I don't have an acct on the freebsd bugzilla, would you mind relaying the workaround we discussed there?
[22:28] <faa> always write
[22:38] <holmanb> faa: https://github.com/canonical/cloud-init/pull/1587
[22:38] <ubottu> Pull 1587 in canonical/cloud-init "freebsd: Don't assume that root user is in root group" [Open]
[22:39] <holmanb> faa: Also, thanks for relaying that to bugzilla :D
[22:41] <faa> done
[23:29] <holmanb> faa: appreciated :)