| === djhankb6 is now known as djhankb | ||
| === esv__ is now known as esv | ||
| quintasan | blackboxsw: holy cow, thanks, I wouldn't have figured it out myself | 13:51 |
|---|---|---|
| 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] | 21:52 |
| holmanb | faa: thanks for reporting. Is this a bug in python' | 22:01 |
| holmanb | s grp module on freebsd? | 22:02 |
| 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:04 |
| 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:08 |
| faa | https://pastebin.com/wm7i9anS log 21.2 and 22.2.2 | 22:10 |
| holmanb | faa: freebsd has no root user? what do you call it, wheel? | 22:12 |
| faa | no, file created ls -lah test-file.sh -rwxrw-r-- 1 root wheel 18B Jul 12 21:30 test-file.sh | 22:13 |
| faa | this standart cloud-init.log part, service startet root user | 22:14 |
| faa | may be cloud init set group roo by default? | 22:15 |
| faa | root | 22:15 |
| 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:16 |
| 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:18 |
| 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:19 |
| 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:21 |
| holmanb | nice, glad there's a workaround | 22:22 |
| faa | yes, thank for help | 22:24 |
| 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:27 |
| 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:28 |
| 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:38 |
| holmanb | faa: Also, thanks for relaying that to bugzilla :D | 22:39 |
| faa | done | 22:41 |
| holmanb | faa: appreciated :) | 23:29 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!