diff options
| author | andrew <andrew@web> | 2016-09-19 04:29:15 +0000 |
|---|---|---|
| committer | admin <admin@branchable.com> | 2016-09-19 04:29:15 +0000 |
| commit | e919b21b83e4d9797aa174afdb89fb86a76d4f66 (patch) | |
| tree | 6c2d09974608dcf267ad5ab20bf514054bafa1e1 /doc | |
| parent | 9094cf25e7ebd69db37ae576d0123971e7fc19bf (diff) | |
Added a comment
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/forum/Systemd_container_pre-setup_properties/comment_2_92d1deb8cb4a913d30cbc94de4177575._comment | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/doc/forum/Systemd_container_pre-setup_properties/comment_2_92d1deb8cb4a913d30cbc94de4177575._comment b/doc/forum/Systemd_container_pre-setup_properties/comment_2_92d1deb8cb4a913d30cbc94de4177575._comment new file mode 100644 index 00000000..f5ceb075 --- /dev/null +++ b/doc/forum/Systemd_container_pre-setup_properties/comment_2_92d1deb8cb4a913d30cbc94de4177575._comment @@ -0,0 +1,11 @@ +[[!comment format=mdwn + username="andrew" + subject="comment 2" + date="2016-09-19T04:29:14Z" + content=""" +I’ve made the changes you’ve suggested, but I think I’ve run into that tricky bit that you were mentioning. I think I’ve figured out how chroots work in propellor, but I’m not seeing how you were thinking of referencing these extra properties. I know that chroots fork off a new propellor instance inside the chroot which get properties via a folder/hostname lookup in the configuration, but these include the systemd container properties as well. Were you thinking of isolating the container properties and move chroot properties to `hostProperties` or just the opposite? + +[Here](https://github.com/arcticwaters/propellor/commit/605af4cdcf76e728290d242675f3fa3fea9309b5) is what I’ve done so far minus much of the mess I made in `Chroot.hs`. + +I should mention that one unintended consequence of my code is that systemd containers now only accept `DebianLike` properties. This is fine for me and an implicit condition in the code, but isn’t strictly correct. I don’t know haskell or the codebase well enough to fix this. I suppose anyone who tried to use container images which weren't debian like already encountered this issue and made changes to the code base manually. +"""]] |
