Friday, 2019-09-27

« Thursday, 2019-09-26 Index Saturday, 2019-09-28 »
*** gyee has quit IRC00:05
*** dsneddon has joined #openstack-ironic00:11
*** BjoernT has joined #openstack-ironic00:14
*** kaifeng|afk is now known as kaifeng00:41
kaifengmorning ironic00:46
*** mkrai has joined #openstack-ironic00:49
*** chihhsin has quit IRC00:50
*** rloo has quit IRC00:57
*** rloo has joined #openstack-ironic00:57
kaifengrpittau: thanks, so it's an external dependency by the sphinx extension01:04
*** kiseok7 has joined #openstack-ironic01:13
*** dsneddon has quit IRC01:16
*** dsneddon has joined #openstack-ironic01:17
*** dsneddon has quit IRC01:22
*** Goneri has joined #openstack-ironic01:30
*** mkrai has quit IRC01:37
*** Goneri has quit IRC01:39
*** tzumainn has quit IRC01:44
*** dsneddon has joined #openstack-ironic01:55
*** rh-jelabarre has quit IRC01:58
*** Goneri has joined #openstack-ironic02:13
*** mkrai has joined #openstack-ironic02:13
*** rloo has quit IRC02:23
*** BjoernT has quit IRC02:38
*** dsneddon has quit IRC02:42
*** dsneddon has joined #openstack-ironic02:45
*** Goneri has quit IRC02:47
*** ricolin has joined #openstack-ironic02:49
*** mkrai has quit IRC02:51
*** gregoryo has joined #openstack-ironic03:10
*** kiseok7 has quit IRC03:23
*** binhnd108 has joined #openstack-ironic03:47
binhnd108hi everyone03:48
binhnd108i have an issue with supermicro server and software raid with ironic03:48
binhnd108config software raid with ironic so ok03:48
binhnd108but when i launch the instance with whole disk ubuntu 18.04, the server boot hang at booting from san device 0x8003:50
*** dsneddon has quit IRC03:50
binhnd108is there anyone have the same issue?03:50
*** dsneddon has joined #openstack-ironic03:57
*** dsneddon has quit IRC04:02
*** igordc has quit IRC04:08
*** mkrai has joined #openstack-ironic04:17
*** dsneddon has joined #openstack-ironic04:38
*** whoami-rajat has joined #openstack-ironic05:18
*** ociuhandu has joined #openstack-ironic05:31
*** ociuhandu has quit IRC05:35
*** Guest29634 has quit IRC06:16
*** gregoryo has quit IRC06:16
openstackgerritMerged openstack/ironic-inspector master: Don't use configuration options in function signatures  https://review.opendev.org/68508406:21
*** rcernin has quit IRC06:29
arne_wiebalckGood morning, ironic!06:32
*** benj has joined #openstack-ironic06:32
*** benj is now known as Guest5786106:32
openstackgerritMerged openstack/ironic master: Run DIB with tracing enabled and increase the DHCP timeout  https://review.opendev.org/68476506:34
arne_wiebalckbinhnd108: I haven't seen this issue, but is it possible that your server is trying to boot from a device on the network, rather than the local drive where you set up the software RAID?06:42
*** e0ne has joined #openstack-ironic06:45
*** e0ne has quit IRC06:50
*** rpittau|afk is now known as rpittau07:05
rpittaugood morning ironic! o/07:05
rpittauhappy Friday!07:05
rpittaukaifeng: yes, it's an external package that is part of an svg rendering library that contains a tool to convert svg into raster images07:07
*** tesseract has joined #openstack-ironic07:20
*** pcaruana has joined #openstack-ironic07:21
*** ociuhandu has joined #openstack-ironic07:27
*** ociuhandu has quit IRC07:28
kaifengmorning arne_wiebalck, rpittau o/07:41
kaifengrpittau: i didn't build pdf locally since the svg converter switch to the one from the doc team, it's all done at the ci07:41
kaifengotherwise i should be aware of it07:42
*** priteau has joined #openstack-ironic07:43
rpittaukaifeng: yeah, in deb-based distro the binary is integrated directly in the lib package, that's why it gave that issue in fedora07:45
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Limit lower-constraints.txt to only direct dependencies  https://review.opendev.org/68470807:47
kaifenghmm, that's a rpm spec issue07:49
kaifengmissing the necessary dependency07:49
*** alexmcleod has joined #openstack-ironic07:51
*** Emine has joined #openstack-ironic07:53
*** S4ren has joined #openstack-ironic07:59
*** ociuhandu has joined #openstack-ironic08:02
mgoddardmorning08:07
*** ociuhandu has quit IRC08:07
openstackgerritMerged openstack/networking-baremetal master: Add versions to release notes series  https://review.opendev.org/68508508:08
*** priteau has quit IRC08:09
openstackgerritMerged openstack/ironic-ui master: Update master for stable/train  https://review.opendev.org/68516808:10
openstackgerritMerged openstack/ironic-ui stable/train: Update .gitreview for stable/train  https://review.opendev.org/68516608:10
openstackgerritMerged openstack/ironic-ui stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68516708:12
arne_wiebalckhey kaifeng rpittau mgoddard o/08:12
rpittauhi arne_wiebalck mgoddard :)08:12
openstackgerritMerged openstack/ironic-prometheus-exporter master: Add versions to release notes series  https://review.opendev.org/68508708:17
openstackgerritMerged openstack/bifrost stable/train: Update .gitreview for stable/train  https://review.opendev.org/68512308:23
openstackgerritMerged openstack/bifrost stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68512408:23
kaifengmorning mgoddard o/08:24
mgoddardmorning arne_wiebalck rpittau & kaifeng08:24
*** derekh has joined #openstack-ironic08:33
openstackgerritMark Goddard proposed openstack/bifrost master: Add versions to release notes series  https://review.opendev.org/68523508:41
openstackgerritMerged openstack/python-ironic-inspector-client master: Recover the functional test  https://review.opendev.org/68509908:41
openstackgerritMerged openstack/python-ironic-inspector-client master: Add versions to release notes series  https://review.opendev.org/68508308:41
openstackgerritMark Goddard proposed openstack/ironic master: Remove old online migrations and new models  https://review.opendev.org/68509108:44
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui stable/train: Imported Translations from Zanata  https://review.opendev.org/68523708:48
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.opendev.org/68523808:49
*** pcaruana has quit IRC08:52
*** rcernin has joined #openstack-ironic08:53
*** pcaruana has joined #openstack-ironic09:03
openstackgerritMerged openstack/ironic-lib master: Move ironic-lib.filters to etc/ironic/rootwrap.d  https://review.opendev.org/68477109:04
openstackgerritMerged openstack/ironic master: Remove earliest version from releasing docs, update examples  https://review.opendev.org/68507609:04
openstackgerritMerged openstack/ironic stable/train: Update docs for train release  https://review.opendev.org/68507409:04
openstackgerritMerged openstack/python-ironicclient master: Add versions to release notes series  https://review.opendev.org/68508109:04
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client stable/train: Recover the functional test  https://review.opendev.org/68525509:06
openstackgerritMerged openstack/virtualbmc master: Update the constraints url  https://review.opendev.org/68416309:13
*** dtantsur|afk is now known as dtantsur09:18
dtantsurmorning ironic09:19
dtantsurmgoddard: seems that we can proceed with inspector finally09:19
mgoddarddtantsur: \o/09:20
mgoddardI'll get on it09:20
dtantsurmgoddard: if you approve https://review.opendev.org/#/c/685059/, we can proceed with IPA-builder as well09:20
patchbotpatch 685059 - ironic-python-agent-builder - Recover iso-image-create script - 1 patch set09:20
dtantsurI think other outstanding patches can wait for post-1.0.009:21
openstackgerritMark Goddard proposed openstack/ironic-python-agent-builder master: Recover iso-image-create script  https://review.opendev.org/68505909:21
dtantsurah, nice one09:22
*** rpittau is now known as rpittau|bbl09:24
*** ricolin has quit IRC09:24
*** dtantsur is now known as dtantsur|lunch09:24
*** e0ne has joined #openstack-ironic09:25
*** iurygregory has joined #openstack-ironic09:28
*** mkrai has quit IRC09:32
openstackgerritMerged openstack/ironic-python-agent-builder master: Recover iso-image-create script  https://review.opendev.org/68505909:41
kaifengbye guys, have a nice weekend o/09:43
*** kaifeng has quit IRC09:43
*** Guest57861 has quit IRC09:44
*** e0ne has quit IRC09:45
*** benj has joined #openstack-ironic09:47
*** benj is now known as Guest9408309:48
mgoddarddtantsur|lunch: 1.0.0 for IPAB?09:48
openstackgerritMerged openstack/networking-generic-switch master: Update master for stable/train  https://review.opendev.org/68517109:49
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Add versions to release notes series  https://review.opendev.org/68530309:51
*** mkrai has joined #openstack-ironic09:53
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Fix a bug for job creation when only required  https://review.opendev.org/68237610:17
openstackgerritMark Goddard proposed openstack/ironic-tempest-plugin master: Add CI jobs for Train stable branches  https://review.opendev.org/68531110:22
*** rachit7 has joined #openstack-ironic10:23
*** iurygregory has quit IRC10:27
*** iurygregory has joined #openstack-ironic10:27
openstackgerritMark Goddard proposed openstack/ironic-tempest-plugin master: Add CI jobs for Train stable branches  https://review.opendev.org/68531110:30
*** tesseract has quit IRC10:35
*** tesseract has joined #openstack-ironic10:35
*** S4ren has quit IRC10:45
*** dtantsur|lunch is now known as dtantsur11:03
dtantsurmgoddard: yeah, I think it makes sense11:03
*** iurygregory has quit IRC11:03
dtantsurTheJulia: something seems quite broken around installing bootloader https://23ac9a902a14c73a5e88-5cb3754a5639f7be0289e9ab84888d65.ssl.cf5.rackcdn.com/685100/1/check/metalsmith-integration-glance-localboot-centos7/290559a/controller/ironic-bm-logs/node-0_no_ansi_2019-09-26-16:12:41.log11:13
*** mkrai has quit IRC11:13
dtantsurrechecking to see if it's a regression or just a temporary issue11:13
*** mkrai has joined #openstack-ironic11:13
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add a CI job with a DIB-built ramdisk  https://review.opendev.org/68430411:21
*** S4ren has joined #openstack-ironic11:22
dtantsurblkid returns a UUID on the conductor side, but no UUID in the output of lsblk later on the ramdisk11:26
dtantsur"fun"11:26
*** gaotan has quit IRC11:29
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Try calling partprobe before installing bootloader  https://review.opendev.org/68532411:34
*** jhesketh has quit IRC11:34
dtantsurdunno, maybe ^^11:34
*** jhesketh has joined #openstack-ironic11:35
*** dhellmann_ has joined #openstack-ironic11:37
*** dhellmann has quit IRC11:37
*** dhellmann_ is now known as dhellmann11:37
openstackgerritPradip Kadam proposed openstack/ironic master: DRAC: Fix a bug for clear_job_queue clean step with non-BIOS pending job  https://review.opendev.org/68316111:45
*** mkrai has quit IRC11:46
*** ociuhandu has joined #openstack-ironic11:47
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.opendev.org/68523811:48
*** mkrai has joined #openstack-ironic11:55
openstackgerritMerged openstack/sushy master: Add versions to release notes series  https://review.opendev.org/68507811:59
*** e0ne has joined #openstack-ironic12:04
*** dsneddon has quit IRC12:09
*** dsneddon has joined #openstack-ironic12:09
*** bnemec has quit IRC12:10
*** bnemec has joined #openstack-ironic12:11
*** rloo has joined #openstack-ironic12:13
*** goldyfruit_ has quit IRC12:14
*** rh-jelabarre has joined #openstack-ironic12:21
*** rh-jelabarre has quit IRC12:23
*** rh-jelabarre has joined #openstack-ironic12:23
*** bobmel has quit IRC12:24
*** bobmel has joined #openstack-ironic12:29
*** iurygregory has joined #openstack-ironic12:31
*** bobmel has quit IRC12:33
*** rpittau|bbl is now known as rpittau12:36
*** ociuhandu has quit IRC12:37
*** ociuhandu has joined #openstack-ironic12:41
*** mkowalski has quit IRC12:46
*** mkowalski has joined #openstack-ironic12:46
*** mkrai has quit IRC12:55
openstackgerritMerged openstack/ironic-ui stable/train: Imported Translations from Zanata  https://review.opendev.org/68523712:59
*** lucasagomes has joined #openstack-ironic13:02
*** mkrai has joined #openstack-ironic13:02
*** tkajinam has joined #openstack-ironic13:05
*** pcaruana has quit IRC13:09
*** rcernin has quit IRC13:18
*** tzumainn has joined #openstack-ironic13:18
TheJuliagood morning13:20
*** BjoernT has joined #openstack-ironic13:21
TheJuliadtantsur: ugh... cursed we are13:21
dtantsurindeed13:21
dtantsurmorning TheJulia13:21
* TheJulia makes coffee13:21
TheJuliaGood morning everyone13:21
*** goldyfruit_ has joined #openstack-ironic13:22
etingofo/ TheJulia13:22
openstackgerritMark Goddard proposed openstack/ironic master: Remove old online migrations and new models  https://review.opendev.org/68509113:23
*** ociuhandu has quit IRC13:25
iurygregorymorning TheJulia o/13:26
goldyfruit_Morning guys13:28
dtantsurhttps://review.opendev.org/#/c/685324/ doesn't help.. thinking more13:28
patchbotpatch 685324 - ironic-python-agent - [WIP] Try calling partprobe before installing boot... - 1 patch set13:28
*** ricolin has joined #openstack-ironic13:30
*** gaotan has joined #openstack-ironic13:33
*** gaotan has left #openstack-ironic13:34
*** gaotan has joined #openstack-ironic13:34
*** mkrai has quit IRC13:41
*** sthussey has joined #openstack-ironic13:45
tzumainnhi! random question: is it possible to set the instance_info image_source parameter to a glance image UUID, run 'openstack baremetal node deploy' and just have things magically work? I saw some things in the documentation that made it seem like maybe...? but I wasn't sure13:46
dtantsurtzumainn: there may be a few more things to set. if you're running without nova, you can give https://docs.openstack.org/metalsmith/latest/ a try13:52
dtantsurit has a more human-friendly CLI13:52
dtantsura recent example I used on my devstack: metalsmith deploy --resource-class baremetal --image CentOS7 --network private --ssh-public-key ~/.ssh/id_rsa.pub --root-size 10 --hostname my-node.example.com13:52
dtantsurfor a whole disk image and without using neutron, setting image_source should be enough13:53
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID  https://review.opendev.org/67909313:57
* TheJulia starts sorting through email13:57
TheJuliatzumainn: image_source should work just fine with a uuid if memory serves.. :)13:58
TheJuliatzumainn: that being said, that glance image has to be properly configured13:58
dtantsurand partition images require root_gb13:59
TheJulia++13:59
*** goldyfruit_ has quit IRC13:59
*** goldyfruit_ has joined #openstack-ironic14:00
*** goldyfruit_ has quit IRC14:00
dtantsurTheJulia: do you have any opinion on https://bugzilla.redhat.com/show_bug.cgi?id=1751918 ?14:01
openstackbugzilla.redhat.com bug 1751918 in openstack-ironic-python-agent "Openstack qcow2 images create poor filesystem layouts when installed" [High,New] - Assigned to rhos-maint14:01
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID  https://review.opendev.org/67909314:01
dtantsurif true, it seems a downside of the image-based approach14:02
dtantsurwhich, as the disks grow, may make advanced partitioning more important14:02
tzumainndtantsur, TheJulia thanks! I'll take a look at that doc14:02
*** nick_kar has quit IRC14:02
*** pcaruana has joined #openstack-ironic14:04
etingofrpioso, cleaning finally passes like a charm \o/14:04
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID  https://review.opendev.org/67909314:04
clarkbdtantsur: the infra team sets journal sizes for ext4 on our images to match the target disk sizerather than build size. Basically you need to preprep the image fs for that14:06
dtantsurclarkb: yeah, it's probably the one-size-doesnt-fit-all case14:06
dtantsurrealistically, we should be able to keep root small and make ironic allocate /srv or whatever is needed14:06
dtantsurbut we're not really close to that yet14:06
* dtantsur wonders if clarkb actually reads all channels or just has a lot of highlight words :)14:07
clarkbI defibitely dont read them all, byt it is morning tea time so easy to catach up here and there14:08
dtantsurI see :)14:08
*** TxGirlGeek has joined #openstack-ironic14:13
NobodyCamGood Morning Ironic Folks,14:15
NobodyCamand OFC14:15
NobodyCamTGIF!14:15
rpittauhi NobodyCam :)14:15
iurygregoryMorning NobodyCam o/14:15
NobodyCamGood Morning rpittau and iurygregory :) o/14:16
dtantsurmorning NobodyCam14:19
dtantsuroh yeah, TGIF14:19
NobodyCamhey hey dtantsur :) Good Morning :)14:19
*** munimeha1 has joined #openstack-ironic14:24
*** tkajinam has quit IRC14:24
*** jtwill98 has joined #openstack-ironic14:30
*** rachit7 has quit IRC14:33
*** pcaruana has quit IRC14:35
*** _mmethot_ has quit IRC14:50
*** BjoernT has quit IRC14:53
*** whoami-rajat has quit IRC14:54
*** BjoernT has joined #openstack-ironic14:56
* TheJulia finally seems to be done with emails/calls and looks at the bug15:04
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector stable/train: Update .gitreview for stable/train  https://review.opendev.org/68536915:06
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68537015:07
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/train  https://review.opendev.org/68537115:07
TheJuliadtantsur: so looking at that bug filing. Yeah, it is one size doesn't fit all kind of thing. The only real "fix" as you suggested is build your own, or we stop shipping filesystem images and we move to something like tar files with logic to support putting the bootloader in place and finishing the install15:09
dtantsurtar files.. will keep us in the business of knowing about filesystems etc15:10
TheJuliayup15:10
dtantsurif we go down that path, I'd surrender to kickstart15:10
TheJuliaNot really a fan of that and having OS specific stuff.15:10
TheJuliawell, technically someone can do that partially now15:10
*** mmethot has joined #openstack-ironic15:11
TheJuliathe extra pxe parameter pass through :)15:11
dtantsurright15:11
dtantsurdid you have a chance to run the idea of dropping partition images through the operators?15:11
TheJuliaI'd like to see someone doing that to solve their use case15:11
TheJuliadtantsur: no, they... didn't really feel up to talking about baremetal15:11
dtantsurI see15:12
dtantsuron the other hand, people shouldn't have 1.8T root partition15:12
TheJuliaBasically favorable view of ironic, some have legacy deployment systems for $reasons that they still have to use, but want them gone15:12
TheJuliayeah15:12
dtantsurit's only because we don't have a way to provide advanced partitioning15:12
dtantsurwe should at least make deploy steps work in-band :)15:13
TheJulia++15:13
* TheJulia sighs at moved meeting15:13
dtantsurTheJulia: will you be able to chair the next meeting? I have no idea how my Monday will look like15:14
dtantsur(likely messed up)15:14
*** dtantsur is now known as dtantsur|afk15:20
dtantsur|afkanyway, have a great weekend, all15:20
TheJuliadtantsur|afk:  I believe I board an aircraft during the meeting :(15:21
*** arxcruz|rover is now known as arxcruz|zzz15:22
*** gyee has joined #openstack-ironic15:30
*** pcaruana has joined #openstack-ironic15:36
*** alexmcleod has quit IRC15:39
arne_wiebalckxXraphXx: I have a new image which I try to use with s/w RAID. This new image does not have the root fs in the 1st partition anymore. I remembered you have submitted  https://review.opendev.org/#/c/674879/ to handle this, but it does not seem to work for me: 'No partition with UUID 0x00000000 found on device /dev/md126' ... before I look more into this: is there sth needed in addition?15:46
patchbotpatch 674879 - ironic-python-agent - image extension, install_bootloader improvements - 10 patch sets15:46
rpittauhave a great weekend everyone! o/15:48
*** rpittau is now known as rpittau|afk15:48
xXraphXxarne_wiebalck, hum no nothing more needed than usual15:58
arne_wiebalckxXraphXx: I guess the partition does not have UUID 0x0000 ... need to check where this comes from.16:02
arne_wiebalckxXraphXx: Or where it should be filled.16:02
arne_wiebalckxXraphXx: Did you explicitly specify this anywhere (or the root device itself, e.g. via root device hints) when testing?16:03
xXraphXxchecking16:03
*** e0ne has quit IRC16:04
*** mriedem has joined #openstack-ironic16:05
mriedemhi ironicites,16:05
mriedemso nova broke ironic serial console support in train and backported that down through rocky16:06
mriedemmelwitt has a fix here https://review.opendev.org/#/c/685194/16:06
patchbotpatch 685194 - nova - Reduce scope of 'path' query parameter to noVNC co... - 2 patch sets16:06
mriedembut looking for someone to actually verify the fix since we don't have an ironic setup to test that and it's not verified in the gate (obviously)16:06
mriedemis there anyone around that has an env handy that could test that?16:06
xXraphXxarne_wiebalck right -> when I called the agent function I specified the rootfs uuid as done in the last call in the story description here16:09
xXraphXxhttps://storyboard.openstack.org/#!/story/200637916:10
*** Emine has quit IRC16:10
* arne_wiebalck checking story16:10
arne_wiebalckhmm, I see16:12
xXraphXxcode snippet where i use it: https://paste.ubuntu.com/p/yQ3WpN6mQv/16:12
arne_wiebalckhow does this work in the general case, i.e. when there is no s/w RAID, where does the UUID come from?16:13
arne_wiebalckI think it's the conductor triggering the bootloader installation ... I'll need to check where it gets the ID from ...16:14
arne_wiebalckpassing the rootfs uuid is not meant to be used with whole disk images (as these have the bootloader already installed)16:17
arne_wiebalckatm (and for s/w RAID), the uuid would come from the node's internal driver info16:21
openstackgerritIlya Etingof proposed openstack/ironic master: Fix drive sensors collection in `redfish` mgmt interface  https://review.opendev.org/68539816:22
*** hwoarang has quit IRC16:25
*** hwoarang has joined #openstack-ironic16:25
*** bdodd has joined #openstack-ironic16:29
*** tesseract has quit IRC16:32
TheJuliaarne_wiebalck: I'm not sure anything else is needed in regards to that, but I guess I'm going to have to have some time where I can spend and wrap my brain around that entire path soon :(16:32
arne_wiebalckfrom what I see the conductor explicitly sets the UUID of the partition for the config drive, so if the conductor should know (or could set) the UUID for the deploy partition and pass this on to the IPA16:33
arne_wiebalckTheJulia: Thanks, not super urgent, we can work around this for now :)16:34
TheJuliait knows it to support network booting the entire thing16:35
arne_wiebalckfrom what I see == what I think when I see a config drive UUID that is "2019-09-26-15-16-15-00"16:35
TheJulianice!16:36
arne_wiebalckTheJulia: I think one issue here is that except for s/w RAID this info does not need to be passed for whole disk images ...16:37
arne_wiebalckTheJulia: So, if it knows we can probably use it somehow ...16:37
* arne_wiebalck hears some evil laughter16:38
TheJuliahehe16:38
arne_wiebalckTheJulia: Setting it is probably not a good idea, as the conductor does not know when dealing with a s/w RAID at the moment it creates the partitions ... it could though ...16:39
xXraphXxarne_wiebalck, on my side i do not use it for non softraid (since as you said not needed for whole disk images), but i store it in the glance image metadata so that they end up in the node instance_info (since the rootfs uuid will depend on the chosen image)16:39
xXraphXxinstead of the driver info16:39
xXraphXxfor it to be available whenever needed (for softraid, for example)16:40
arne_wiebalckisn't there a difference between the fs UUID and the device UUID?16:41
* arne_wiebalck is probably confused16:41
xXraphXxyes there is but what needs to be passed to the function is the fs uuid16:41
arne_wiebalckwhat the IPA searches is the device UUID (as given by lsblk, no?)16:41
arne_wiebalckso what lsblk returns is the fs UUID?16:42
* TheJulia thinks we need flow diagrams or something16:43
* arne_wiebalck or some drinks16:43
TheJulialikely both16:44
arne_wiebalckif this uuid is the fs uuid which comes from the image, then we can do what xXraphXx does and grab it from the instance info16:44
* arne_wiebalck never thought about combining flow charts with drinks16:45
xXraphXxat least currently, this is what the install_bootloader extension in the ipa expects (fs uuid as the docstring says), so i used it this way. But it might be worth reconsidering this indeed16:46
arne_wiebalckI think this is correct: I just checked two instances and lsblk reports the *same* UUID for the devices, so it reports the fs UUID (which comes from the image).16:52
arne_wiebalckSo, the conductor could get this from the instance and pass it to the IPA which should be able to find it.16:53
arne_wiebalckSo, the conductor could get the (fs) UUID he instance and pass it to the IPA which should be able to find the corresponding partition.16:54
TheJuliacorrect, I think16:54
arne_wiebalckxXraphXx: I think this ^ is what you suggested / do, right?16:54
xXraphXxyep16:54
arne_wiebalcknice16:55
arne_wiebalckbut then the aforementioned patch is not enough to find the right partition, we need to add the logic to get it from the instance16:55
* arne_wiebalck should stop using "it" all the time and be more precise16:56
*** mriedem has left #openstack-ironic16:56
xXraphXxright -> note, that since image properties are included in instance info (prefixing with image_) a possible way is to directly expect it to come from the glance image property16:57
arne_wiebalckok!16:59
arne_wiebalckI guess we would then require images for s/w to have that property?16:59
arne_wiebalckfor s/w RAID17:00
*** derekh has quit IRC17:01
xXraphXxah my bad -> I just mixed -> about the image properties, it's not at the conductor level, it actually depends on the driver17:02
arne_wiebalck?17:02
* arne_wiebalck has to go now17:06
xXraphXxno forget my last comment -> bit tired (I remember having seen something fill node.instance_info['image_*'] with image properties during deployment17:06
arne_wiebalckdone :)17:06
openstackgerritMerged openstack/ironic-inspector master: Update master for stable/train  https://review.opendev.org/68537117:06
arne_wiebalckthanks xXraphXx and TheJulia, I'll try to have another look next week, have a good weekend o/17:07
xXraphXxsame to you arne_wiebalck17:07
*** igordc has joined #openstack-ironic17:09
TheJuliaI think nova only sends the specific image uuid across17:14
TheJuliafwiw17:14
*** pcaruana has quit IRC17:23
*** e0ne has joined #openstack-ironic17:50
*** bfournie has quit IRC17:56
*** bfournie has joined #openstack-ironic18:09
*** jtwill98 has quit IRC18:52
*** zacf has joined #openstack-ironic19:00
*** lucasagomes has quit IRC19:01
*** ricolin has quit IRC19:03
*** zacf has quit IRC19:15
*** zacf has joined #openstack-ironic19:15
*** rh-jelabarre has quit IRC19:16
*** igordc has quit IRC19:52
*** TxGirlGeek has quit IRC19:52
*** rh-jelabarre has joined #openstack-ironic19:53
*** rh-jelabarre has quit IRC20:06
*** rh-jelabarre has joined #openstack-ironic20:06
openstackgerritPradip Kadam proposed openstack/ironic master: DRAC: Fix a bug for clear_job_queue clean step with non-BIOS pending job  https://review.opendev.org/68316120:09
*** etingof has quit IRC20:20
*** igordc has joined #openstack-ironic20:23
*** dsneddon has quit IRC20:28
*** e0ne has quit IRC20:45
*** TxGirlGeek has joined #openstack-ironic20:46
*** jtomasek has quit IRC20:55
*** dsneddon has joined #openstack-ironic20:57
*** hwoarang has quit IRC21:01
openstackgerritPradip Kadam proposed openstack/ironic master: DRAC: Fix a bug for clear_job_queue clean step with non-BIOS pending job  https://review.opendev.org/68316121:03
*** hwoarang has joined #openstack-ironic21:04
*** ociuhandu has joined #openstack-ironic21:27
openstackgerritMerged openstack/ironic-inspector stable/train: Update .gitreview for stable/train  https://review.opendev.org/68536921:28
openstackgerritMerged openstack/ironic-inspector stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68537021:28
*** ociuhandu has quit IRC21:33
*** munimeha1 has quit IRC21:35
*** BjoernT has quit IRC21:40
*** igordc has quit IRC22:01
*** igordc has joined #openstack-ironic22:01
*** mmethot has quit IRC22:04
*** igordc has quit IRC22:17
*** rcernin has joined #openstack-ironic22:24
*** dsneddon has quit IRC22:29
*** igordc has joined #openstack-ironic22:29
*** rcernin has quit IRC22:48
*** zacf has quit IRC23:15
*** dsneddon has joined #openstack-ironic23:18
*** dsneddon has quit IRC23:23
*** igordc has quit IRC23:32
*** igordc has joined #openstack-ironic23:34
*** igordc has quit IRC23:38
*** sthussey has quit IRC23:43
*** mmethot has joined #openstack-ironic23:47
*** mmethot has quit IRC23:57
*** rloo has quit IRC23:57
« Thursday, 2019-09-26 Index Saturday, 2019-09-28 »

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