Thursday, 2018-05-17

« Wednesday, 2018-05-16 Index Friday, 2018-05-18 »
*** rbudden has joined #openstack-ironic00:07
*** baoli has quit IRC00:08
*** baoli has joined #openstack-ironic00:09
*** baoli has quit IRC00:14
*** liuzz has quit IRC00:17
*** liuzz has joined #openstack-ironic00:19
*** rbudden has quit IRC00:19
*** cjloader has joined #openstack-ironic00:27
*** r-daneel has quit IRC00:29
*** cjloader has quit IRC00:31
*** jcoufal has joined #openstack-ironic00:47
*** trungnv has quit IRC00:53
*** trungnv has joined #openstack-ironic00:54
*** rbudden has joined #openstack-ironic00:58
*** rbudden has quit IRC00:59
*** phuongnh has joined #openstack-ironic01:02
*** baoli has joined #openstack-ironic01:05
*** tiendc has joined #openstack-ironic01:06
*** rbudden has joined #openstack-ironic01:10
*** zshi has joined #openstack-ironic01:14
*** fragatina has quit IRC01:16
*** fragatina has joined #openstack-ironic01:18
*** fragatina has quit IRC01:19
*** hemna_ has quit IRC01:32
*** namnh has joined #openstack-ironic01:40
*** fragatin_ has joined #openstack-ironic01:52
*** rwsu has joined #openstack-ironic02:03
openstackgerritzenghui.shi proposed openstack/ironic master: Change exception msg of BIOS caching  https://review.openstack.org/56819302:07
*** zhangfei has joined #openstack-ironic02:09
*** cjloader has joined #openstack-ironic02:19
openstackgerritTuan Luong-Anh proposed openstack/ironic master: [WIP] Implement iRMC BIOS configuration  https://review.openstack.org/53459502:23
*** cjloader has quit IRC02:24
openstackgerritTuan Luong-Anh proposed openstack/ironic master: [WIP] Implement iRMC BIOS configuration  https://review.openstack.org/53459502:27
openstackgerritMerged openstack/ironic master: Add reno for new config [disk_utils]partprobe_attempts  https://review.openstack.org/55977902:35
*** jcoufal has quit IRC02:35
*** lbragstad has quit IRC02:39
*** masber has quit IRC02:49
*** baoli has quit IRC02:51
*** baoli has joined #openstack-ironic02:51
*** masber has joined #openstack-ironic02:52
openstackgerritTuan Luong-Anh proposed openstack/ironic master: [WIP] Implement iRMC BIOS configuration  https://review.openstack.org/53459502:56
*** baoli has quit IRC02:56
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: add admin doc  https://review.openstack.org/56850402:56
openstackgerritTuan Luong-Anh proposed openstack/ironic master: [WIP] Implement iRMC BIOS configuration  https://review.openstack.org/53459503:03
*** akhilaki has quit IRC03:07
*** jcoufal has joined #openstack-ironic03:13
*** cjloader has joined #openstack-ironic03:19
*** baoli has joined #openstack-ironic03:22
*** cjloader has quit IRC03:24
*** jcoufal has quit IRC03:30
*** baoli has quit IRC03:30
*** cjloader has joined #openstack-ironic03:46
*** masuberu has joined #openstack-ironic03:56
*** gyankum has joined #openstack-ironic03:59
*** masber has quit IRC04:00
*** zhangfei has quit IRC04:01
*** zhangfei has joined #openstack-ironic04:02
*** masber has joined #openstack-ironic04:02
*** masuberu has quit IRC04:05
*** links has joined #openstack-ironic04:24
*** gyee has quit IRC04:26
openstackgerritMerged openstack/ironic master: Remove excessive usage of mock_the_extension_manager in unit tests - part 1  https://review.openstack.org/56858504:28
*** jesusaur has quit IRC04:39
*** jesusaur has joined #openstack-ironic04:42
*** cjloader has quit IRC04:47
*** cjloader has joined #openstack-ironic04:48
*** cjloader has quit IRC04:53
*** marios has joined #openstack-ironic05:05
*** jtomasek has quit IRC05:36
*** liuzz has quit IRC05:47
*** liuzz has joined #openstack-ironic05:48
*** mjura has joined #openstack-ironic05:48
*** penick has quit IRC05:53
*** penick has joined #openstack-ironic05:58
*** jaganathan has joined #openstack-ironic06:15
*** liuzz_ has joined #openstack-ironic06:15
*** marios has quit IRC06:16
*** liuzz has quit IRC06:16
*** rbartal has joined #openstack-ironic06:17
*** marios has joined #openstack-ironic06:17
*** liuzz_ has quit IRC06:20
*** liuzz has joined #openstack-ironic06:21
*** liuzz has quit IRC06:23
*** liuzz has joined #openstack-ironic06:25
*** jaganathan has quit IRC06:28
*** jaganathan has joined #openstack-ironic06:31
*** liuzz_ has joined #openstack-ironic06:32
*** liuzz has quit IRC06:33
*** Arkady has joined #openstack-ironic06:43
*** rbudden has quit IRC06:46
olivierb-good morning everyone06:57
*** tesseract has joined #openstack-ironic07:01
*** cjloader has joined #openstack-ironic07:18
*** ajya has joined #openstack-ironic07:20
*** cjloader has quit IRC07:23
*** mjura has quit IRC07:38
*** mjura has joined #openstack-ironic07:38
*** AlexeyAbashkin has joined #openstack-ironic07:47
*** racedo has joined #openstack-ironic07:51
*** racedo has quit IRC07:51
*** milan has joined #openstack-ironic08:04
*** mgoddard has joined #openstack-ironic08:05
*** rcernin has quit IRC08:12
*** dougsz has joined #openstack-ironic08:14
*** jtomasek has joined #openstack-ironic08:16
*** cjloader has joined #openstack-ironic08:19
*** cjloader has quit IRC08:23
*** dtantsur|afk is now known as dtantsur08:26
dtantsurmorning ironic08:26
*** zhangfei has quit IRC08:28
etingof|afkgood [almost Friday] morning everyone o/08:29
*** etingof|afk is now known as etingof08:30
*** links has quit IRC08:32
*** gyankum has quit IRC08:33
*** jtomasek has quit IRC08:37
*** e0ne has joined #openstack-ironic08:39
*** jtomasek has joined #openstack-ironic08:39
*** zhangfei has joined #openstack-ironic08:41
*** MattMan has quit IRC08:42
*** MattMan has joined #openstack-ironic08:42
olivierb-any of you having more clue today about the CI failures in ironic-lib ???08:45
olivierb-TheJulia made a tentative (partial?) patch with https://review.openstack.org/568828 but seems like it still fails08:45
patchbotpatch 568828 - ironic-lib - Slightly increase job timeouts08:45
*** jcoufal has joined #openstack-ironic08:45
*** links has joined #openstack-ironic08:49
*** jcoufal has quit IRC08:50
*** d0ugal__ has quit IRC08:51
*** d0ugal has joined #openstack-ironic08:51
*** d0ugal has quit IRC08:51
*** d0ugal has joined #openstack-ironic08:51
*** derekh has joined #openstack-ironic08:51
*** gyankum has joined #openstack-ironic09:04
*** gyankum has quit IRC09:08
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: add admin doc  https://review.openstack.org/56850409:12
*** dims has quit IRC09:20
*** rwsu has quit IRC09:28
openstackgerritzenghui.shi proposed openstack/ironic master: Change exception msg of BIOS caching  https://review.openstack.org/56819309:43
*** gyankum has joined #openstack-ironic09:46
*** gyankum has quit IRC09:50
*** AlexeyAbashkin has quit IRC09:53
*** gyankum has joined #openstack-ironic09:56
openstackgerrityolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257909:57
*** namnh has quit IRC10:00
*** stephenbm has joined #openstack-ironic10:05
*** stephenbm has quit IRC10:07
*** dims has joined #openstack-ironic10:07
*** dims has quit IRC10:17
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/queens: Mark xclarity password as secret  https://review.openstack.org/56763910:22
*** hoonetorg has quit IRC10:43
*** jcoufal has joined #openstack-ironic10:46
*** jcoufal has quit IRC10:51
*** rwsu has joined #openstack-ironic10:51
*** AlexeyAbashkin has joined #openstack-ironic10:54
openstackgerritOlivier Bourdon proposed openstack/ironic-python-agent master: Fix for fatal error when GPT was used and only MBR was cleaned  https://review.openstack.org/56908610:58
*** hoonetorg has joined #openstack-ironic11:00
*** gyan_ has joined #openstack-ironic11:14
*** jcoufal has joined #openstack-ironic11:15
*** gyankum has quit IRC11:16
*** links has quit IRC11:16
*** cjloader has joined #openstack-ironic11:19
openstackgerritSam Betts proposed openstack/ironic-tempest-plugin master: Stop double json encoding the error responses  https://review.openstack.org/56908911:19
openstackgerritSam Betts proposed openstack/ironic master: Stop double json encoding the error message responses  https://review.openstack.org/56783711:19
*** jcoufal has quit IRC11:20
*** cjloader has quit IRC11:23
*** rh-jelabarre has joined #openstack-ironic11:24
*** phuongnh has quit IRC11:28
*** links has joined #openstack-ironic11:29
*** zhangfei has quit IRC11:32
*** lbragstad has joined #openstack-ironic11:33
*** UForgotten has quit IRC11:35
*** UForgotten has joined #openstack-ironic11:35
*** bfournie has quit IRC11:38
*** tiendc has quit IRC11:44
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387911:49
*** dprince has joined #openstack-ironic11:56
*** milan_ has joined #openstack-ironic11:57
*** milan has quit IRC11:57
*** liuzz_ has quit IRC11:57
*** milan_ is now known as milan11:59
*** trown|outtypewww is now known as trown12:03
*** jcoufal has joined #openstack-ironic12:07
*** bfournie has joined #openstack-ironic12:21
olivierb-Do you think I need to adjust some timeouts to have https://review.openstack.org/#/c/569086/ pass CI. because in the log trace I can see:12:31
patchbotpatch 569086 - ironic-python-agent - Fix for fatal error when GPT was used and only MBR...12:31
olivierb-this takes 10 - 15 minutes (logs in logs/devstacklog.txt.gz)12:31
olivierb-however next trace TS is +29mn after so I am suspecting a timeout furthermore as in the other logs I see broken pipe.12:31
olivierb-however as this is the 1st time I am dealing with these tests I might very well be wrong12:32
*** sapd_ has quit IRC12:34
*** sapd_ has joined #openstack-ironic12:34
*** sapd_ has quit IRC12:35
*** sapd_ has joined #openstack-ironic12:35
*** sapd_ has quit IRC12:36
*** sapd_ has joined #openstack-ironic12:36
*** rbudden has joined #openstack-ironic12:43
*** dims has joined #openstack-ironic12:43
*** Goneri has joined #openstack-ironic12:46
*** gyan_ has quit IRC12:54
*** rloo has joined #openstack-ironic12:55
vdrokolivierb-: this info is just out of date, it is normal for devstack setup take around 30 min I think12:55
vdrokalso morning12:55
olivierb-morning vdrok12:56
vdrokolivierb-: the actual error seems to be http://logs.openstack.org/86/569086/1/check/ipa-tempest-dsvm-wholedisk-bios-pxe_ipmitool-tinyipa-src/54cfe1f/logs/devstacklog.txt.gz#_2018-05-17_11_33_14_54312:57
vdrokseems like some kind of problem with scsi package in tinycore12:57
vdrokah, it's just download time out12:57
jrollmorning everyone12:58
vdrokmorning jroll12:58
olivierb-and thx for the info, I now see what you mean12:58
olivierb-I'll launch rechech then12:58
vdrokyup12:58
olivierb-many thx again12:59
vdrokno problem, you're welcome12:59
openstackgerritKaifeng Wang proposed openstack/ironic master: Power fault recovery: db and rpc implementation  https://review.openstack.org/55570813:00
openstackgerritDmitry Tantsur proposed openstack/ironic master: Remove excessive usage of mock_the_extension_manager in unit tests - part 2  https://review.openstack.org/56910513:08
*** links has quit IRC13:09
TheJuliagood morning13:18
openstackgerritJim Rollenhagen proposed openstack/ironic master: Remove endpoint_type from configuration  https://review.openstack.org/56762813:21
jrolldtantsur: fixed those nits for you, thanks :)13:21
dtantsurthnx!13:21
jrollmorning TheJulia13:21
rloogood morning everyone, jroll, dtantsur, TheJulia, vdrok, olivierb-13:22
dtantsurmorning rloo, jroll, TheJulia13:22
vdrokMorning dtantsur TheJulia and rloo !13:24
dtantsurhey vdrok13:24
olivierb-hello rloo13:25
* rloo such a wonderful group of people to work with :)13:25
olivierb-yep, very helpful and so kind with newbies13:26
*** baoli has joined #openstack-ironic13:28
dtantsurugh, is ironic removing neutron ports on undeploy? Oo13:29
*** sambetts|afk is now known as sambetts13:29
sambettsdtantsur: it definatly should not be13:30
sambettsdtantsur: in an old version it used too13:30
dtantsurwell, it's master...13:30
dtantsurand I see a port disappearing, I cannot get where13:30
sambettsdtantsur: nova will delete the port if you have deployed only giving it a network13:31
dtantsursambetts: there's no nova..13:31
jrollsambetts: I think you have it backward, we used to not do it, now we do13:31
sambettsjroll: not the neutron port, we remove the vif from ironic13:31
dtantsurjroll: VIFs attachment or port themselves?13:31
jrolloh yes, only vifs13:31
*** baoli has quit IRC13:32
jrollit's deleting the neutron port? O_o13:32
sambettsonly time we should delete a neutron port is during cleaning or provisioning tear down13:32
*** baoli has joined #openstack-ironic13:32
sambettsand we only delete it in provisioning in the neutron driver13:32
dtantsurwelllllll13:33
jrollhttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/network/neutron.py#L10413:33
jrollwe do delete them13:33
sambettsjroll: only for provisioning like I said13:33
sambettsjroll: we never delete a tenant port13:33
jrolloh, oops13:34
dtantsurwell, here we go http://logs.openstack.org/70/569070/4/check/metalsmith-integration-glance/bf54a52/controller/logs/screen-ir-cond.txt.gz#_May_17_13_19_14_67934413:34
jrollunless provisioning net and tenant net are the same :)13:34
dtantsurthis is flat networking (I hope)13:34
sambettsjroll: thats why this does nothing: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/network/flat.py#L9113:34
dtantsurbut we delete the port13:34
* dtantsur wonders who is crazy: ironic, himself or both?13:35
jrollsambetts: right, I was implying neutron network driver but using the same networks (which is a mistake, but a valid mistake)13:35
sambettsjroll: that will never work because we try to create a port in neutron in that network but there is already one so you get a mac address conflict13:35
jrollfair13:36
sambettsso tenant can't == provisioning13:36
dtantsurfolks, we're deleting ports in *flat* networking, which I don't think is correct..13:36
jrolldtantsur: yes, understand, looking13:37
sambettsdtantsur: we only remove the neutron port we create in neutron13:37
jrollI was explaining my comment before you said it was flat13:37
sambettswe create one for cleaning, and then delete it13:37
dtantsursambetts: this is not what I see13:37
sambettsthat log you sent the node has just finished cleaning13:37
dtantsursambetts: nope, it's starting cleaning13:38
dtantsur(and I know this port's UUID)13:38
sambettsoooooo I know what happening!13:38
TheJuliavif teardown13:38
sambettsnope13:38
sambettsI think add_cleaning_network removes all ports from the network first to prevent mac address conflcits13:39
sambettswhen we create a port we own13:39
sambettsbecause there shouldn't be a tenant port at cleaning time13:39
sambettsin the cleaning network13:39
TheJuliaYeah, there is that13:39
*** sdake_ is now known as sdake13:39
dtantsurit means, we delete tenant ports before cleaning?13:39
sambettsits this line: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/network/flat.py#L115-L11613:40
sambettsit ensures we can proceed through cleaning13:41
sambettsbut results in the tenant port getting deleted if its left in the cleaning network13:41
sambettswithout that we'll fail with a mac address conflict when attempting to add a cleaning port13:42
* dtantsur also screwed up something around creating the port apparently: http://logs.openstack.org/70/569070/4/check/metalsmith-integration-glance/bf54a52/controller/logs/screen-neutron-api.txt.gz#_May_17_13_15_49_95690713:42
jrollsambetts: yep, that's it13:43
jrollpossible solution is to check if there's a port, update it if needed13:43
sambettsdtantsur: that error is expected if your using devstack and hacking baremetal into a vxlan network13:43
jrollcould probably do that for all of flat networking tbh13:43
TheJuliadtantsur: vlan instead of vxlan maybe?13:43
sambettsdtantsur: and not running a real flat network13:44
dtantsursambetts: I guess I should modify some devstack setting? or just set --provider stuff when creating a port?13:44
dtantsur(this is re vxlan, dunno what to do about deletion)13:44
sambettsdtantsur: its the network type not the port type13:44
sambettsdtantsur: the network type needs to be "flat" but in ironic devstack stuff we cxreate a vxlan network and then hack our way into it13:44
sambettsdtantsur: resulting in weird errors becuase the network type is wrong13:45
dtantsursambetts: oh, so it's normal for the default devstack?13:45
sambettsdtantsur: yup13:45
dtantsurok, thanks! one problem less :)13:45
dtantsurnow, the port deletion... I guess we don't have a CI job using a pre-created port?13:45
sambettsI had many patches to resolve that and change the network to a real network, but they became inpossible to maintain13:45
sambettsdtantsur: we don't have on that reuses a port no13:46
dtantsurnice, so I'm the first to test it in the CI :D13:46
sambettsdtantsur: I was working on a fix for this though13:46
sambettsthe deletion thing13:46
dtantsurhttps://review.openstack.org/#/c/569070/4/playbooks/integration/exercise.yaml13:46
patchbotpatch 569070 - metalsmith - Allow attaching existing ports13:46
dtantsur(the last task fails with port not found)13:46
sambettsdtantsur: https://review.openstack.org/#/c/539981/13:46
patchbotpatch 539981 - ironic - Ensure we unbind flat network ports and clear BM m...13:46
sambettsdtantsur: basically instead of deleting the ports we just need to reset their mac addresses from the baremetal mac address so they aren't associated any more13:47
sambettsdtantsur: because otherwise if you use pre-created ports, you run into mac address conflicts the next time you deploy the same node with a different port13:48
dtantsursambetts: need help with that patch? I can throw a depend-on13:48
TheJuliasambetts: I had been thinkign about that with recent issues with heat/nova that I 've been chasing down, but I'm worried that we may confuse SDN integration by doing updates that change the mac and that port could have been wired elsewhere... my worry really is about long term port re-use13:48
sambettsdtantsur: Yeah, any help would be good to get this resolved13:49
sambettsTheJulia: thats why we need to reset the mac when we unbind, because the port is no longer associated with the baremetal13:49
dtantsurTheJulia: removing the pre-created port is also confusing..13:49
* TheJulia is on a call at the moment13:50
sambettsbut currently we leave the mac address behind which still ties the port to the baremetal node, even though its not bound for that node any more13:50
dtantsursambetts: do you have a bug number for all this?13:50
sambettsI thought I created one but I don't see it on the gerrit patch :(13:51
* sambetts fails at searching storyboard for it 13:52
dtantsursambetts: I guess we can remove https://review.openstack.org/#/c/539981/4/ironic/drivers/modules/network/flat.py@138 now, right?13:52
patchbotpatch 539981 - ironic - Ensure we unbind flat network ports and clear BM m...13:52
dtantsurhmm, and https://review.openstack.org/#/c/539981/4/ironic/drivers/modules/network/flat.py@159 will probably try the same...13:53
patchbotpatch 539981 - ironic - Ensure we unbind flat network ports and clear BM m...13:53
sambettsdtantsur: with this change unconfigure tenant networks should reset the mac address of the tenant port, but I think the rollback is there for the case of retrying cleaning too13:54
TheJuliasambetts: We would need community consensus w/r/t ml2 driver owners to ensure that it is sane to them because they might be using the mac address to track the ports existence13:54
sambettsits just an accident that the tenant network is getting rolled into it13:54
dtantsursambetts: then we need to record which ports we created13:54
sambettsTheJulia: at this point the port is unbound, the ml2 drivers have no idea it even exists13:54
sambettsdtantsur: we have that already, cleaning_vif_id, so yeah rollback should only deal with that13:55
dtantsurTheJulia: I think the problem with user experience is worse, to be honest. at least assuming that nova does not delete ports on server deletion13:55
dtantsurright13:55
TheJuliaI'm not 100% sure about that13:55
TheJuliadtantsur: nova delete the port if you ask for a network attachment because it creates the port13:55
TheJuliaotherwise it does not delete the port13:55
dtantsurTheJulia: right, we're talking about the latter13:55
TheJuliabut the port is functionally unusable13:56
TheJuliawhich goes to sambetts's point that we shoudl be resetting the mac back13:56
dtantsuryep13:56
TheJuliawhich might work just fine, I just have concerns13:56
*** zhangfei has joined #openstack-ironic13:57
sambettswe clear the binding:host_id and binding:profile, that defines a port as unbound, and allows us to change the mac address, neutron disallows changing the mac address of a bound port specifially because of your concern13:58
TheJuliaokay, whew13:59
sambettsbecause if its bound then it might be being used for something, once its unbound its just a blob of data13:59
*** ianychoi_ has quit IRC13:59
TheJuliain theory, I feel like at least one of the ml2 drivers I've looked at does some wonky things still tracking the blobs of data14:00
sambettsTheJulia: the only thing that could go wrong with my code in this case is that we unluckly manage to generate a random mac address that already exists in the network, but I think thats a very small chance14:01
sambettsand we could wrap it in a retry14:01
sambettsthe code I use to generate the mac is taken straight from neutron too, so it uses the exact same scheme14:02
TheJuliaI wonder if we could store the original mac14:02
sambettsTheJulia: neutron might regenerate that mac though14:02
TheJuliatrue14:02
sambettsTheJulia: so its actually safer to generate a new one14:02
TheJuliayeah14:02
TheJuliais it just unsetting the mac fhat forces it to regenerate?14:03
sambettsneutron doesn't let you unset the mac, so we have to generate one on our end14:03
sambettshttps://review.openstack.org/#/c/539981/4/ironic/common/neutron.py @ L9314:03
patchbotpatch 539981 - ironic - Ensure we unbind flat network ports and clear BM m...14:03
sambettsI was hoping you could just set it to null/none and it would generate a new one neutron side14:04
sambettsbut alas nope14:04
TheJuliaugh, fun14:04
sambettsyeah14:04
sambettsthis is one of those patches I was working on and kinda slipped into the background because of other things :( but this bug caused me and massive headache in a system a few months ago that were trying to use pre-existing neutron ports14:06
dtantsurditto :)14:07
TheJuliaYeah, I just got off a call where they were hitting this because of heat orphaned ports :(14:08
sambettswell at least the code is already written, even if it got forgotten for a bit14:09
dtantsurI can try resurrecting it14:09
dtantsurprobably tomorrow with a fresher head14:09
TheJuliadoesn't neutron have a range it generates macs in?14:10
sambettsTheJulia: yup, I basically stole the code striaght from neutron, so we generate the macs in exactly the same way14:10
*** skazi has quit IRC14:10
TheJuliaok14:10
sambettsTheJulia: and its configurable in the same way14:10
TheJuliaokay, yeah, see the base mac now14:11
sambettsI believe all thats missing from that patch is UTs14:11
jrollso we used to generate macs in exactly the same way until the copy/paste evolved on both ends*14:11
jroll:(14:11
sambettsjroll: yeah thats the only thing thats a pain ...14:12
* TheJulia goes and gets coffee14:12
jrollsambetts: it's not a pain, it's bad practice, why not pull that out into neutron-lib or something?14:12
jrollprobably a separate library actually14:12
jrollidk14:13
jrollany reason we can't just null the mac out?14:13
sambettsactually its already in neutron-lib https://github.com/openstack/neutron-lib/blob/master/neutron_lib/utils/net.py#L2814:13
sambettsjroll: it won't let us :( that was my original goal14:14
jrollokay, why not import neutron-lib? :)14:14
sambettsjroll: new dep for ironic14:14
jrollyes, and?14:14
jrollare dependencies bad again?14:14
jrollbetter question, are dependencies worse than copy/paste?14:14
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387914:14
jrollit doesn't require much that ironic doesn't have already: https://github.com/openstack/neutron-lib/blob/master/requirements.txt14:15
sambettstbh I wasn't even sure this patch would fly, so yeah totally can use neutron-lib to avoid the copy/paste14:15
TheJulia+1 to using neutron-lib and not copy/paste14:16
sambettscool left a comment to that affect on the patch14:17
jrollthanks14:18
*** ianychoi has joined #openstack-ironic14:18
*** baha has joined #openstack-ironic14:20
*** hemna_ has joined #openstack-ironic14:24
*** hamzy_ has joined #openstack-ironic14:28
*** hamzy has quit IRC14:29
openstackgerritSam Betts proposed openstack/ironic-tempest-plugin master: Stop double json encoding the error responses  https://review.openstack.org/56908914:32
*** jaypipes has quit IRC14:34
openstackgerritMark Hamzy proposed openstack/bifrost master: pip version 10.0.0b1 will not uninstall system packages  https://review.openstack.org/56871914:43
*** hamzy_ is now known as hamzy14:43
olivierb-TheJulia once back from coffee could you please have a brief look at the very short fix I made concerning what we talked about yesterday and is resumed in commit message of https://review.openstack.org/569086, many thanks. I have successfully tested this several times today on my test environments. Comments from others also very welcomed14:44
patchbotpatch 569086 - ironic-python-agent - Fix for fatal error when GPT was used and only MBR...14:44
TheJuliaolivierb-: in the middle of helping someone with a deployment issue in their system, when I circle back from that14:54
*** cjloader has joined #openstack-ironic14:54
olivierb-no pb at all, take your time TheJulia I am in absolute no hurry on my side as I already patched my env14:55
*** mjura has quit IRC15:01
*** rbartal has quit IRC15:07
*** fragatin_ has quit IRC15:08
*** fragatina has joined #openstack-ironic15:10
*** AlexeyAbashkin has quit IRC15:12
*** AlexeyAbashkin has joined #openstack-ironic15:15
dtantsurone more big boring but necessary patch: https://review.openstack.org/56910515:19
patchbotpatch 569105 - ironic - Remove excessive usage of mock_the_extension_manag...15:19
*** dprince has quit IRC15:24
*** cz2 has quit IRC15:24
*** zhangfei has quit IRC15:25
prometheanfirehttps://paste.pound-python.org/show/roqd3nvt1QhalGZUeKpa/ helps, but still getting what looks like a unicode error15:35
prometheanfirehttps://gist.githubusercontent.com/prometheanfire/d4bc95be41d401471b040e72ab82d7e7/raw/d6aff3e1f00c80b5642bfc379d998162d7d59709/gistfile1.txt15:35
*** gyee has joined #openstack-ironic15:41
*** dprince has joined #openstack-ironic15:43
*** marios has quit IRC15:45
*** lucasagomes is now known as lucas-afk15:48
openstackgerritKaifeng Wang proposed openstack/ironic master: Power fault recovery: apply fault  https://review.openstack.org/55675815:49
*** e0ne has quit IRC15:51
*** cz2 has joined #openstack-ironic15:52
*** baha has quit IRC15:53
rlooTheJulia: if you have a few min to hopefully put your mark so we can proceed: https://review.openstack.org/#/c/568627/15:56
patchbotpatch 568627 - ironic-specs - Use node.fault field for power fault recovery work15:56
*** milan has quit IRC15:59
TheJuliaas long as my mark is not signing away my soul to sail on a ship for the rest of time16:01
*** trown is now known as trown|lunch16:02
TheJuliarloo: did you see Kaifeng's most recent comment?16:04
rlooTheJulia: no, i didn't. I don't think we should change it to a json field; we haven't spent time thinking about what it means to support more than one fault at a time.16:05
TheJuliaThat too is reasonable :)16:06
rlooTheJulia: we could add new field .faults for that. and/or look at the orig spec that proposes a more robust solution.16:06
rlooTheJulia: i will respond; thx for pointing it out!16:06
TheJuliayeah, and that actually may be a better path, that way we have a clean migration there if we ever get there16:06
TheJuliarloo: no problem, let me know when you want me to get out the +A stamp16:07
rlooTheJulia: oh, you can do the +A now :)16:07
rlooTheJulia: thx!16:07
openstackgerritMatthew Thode proposed openstack/ironic-lib master: Fall back to PARTUUID if UUID returns nothing.  https://review.openstack.org/56915816:09
TheJuliaStamped16:09
openstackgerritMatthew Thode proposed openstack/ironic-python-agent master: write byte objects when using os.write  https://review.openstack.org/56915916:12
TheJuliaolivierb-: your ipa change lgtm, super simple, although I did have to go double check the manual for sgdisk16:15
prometheanfiremind reviewing https://review.openstack.org/569160 as well?16:16
patchbotpatch 569160 - diskimage-builder -   IPA requires iptables16:16
openstackgerritMerged openstack/ironic-specs master: Use node.fault field for power fault recovery work  https://review.openstack.org/56862716:19
TheJuliaprometheanfire: sure in a little bit16:20
prometheanfireyes16:20
*** baha has joined #openstack-ironic16:20
TheJuliaoh jeeze16:21
TheJuliadone btw :)16:21
prometheanfire?16:21
TheJuliaiptables missing16:21
prometheanfire:D16:22
*** jaganathan has quit IRC16:23
*** jaganathan has joined #openstack-ironic16:23
*** jrist has quit IRC16:24
TheJuliastepping away for realz for a little while16:25
*** Arkady has quit IRC16:27
* sambetts --> home 16:33
sambettsnight all16:33
*** sambetts is now known as sambetts|afk16:33
*** mjturek has joined #openstack-ironic16:38
*** tesseract has quit IRC16:38
openstackgerritRuby Loo proposed openstack/ironic master: Disable periodic tasks if interval set to 0  https://review.openstack.org/56916616:46
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Convert test_manager to hardware types  https://review.openstack.org/56916716:47
*** trown|lunch is now known as trown|mtg17:02
openstackgerritKaifeng Wang proposed openstack/ironic master: Power fault recovery: API implementation  https://review.openstack.org/55601517:03
dtantsurg'night17:04
*** AlexeyAbashkin has quit IRC17:04
*** dtantsur is now known as dtantsur|afk17:04
*** dougsz has quit IRC17:04
*** derekh has quit IRC17:05
*** mgoddard has quit IRC17:06
*** jtomasek has quit IRC17:07
anupnHi TheJulia: I was able to provision the machine correctly. It has to have separate provisioning network specified and need OVS_BRIDGE_MAPPINGS to be specified. That way br-int and the ovs bridge connected to the physical network interface can talk to each other17:36
*** akhilaki has joined #openstack-ironic17:38
TheJulia\o/17:42
*** baha has quit IRC17:44
*** retr0h has quit IRC17:44
*** retr0h has joined #openstack-ironic17:44
*** baha has joined #openstack-ironic17:46
*** mgoddard has joined #openstack-ironic17:46
*** dprince has quit IRC17:56
*** links has joined #openstack-ironic17:59
*** cjloader_ has joined #openstack-ironic18:07
*** cjloader has quit IRC18:07
*** cjloader_ has quit IRC18:07
*** cjloader has joined #openstack-ironic18:07
*** trown|mtg is now known as trown18:22
openstackgerritMatthew Thode proposed openstack/ironic-lib master: Fall back to PARTUUID if UUID returns nothing.  https://review.openstack.org/56915818:32
*** ElCoyote_ has joined #openstack-ironic18:33
*** cjloader has quit IRC18:34
*** cjloader has joined #openstack-ironic18:35
*** cjloader has quit IRC18:35
*** links has quit IRC18:35
*** cjloader has joined #openstack-ironic18:36
TheJuliaprometheanfire: I think it should...19:13
prometheanfirethink what should?19:14
*** mjturek has quit IRC19:25
*** dtantsur|afk has quit IRC19:29
TheJuliathe patch, that it should work. Unit tests likely won't like it much :)19:30
*** dtantsur has joined #openstack-ironic19:36
*** jrist has joined #openstack-ironic19:36
*** ajya has quit IRC19:37
*** ajya has joined #openstack-ironic19:37
openstackgerritMerged openstack/ironic-python-agent master: rework ATA secure erase  https://review.openstack.org/55775519:39
*** dtantsur has quit IRC19:40
*** jrist has quit IRC19:48
*** dtantsur has joined #openstack-ironic19:51
*** jrist has joined #openstack-ironic19:51
*** zshi has quit IRC19:54
*** zshi has joined #openstack-ironic19:54
prometheanfireTheJulia: ah, question, should this image work with ironic-python-agent?19:55
prometheanfiredisk-image-create -o baremetal-$DISTRO_NAME-$DIB_RELEASE ubuntu baremetal block-device-gpt bootloader dhcp-all-interfaces local-config slow-network devuser19:56
TheJuliaprometheanfire: I believe so yes19:56
prometheanfireok, I'm getting this, it's with my patches https://paste.pound-python.org/show/MwRhLPfXUrChtIiKUp8t/19:58
TheJuliathat is... bizzar20:08
prometheanfireyep20:12
prometheanfiretrying with bionic instead of fedora next20:12
*** ajya has quit IRC20:13
prometheanfireor not, dib doesn't support it yet20:15
prometheanfirewhat do we do with stuff like this? do we want UUID or PARTUUID?20:19
prometheanfireKNAME="nvme0n1p1" UUID="F0CC-7859" PARTUUID="69497f55-a6e8-46ff-8c30-eb01a2ab9854" TYPE="part"20:19
TheJuliaI think we want part uuid, but it looks like it is not reading the underlying uuid correctly20:20
prometheanfireya, IPA needs a change for PARTUUID as well20:20
prometheanfirethe log showed it mapping the UUID to /dev/sda2, wasn't the right?20:20
TheJuliadepends on the image20:21
prometheanfireso... I'll check for PARTUUID and use that, if that doesn't exist try UUID20:21
TheJuliawhat it thought was correct matched what it wrote I beleive20:21
prometheanfirerather than the other way around20:21
TheJuliaokay20:24
openstackgerritMatthew Thode proposed openstack/ironic-python-agent master: fall back to PARTUUID if UUID not found.  https://review.openstack.org/56925020:25
prometheanfirethough for ipa the order doesn't mater, just the matching20:25
*** fragatina has quit IRC20:32
*** dprince has joined #openstack-ironic20:39
prometheanfireI'm surprised no one else ran into this20:40
prometheanfiremakes me think I'm doing something wrong20:40
*** jcoufal has quit IRC20:50
TheJuliacould be your hardware or kernel...20:51
*** ElCoyote_ has quit IRC20:51
*** fragatina has joined #openstack-ironic20:59
prometheanfirejust dell and hp servers21:01
*** fragatina has quit IRC21:01
*** fragatina has joined #openstack-ironic21:02
*** dtantsur has quit IRC21:02
*** milan has joined #openstack-ironic21:02
*** dtantsur has joined #openstack-ironic21:03
*** baha has quit IRC21:03
*** fragatina has quit IRC21:03
*** fragatina has joined #openstack-ironic21:04
*** ElCoyote_ has joined #openstack-ironic21:08
*** Goneri has quit IRC21:08
*** trown is now known as trown|outtypewww21:09
*** rh-jelabarre has quit IRC21:15
*** rh-jelabarre has joined #openstack-ironic21:15
NobodyCamGood Afternoon Ironic'ers21:27
*** cjloader_ has joined #openstack-ironic21:28
*** cjloader has quit IRC21:28
*** dprince has quit IRC21:28
*** bfournie has quit IRC21:30
*** bfournie has joined #openstack-ironic21:31
*** bfournie has quit IRC21:31
*** cjloader_ has quit IRC21:33
*** hemna_ has quit IRC21:34
*** fragatina has quit IRC21:39
*** mgoddard has quit IRC21:40
*** fragatina has joined #openstack-ironic21:46
*** rcernin has joined #openstack-ironic22:04
*** hemna_ has joined #openstack-ironic22:05
*** baoli has quit IRC22:09
*** baoli has joined #openstack-ironic22:10
*** baoli has quit IRC22:14
*** milan has quit IRC22:15
*** hemna_ has quit IRC22:24
*** hemna_ has joined #openstack-ironic22:24
*** cjloader has joined #openstack-ironic22:26
*** cjloader has quit IRC22:28
*** cjloader has joined #openstack-ironic22:28
*** ElCoyote_ has quit IRC22:33
*** threestrands has joined #openstack-ironic22:37
*** rpioso|afk is now known as rpioso22:41
rpiosoTheJulia: I have a favor to ask. Do you have the link to Jimmy McArthur’s announcement about the revised early PTG registration deadline handy?22:45
rpiosoTheJulia: I don’t recall which list he sent it to.22:46
TheJuliaNot readily, did you check the openstack-dev archive?22:46
rpiosoTheJulua: y22:46
TheJuliaHmmmm22:47
rpiosoTheJulia: Oops22:47
rpiosoTheJulia: He sent it in April.22:47
*** ElCoyote_ has joined #openstack-ironic22:51
prometheanfireis ironic tested with diskimage built images?22:54
* prometheanfire is really confused how anyone is using ironic with the current IPA issues, maybe they are using old DIB?22:57
*** ElCoyote_ has quit IRC23:03
*** rbudden has quit IRC23:03
rpiosoTheJulia: Never mind. I found it.23:11
TheJuliaprometheanfire: bifrost has a job that does it. I can check it when i get back in front of a computer23:12
prometheanfireTheJulia: thanks, atm I just want to find something that works, just to compare the current deploy to it23:14
TheJuliaprometheanfire: http://logs.openstack.org/19/568719/4/check/bifrost-integration-dibipa-debian/d1a49a5/23:16
prometheanfireok, I'll work with that, thanks23:24
*** baoli has joined #openstack-ironic23:25
*** baoli has quit IRC23:25
*** cjloader has quit IRC23:29
*** bfournie has joined #openstack-ironic23:35
« Wednesday, 2018-05-16 Index Friday, 2018-05-18 »

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