Thursday, 2017-05-18

*** early has quit IRC00:00
*** ijw has quit IRC00:03
*** early has joined #openstack-ironic00:03
*** sasha2 has quit IRC00:05
*** ijw has joined #openstack-ironic00:31
*** ijw has quit IRC00:31
*** ijw has joined #openstack-ironic00:32
*** causten_ has quit IRC00:34
*** rbudden_ has joined #openstack-ironic00:34
*** causten has joined #openstack-ironic00:34
*** rbudden has quit IRC00:36
*** rbudden_ is now known as rbudden00:36
*** causten has quit IRC00:38
*** dsneddon has quit IRC00:40
*** pmannidi has quit IRC00:41
*** Sukhdev has joined #openstack-ironic00:43
*** jrist has joined #openstack-ironic00:45
*** jrist has quit IRC00:45
*** jrist has joined #openstack-ironic00:45
*** ijw has quit IRC00:48
*** ijw has joined #openstack-ironic00:49
*** ijw has quit IRC00:49
*** ijw has joined #openstack-ironic00:50
*** ijw has quit IRC00:51
*** ijw has joined #openstack-ironic00:52
*** ijw has quit IRC00:53
*** ijw has joined #openstack-ironic00:53
*** ram5391 has quit IRC00:58
*** ijw has quit IRC00:58
*** pmannidi has joined #openstack-ironic01:03
*** Sukhdev has quit IRC01:06
*** jaypipes has joined #openstack-ironic01:12
*** tuanluong has joined #openstack-ironic01:15
*** mat128 has joined #openstack-ironic01:16
*** jaypipes has quit IRC01:19
*** karthiks has joined #openstack-ironic01:20
*** WeiA has joined #openstack-ironic01:25
*** fragatina has quit IRC01:27
*** ijw has joined #openstack-ironic01:30
*** ijw has quit IRC01:36
*** dsneddon has joined #openstack-ironic01:52
*** rbudden has quit IRC01:55
*** baoli has quit IRC01:56
*** baoli has joined #openstack-ironic01:57
*** crushil has joined #openstack-ironic02:10
*** jaypipes has joined #openstack-ironic02:22
*** zackf has quit IRC02:23
*** zackf has joined #openstack-ironic02:27
*** zackf has quit IRC02:31
*** shobhar_ has quit IRC02:32
*** dsneddon has quit IRC02:32
*** jaypipes has quit IRC02:41
*** crushil has quit IRC02:43
*** baoli has quit IRC02:46
*** baoli has joined #openstack-ironic03:02
*** links has joined #openstack-ironic03:14
*** zackf has joined #openstack-ironic03:15
*** crushil has joined #openstack-ironic03:18
*** vikrant has joined #openstack-ironic03:19
*** baoli has quit IRC03:19
*** causten has joined #openstack-ironic03:20
*** causten has quit IRC03:24
*** ykarel has joined #openstack-ironic03:36
*** ykarel_ has joined #openstack-ironic03:53
*** ykarel has quit IRC03:53
*** litao__ has joined #openstack-ironic04:01
*** ijw has joined #openstack-ironic04:01
litao__Any high level bugs to fix for bug smash?04:03
*** ijw has quit IRC04:06
*** mat128 has quit IRC04:07
*** mat128 has joined #openstack-ironic04:08
*** mat128 has quit IRC04:12
*** Sukhdev has joined #openstack-ironic04:13
*** ykarel_ has quit IRC04:14
*** links has quit IRC04:20
*** shobhar_ has joined #openstack-ironic04:25
*** fragatina has joined #openstack-ironic04:30
*** fragatina has quit IRC04:30
*** fragatina has joined #openstack-ironic04:31
*** jaganathan has joined #openstack-ironic04:37
*** jaganathan has quit IRC04:45
*** jaganathan has joined #openstack-ironic04:45
*** adisky_ has joined #openstack-ironic04:50
*** crushil has quit IRC04:52
*** crushil has joined #openstack-ironic04:54
*** crushil has quit IRC04:56
*** links has joined #openstack-ironic04:58
*** jaosorior_away is now known as jaosorior05:00
*** vmud213 has joined #openstack-ironic05:04
*** crushil has joined #openstack-ironic05:06
*** jaganathan has quit IRC05:07
*** dsneddon has joined #openstack-ironic05:10
*** shobhar_ has quit IRC05:10
*** shobhar__ has joined #openstack-ironic05:11
*** ChubYann has quit IRC05:11
*** pc_m has quit IRC05:11
*** fragatina has quit IRC05:11
*** udesale has joined #openstack-ironic05:18
*** crushil has quit IRC05:23
*** Sukhdev has quit IRC05:28
*** mvpnitesh has joined #openstack-ironic05:37
*** jaganathan has joined #openstack-ironic05:41
*** mjura has joined #openstack-ironic05:43
*** rcernin has joined #openstack-ironic05:46
*** shobhar__ has quit IRC05:54
*** mvpnitesh has quit IRC06:04
*** ijw has joined #openstack-ironic06:05
*** ccamacho has quit IRC06:06
*** links has quit IRC06:08
*** ykarel_ has joined #openstack-ironic06:08
*** mvpnitesh has joined #openstack-ironic06:09
*** ijw has quit IRC06:10
*** Lambert has joined #openstack-ironic06:15
*** zackf has quit IRC06:18
*** pc_m has joined #openstack-ironic06:19
openstackgerritparesh sao proposed openstack/ironic master: Adds parameters to run CI with hardware types.  https://review.openstack.org/46548406:22
*** links has joined #openstack-ironic06:24
*** pcaruana has joined #openstack-ironic06:24
openstackgerritparesh sao proposed openstack/ironic master: Adds parameters to run CI with hardware types.  https://review.openstack.org/46548406:27
pas-hamorning ironic :)06:27
*** livelace has joined #openstack-ironic06:29
openstackgerritlidong proposed openstack/bifrost master: Fix a typo  https://review.openstack.org/46583106:32
*** aparnav has joined #openstack-ironic06:39
*** rbartal has joined #openstack-ironic06:41
*** skazi has joined #openstack-ironic06:41
*** e0ne has joined #openstack-ironic06:49
*** ralonsoh has joined #openstack-ironic06:50
*** e0ne has quit IRC06:56
openstackgerritAparna proposed openstack/ironic master: Agent driver rescue implementation  https://review.openstack.org/40043707:04
*** Serlex has joined #openstack-ironic07:10
*** ccamacho has joined #openstack-ironic07:11
*** links has quit IRC07:16
*** zackf has joined #openstack-ironic07:19
*** zackf has quit IRC07:23
*** mgoddard__ has joined #openstack-ironic07:28
*** adisky_ has quit IRC07:29
*** links has joined #openstack-ironic07:30
openstackgerritparesh sao proposed openstack/ironic-specs master: Out-of-band Boot from Volume using UEFI iSCSI Boot.  https://review.openstack.org/20733707:44
*** ijw has joined #openstack-ironic07:54
*** shuyingya has joined #openstack-ironic07:58
*** ijw has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** SerenaFeng has joined #openstack-ironic08:02
*** athomas has joined #openstack-ironic08:07
*** mgoddard__ has quit IRC08:10
*** adisky_ has joined #openstack-ironic08:10
*** aarefiev_afk is now known as aarefiev08:16
aarefievmorning all!08:16
*** lucas-afk is now known as lucasagomes08:16
*** SerenaFeng has quit IRC08:22
*** stendulker has joined #openstack-ironic08:24
*** racedo has joined #openstack-ironic08:25
openstackgerritJun Yao proposed openstack/ironic master: Validating ipmi_address to accept unicast IP address and resolvable hostname.  https://review.openstack.org/46586608:30
*** mvpnitesh has quit IRC08:33
*** shuyingya has quit IRC08:35
*** shuyingya has joined #openstack-ironic08:35
*** mvpnitesh has joined #openstack-ironic08:40
*** zhurong has joined #openstack-ironic08:40
*** jaganathan has quit IRC08:43
*** derekh has joined #openstack-ironic08:45
*** mgoddard__ has joined #openstack-ironic08:46
*** mvpnitesh has quit IRC08:49
*** jaosorior is now known as jaosorior_lunch08:57
*** SerenaFeng has joined #openstack-ironic08:58
*** shuyingy_ has joined #openstack-ironic09:00
*** amarao has joined #openstack-ironic09:00
*** ykarel__ has joined #openstack-ironic09:00
*** ykarel_ has quit IRC09:03
*** shuyingya has quit IRC09:03
pas-hamgoddard__:  left a comment on https://review.openstack.org/#/c/458425, looks good except a requirements thing09:04
patchbotpatch 458425 - networking-generic-switch - Workaround switch SSH connection rate limiting09:04
*** zhurong has quit IRC09:05
*** mvpnitesh has joined #openstack-ironic09:09
pas-havdrok: please take a look at https://review.openstack.org/#/c/465850, trivial but crucial typo fix09:10
patchbotpatch 465850 - networking-generic-switch - fail to config dell_force10 switch09:10
*** shuyingy_ has quit IRC09:10
*** shuyingya has joined #openstack-ironic09:10
*** ricardoas has quit IRC09:12
*** ykarel__ has quit IRC09:12
*** ykarel__ has joined #openstack-ironic09:13
*** zackf has joined #openstack-ironic09:14
vdrokpas-ha: done09:15
vdrokmorning ironic, pas-ha and aarefiev !09:15
aarefievo/09:15
pas-haO/ thanks09:16
* pas-ha having bigger head than usually.. :)09:16
openstackgerritparesh sao proposed openstack/ironic master: Adds parameters to run CI with hardware types.  https://review.openstack.org/46548409:20
*** ykarel__ has quit IRC09:23
*** e0ne has joined #openstack-ironic09:23
*** Lambert has quit IRC09:31
*** udesale__ has joined #openstack-ironic09:31
*** udesale has quit IRC09:31
*** shuyingy_ has joined #openstack-ironic09:31
litao__https://etherpad.openstack.org/p/OpenStack-Bug-Smash-Pike-Suzhou-Bug-List09:33
*** e0ne has quit IRC09:34
*** jaganathan has joined #openstack-ironic09:34
*** shuyingya has quit IRC09:35
*** e0ne has joined #openstack-ironic09:35
*** ykarel__ has joined #openstack-ironic09:35
*** skazi has quit IRC09:39
*** ykarel__ has quit IRC09:40
*** sambetts|afk is now known as sambetts09:44
*** pmannidi has quit IRC09:45
sambettsMorning all09:45
vdrokmorning sambetts09:48
openstackgerritM V P Nitesh proposed openstack/ironic master: Ironic conductor works correctly on changes of networks  https://review.openstack.org/46293109:50
*** tuanluong has quit IRC10:09
*** dtantsur|afk is now known as dtantsur10:09
dtantsurmorning ironic10:09
sambettshey dtantsur10:13
vdrokgood morning dtantsur10:15
dtantsuro/10:16
*** ykarel__ has joined #openstack-ironic10:22
*** milan has joined #openstack-ironic10:22
*** milan has quit IRC10:22
vdrokdtantsur: please take a look at https://review.openstack.org/462931 if you have a bit of time, the change is pretty small and easy, but it might be not what we want to do10:27
patchbotpatch 462931 - ironic - Ironic conductor works correctly on changes of net...10:27
mgoddard__pas-ha: thanks for the review. any thoughts on how to reconcile with the locking change?10:27
pas-haI honestly have not thought on that yet, but do not forsee any actual troubles - the locking part will kick in before connection attempts, so those are independent. Plus these two 'features' could be enabled/configured separately, so if your switch works fine with only one of them you can enable only this feature alone.10:30
sambettsdtantsur: the reason we need that patch to update the network cache is for cases when you use a named network and then someone deletes and recreates in on the neutron side then the UUID the conductors have cached is wrong10:32
vdrok++ just commented the same10:33
sambettsand currently all conductors will break and require a restart if that happens10:33
vdroksambetts: do they actually "break" ? I thought that provisioning actions will fail, not the conductors themselves10:34
vdrokmaybe we need to add some refresh flag? and set it to True when doing things like add_cleaning_network etc.10:34
vdrokbut still in some cases it will not help and network interface validation may still fail :(10:35
mgoddard__pas-ha: sounds reasonable to me10:37
jaganathandtantsur, good morning, please guide us for the next steps to do https://review.openstack.org/#/c/424729/10:38
patchbotpatch 424729 - ironic-python-agent - NUMA-topology collector10:38
sambettsvdrok: I think having it in validate makes sense, currently the validation is broken because if someone deletes the neutron network after one validation, then the conductor will validate to true forever even if that isn't the case10:39
*** jkilpatr has quit IRC10:39
sambettsfor node validate10:39
dtantsurjaganathan, you need to wait for someone else (not from Red Hat) to also review and approve it10:40
vdroksambetts: ah, right, the cached one will be used. tho when the actions will actually happen they will fail10:40
sambettsvdrok: we could have a periodic task that updates the network uuid cache on a set interval10:40
jaganathandtantsur, ok, thanks10:40
sambettsvdrok: yup10:40
sambettsvdrok: periodic would be nasty though10:40
vdroksambetts: so the refresh=True will do basically the same thing. yeah, I'd prefer no more periodics :)10:41
*** mat128 has joined #openstack-ironic10:41
sambettsI'd prefer to bite the bullet now and have more calls to neutron10:41
sambettsand then maybe we can hook into some neutron notifications and listen out for a network delete in the future10:42
vdrokso, we'll start doing calls to neutron in 4 cases - do_node_clean, vif list/attach/detach10:42
vdrokin port/portgroup update we call it anyway10:43
dtantsurthis is not so bad, I guess10:43
*** openstackgerrit has quit IRC10:48
*** phuongnh has joined #openstack-ironic10:48
*** mat128_ has joined #openstack-ironic10:54
*** WeiA has quit IRC10:55
*** mat128 has quit IRC10:56
*** mvpnitesh has quit IRC10:59
*** mat128 has joined #openstack-ironic11:01
*** mat128_ has quit IRC11:02
*** xavierr-afk has joined #openstack-ironic11:13
*** udesale__ has quit IRC11:15
*** jkilpatr has joined #openstack-ironic11:15
*** mgoddard__ has quit IRC11:22
*** stendulker_ has joined #openstack-ironic11:24
*** stendulker has quit IRC11:26
*** stendulker_ has quit IRC11:28
*** jaosorior_lunch is now known as jaosorior11:29
*** ijw has joined #openstack-ironic11:30
*** ykarel_ has joined #openstack-ironic11:32
*** ykarel__ has quit IRC11:34
*** ijw has quit IRC11:34
*** phuongnh has quit IRC11:39
*** aparnav has quit IRC11:44
*** ykarel_ has quit IRC11:45
*** chlong has joined #openstack-ironic11:48
*** openstackgerrit has joined #openstack-ironic11:49
openstackgerritMerged openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/46524911:49
*** mvpnitesh has joined #openstack-ironic11:55
openstackgerritSam Betts proposed openstack/ironic-python-agent master: Add option for building IPMItool into TinyIPA  https://review.openstack.org/46593211:55
sambettsdtantsur: ^^11:55
sambettsdtantsur: is that something we can backport too??11:55
dtantsursambetts: I'd say yes, because tinyipa is for CI mostly, so it does not count as feature11:57
sambettsawesome :)11:58
openstackgerritSam Betts proposed openstack/ironic-python-agent master: Add option for building IPMItool into TinyIPA  https://review.openstack.org/46593212:00
*** litao__ has quit IRC12:01
*** mat128 has quit IRC12:01
sambettsdtantsur: https://review.openstack.org/#/c/396406 looking at this patch, do you think that we should merge something we don't have CoreOS support for?12:05
patchbotpatch 396406 - ironic-python-agent - Collect NIC name given by BIOS12:05
*** parlos has joined #openstack-ironic12:05
*** zhurong has joined #openstack-ironic12:09
*** zhurong has quit IRC12:10
TheJuliaGood morning12:10
sambettsHi TheJulia12:10
*** ykarel_ has joined #openstack-ironic12:11
*** SerenaFeng has quit IRC12:16
vdrokmorning TheJulia12:16
TheJuliaSo anyone else want me to send them stickers today?12:19
dtantsurTheJulia: morning :) did you see my message re stickers yesterday?12:20
*** bfournie has quit IRC12:20
TheJuliadtantsur: yup12:20
dtantsurcool12:20
vdrokoh, stickers12:21
openstackgerritM V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info  https://review.openstack.org/46056412:27
lucasagomesTheJulia, oh which stickers ?12:28
*** rbudden has joined #openstack-ironic12:28
lucasagomespixie boots ?12:28
TheJulialucasagomes: The new project logo12:28
lucasagomesoh right12:28
vdroklucasagomes: btw, this can be closed right? https://bugs.launchpad.net/ironic/+bug/166295912:30
openstackLaunchpad bug 1662959 in Ironic ocata "OSC: NMI requires the 1.29 version of the API which is not present, needs to be bumped" [Medium,Triaged] - Assigned to Lucas Alvares Gomes (lucasagomes)12:30
lucasagomesvdrok, oh yeah12:31
lucasagomesit's being fixed already12:31
vdrokok, closing then, thanks12:31
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Follow up: conditions optional fields  https://review.openstack.org/46594312:32
*** skazi has joined #openstack-ironic12:33
*** milan has joined #openstack-ironic12:33
milanmorning Ironic! :)12:34
dtantsurhey-hey, milan12:34
milanhey dtantsur! :D12:35
milancoffee time?12:35
dtantsurmilan: I'm not around. Got a new monitor for my home office today :)12:35
* milan afraid he won't see dtantsur till paperwork needed :D12:36
* dtantsur too12:36
dtantsur;)12:37
milanlol12:37
milandtantsur, could you please revisit https://review.openstack.org/#/c/457765 ? aarefiev +2, thanks aarefiev! :)12:38
patchbotpatch 457765 - ironic-inspector-specs - Follow up PXE filter driver12:38
milanand good morning :D12:38
aarefievhey milan, morning :)12:38
milanaarefiev, thanks for considering the comments on the api--worker split! :)12:39
aarefievmilan: thx for review, I'll update today12:39
dtantsursure, omw12:39
milanI think it's going the right way, it was just a bit confusing with the organization12:39
milanaarefiev, thx12:39
milandtantsur, thx12:39
milan:D12:40
aarefievyeah, it was molothic patch first12:40
aarefievbiiiig patch12:40
aarefievhuge I'd say12:40
* milan remembers his state patch :D12:40
sambetts+2 from me, I'll let dtantsur review before +A12:40
milansambetts, morning! :)12:41
dtantsursambetts: feel free to approve, it was ok the last time I checked12:41
sambettsokie doke :D12:41
milansambetts, thx for the +2 :D12:41
milan\o/ spec merges!12:42
*** ykarel_ has quit IRC12:43
openstackgerritMerged openstack/ironic-inspector-specs master: Follow up PXE filter driver  https://review.openstack.org/45776512:44
*** links has quit IRC12:45
*** ykarel_ has joined #openstack-ironic12:45
*** jrist has quit IRC12:45
vdrokgood morning milan :)12:46
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/46524812:46
milanmorning vdrok! :)12:46
*** trown|outtypewww is now known as trown12:52
*** shuyingy_ has quit IRC12:52
*** shuyingya has joined #openstack-ironic12:53
*** bfournie has joined #openstack-ironic12:55
*** shuyingya has quit IRC13:00
*** sasha2 has joined #openstack-ironic13:04
*** mgoddard__ has joined #openstack-ironic13:05
*** mat128 has joined #openstack-ironic13:07
*** mvpnitesh has quit IRC13:08
*** mvpnitesh has joined #openstack-ironic13:09
*** baoli has joined #openstack-ironic13:13
*** jrist has joined #openstack-ironic13:14
*** mvpnitesh has quit IRC13:15
*** rloo has joined #openstack-ironic13:17
*** ijw has joined #openstack-ironic13:18
*** dprince has joined #openstack-ironic13:22
*** jcoufal has joined #openstack-ironic13:22
*** ijw has quit IRC13:23
openstackgerritMerged openstack/ironic master: Refactor install-guide: update node enrollment  https://review.openstack.org/46360913:25
*** parlos has quit IRC13:29
*** vikrant has quit IRC13:31
*** Goneri has joined #openstack-ironic13:32
openstackgerritPeter Piela proposed openstack/ironic-ui master: Consolidate code for setting node maintenance state  https://review.openstack.org/46597213:33
*** srobert has joined #openstack-ironic13:38
*** trown is now known as trown|brb13:41
*** livelace has quit IRC13:41
*** hshiina has joined #openstack-ironic13:43
*** baoli_ has joined #openstack-ironic13:43
*** trown|brb is now known as trown13:45
*** mvpnitesh has joined #openstack-ironic13:45
*** baoli has quit IRC13:46
*** sasha2 has quit IRC13:47
openstackgerritMerged openstack/bifrost master: Updated from global requirements  https://review.openstack.org/46522813:52
*** vmud213 has quit IRC13:57
*** sasha2 has joined #openstack-ironic13:58
*** sasha2 has quit IRC13:59
*** sasha2 has joined #openstack-ironic13:59
*** shuyingya has joined #openstack-ironic14:04
*** chlong has quit IRC14:04
*** mvpnitesh has quit IRC14:04
*** shuyingy_ has joined #openstack-ironic14:05
*** shuyingya has quit IRC14:09
*** shuyingya has joined #openstack-ironic14:11
openstackgerritPeter Piela proposed openstack/ironic-ui master: Modify regex pattern used to validate node ids  https://review.openstack.org/46598214:12
dtantsurlucasagomes, vsaienk0: wdyt about reviving https://review.openstack.org/#/c/443649/ ? I think we hit it again downstream.. I can update the patch.14:13
*** krtaylor has quit IRC14:13
patchbotpatch 443649 - ironic-python-agent - Change the _wait_for_disks behavior to wait for a ...14:13
lucasagomesdtantsur, yeah if you can take it over it would be great14:14
lucasagomesI could rebase it but, it wil be hard to find the time to actually test it out14:14
*** shuyingy_ has quit IRC14:14
dtantsurlucasagomes: dunno if I'll be able to test it.. I don't have suitable hardware. but I can simulate it with wrong root device hints, yeah14:15
lucasagomesright, yeah, I remmeber when i was doing it I was testing with a machine with 30+ disks in it14:16
dtantsurI'm nowhere near such a machine :)14:16
lucasagomesbut now, I don't know how I would test it either14:16
*** shuyingy_ has joined #openstack-ironic14:18
*** shuyingya has quit IRC14:19
*** mat128 has quit IRC14:29
*** rbartal has quit IRC14:30
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Change the _wait_for_disks behavior to wait for a specific disk  https://review.openstack.org/44364914:34
*** MattMan_1 has quit IRC14:34
dtantsurlucasagomes, vsaienk0 ^^^14:34
*** bcornec has joined #openstack-ironic14:34
*** MattMan_1 has joined #openstack-ironic14:34
NobodyCamGood Morning Ironic'ers14:35
dtantsurmorning NobodyCam14:35
NobodyCamhey hey dtantsur :) good Morning14:36
*** krtaylor has joined #openstack-ironic14:42
*** crushil has joined #openstack-ironic14:43
*** krtaylor has quit IRC14:44
dtantsurlucasagomes: btw did you actually test the change back then?14:46
lucasagomesdtantsur, yeah, but, AFAIR there was more things involving that machine that was causing the deployment to fail14:47
lucasagomesdtantsur, I think I was working with sai (irc handle downstream)14:47
lucasagomeshe might remember it better14:47
dtantsurlucasagomes: yeah, at least if you tried this particular bit, I won't re-test it again14:48
sailucasagomes: dtantsur the patch didnt solve the issue we had AFAIR14:48
*** krtaylor has joined #openstack-ironic14:48
dtantsursai: so, the disk still was not found?14:49
saidtantsur: correct14:49
saii dont have access to the same hardware to reproduce though14:49
dtantsurOo14:49
saithese were machines with 36 disks14:49
lucasagomes:-/14:51
lucasagomesI remember it was kinda trick, I don't think we got to the bottom of that problem14:52
dtantsurmeh..14:54
dtantsurthen I don't know if we really need this change14:54
*** fellypefca has joined #openstack-ironic14:55
*** shuyingy_ has quit IRC14:55
dtantsursai: so, how did it end up?14:58
saidtantsur: i never ended up deploying on those nodes and lost access to the hardware (had it for a limited time)14:58
dtantsurugh14:58
*** jaosorior is now known as jaosorior_away15:00
openstackgerritFellype Cavalcante proposed openstack/ironic master: Introduces onetime boot in oneview drivers  https://review.openstack.org/43646915:05
openstackgerritXavier proposed openstack/ironic master: Remove agent mixin from agent pxe oneview driver  https://review.openstack.org/46421215:06
dtantsurlucasagomes: when exactly do we pass root device hints? it was via command line previously, but it's no longer, right?15:06
lucasagomesdtantsur, yeah no longer via the command line, in IPA, the node is passed via IPA's API15:07
lucasagomesthe cmdline was used for the bash ramdisk15:07
dtantsurlucasagomes: so it's useless to call get_os_install_device so early, right?15:08
dtantsurwe don't have root device hints anyway...15:08
dtantsurmaybe we should add retry to the deployment part instead...15:08
lucasagomesdtantsur, the root device should be there prior to the deployment, no?15:08
lucasagomesI mean, one should set it before the setting the provision_state to active15:09
dtantsurlucasagomes: I mean, in your patch, the retries happen long before IPA knows root device hints15:09
lucasagomesdtantsur, oh right, ok I have to check... I don't remember off the top of my head when the Ironic node is passed to IPA15:10
* dtantsur builds tinyipa to verify his guess15:15
*** aarefiev is now known as aarefiev_afk15:15
*** kromanenko has quit IRC15:16
*** jkilpatr has quit IRC15:16
*** jkilpatr has joined #openstack-ironic15:16
JayFlucasagomes: dtantsur: Lookup sends back the node in the reply15:19
lucasagomesJayF, thanks15:20
lucasagomesyeah now we need to know if that method is called after that ^ happens15:21
lucasagomesdtantsur, is probably looking into it already15:21
JayFevaluate_hardware_support is called well before lookup15:24
JayFI beleive15:24
openstackgerritMerged openstack/ironic-inspector master: Remove rollback support from introspection rules  https://review.openstack.org/46566615:24
JayFbceause we precache hardware managers explicitly on startup, so we don't leave nodes stuck in *wait states15:24
openstackgerritMerged openstack/ironic-inspector stable/ocata: Do not fail rules rollback on bad formatting key  https://review.openstack.org/46551615:24
JayF(think: if you have a bad hardware manager, and you complete the lookup, then crash, the node is stuck in an unrecoverable state)15:25
*** ccamacho has quit IRC15:26
*** mjura has quit IRC15:27
* JayF slithers back into the inky shadows15:27
dtantsuryeah, this is my finding too15:28
*** Serlex has quit IRC15:33
*** fragatina has joined #openstack-ironic15:33
*** fragatina has quit IRC15:33
*** fragatina has joined #openstack-ironic15:34
vdrokgood morning NobodyCam and JayF !15:35
NobodyCamGood Morning vdrok :) and JayF too :)15:35
*** rbudden has quit IRC15:38
* dtantsur used horizon to provision an instance for the first time in his life :)15:41
lucasagomesdtantsur, hah o/15:42
*** chlong has joined #openstack-ironic15:43
rama_yHello sambetts, this is regarding https://review.openstack.org/#/c/396406/6015:43
TheJuliadtantsur: vdrok: lucasagomes: JayF: jlvillal: stickers mailed.15:43
patchbotpatch 396406 - ironic-python-agent - Collect NIC name given by BIOS15:43
dtantsur\o/ thanks TheJulia!15:43
rama_yThanks for the review; I will fix the nits.15:43
JayFTheJulia: thanks!15:43
vdrokTheJulia: wheee :)15:43
TheJuliaLet me know if they don't arrive in the next week or so15:44
rama_ysambetts, regarding CoreOS, I was wondering if you would like more details in the release note?15:44
*** rcernin has quit IRC15:47
vdrokTheJulia: in my case, it can easily be 3 weeks :)15:48
TheJuliavdrok: Eek.  Okay, 3 weeks then :)15:51
* TheJulia forgot how much paperwork was involved in mailing something out of the states15:51
sambettsrama_y: its more that I wonder if we should build it from source for the CoreOS build like we do for tinyIPA15:52
jlvillalTheJulia: You rock, you rock hard! :)15:52
dtantsur++15:52
JayFTheJulia: for me, I just put stickers in an envelope, put the intl' address on the front, and just bought extra stamps15:53
JayFTheJulia: they didn't make me fill out any paperwork15:53
jlvillalIn case people want to download the mascot: https://www.openstack.org/project-mascots/15:53
TheJuliaYeah, jlvillal and JayF were super quick :)15:53
TheJuliaJayF: I shipped them with tracking just in case, which is likely why15:54
JayFaha15:54
JayFI just mailed them to dtantsur over and over until the packet wasn't dropped :P15:54
dtantsurI think it was just the 2nd attempt :)15:55
JayFI don't remember or mind really :)15:55
JayFjust strange to think about mail being unreliable, because USPS is super duper reliable in the states.15:55
dtantsurhah, Czech mail it not quite (still not as bad as Russian one)15:56
*** e0ne has quit IRC15:56
JayFSo what you're saying is, don't mail any checks? <.< >.>15:56
dtantsurheh, yeah15:57
openstackgerritMerged openstack/ironic master: Remove unit tests that test oslo_concurrency.processutils.execute  https://review.openstack.org/46519315:57
anupnTheJulia: Could you please send me one sticker? I will email you the details15:58
TheJuliaanupn: Sure, it will be a week or two before I get back to the post office.15:59
anupnTheJulia: no hurry :)15:59
dtantsurTheJulia: time for BFV meeting?16:00
dtantsuror am I confusing something?16:00
TheJuliayup, just switching screens16:00
*** karthiks has quit IRC16:01
rama_yHi Sambetts, thanks. Sorry, went to get a cup of coffee;16:01
rama_yHi Sambetts, would you suggest that for this patch? Or, a separate patch? i.e., CoreOS building from sources part16:02
*** shuyingya has joined #openstack-ironic16:07
sambettsI think if we're including the changes for tinyIPA in that patch that it's probably worth including the changes for CoreOS too, dtantsur, JayF thoughts on https://review.openstack.org/#/c/396406, its really close to merging but I'm concerned about not having a way to include the required tool in CoreIPA16:08
patchbotpatch 396406 - ironic-python-agent - Collect NIC name given by BIOS16:08
*** ykarel_ has quit IRC16:09
dtantsursambetts: this is a fair point, but I'm fine with a follow-up (though I'm not sure who is going to do it)16:09
JayFdtantsur: it can't be done in a followup. It's just not packaged for coreos.16:10
JayFdtantsur: erm, I mean, the container coreos runs16:10
JayFsambetts: dtantsur: My vote: it doesn't matter what's in coreos or not, bceause if you all can, you should urgently get into a world where we have more reasonable ramdisks -- all the ones we publish are, at this point, years out of date security-wise (or in TinyIPA's case, built insecurely)16:11
JayFso when that was getting dev'd, my thought was that coreos missing it was a small issue since hopefully I would deprecate that in 18m or so if I got attention on ramdisks16:11
dtantsurJayF: well, it's installed from source for TinyIPA.. though I kind of feel that installing something from source in a packaged environment is not great16:11
JayFbut I'm not here so YMMV16:11
dtantsurwhich reminds me that I should talk to DIB folks about our element16:11
JayFdtantsur: I'm fairly certain it wouldn't even be able to work in a container, but not certain16:11
sambettsthats a good point, I didn't think about that16:12
*** shuyingya has quit IRC16:12
sambettsJayF: what's your thoughts on buildroot ramdisk in terms of security etc would you use it in prod?16:13
openstackgerritJulia Kreger proposed openstack/ironic master: Wire in storage interface attach/detach operations  https://review.openstack.org/40629016:13
dtantsurI guess DIB is the only ramdisk we have that is built from recent source..16:13
*** jaganathan has quit IRC16:13
JayFsambetts: I'd talk to csmart about that if I were you, but IDK if he's still around as he was laid off as well.16:13
JayFsambetts: not in IRC now for sure.16:13
*** Serlex has joined #openstack-ironic16:13
* sambetts is looking to take it over thats why I ask if you personally would have used it if it was avaible 16:14
JayFI would personally think a buildroot ramdisk would be a lot more secure, if for no other reason than minimal configuration means less security surface exposed16:14
JayFsambetts: I mean, I would prefer a buildroot ramdisk to all the options we have16:14
sambettsawesome, then its worth spending time on16:14
sambettsdidn't want to put cycles into something that would be rejected by ops for one reason or another16:15
*** fragatina has quit IRC16:25
*** milan has quit IRC16:32
*** lucasagomes is now known as lucas-afk16:35
*** amarao has quit IRC16:36
*** mat128 has joined #openstack-ironic16:39
*** pcaruana has quit IRC16:40
fellypefcajroll Hi! :) We need to test a feature in different environments. To do this, we need backports of this feature of Ocata and Newton releases. Could you take a look at these patches? Ocata - https://review.openstack.org/#/c/459325/ , Newton - https://review.openstack.org/#/c/459415/)16:45
patchbotpatch 459325 - ironic (stable/ocata) - Validation before perform node deallocation16:45
patchbotpatch 459415 - ironic (stable/newton) - Validation before perform node deallocation16:45
*** mgoddard__ has quit IRC16:48
jlvillaldtantsur: rloo ^^^16:48
dtantsuron my radar. not a trivial patch, so will need some effort.16:49
dtantsur(to figure out if it's an acceptable backport)16:49
rloohuh, oh. i already reviewed the ocata one. not going to look at newton until ocata lands16:49
* dtantsur was promised some beers, but only if he comes right now, soooo16:50
dtantsursee you tomorrow :)16:50
*** dtantsur is now known as dtantsur|afk16:50
rloociao dtantsur|afk, enjoy your beers :)16:50
*** karthiks has joined #openstack-ironic16:54
fellypefcadtantsur ok! thanks :)16:54
*** derekh has quit IRC16:58
*** fragatina has joined #openstack-ironic16:59
*** ralonsoh has quit IRC17:00
sambettso/ night all17:03
*** hshiina has quit IRC17:03
*** sambetts is now known as sambetts|afk17:03
*** fragatina has quit IRC17:04
*** rbudden has joined #openstack-ironic17:07
*** bcornec has quit IRC17:09
*** fultonj has joined #openstack-ironic17:11
*** dprince has quit IRC17:15
*** dprince has joined #openstack-ironic17:15
*** karthiks has quit IRC17:19
*** racedo has quit IRC17:20
*** fragatina has joined #openstack-ironic17:22
*** harlowja has joined #openstack-ironic17:30
*** trown is now known as trown|lunch17:30
*** ijw has joined #openstack-ironic17:38
*** mgoddard__ has joined #openstack-ironic17:40
*** ijw has quit IRC17:51
*** ijw has joined #openstack-ironic17:51
*** harlowja has quit IRC18:01
*** fragatina has quit IRC18:07
*** fragatina has joined #openstack-ironic18:08
*** harlowja has joined #openstack-ironic18:15
*** Goneri has quit IRC18:22
*** Sukhdev has joined #openstack-ironic18:30
*** racedo has joined #openstack-ironic18:36
*** jcoufal_ has joined #openstack-ironic18:39
*** jcoufal has quit IRC18:42
*** trown|lunch is now known as trown18:44
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Raise HTTP 400 rather than 500 error  https://review.openstack.org/46103618:49
*** ChubYann has joined #openstack-ironic18:51
*** racedo has quit IRC18:52
*** harlowja has quit IRC18:56
*** racedo has joined #openstack-ironic18:56
anupnTheJulia, ppiela: so wrt driver composition support in ui, should it show all the interfaces that we see on "ironic driver-show <NAME>" or just enabled interfaces and the value?18:57
*** karthiks has joined #openstack-ironic18:58
*** fellypefca has quit IRC19:00
*** Goneri has joined #openstack-ironic19:03
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers  https://review.openstack.org/38199119:04
*** racedo has quit IRC19:05
*** harlowja has joined #openstack-ironic19:07
ppielaanupn: Driver composition effects node creation, editing, and viewing19:08
ppielaanupn: I think that for viewing driver composition should be integrated with the existing node-details - Driver Info and Driver validation display areas19:11
*** fultonj has quit IRC19:13
anupnppiela: yes for i was talking about viewing, but still there is a list of interfaces for a specific driver and drive validation shows what is supported19:14
*** crushil has quit IRC19:14
*** fultonj has joined #openstack-ironic19:15
anupnppiela: if we show interfaces again in driver info then wouldn't it be repetitive?19:15
ppielaanupn: yes19:15
anupnppiela: hmm, so should driver info show a list of interfaces for that driver (deploy, management, power, etc.) and their value when enabled, does that sound fine?19:18
*** adisky_ has quit IRC19:19
anupnppiela: and of course for editing and creating node drop box or something can be used19:19
*** Goneri has quit IRC19:20
anupnppiela: but first thought to go with viewing interfaces19:21
ppielaanupn: One option would be to add another column into the Driver Validation table for "provider"19:23
*** mgoddard_ has joined #openstack-ironic19:23
anupnppiela: yeah that can be also good, the interfaces that are supported and validated we can show the enabled ones for that interface19:25
anupnppiela: i think actually that is better bcoz otherwise driver info area will get too big19:25
*** mgoddard__ has quit IRC19:26
ppielaanupn: Yes. Maybe the table becomes "Hardware Interface" with validity being one atrribute19:27
anupnppiela: Right now also the driver validation table has interfaces and other essential columns (valid, reason), so I think adding one more column like "Provider" or "Enabled" will show enabled ones and may be change the table name to "Hardware interface"19:31
*** krtaylor has quit IRC19:33
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode  https://review.openstack.org/43557319:34
*** crushil has joined #openstack-ironic19:35
*** Sukhdev has quit IRC19:35
*** mgoddard__ has joined #openstack-ironic19:37
*** krtaylor has joined #openstack-ironic19:37
*** mgoddard_ has quit IRC19:39
*** krtaylor has quit IRC19:40
*** krtaylor has joined #openstack-ironic19:41
*** e0ne has joined #openstack-ironic19:41
openstackgerritChristopher Dearborn proposed openstack/ironic master: Wait until iDRAC is ready before out-of-band cleaning  https://review.openstack.org/46608619:45
openstackgerritChristopher Dearborn proposed openstack/ironic master: Wait until iDRAC is ready before out-of-band cleaning  https://review.openstack.org/46608619:47
*** ijw has quit IRC20:00
*** Goneri has joined #openstack-ironic20:00
*** mgoddard__ has quit IRC20:01
*** harlowja has quit IRC20:03
*** mat128 has quit IRC20:11
*** ijw has joined #openstack-ironic20:16
*** jcoufal has joined #openstack-ironic20:26
*** jcoufal_ has quit IRC20:29
*** harlowja has joined #openstack-ironic20:30
*** Serlex has quit IRC20:31
*** e0ne has quit IRC20:31
*** krtaylor has quit IRC20:34
*** yee379 has quit IRC20:35
*** rbudden has quit IRC20:35
*** yee379 has joined #openstack-ironic20:35
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: WIP: Some utils.execute() stuff  https://review.openstack.org/46610020:35
*** harlowja has quit IRC20:39
*** jkilpatr has quit IRC20:41
*** jcoufal_ has joined #openstack-ironic20:42
*** jrist has quit IRC20:42
*** srobert has quit IRC20:43
*** jcoufal has quit IRC20:44
*** Serlex has joined #openstack-ironic20:46
*** Serlex has quit IRC20:46
*** Sukhdev has joined #openstack-ironic20:49
*** hamzy has quit IRC20:52
*** rbudden has joined #openstack-ironic20:54
*** jrist has joined #openstack-ironic20:56
*** trown is now known as trown|outtypewww20:57
*** jcoufal_ has quit IRC20:59
*** athomas has quit IRC21:10
*** baoli_ has quit IRC21:12
*** jhesketh_ has joined #openstack-ironic21:14
*** jrist has quit IRC21:15
*** jkilpatr has joined #openstack-ironic21:16
*** jhesketh has quit IRC21:18
*** bfournie has quit IRC21:21
*** ijw has quit IRC21:29
*** ijw has joined #openstack-ironic21:30
*** crushil_ has joined #openstack-ironic21:30
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/46524821:31
*** ijw has quit IRC21:35
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS  https://review.openstack.org/39640621:36
*** baoli has joined #openstack-ironic21:37
*** fragatin_ has joined #openstack-ironic21:39
*** dprince has quit IRC21:40
*** baoli has quit IRC21:42
*** fragatina has quit IRC21:42
*** fragatin_ has quit IRC21:45
*** fragatina has joined #openstack-ironic21:46
*** crushil_ has quit IRC21:53
*** rbudden has quit IRC21:53
*** yee379 has quit IRC22:01
*** yee379 has joined #openstack-ironic22:01
*** baoli has joined #openstack-ironic22:01
openstackgerritLin Yang proposed openstack/sushy master: Parse generic attributions for redfish resource  https://review.openstack.org/44883422:04
*** ijw has joined #openstack-ironic22:05
srart'𝄡𝄢𝄞'22:09
*** crushil_ has joined #openstack-ironic22:11
*** yuanying has quit IRC22:12
*** crushil_ has quit IRC22:12
*** yuanying has joined #openstack-ironic22:12
*** bfournie has joined #openstack-ironic22:16
*** bfournie has quit IRC22:17
*** Sukhdev has quit IRC22:20
*** Sukhdev has joined #openstack-ironic22:36
*** chlong has quit IRC22:45
openstackgerritPeter Piela proposed openstack/ironic-ui master: Setup JavaScript test environment  https://review.openstack.org/45373022:56
*** pmannidi has joined #openstack-ironic23:11
*** rloo has quit IRC23:15
*** Goneri has quit IRC23:21
*** baoli has quit IRC23:23
*** ijw has quit IRC23:35
*** Fdaisuke_ has joined #openstack-ironic23:43
*** Fdaisuke has quit IRC23:43
*** zackf has quit IRC23:48
*** zackf has joined #openstack-ironic23:51
*** ijw has joined #openstack-ironic23:56
*** ijw has quit IRC23:56
*** zackf has quit IRC23:56
*** ijw has joined #openstack-ironic23:57
*** ijw has quit IRC23:58
*** ijw has joined #openstack-ironic23:59

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