Thursday, 2017-03-23

openstackgerritMerged openstack/diskimage-builder master: Fix typo in CMDLINE env var for bootloader  https://review.openstack.org/44850800:02
ianwmmedvede: do you run ubuntu or debian?00:28
mmedvedeianw: Ubuntu00:28
ianwmmedvede: ok, i'm just seeing if we can do a bit better testing these bootloader elements ... change forthcoming00:29
openstackgerritIan Wienand proposed openstack/diskimage-builder master: functests: skip qcow2 generically but add specific test  https://review.openstack.org/44883700:57
ianwmmedvede: ^ ok, actually hoping that fails ppc CI :)00:57
ianwthat rolls in an old change00:57
ianwhopefully we both speed up ci and get better coverage all at once00:58
mmedvedeianw: oh, so you only enable it on xenial, that is why you asked? I thought you asked what we use in CI in general00:59
mmedvedeianw: the ppc job currently whitelists only debian01:00
openstackgerritMerged openstack/diskimage-builder master: Capture output in _exec_sudo  https://review.openstack.org/44881001:01
ianwahh, ok, we could do it for debian i guess01:02
mmedvedeianw: I can just switch it to xenial01:02
mmedvedethat probably is better01:02
mmedvedecan't use both at the same time01:02
ianwi'm just thinking because that's what we use in the gate01:02
ianwso has the most eyes on it01:02
openstackgerritMerged openstack/diskimage-builder master: Add default PPC block-device layout  https://review.openstack.org/44881101:03
mmedvedeI am rechecking https://review.openstack.org/#/c/44883701:07
mmedvedeianw: it is failing on ^ as expected!01:11
ianwmmedvede: that's a good sign :)  i don't think testing the qcow conversion path over and over again was helping anything01:15
mmedvedeianw: so it does not find the block-device-ppc64el.yaml01:15
mmedvedeI have no opinion on testing qcow conversion in functional tests :)01:16
ianwohhh, doh, i don't think i meant to add that01:19
mmedvedeianw: right, the ARCH is ppc64el, and file is named as if it is 'ppc'01:19
ianwheh, yeah i didn't mean for that to get into that change01:20
mmedvedeyou did not mean to add the 10-partitoining change?01:20
ianwnope01:23
ianwaccidentally staged it01:23
openstackgerritIan Wienand proposed openstack/diskimage-builder master: functests: skip qcow2 generically but add specific test  https://review.openstack.org/44883701:24
mmedvedeianw: do you think it is worth adding distro url autodetection for Ubuntu into dib? I think I can use DIB_UBUNTU_DISTRIBUTION_MIRROR on ppc to be able to test both debian and ubuntu. But maybe it should be done automatically for known arches01:28
mmedvedeI guess there is a fine line there on how much should be left as an exercise to the end user01:30
ianwumm, yeah, i feel like it's probably ok ...01:34
ianwbut if users want to use specific mirrors etc, they will have to figure that out themselves a bit01:34
ianwbut it seems reasonable to choose the ports mirror when arch is set01:35
*** pmannidi_ has joined #openstack-dib01:40
*** pmannidi has quit IRC01:43
openstackgerritMikhail S Medvedev proposed openstack/diskimage-builder master: Use correct Ubuntu distro url on non-x86 arches  https://review.openstack.org/44884801:57
*** owalsh has quit IRC02:20
*** owalsh has joined #openstack-dib02:21
openstackgerritIan Wienand proposed openstack/diskimage-builder master: functests: skip qcow2 generically but add specific test  https://review.openstack.org/44883702:49
*** pmannidi_ has quit IRC04:11
*** aparnav has joined #openstack-dib04:23
*** pmannidi_ has joined #openstack-dib04:28
*** pmannidi_ has quit IRC04:43
*** pmannidi_ has joined #openstack-dib05:00
*** venki has quit IRC05:32
*** andreas-f has joined #openstack-dib05:41
*** venki has joined #openstack-dib05:47
*** venki has quit IRC05:54
*** venki has joined #openstack-dib06:07
*** appu103 has joined #openstack-dib06:09
openstackgerritIan Wienand proposed openstack/diskimage-builder master: [DNM] Try symlinks for grub-install on PPC  https://review.openstack.org/44891406:10
openstackgerritIan Wienand proposed openstack/diskimage-builder master: [DNM] Try symlinks for grub-install on PPC  https://review.openstack.org/44891406:12
*** aparnav has quit IRC06:13
*** venki has quit IRC06:18
yolandahi, so i finally don't need an isolated /boot partition, at least for my security use case. But enabling that possibility is good06:36
*** venki has joined #openstack-dib06:37
yolandai need partitions for /var, /var/log, /var/log/audit, /tmp06:37
*** venki has quit IRC06:41
prometheanfireyolanda: seperate /usr kthnx :D06:43
prometheanfirelets bring up that debate again06:43
openstackgerritAndreas Florath proposed openstack/diskimage-builder master: Use stevedore for module config of block device  https://review.openstack.org/44709006:44
*** venki has joined #openstack-dib06:55
*** jasondotstar has quit IRC07:03
openstackgerritgecong proposed openstack/diskimage-builder master: Typo fix: curent => current  https://review.openstack.org/44896607:03
*** jasondotstar has joined #openstack-dib07:08
yolandaprometheanfire, you need separate /usr? this can be done with all the changes in block device07:24
yolandaprometheanfire, from my side i didn't get /usr partition as a security requirement, you need it for security or for something else?07:24
*** venki has quit IRC08:08
*** venki has joined #openstack-dib08:13
*** pmannidi_ has quit IRC08:13
ianwyolanda: that seems good ... though yeah we kind of have to fiddle things for this ppc issue anyway08:13
ianwwouldn't you need a unencrypted /boot for encrypted root applications?08:14
*** Administrator_ has quit IRC08:20
*** Administrator_ has joined #openstack-dib08:21
yolandai got requests for an encrypted home so far08:39
yolandanot encrypting the whole system08:39
*** makowals_ has quit IRC09:00
*** makowals has joined #openstack-dib09:01
*** openstackgerrit has quit IRC09:03
*** hashar has joined #openstack-dib09:03
*** openstackgerrit has joined #openstack-dib09:11
openstackgerrityolanda.robla proposed openstack/diskimage-builder master: Refactor: block-device filesystem creation, mount and fstab  https://review.openstack.org/44458609:11
openstackgerrityolanda.robla proposed openstack/diskimage-builder master: Add lvm management to diskimage-builder  https://review.openstack.org/44440309:13
*** venki has quit IRC09:30
*** venki has joined #openstack-dib09:38
*** venki has quit IRC10:11
*** makowals has quit IRC10:38
*** makowals has joined #openstack-dib10:43
*** appu103 has quit IRC11:15
prometheanfireyolanda: I was joking, but seperate /boot would be nice13:44
prometheanfireseperate /var /opt /home /tmp (and possibly /usr ) is nice though13:45
prometheanfiremounting with nodev13:45
yolandawell yes, support for any different mountpoints, with different flags, is covered13:50
yolanda /boot is a particular use case, because bootloader element needs to be refactored13:50
*** aparnav has joined #openstack-dib14:18
*** aparnav has quit IRC14:27
*** aparnav has joined #openstack-dib14:32
openstackgerritMikhail S Medvedev proposed openstack/diskimage-builder master: Use correct Ubuntu distro url on non-x86 arches  https://review.openstack.org/44884814:34
openstackgerrityolanda.robla proposed openstack/diskimage-builder master: Refactor: block-device filesystem creation, mount and fstab  https://review.openstack.org/44458615:45
*** aparnav has quit IRC17:03
*** hashar has quit IRC17:58
openstackgerritAndreas Florath proposed openstack/diskimage-builder master: Refactor: block-device filesystem creation, mount and fstab  https://review.openstack.org/44458619:17
yolandahi andreas-f , ianw , i started testing on ironic of my whole disk images, but getting that error20:13
yolandanstanceDeployFailure: Deploy failed for instance 8322306c-a748-4a43-aad7-6000b995fa61. Error: Failed to check the number of primary partitions present on /dev/disk/by-path/ip-192.168.24.9:3260-iscsi-iqn.2008-10.org.openstack:c0c6a5bf-cc82-45df-99c3-30b75b8f57d4-lun-1 for node c0c6a5bf-cc82-45df-99c3-30b75b8f57d4. Error: The device /dev/disk/by-path/ip-192.168.24.9:3260-iscsi-iqn.2008-10.org.openstack:c0c6a5bf-cc82-45df-99c3-30b75b8f57d4-lun-1 does20:13
yolandanot have a valid MBR partition table20:13
yolandathe image is booting ok with virt-install, but fails on ironic20:13
yolandaso need to debug more20:13
*** hashar has joined #openstack-dib20:30
yolandaandreas-f, ianw , so i saw the issue. I cannot define the env vars on my deploy, because it was giving conflicts with nested yamls. Instead, i created an element, with the hope to provide the right env var: https://review.openstack.org/44912220:36
yolandahowever, it doesn't seem to be applied properly, maybe the env var declared too late20:36
yolandahow can i pass the BLOCK_DEVICE_CONFIG properly from an element, is that possible?20:37
ianwyolanda: hmm ... that seems to be one of the old issues of passing things back from elements -> main.  env doesn't work20:53
yolandai saw that vm is just defining a block-device-config.yaml instead20:55
yolandaso trying the same20:55
andreas-fyolanda: please have a look at https://review.openstack.org/#/c/444586/7/diskimage_builder/lib/common-functions line 357ff how the block-device-config.yaml is used in my patch.21:02
andreas-fThere is a patch from ian which does something very similar: https://review.openstack.org/#/c/448811/21:02
andreas-fyolanda: I'm confused about your earlier statement (9:13) that the boot loader is missing an MBR. I thought only a file system image was needed for ironic?21:04
openstackgerritAndreas Florath proposed openstack/diskimage-builder master: Refactor: block-device filesystem creation, mount and fstab  https://review.openstack.org/44458621:07
*** pmannidi has joined #openstack-dib22:15
*** hashar has quit IRC23:07
ianwmmedvede: hmm, what's the deal with the ppc failures on the ports patch -> http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/diskimage-builder/48/448848/2/check/dib-dsvm-functests-devstack-xenial/d665374/23:52
ianwi'd really like to get the master tree into the state where the only bit failing for ppc is the bootloader changes23:52
ianwahh, it's passed now.  i guess just an old mirror failure23:56
ianwexcellent, so xenial-build works there, for now ... we should merge the bootloader test for that and then it will start failing.  then we can actually fix it23:56
mmedvedeianw: yeah, I have seen a few of the mirror failures recently23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!