Monday, 2018-01-29

*** cjloader has joined #openstack-ironic00:02
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/53705200:09
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements  https://review.openstack.org/53493500:10
*** lbragstad has joined #openstack-ironic00:14
openstackgerritOpenStack Proposal Bot proposed openstack/networking-baremetal master: Updated from global requirements  https://review.openstack.org/53707800:16
*** yangyapeng has quit IRC00:29
*** yangyapeng has joined #openstack-ironic00:30
*** yangyapeng has quit IRC00:34
*** cjloader has quit IRC00:40
*** cjloader has joined #openstack-ironic00:40
*** cjloader has quit IRC00:45
*** gcb has joined #openstack-ironic00:53
*** hieulq__ has quit IRC00:55
*** cjloader has joined #openstack-ironic00:56
*** tuanla____ has joined #openstack-ironic01:00
*** cjloader has quit IRC01:01
*** phuongnh has joined #openstack-ironic01:03
*** tiendc has joined #openstack-ironic01:07
*** hieulq__ has joined #openstack-ironic01:11
*** rbudden has joined #openstack-ironic01:12
openstackgerritDao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode  https://review.openstack.org/52411801:14
*** yangyapeng has joined #openstack-ironic01:16
openstackgerritDao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode  https://review.openstack.org/52411801:17
*** gcb has quit IRC01:21
*** zhangfei has joined #openstack-ironic01:42
openstackgerritShangXiao proposed openstack/ironic master: [api-ref] Add fault names for the error response codes  https://review.openstack.org/53871701:58
openstackgerritDongcan Ye proposed openstack/ironic master: Add meaningful exception in Neutron port show  https://review.openstack.org/53364002:10
*** phuongnh has quit IRC02:15
*** cjloader has joined #openstack-ironic02:15
*** phuongnh has joined #openstack-ironic02:15
openstackgerritMerged openstack/ironic-lib master: Use the 'ironic' queue for the gate  https://review.openstack.org/53797202:18
*** zhangfei has quit IRC02:20
*** crushil has joined #openstack-ironic02:21
*** crushil_ has joined #openstack-ironic02:21
openstackgerritJames E. Blair proposed openstack/sushy master: Zuul: Remove project name  https://review.openstack.org/53871902:26
*** tuanla____ has quit IRC02:27
*** tuanla____ has joined #openstack-ironic02:28
*** crushil_ has quit IRC02:33
*** crushil has quit IRC02:33
*** zhangfei has joined #openstack-ironic02:37
*** cjloader has quit IRC02:40
openstackgerritJames E. Blair proposed openstack/sushy stable/pike: Zuul: Remove project name  https://review.openstack.org/53872402:46
*** rcernin has quit IRC03:08
*** harlowja has joined #openstack-ironic03:11
*** harlowja has quit IRC03:18
*** zhangfei has quit IRC03:23
*** jgwentworth is now known as melwitt03:26
*** zhangfei has joined #openstack-ironic03:35
*** rbudden has quit IRC04:03
*** rcernin has joined #openstack-ironic04:04
*** stendulker has joined #openstack-ironic04:06
openstackgerritDao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests  https://review.openstack.org/52869904:17
openstackgerritDao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode standalone tests  https://review.openstack.org/53811904:18
openstackgerritDao Cong Tien proposed openstack/ironic master: CI job for rescue mode  https://review.openstack.org/52870404:19
openstackgerritDao Cong Tien proposed openstack/ironic master: DNM: test rescue mode  https://review.openstack.org/53318704:20
openstackgerritDao Cong Tien proposed openstack/ironic master: Update "standalone" job for supporting rescue mode  https://review.openstack.org/53782104:21
openstackgerritDao Cong Tien proposed openstack/ironic master: Update "standalone" job for supporting rescue mode  https://review.openstack.org/53782104:22
openstackgerritDao Cong Tien proposed openstack/ironic master: DNM: standalone test rescue mode  https://review.openstack.org/53475104:23
*** Nisha_Agarwal has joined #openstack-ironic04:25
Nisha_AgarwalMorning Ironic!!!04:25
*** stendulker has quit IRC04:28
*** stendulker has joined #openstack-ironic04:28
*** pmannidi has quit IRC04:47
*** AlexeyAbashkin has joined #openstack-ironic05:06
*** AlexeyAbashkin has quit IRC05:10
*** AlexeyAbashkin has joined #openstack-ironic05:31
*** stendulker_ has joined #openstack-ironic05:33
*** stendulker has quit IRC05:34
*** AlexeyAbashkin has quit IRC05:46
openstackgerritHoang Trung Hieu proposed openstack/ironic master: Zuul: Remove project name  https://review.openstack.org/53875905:59
openstackgerritHoang Trung Hieu proposed openstack/ironic-inspector master: Zuul: Remove project name  https://review.openstack.org/53876006:03
*** e0ne has joined #openstack-ironic06:04
openstackgerritHoang Trung Hieu proposed openstack/ironic-lib master: Zuul: Remove project name  https://review.openstack.org/53876106:05
*** rbartal has joined #openstack-ironic06:07
openstackgerritHoang Trung Hieu proposed openstack/ironic-python-agent master: Zuul: Remove project name  https://review.openstack.org/53876406:07
*** AJaeger has left #openstack-ironic06:08
*** AlexeyAbashkin has joined #openstack-ironic06:17
*** dpawar has joined #openstack-ironic06:18
openstackgerritTuan Luong-Anh proposed openstack/bifrost master: Zuul: Remove project name  https://review.openstack.org/53878106:33
*** phuongnh has quit IRC06:37
*** yolanda_ has quit IRC06:44
*** marios has joined #openstack-ironic06:53
*** e0ne has quit IRC06:56
*** makowals has joined #openstack-ironic06:56
*** Tengu has quit IRC06:58
*** e0ne has joined #openstack-ironic06:58
openstackgerritDao Cong Tien proposed openstack/ironic master: Update "standalone" job for supporting rescue mode  https://review.openstack.org/53782107:02
openstackgerritDao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode standalone tests  https://review.openstack.org/53811907:03
*** rbartal has quit IRC07:04
*** rcernin has quit IRC07:11
*** e0ne has quit IRC07:11
*** mjura has joined #openstack-ironic07:13
*** namnh has joined #openstack-ironic07:21
*** vsaienk0 has joined #openstack-ironic07:30
*** hoonetorg has quit IRC07:35
*** gcb has joined #openstack-ironic07:39
*** stendulker has joined #openstack-ironic07:41
*** hjensas has joined #openstack-ironic07:43
*** pcaruana has joined #openstack-ironic07:44
*** stendulker_ has quit IRC07:44
*** fragatina has joined #openstack-ironic07:44
*** hoonetorg has joined #openstack-ironic07:52
*** rbartal has joined #openstack-ironic07:53
*** AlexeyAbashkin has quit IRC07:55
*** jtomasek has joined #openstack-ironic08:01
pas-hamorning ironic :)08:02
*** Nisha_Agarwal has quit IRC08:04
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Follow-up of rescue mode  https://review.openstack.org/53881908:07
*** links has joined #openstack-ironic08:09
etingofmorning everyone o/08:10
*** zhenguo has joined #openstack-ironic08:16
*** Nisha_Agarwal has joined #openstack-ironic08:20
*** d0ugal has quit IRC08:22
*** tesseract has joined #openstack-ironic08:26
*** d0ugal has joined #openstack-ironic08:27
openstackgerritJames E. Blair proposed openstack/ironic-python-agent master: Zuul: Remove project name  https://review.openstack.org/53882608:28
*** athomas has joined #openstack-ironic08:33
*** AlexeyAbashkin has joined #openstack-ironic08:35
*** vsaienk0 has quit IRC08:35
*** vsaienk0 has joined #openstack-ironic08:36
*** amarao has joined #openstack-ironic08:36
*** AlexeyAbashkin has quit IRC08:40
*** racedo has joined #openstack-ironic08:41
*** masber has quit IRC08:41
*** masber has joined #openstack-ironic08:42
*** mgoddard_ has joined #openstack-ironic08:45
*** cjloader has joined #openstack-ironic08:48
openstackgerritJames E. Blair proposed openstack/ironic-python-agent stable/ocata: Zuul: Remove project name  https://review.openstack.org/53883708:48
*** makowals has quit IRC08:50
*** cjloader has quit IRC08:53
*** andyjones2001 has joined #openstack-ironic08:54
*** fragatina has quit IRC08:55
*** makowals has joined #openstack-ironic08:58
*** d0ugal has quit IRC09:00
*** e0ne has joined #openstack-ironic09:04
*** mgoddard_ has quit IRC09:07
openstackgerritJames E. Blair proposed openstack/ironic-python-agent stable/pike: Zuul: Remove project name  https://review.openstack.org/53884509:08
hjensasmorning etingof \o09:10
etingofhjensas, ;)09:10
*** zshi has quit IRC09:17
*** zshi has joined #openstack-ironic09:18
*** makowals has quit IRC09:18
*** d0ugal has joined #openstack-ironic09:20
*** AlexeyAbashkin has joined #openstack-ironic09:22
*** derekh has joined #openstack-ironic09:23
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Abstract away the virtualization management interface  https://review.openstack.org/52332109:27
openstackgerritNguyen Hung Phuong proposed openstack/ironic-inspector master: Remove empty files  https://review.openstack.org/53885309:31
*** MattMan has quit IRC09:32
*** MattMan has joined #openstack-ironic09:33
*** e0ne has quit IRC09:35
*** rbartal has quit IRC09:37
*** vsaienk0 has quit IRC09:41
*** mgoddard_ has joined #openstack-ironic09:43
*** vsaienk0 has joined #openstack-ironic09:45
*** makowals has joined #openstack-ironic09:45
*** makowals has quit IRC09:45
*** cjloader has joined #openstack-ironic09:48
*** Nisha_Agarwal has quit IRC09:49
*** rbartal has joined #openstack-ironic09:49
*** cjloader has quit IRC09:52
*** makowals has joined #openstack-ironic09:58
*** e0ne has joined #openstack-ironic10:00
*** tuanla_____ has joined #openstack-ironic10:03
openstackgerritHarald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports.  https://review.openstack.org/53604010:04
openstackgerritHarald Jensås proposed openstack/ironic master: Add documentation for baremetal mech  https://review.openstack.org/53735310:04
*** tuanla____ has quit IRC10:05
openstackgerritparesh sao proposed openstack/ironic master: Follow-up for Implementation for UEFI iSCSI boot for ILO  https://review.openstack.org/53886310:07
openstackgerritHarald Jensås proposed openstack/ironic master: Devstack - use neutron segments (routed provider networks)  https://review.openstack.org/53127510:08
openstackgerritHarald Jensås proposed openstack/ironic master: Wait for ironic-neutron-agent to report state  https://review.openstack.org/53163710:08
openstackgerritHarald Jensås proposed openstack/ironic master: [WIP] Add baremetal mechanism driver to gate jobs  https://review.openstack.org/53735410:08
*** namnh has quit IRC10:09
*** namnh has joined #openstack-ironic10:09
*** AlexeyAbashkin has quit IRC10:13
*** AlexeyAbashkin has joined #openstack-ironic10:13
*** gcb has quit IRC10:29
*** threestrands_ has joined #openstack-ironic10:33
*** threestrands has quit IRC10:36
*** lucas-afk is now known as lucasagomes10:37
*** mfranczy has joined #openstack-ironic10:37
*** tiendc has quit IRC10:37
*** jappleii__ has joined #openstack-ironic10:37
*** jappleii__ has quit IRC10:38
*** makowals has quit IRC10:39
*** jappleii__ has joined #openstack-ironic10:39
*** milan has joined #openstack-ironic10:40
*** threestrands_ has quit IRC10:41
*** rbartal has quit IRC10:45
*** cjloader has joined #openstack-ironic10:48
*** namnh has quit IRC10:52
*** cjloader has quit IRC10:53
*** gcb has joined #openstack-ironic10:53
*** zhangfei has quit IRC10:54
*** makowals has joined #openstack-ironic10:56
*** gcb has quit IRC10:59
hjensasany cores around that can review https://review.openstack.org/#/c/521838/ ? The previous two changes merged are useless without it ...11:00
patchbotpatch 521838 - networking-baremetal - Switch from MechanismDriver to SimpleAgentMechanis...11:00
-openstackstatus- NOTICE: Zuul is currently under heavy load. Do not *recheck* or *approve* any changes.11:03
*** stendulker_ has joined #openstack-ironic11:04
*** stendulker has quit IRC11:07
*** stendulker_ has quit IRC11:09
*** gcb has joined #openstack-ironic11:17
*** gcb has quit IRC11:18
*** AlexeyAbashkin has quit IRC11:23
sambetts|afkMorning all11:23
*** sambetts|afk is now known as sambetts|11:24
*** sambetts| is now known as sambetts11:24
*** marios has quit IRC11:25
*** nicodemos has joined #openstack-ironic11:26
nicodemosmorning, ironic.11:27
*** tuanla_____ has quit IRC11:36
openstackgerritDebo Zhang proposed openstack/ironic master: Correct link address  https://review.openstack.org/53888611:40
*** cjloader has joined #openstack-ironic11:48
*** cjloader has quit IRC11:52
*** AlexeyAbashkin has joined #openstack-ironic11:59
openstackgerritHugo Nicodemos proposed openstack/ironic master: Soft power operations for OneView hardware type  https://review.openstack.org/51068512:02
*** fragatina has joined #openstack-ironic12:02
*** charcol-v2 has quit IRC12:06
*** marios has joined #openstack-ironic12:18
*** bfournie has quit IRC12:33
*** bfournie has joined #openstack-ironic12:33
*** bfournie has quit IRC12:37
*** vsaienk0 has quit IRC12:39
*** vsaienk0 has joined #openstack-ironic12:39
*** jlabarre has joined #openstack-ironic12:39
jrollmorning everyone12:46
sambettsHey jroll, zuul is still sad today12:47
jrolljoy12:47
*** cjloader has joined #openstack-ironic12:48
*** vsaienk0 has quit IRC12:49
*** cjloader has quit IRC12:52
*** dpawar has quit IRC12:54
*** dpawar has joined #openstack-ironic12:56
*** dpawar has quit IRC13:14
*** vsaienk0 has joined #openstack-ironic13:17
*** bfournie has joined #openstack-ironic13:18
*** dprince has joined #openstack-ironic13:19
*** rbartal has joined #openstack-ironic13:23
openstackgerritJames E. Blair proposed openstack/ironic-tempest-plugin master: Zuul: Remove project name  https://review.openstack.org/53891913:30
*** vsaienk0 has quit IRC13:30
-openstackstatus- NOTICE: Zuul is currently under heavy load. Do not *recheck* or *approve* any changes until we give the go ahead.13:32
*** ChanServ changes topic to "Zuul is currently under heavy load. Do not *recheck* or *approve* any changes until we give the go ahead."13:32
*** vsaienk0 has joined #openstack-ironic13:33
*** d0ugal has quit IRC13:38
*** d0ugal has joined #openstack-ironic13:38
*** trown|outtypewww is now known as trown13:42
*** trown is now known as trown|rover13:42
*** vsaienk0 has quit IRC13:42
*** mjturek has joined #openstack-ironic13:42
*** rloo has joined #openstack-ironic13:42
TheJuliagood morning13:47
*** cjloader has joined #openstack-ironic13:48
TheJuliaoh my, zuul does not sound happy13:48
sambettsTheJulia: no :(13:48
sambettsTheJulia: apprently there was an outtage in one of the cloud providers13:48
sambettsso zuul is really backed up13:48
TheJuliain addition to maintenance?13:49
sambettsin addition to everything else13:49
sambettsyeah13:49
sambettsnot great :(13:49
TheJuliaugh13:49
* TheJulia waits for the coffee to finish brewing13:50
*** cjloader has quit IRC13:53
*** rbudden has joined #openstack-ironic13:55
*** cdearborn has joined #openstack-ironic13:59
*** ricardoas has quit IRC13:59
*** dpawar has joined #openstack-ironic14:02
*** dpawar has quit IRC14:03
*** zhenguo has quit IRC14:05
* TheJulia takes a few minutes updates etherpad14:10
*** fragatina has quit IRC14:10
*** fragatina has joined #openstack-ironic14:11
TheJuliaa LOT landed last week14:15
rlooGood Monday morning TheJulia, sambetts, and everyone else!14:16
rlooTheJulia: yes, I'm still catching up, lots of stuff...14:16
* TheJulia is still trying to wake up, but notices the day star is starting to rise14:16
rloosambetts, TheJulia: if zuul is really backed up, should we hold off on pushing up revisions/approving/ etc, except for 'urgent' patches?14:17
sambettsrloo: infra have asked for no rechecks or approvals14:18
sambettsrloo: no sure about revisions, personally I'm holding off14:18
sambettspushing any new patch sets for now14:18
TheJuliarloo: what sambetts said and I think it would be a good idea to hold off. Seems like today is a good day to get lost in a feature or some local code, or something14:18
rloosambetts: thx -- i need to actually read the Topic, cuz it sez it there14:18
TheJuliaPerhaps write a novel, etc.14:19
rlooTheJulia: or just catching up/planning the ironic release? :) bugs, etc? i can think of lots to do14:19
rlooTheJulia: i guess the FFEs still need to be finished, so maybe more reviews :-(14:20
*** r-daneel has joined #openstack-ironic14:20
*** vsaienk0 has joined #openstack-ironic14:20
TheJuliaYeah14:22
*** Goneri has joined #openstack-ironic14:24
*** jcoufal has joined #openstack-ironic14:24
*** mrunge has joined #openstack-ironic14:25
sambettsTheJulia, rloo do we have a list of the remaining patches we want to merge?14:28
*** superdan is now known as dansmith14:28
sambettsI see the list on the whiteboard they are all marked as merged14:28
TheJuliaWe need to make a new list14:29
-openstackstatus- NOTICE: we've been able to restart zuul, and re-enqueue changes for gate. Please hold off on recheck or approves, we are still recovering. More info shortly.14:29
rloosambetts, TheJulia: yeah. i think we can put them under the priorities as proposed priorities, at the meeting we can go over them and ack14:29
*** makowals has quit IRC14:29
*** milan_ has joined #openstack-ironic14:29
*** milan has quit IRC14:30
rloosambetts, TheJulia: and leave existing priorities as-is until we update in today's meeting14:30
*** dpawar has joined #openstack-ironic14:30
rloooh yeah, dmitry might not be at the meeting. TheJulia, you're in charge :)14:30
TheJuliarloo: I was kind of thinking the same thing at this point.14:30
TheJulia\o/14:30
rlooTheJulia: i'll look/update that list in about 30min, still going through my emails etc14:31
*** makowals has joined #openstack-ironic14:31
*** makowals has quit IRC14:32
sambettssounds good, it would be really helpful for me, I've been distracted by downstream/networking-cisco priorities since just before Christmas trying to get releases out so I'm kind of out of sync with which things are priority now14:32
TheJuliarloo: ack, I'm thinking for rescue we might not be able to land much, that is unless all client actions are done in tempest instead.14:32
rloosambetts: for you, the network-related stuff to review14:33
TheJuliaheh14:33
sambettsack :)14:33
*** makowals has joined #openstack-ironic14:34
rloosambetts: you know which ones I mean? I think from here but I don't actually know: https://review.openstack.org/#/q/topic:bug/1658964+(status:open+OR+status:merged)14:34
sambettsthose were the ones I was think of, plus TheJulia's vif lock patch14:35
sambettsI think that the vif lock patch is really needed sooner rather than later14:36
TheJuliaYeah, that might be a good one to sort out this week since that _might_ just cause less false positive failures as well14:36
rloosambetts: ok, i trust your judgement !14:36
sambettsTheJulia: I agree14:36
TheJuliasambetts: Also if you find free time, you may want to take a look at a spec, https://review.openstack.org/#/c/532468/14:37
patchbotpatch 532468 - ironic-specs - Support ISCDHCPDriver for Standalone Deployment14:37
sambettsTheJulia: I'll try to get a momment to go through it14:37
TheJuliasambetts: no rush, it already needs a revision :(  But conceptually, given networking involvement and integration/use cases, it might be interesting to you14:38
sambettsTheJulia: based on the title I'm interested, I've also been thinking about additional network stuff for standalone recently14:38
*** baha has joined #openstack-ironic14:40
TheJuliaditto, I remember we tried to change the flags/config around for dnsmasq to be the numbers instead of strings to be more friendly to non-dnsmasq neutron installs.... except I seem to remember we backed off14:41
*** jlabarre has quit IRC14:41
sambettsI think that patch landed, it might have been reverted though14:41
TheJuliaYeah, I'm fairly sure we had to revert it14:42
*** jlabarre has joined #openstack-ironic14:42
rlooTheJulia, sambetts: If I push a patch up and then abandon it right away, will it be removed from zuul?14:42
sambettsTheJulia: just going through your vif patch, qq about the locks in Ironic, can we hold a shared=True tasks then there is an exlusive lock held somewhere else?14:42
TheJuliarloo: I don't think so.....14:42
TheJuliarloo: best to ask infra if zuul acts upon abandon actions14:42
sambettsrloo: I'm not sure actually, I think the only thing that removes a patch from zuul is a new patch14:42
rlooTheJulia, sambetts: I'll ask on #infra...14:43
TheJuliasambetts: So, If I understand the code correctly, we can hold a shared lock running, and then another task obtain an exclusive lock. Both will continue to run, since we only record/track the exclusive locking. Really, shared task locking should be called "I just want to create a task...."14:44
rloopabelanger: rloo: yes, abandon will dequeue it from zuul14:45
rlooTheJulia, sambetts: ^^14:45
sambettsrloo: Nice!14:45
TheJuliasambetts: other way around, with an exclusive lock present, I think we're still able to launch a new task that is shared, which can then try to upgrade the lock and fail then14:45
*** SerenaFeng has joined #openstack-ironic14:46
sambettsTheJulia: yeah thats what I thought, its always confused me because I orginally thought it would work like you can hold multiple shared = true, but if there were any shared happening an excludsive wouldn't be able to claim it14:46
sambettsbecause its exclusive, then then the code made me doubt it when I looked so wanted to check14:47
TheJuliayeah, except aiui it can at present since the actual existence of the lock is not recorded14:47
TheJuliawhen "shared"14:47
sambettsso basically exclusive lock =  a lock, and shared actually mean no lock :-P14:47
TheJuliayup14:47
sambettsok cool, that makes the vif patch make more sense, because I was thinking if shared=True blocked anything grabbing an exclusive lock then we've not really helped the problem14:48
*** jcoufal has quit IRC14:49
TheJuliaYeah, the locking code is actually super simple and quick to read through, fwiw14:51
*** gcb has joined #openstack-ironic14:52
*** cjloader has joined #openstack-ironic14:53
openstackgerritRuby Loo proposed openstack/ironic master: Remove unused code in common/neutron.py  https://review.openstack.org/53893614:56
*** cjloader has quit IRC14:57
rloosambetts: ^^. i abandoned it, does it mean you can't vote on it?14:57
sambettsrloo: no can't vote on it14:57
*** cjloader has joined #openstack-ironic14:57
rloosambetts: follow up to your patch14:57
rloosambetts: i'll ping you when i unabandon it :)14:58
rloosambetts: oh, i see your comment now :)14:58
sambettsrloo: just left a comment on it saying a vocal +214:58
*** hw_wutianwei has joined #openstack-ironic14:58
rloosambetts: thx. ok, i'll look into list of patches to review...14:58
sambettsI'm surprised pep8 doesn't catch a flow issue like that, i.e. an unreachable return statement14:59
sambettsI'm sure I've seen something catch an error like that before14:59
rloosambetts: i was surprised too. by two things; unreachable code, and non-existing variable.14:59
*** jcoufal has joined #openstack-ironic15:00
*** yangyapeng has quit IRC15:01
*** lucasagomes is now known as lucas-hungry15:01
*** r-daneel_ has joined #openstack-ironic15:02
etingofpep8 is more focused on code style. pylint might have caught those bugs...15:03
*** Goneri has quit IRC15:03
*** r-daneel has quit IRC15:04
*** r-daneel_ is now known as r-daneel15:04
*** SerenaFeng has quit IRC15:06
*** crushil_ has joined #openstack-ironic15:07
*** crushil has joined #openstack-ironic15:07
*** crushil_ has quit IRC15:08
*** rbartal has quit IRC15:09
*** dpawar has quit IRC15:10
*** r-daneel has quit IRC15:11
*** r-daneel has joined #openstack-ironic15:11
TheJuliahjensas: Do you still feel there is need to discuss the process addition for flat networking?15:11
hjensasTheJulia: I submitted another patch this morning. I brought the config option back in, but I set it deprecated and the default is to use the new behaviour.15:13
hjensasTheJulia: Not sure it is ok to introduce a new option, and deprecate it at the same time ... :)15:13
*** makowals_ has joined #openstack-ironic15:13
TheJuliahjensas: putting in a deprecation right now is a little tricky because realistically it wouldn't be wrapped up until s*15:14
TheJuliabecause landing stuff at the end of a cycle is... difficult15:14
sambettspersonally I'm not sure we need the config option, the current behaviour is that neutron won't bind the port, and once we switch to baremetal and the uuid for host id, then it will still fail to bind the port15:14
sambettsso no change in bahviour15:14
sambettsunless they then load the networking-baremetal driver, to rubber stamp the ports15:15
sambettsthen it will be bound correctly15:15
TheJuliaAgreed the port binding will fail, but there was (and this is going way back in memory) a time where if the port binding failed the dhcp config was ripped out15:16
*** makowals has quit IRC15:16
sambettsthe port binding failing used to be an issue when it was done in nova, because nova checked it and made a failure to bind fatal15:17
* TheJulia would have to go hunt down the original binding failure bug at this point to remember that context15:17
sambettsnow we bind in ironic, we don't check if its successful or not15:17
sambetts(the neutron events work is to improve that)(15:17
*** Goneri has joined #openstack-ironic15:17
sambettsso a failure to port bind is ok15:18
TheJuliaAhh, so that was the behavior i was remembering15:18
sambettswell no ok, more not fatal15:18
sambettsnot*15:18
TheJuliayeah15:18
*** hshiina has joined #openstack-ironic15:18
TheJuliao/ hshiina15:20
hshiinaTheJulia, o/15:21
TheJuliasambetts: hjensas: Since I'm remembering really old behavior I guess it is okay...  Although I need to think about it some more15:21
sambettsTheJulia: yeah, I think hjensas proved that its OK without the config option, because the current devstack setup runs with a VM VXLAN neutron network, and then his patch binds with the baremetal vnic type and node uuid for host id, and it just logs then warning in neutron but continues to work, right hjensas?15:23
hjensasTheJulia: ok, I'll leave it as is for now. Just let me know if I should rip the opt out again.15:23
hjensassambetts: TheJulia: yes, it still works. The only difference is in CI we get some errors in the neutron logs that it does not bind since no mech driver can support the combined binding properties.15:24
sambettsyeah I think we get some of those today anyway, because even though neutron tries to get it agent to bind the port, nova never connects the other end15:26
sambettsso it nevers binds15:26
*** ijw has joined #openstack-ironic15:28
*** mfranczy has quit IRC15:33
TheJuliaI suspect I'm going to have to go hunt down the bug from long ago to put my mind at ease15:33
*** Goneri has quit IRC15:36
rlooTheJulia: sorry to bug you but do you know what the status is wrt this? cuz I don't know if we granted a FFE to them, no one objected, but no one +1'd either: http://lists.openstack.org/pipermail/openstack-dev/2018-January/126508.html15:38
rlooTheJulia: hoping it is already merged so I don't need to look into it15:38
*** Goneri has joined #openstack-ironic15:39
*** r-daneel_ has joined #openstack-ironic15:39
*** r-daneel has quit IRC15:40
*** r-daneel_ is now known as r-daneel15:40
TheJuliaThey were asking around in IRC which might be why15:40
TheJuliarloo: going through the FFE emails and making a list?15:40
rlooTheJulia: if it was granted, would you please send email about it?15:41
rlooTheJulia: yeah, updating the whiteboard now with patches.15:41
TheJuliarloo: trying to remember when off-hand, but sure15:41
rlooTheJulia: just have routed networks left. and that ^^, which is why i'm checking. do i need to look at that?15:41
TheJuliarloo: thanks!15:41
hjensassambetts: not, sure but looks like https://review.openstack.org/#/c/392959/ worked well in the CI. All green, but still the Zuul -1. Pity there are no logs ...15:42
patchbotpatch 392959 - ironic - Rework Ironic devstack baremetal network simulation15:42
TheJuliaI seemed to remember they were 1 or 2 patches away15:42
rlooTheJulia: did we grant them a FFE? If not, I'm not going to look...15:42
*** jlvillal has left #openstack-ironic15:42
*** jlvillal has joined #openstack-ironic15:43
TheJuliaI indicated that I would review their patches, I'm wondering if anyone else did publicly which would essentially be approval for them15:43
sambettshjensas: if you go down to the bottom of the page and toggle CI, the last Zuul result had POST_FAILURES causing it to -115:43
* TheJulia searching irc logs15:43
rlooTheJulia: the email said that you and dmitry agreed to review. i don't know if that is an explicit agreement to FFE15:43
rlooTheJulia: but if you think it is... maybe someone could just reply to that email to confirm.15:44
TheJuliarloo: Yeah, I think it is agreement. Sure15:44
rlooTheJulia: personally i was not for it. but not against it either. given what happened before, i do not want to get involved at this late time.15:44
hjensassambetts: oh, neat. I never tried that button before.15:44
sambettshjensas: really useful for finding logs from old runs15:45
hjensassambetts: yeah, thanks for pointing it out. Now I can stop going to e-mails to figure out it was due to merge issue as well. :D15:46
sambettsno probs :)15:47
*** gcb has quit IRC15:47
*** namnh has joined #openstack-ironic15:49
*** lucas-hungry is now known as lucasagomes15:59
*** hw_wutianwei has quit IRC16:01
*** tesseract has quit IRC16:03
*** mjura has quit IRC16:04
*** namnh has quit IRC16:04
*** jappleii__ has quit IRC16:05
*** fragatina has quit IRC16:08
NobodyCamGood Morning Ironic'ers16:09
rloomorning NobodyCam!16:11
NobodyCamGood Morning rloo :) happy Monday!16:11
rloohjensas, sambetts: wrt the routed networks support. i was about to look at the emails to figure out which patches need to be reviewed, but I think I'd like to delegate, ask (one of) you to do that and update L187ish, https://etherpad.openstack.org/p/IronicWhiteBoard with list16:12
rloohjensas, sambetts: thx in advance!16:13
hjensasrloo: sambetts: I can do it.16:13
sambettshjensas: thanks!16:14
*** vsaienk0 has quit IRC16:16
*** rbartal has joined #openstack-ironic16:18
* TheJulia watches the web browser cog just kind of spin and wonders if it is going to be one of those days16:19
*** links has quit IRC16:20
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"16:21
-openstackstatus- NOTICE: zuul.o.o is back online, feel free to recheck / approve patches.16:21
*** mjturek has quit IRC16:21
TheJulia\o/16:21
hjensas\o\ \o/ /o/16:22
*** mjturek has joined #openstack-ironic16:24
*** pcaruana has quit IRC16:25
*** yangyapeng has joined #openstack-ironic16:30
sambetts\o/16:31
*** yangyapeng has quit IRC16:34
openstackgerritMerged openstack/ironic-specs master: Future of classic drivers: update the migration proposal  https://review.openstack.org/53629816:35
*** ricardoas has joined #openstack-ironic16:39
TheJuliaOh the fun of pre-typing meeting commands :)16:40
rlooTheJulia: qq, i had commented that I didn't like 'rescuepassword', this patch changes it, but I think it is too late? https://review.openstack.org/#/c/538819/1/ironicclient/v1/node.py16:41
patchbotpatch 538819 - python-ironicclient - Follow-up of rescue mode16:41
rlooTheJulia: def too late for the others unless we deprecate. i think too late for rescuepassword cuz the patch landed already?16:42
TheJuliaThat can be changed, since we'e already released stable/queens version of ironicclient and resuce cli stuff didn't make it16:42
TheJuliarloo: it has only been on master for the weekend... and is unreleased16:42
rlooTheJulia: ok. i don't like the inconsistency but i prefer the change.16:42
TheJuliaA quick reno and I think we're good16:42
rlooTheJulia: thx!16:42
TheJulianp16:42
jlvillalo/16:45
TheJuliaI only landed them to master because we branched, and we needed them to get out of the queue otherwise we needlessly block ourselves. We just have to be careful with testing/devstack changes since it will have to finish landing next cycle :(16:45
*** e0ne has quit IRC16:46
*** vsaienk0 has joined #openstack-ironic16:47
*** tesseract has joined #openstack-ironic16:47
rlooTheJulia: oh. we can't use the client changes for rescue until the next cycle? Yeah, that makes sense.16:49
TheJuliaexactly16:52
TheJuliaOn a plus side, hard freeze is friday16:52
jlvillallucasagomes, jroll, I added you to: https://review.openstack.org/#/c/538970/  Since I think you were involved in that code.16:54
patchbotpatch 538970 - oslo.utils - Document specs_matcher.py functions16:54
jrolloh no16:56
jroll:P16:56
jlvillalheh ;)16:56
jlvillalLet's just say I was 100% confused when I wandered into that code. And now I am at about 50% confused :)16:56
lucasagomesjlvillal, hey16:57
lucasagomeshaha will take a look16:57
lucasagomesoh man16:57
jlvillallucasagomes, Thanks!16:57
jrolljlvillal: you're less confused than most of the community, then, congrats!16:57
TheJuliaIs it just me, or is gerrit lagging horribly?16:57
*** vsaienk0 has quit IRC16:57
lucasagomesjroll, ++16:57
jrollTheJulia: fine for me16:57
jlvillalTheJulia, I haven't noticed any horrible lagging. Seems normal-ish to me.16:58
TheJuliaIt might just be the fun of cellular internet16:59
*** rpioso|afk is now known as rpioso16:59
*** stendulker has joined #openstack-ironic17:00
TheJulia#startmeeting ironic17:00
openstackMeeting started Mon Jan 29 17:00:32 2018 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
TheJuliao/17:00
*** openstack changes topic to " (Meeting topic: ironic)"17:00
openstackThe meeting name has been set to 'ironic'17:00
jroll\o/17:00
hjensaso/17:00
jlvillalO/ reo17:00
crushil\o17:00
rpiosoo/17:00
rlooo/17:01
hshiinao/17:01
stendulkero/17:01
*** fragatina has joined #openstack-ironic17:01
etingofo/17:01
bahao/17:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic17:01
TheJuliaAs always, our agenda can be found on the wiki17:01
ricardoaso/17:01
bfournieo/17:02
TheJulia#topic Announcements / Reminder17:02
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"17:02
*** srart has joined #openstack-ironic17:02
* rloo thinks folks haven't updated their subteam status reports... do it now, quick... :)17:02
TheJulia#info Feature Freeze started last Thursday17:02
TheJulia#info Hard freeze starts this Friday, Feburary 2nd.17:02
TheJulia#info Please expect our release to be cut anytime between Friday February 2nd and Feburary 8th.17:03
rlooTheJulia: really? given our FFE is until Feb 2...17:04
*** Nisha_Agarwal has joined #openstack-ironic17:04
Nisha_Agarwalo/17:04
TheJulia#info Only bug fixes should be landed after this Friday. All features with Feature Freeze Exceptions must be landed by Feb 2nd.17:04
NobodyCamo/17:05
TheJuliaAnyone have anything else announcement or reminder wise?17:05
rlooTheJulia: hard string freeze is week of Feb 5. what is 'hard freeze' ^^ ?17:05
rlooTheJulia: https://releases.openstack.org/queens/schedule.html17:05
TheJuliarloo: Hard freeze as in we're done for the cycle and are ready to cut our branch.17:06
rlooreminder about ptg? ptl election?17:06
rlooTheJulia: rc1 target week is the hard freeze, right? (next week)17:06
TheJuliaSince we don't cut an RC, we need to be ready to cut our branch early unless we want to have fun cherry picking17:07
rlooTheJulia: right, i think that we want ours to be in sync with rc117:07
mjtureko/17:07
jrollyeah, I believe we did want to cut at the same time that everyone else does rc117:07
TheJulia#info PTL nominations start later today \o/17:07
TheJulia#info The PTG is coming, please submit topics to the etherpad before Feb 15th.17:08
TheJulia#link https://etherpad.openstack.org/p/ironic-rocky-ptg17:08
TheJuliajroll: correct17:08
*** ijw has quit IRC17:09
TheJuliaWow, this was an annoucement/reminder heavy day17:09
*** sauloaislan has joined #openstack-ironic17:09
TheJuliaIf there is nothing else, we should move on17:09
jrollright, so to be clear: feb 2: deadline for landing any feature, FFE or not, to ironic. feb 9: goal to cut stable/queens branch. yes?17:09
TheJuliajroll: Correct17:10
rloojroll: yup. although we should try for feb 8 (Thurs).17:10
jlvillalUnrelated. I have to say I'm glad we have our own 'gate' queue. Otherwise our patches would still be in the process of landing :)17:10
jrollexcellent. thanks17:10
rloothx jlvillal :)17:10
jlvillalI didn't do it. Someone else did it before me. I was just cleaning up some stragglers.17:10
TheJulia#topic Review action items from previous meeting17:11
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"17:11
jlvillalBut nothing of ours was in the 'integrated' queue. Which is the really big one.17:11
TheJulia#link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-22-17.00.html17:11
TheJuliaOur only action item from last week was that mjturek would take care of bug triaging this week. mjturek, anything to report?17:11
*** vsaienk0 has joined #openstack-ironic17:12
rlooTheJulia: should we enable the multinode-grenade job as voting?17:12
mjturekTheJulia nothing really, there were a couple of IPv6 bugs that I looked at17:12
TheJuliamjturek: Awesome, thanks!17:12
mjtureknp!17:13
rloo(sorry, TheJulia, I'll take that off line about the grenade job)17:13
TheJuliarloo: Perhaps discussion, although personally I've not had a chance to dig through logs and really evaluate the failure rate with recent gate issues17:13
TheJulia#topic Review subteam status reports17:13
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"17:13
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard17:13
TheJuliaStarting around line 232 :)17:14
TheJuliaI think the largest thing of note is that we were able to land quite a bit last week, and we were able to release python-ironicclient with client negotiation changes as well as support for traits. Resuce support did not make it into the client :(17:15
rlooTheJulia: could you update version negotiation? i just quickly updated drivers deprecation17:15
*** milan_ has quit IRC17:16
rloojroll: wrt traits, did you push up a patch to test in nova?17:17
rloojroll: ah, i see it. thx :)17:18
jrollrloo: yeah, we decided just to use functional tests for now17:18
TheJuliarloo: done17:18
rloojroll: great. looks like the nova patches have all been approved, just waiting to land!17:18
jrollyep :)17:18
jrollwe dun good17:18
* TheJulia shifts out of the slowly shifting day star beam17:19
TheJuliaOkay, I can see again17:19
TheJuliarloo: I _think_ we are done with the auth rework17:20
rlooTheJulia: ok. i can't even see it on the etherpad. too many things...17:20
TheJuliaYeah, the etherpad needs a good cleanup17:21
TheJuliaI think we are good to move on?17:21
*** etingof is now known as etingof|afk17:21
rlooTheJulia: found it, i updated with DONE17:21
*** r-daneel_ has joined #openstack-ironic17:21
rloo++ to clean up17:21
rloo++ to moving on17:21
jroll+117:22
TheJulia#topic Deciding on priorities for the next week17:22
*** openstack changes topic to "Deciding on priorities for the next week (Meeting topic: ironic)"17:22
rlooTheJulia: darn, i have a question about version negotiation17:22
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard17:22
TheJuliarloo: ask ? :)17:22
rlooTheJulia: is there anything more that needs to be done for that? in nova?17:22
*** vsaienk0 has quit IRC17:22
TheJuliarloo: at this point, negotiation/discussion17:22
rlooTheJulia: ok, no more patches for queens. thx.17:23
TheJuliaI think we're not doing anything else with that until after the PTG17:23
TheJuliarloo: correct17:23
rlooTheJulia: do we need to discuss with nova at ptg?17:23
*** r-daneel has quit IRC17:23
*** r-daneel_ is now known as r-daneel17:23
jrollwe should be able to discuss it outside of the ptg, but yeah, nothing for queens17:23
TheJuliaOkay, as for priorities, rloo created a list of Feature Freeze Exceptions and patches starting on line 15717:23
jrollI thought we had agreed at some level17:23
TheJuliajroll: we kind of had, but at the same time the sands have shifted if that makes sense17:24
jrollI guess, I think folks were just opposed to rushing it :)17:24
rloojroll, TheJulia: I think it makes sense to touch base with nova and make sure we're all in agreement. at or outside ptg :)17:25
jrollyep17:25
TheJuliajroll: that and the original person that drove the ask has stepped out of the community17:25
rlooi'll make a note of it in etherpad17:26
TheJuliaAs for priorities, I think we should work from the FFE list this week17:26
rlooTheJulia: i didn't prioritize that list. it seems like routed networks is the highest?17:27
TheJuliasambetts did propose a vif locking bug fix I submitted as another thing we would likely want to try and land, since it will help improve gate reliability as well17:27
*** rbartal has quit IRC17:27
rlooTheJulia: bugs. yeah... someone needs to go through bugs, see which ones we might want to try to land.17:27
TheJuliarloo: I think routed networks might be good to have as best because traits is basically at docs at this point17:27
TheJulias/best/highest/17:27
rlooTheJulia: oh. and classic drivers deprecation.17:28
TheJuliarescue, we're going to need to sift through that list, and we're blocked on any osc-commands due to them not landing in the client17:28
TheJulia++17:28
rlooTheJulia: that might be #1 if we don't want to delay the deprecation etc.17:28
TheJuliarloo: I agree with that17:28
rlooTheJulia: do you want to massage that for the priorities or do you want me to do it. nowish i guess...17:29
TheJuliarloo: sure17:29
rlooand by the way, there are a lot of patches for routed networks. would be good if someone could indicate which 'need' to land.17:30
TheJuliaOneview and ilo uefi iscsi boot just need follow-ups it looks like.17:30
hjensasfor routed networks at least 521838, 536040 and 537353. since we landed the agent, we need the mech driver updated to actually use the agent_db data.17:30
TheJuliahjensas: would you tag those as needed in the list?17:31
*** andyjones2001 has quit IRC17:31
hjensasTheJulia: yes.17:31
*** yangyapeng has joined #openstack-ironic17:31
TheJuliaI've updated the rescue item to note that we need to review that and block at devstack changes17:32
TheJuliaI can do that after the meeting17:32
rlooThx TheJulia!17:32
rlooTheJulia: do you have a link to the vif bug?17:32
*** mgoddard_ has quit IRC17:33
TheJulia#action TheJulia to review rescue patches and block at appropriate place for devstack to hold for the next cycle.17:33
*** cjloader has quit IRC17:33
TheJuliarloo: https://review.openstack.org/#/c/534441/417:33
patchbotpatch 534441 - ironic - Don't try to lock upfront for vif removal17:33
*** cjloader has joined #openstack-ironic17:33
TheJuliaIs anyone else aware of any bugs that we need to address this week?17:34
*** AlexeyAbashkin has quit IRC17:34
* TheJulia guesses not at this time17:35
rlooTheJulia: I added that bug to the prioriites. we can add more if there are any.17:35
rlooTheJulia: but we don't really have bugs in our code so ...17:35
jrollnever17:35
TheJuliarloo: Thanks! I think the priorities for this week are good, lots of patches, but many of them are smaller changes17:35
TheJuliarloo: only features17:35
rlooheh17:35
TheJuliaI think we're good to move on17:36
TheJuliaAny objections?17:36
*** yangyapeng has quit IRC17:36
rloo++ I'm good with the priorities and to moving on17:36
TheJulia#topic Appointing a bug triaging lead for the coming week17:36
*** openstack changes topic to "Appointing a bug triaging lead for the coming week (Meeting topic: ironic)"17:37
TheJuliaAnyone up for volunteering this week?17:37
mjturekI'm good with continuing17:37
TheJuliamjturek: Awesome, thanks!17:37
mjtureknp17:37
TheJulia#action mjturek to lead bug triage this next week.17:37
*** AlexeyAbashkin has joined #openstack-ironic17:37
*** pc_m has quit IRC17:38
*** amarao has quit IRC17:38
TheJuliaSince we have no discussion topics, I think we should just skip to Open Discussion, unless someone has RFEs that need review?17:38
* TheJulia will accept the crickets as confirmation that we can move to open discussion :)17:38
NobodyCam++17:39
rloo++17:39
*** cjloader has quit IRC17:39
TheJuliaOkay, moving on!17:39
*** cjloader has joined #openstack-ironic17:40
TheJulia#topic Open Discussion17:40
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"17:40
TheJuliaDoes anyone have any topics to bring up for discussion today?17:40
* jroll does not17:40
rlooare these meetings useful?17:40
rloowait, let's not discuss that.17:40
*** mgoddard_ has joined #openstack-ironic17:40
jrollheh17:41
*** rama_y has joined #openstack-ironic17:41
rloocrickets17:41
jrollI suspect they're more useful for people like me that are not plugged in 40+ hours/week17:41
TheJulialol17:41
rloogood, i thought maybe it was only useful for more 'project management' related folks.17:41
TheJuliajroll: Yeah, very much so, and force a checkpoint for those of us who are plugged into it17:41
*** cjloader has quit IRC17:41
*** AlexeyAbashkin has quit IRC17:41
jrollnod17:42
*** cjloader has joined #openstack-ironic17:42
rlooare we waiting now for ... crickets...17:42
TheJuliaYes, the creation of crickets as a service :)17:43
TheJuliaWell, thanks everyone! Have a wonderful week!17:43
TheJulia#endmeeting17:43
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"17:43
openstackMeeting ended Mon Jan 29 17:43:35 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:43
jrollTheJulia: thanks for hosting :)17:43
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-29-17.00.html17:43
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-29-17.00.txt17:43
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-29-17.00.log.html17:43
NobodyCamThank you Everyone! Good meeting17:44
*** pc_m has joined #openstack-ironic17:45
*** cjloader has quit IRC17:45
*** cjloader has joined #openstack-ironic17:46
*** fragatina has quit IRC17:47
*** cjloader has quit IRC17:48
*** cjloader has joined #openstack-ironic17:48
hjensasTheJulia: Regarding the flat change, I did some testing. This is the results when the mech driver is not loaded: http://paste.openstack.org/show/657181/ This is the results when it is loaded: http://paste.openstack.org/show/657273/  - In both cases the ironic node deployed just fine.17:50
*** yangyapeng has joined #openstack-ironic17:52
TheJuliahjensas: awesome!17:53
*** pc_m has quit IRC17:53
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add documentation for rescue interface  https://review.openstack.org/41960617:55
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Do not use asserts with business logic  https://review.openstack.org/53899117:56
*** pc_m has joined #openstack-ironic17:56
*** Nisha_Agarwal has quit IRC17:56
*** yangyapeng has quit IRC17:56
*** hshiina has quit IRC17:57
*** JunOcataGuy is now known as penick18:00
*** derekh has quit IRC18:00
* sambetts ---> home 18:01
sambettsnight all o/18:01
*** sambetts is now known as sambetts|afk18:01
*** marios has quit IRC18:03
*** mgoddard_ has quit IRC18:07
*** weshay|ruck is now known as weshay|ruck|brb18:09
*** trown|rover is now known as trown|lunch18:11
*** stendulker has quit IRC18:13
*** fragatina has joined #openstack-ironic18:18
*** weshay|ruck|brb is now known as weshay18:20
*** weshay is now known as weshay|ruck18:20
jrollhm, release notes for 4.3.0 - 5.0.x are missing here (or under the wrong release, possibly?) https://docs.openstack.org/releasenotes/ironic/mitaka.html18:21
jrollditto for 6.0.0 - 6.1.x: https://docs.openstack.org/releasenotes/ironic/newton.html18:21
jrollall other releases look fine, I assume nobody knows offhand what's up with this?18:21
TheJuliarloo: https://review.openstack.org/#/c/419606/27 suggesting splitting this one into two patches18:21
patchbotpatch 419606 - ironic - Add documentation for rescue interface18:21
TheJuliajroll: I bet it is because we might need to update branch names.. but I thought that was automagic18:22
jrollTheJulia: ah could be, I'll poke around. thanks18:22
rlooTheJulia: yeah, normally I'd suggest that too. At this point, I don't care that much. I guess we have time to do it, not in a hurry any more.18:23
jrollthough newton isn't EOL :/18:24
rloojroll: ugh. it used to work :-(18:24
jrollyeah I know18:24
rloojroll: i wonder if someone put in some logic with the assumption that only a/last major version should be listed.18:24
rloojroll: no, it isn't even that. not sure what is going on.18:25
jrollrloo: yeah, I'll look around, just wondering if this was a known problem or whatever18:25
rloojroll: thx for looking into it18:26
TheJuliajroll: afaik, no known problem, but there is some weird behavior and release note changes could be slightly impacting in that it is really built around projects doing release cantidates18:26
jrollTheJulia: yeah, none of that is intentional though18:27
* jroll digs18:27
TheJuliayeah18:27
TheJuliaAnyway, I should go eat lunch finally18:27
*** baha has quit IRC18:30
*** dsneddon_ has joined #openstack-ironic18:30
*** rama_y has quit IRC18:31
*** dsneddon_ has quit IRC18:32
*** dsneddon_ has joined #openstack-ironic18:33
*** r-daneel_ has joined #openstack-ironic18:33
*** r-daneel has quit IRC18:34
TheJuliaWhite board updated to indicate resuce patches that can, possibly, and not merge18:37
*** r-daneel has joined #openstack-ironic18:37
*** r-daneel_ has quit IRC18:38
*** e0ne has joined #openstack-ironic18:40
*** baha has joined #openstack-ironic18:44
*** rama_y has joined #openstack-ironic18:45
*** mgoddard_ has joined #openstack-ironic18:46
mjturekhey TheJulia, I know things are buuuusy, but the DIB dependency for the lshw patch ( https://review.openstack.org/#/c/501799 ) would you be willing to +A it?18:47
patchbotpatch 501799 - ironic-python-agent - Use lshw in place of dmidecode for the default har...18:47
mjturekfor the lshw patch landed*18:47
*** dsneddon_ has quit IRC18:51
*** dsneddon_ has joined #openstack-ironic18:51
*** yangyapeng has joined #openstack-ironic18:54
*** yangyapeng has quit IRC18:58
*** lucasagomes is now known as lucas-afk19:08
*** cjloader has quit IRC19:08
*** cjloader has joined #openstack-ironic19:09
*** dprince has quit IRC19:10
*** AlexeyAbashkin has joined #openstack-ironic19:11
TheJuliamjturek: sweet19:14
*** trown|lunch is now known as trown|rover19:14
TheJuliamjturek: sure, since it is a bug19:14
*** yangyapeng has joined #openstack-ironic19:14
openstackgerritJim Rollenhagen proposed openstack/ironic master: Explicitly mark earliest-version for release notes  https://review.openstack.org/53900219:15
* jroll waits to ensure that fixes things before pinging people19:15
*** fragatina has quit IRC19:18
* rloo waits for ping from jroll, but doesn't care if there is no ping :)19:18
*** fragatina has joined #openstack-ironic19:18
jrollrloo: an easy review to make docs more correct? I can't think of a patch you'd rather review!19:19
*** yangyapeng has quit IRC19:19
rloojroll: oh, it isn't that. i'm fine if you ask others to review, you know. spread the joy of reviewing etc! :D19:20
rloojroll: i don't want you to feel obligated to ping me, i won't be hurt if you don't. that's all :)19:20
*** harlowja has joined #openstack-ironic19:25
mjturekthanks TheJulia!!19:26
*** tesseract has quit IRC19:31
openstackgerritMark Goddard proposed openstack/ironic master: Fix nits found in node traits  https://review.openstack.org/53738619:33
*** dsneddon_ has quit IRC19:34
*** yangyapeng has joined #openstack-ironic19:35
*** dsneddon_ has joined #openstack-ironic19:36
*** mgoddard_ has quit IRC19:39
*** yangyapeng has quit IRC19:40
*** dprince has joined #openstack-ironic19:45
*** vsaienk0 has joined #openstack-ironic19:49
*** rama_y has quit IRC19:49
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Update docs and generate config file example  https://review.openstack.org/53901019:50
*** rama_y has joined #openstack-ironic19:50
*** e0ne has quit IRC19:54
*** yangyapeng has joined #openstack-ironic19:56
*** e0ne has joined #openstack-ironic19:57
*** vsaienk0 has quit IRC19:59
*** e0ne has quit IRC19:59
*** yangyapeng has quit IRC20:00
*** yangyapeng has joined #openstack-ironic20:16
*** Goneri has quit IRC20:19
*** yangyapeng has quit IRC20:21
*** AlexeyAbashkin has quit IRC20:25
*** jlabarre has quit IRC20:27
*** jlabarre has joined #openstack-ironic20:28
*** sauloaislan has quit IRC20:33
*** namnh has joined #openstack-ironic20:35
*** yangyapeng has joined #openstack-ironic20:37
*** yangyapeng has quit IRC20:41
*** namnh has quit IRC20:41
* TheJulia wonders where she left her brains20:46
rloopas-ha: have you seen this unit test error for ansible hw type? it seems random/sporatic: http://logs.openstack.org/36/538936/1/check/openstack-tox-py27/b6d1655/job-output.txt.gz#_2018-01-29_17_29_02_87444220:54
tonybTheJulia: check under the fridge I often find lost lego there20:54
*** e0ne has joined #openstack-ironic20:54
* rloo sure that TheJulia will find it, maybe you need a pat on the head20:55
*** yangyapeng has joined #openstack-ironic21:01
*** yangyapeng has quit IRC21:05
*** etingof|afk has quit IRC21:05
*** Sukhdev_ has joined #openstack-ironic21:07
jrollrloo: TheJulia: this does indeed fix the issue: https://review.openstack.org/#/c/539002/ . side note: this reno patch also fixes it, but no sense in waiting IMO: https://review.openstack.org/#/c/539009/221:17
patchbotpatch 539002 - ironic - Explicitly mark earliest-version for release notes21:17
patchbotpatch 539009 - reno - support scanning closed stable branches21:17
*** etingof|afk has joined #openstack-ironic21:18
rloojroll: shouldn't we just wait for the reno fix?21:19
rloojroll: why don't you want to wait for it?21:19
jrollrloo: the workaround just makes it explicit, it isn't a bad thing to have anyway (I believe this bug has happened multiple times)21:19
rloojroll: i have seen several issues with reno, don't know if this one has happened before but you might be right.21:20
jrollrloo: as to why not wait, I don't have any control over how long the reno review takes to land, and I tend to think it's worth fixing now (as a person currently going through old release notes for reasons)21:20
rloojroll: it means that when we push out queens, we'll need to add that :earliest-version tag. right now, I think that releasenotes/source/<name>.rst is automatically generated21:20
jrollrloo: the file is automatically generated, but we already modify it later to add the versions (as the rest of that note suggests)21:22
*** yangyapeng has joined #openstack-ironic21:22
*** dprince has quit IRC21:22
rloojroll: i'm just worried about adding more process/overhead for *us*. and yeah, we already modify it. for ironic only.21:22
jrollrloo: anyway, I guess I don't care. I know how to generate the fixed notes locally, so I can get what I need. just thought I'd fix it for others ¯\_(ツ)_/¯21:22
rloojroll: you did get it fixed, didn't you ping doug about it?21:23
jrollrloo: I meant I thought I'd get the published version fixed ASAP.21:23
jrollI don't care enough to argue for it, I just tend to think our users (who might not realize notes are missing) would appreciate the fix asap21:24
jrollso pointed it out. do what you will21:24
*** yangyapeng has quit IRC21:26
TheJuliatonyb: good suggestion, I'll convey that to my other half who is always loosing legos....21:29
tonyb:)21:29
*** e0ne has quit IRC21:32
TheJuliajroll: I agree, it makes sense to fix sooner rather then later... and we do go back and edit the generated file21:32
TheJuliarloo: Ice cream... and ice ream. :)21:32
TheJuliacream21:32
*** r-daneel has quit IRC21:33
*** r-daneel has joined #openstack-ironic21:33
*** Goneri has joined #openstack-ironic21:47
rlooTheJulia: hmm, yes, I think it is time for ice cream :D21:50
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Update docs and generate config file example  https://review.openstack.org/53901021:50
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Update docs and generate config file example  https://review.openstack.org/53901021:52
*** crushil has quit IRC21:55
TheJuliahjensas: Thinking about it (and since I can't seem to find the bug in my email in a reasonable amount of time), I'm okay with https://review.openstack.org/#/c/536040 not including a config option, since the behavior did end up changing from what my brain was recalling with the vif attach/detach work21:59
patchbotpatch 536040 - ironic - Flat networks use node.uuid when binding ports.21:59
hjensasTheJulia: ok, I'll go take it out again. Thanks!22:00
*** trown|rover is now known as trown|outtypewww22:02
TheJuliahjensas: sorry to create more work :(22:02
openstackgerritMerged openstack/ironic-lib master: Add missing 'self' parameter to class methods  https://review.openstack.org/53764022:03
*** baha has quit IRC22:03
hjensasTheJulia: :) No worries.22:04
*** threestrands has joined #openstack-ironic22:05
*** threestrands has quit IRC22:05
*** threestrands has joined #openstack-ironic22:05
*** threestrands_ has joined #openstack-ironic22:07
*** threestrands has quit IRC22:08
*** threestrands_ has quit IRC22:08
*** threestrands has joined #openstack-ironic22:08
*** threestrands has quit IRC22:08
*** threestrands has joined #openstack-ironic22:08
openstackgerritHarald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports.  https://review.openstack.org/53604022:13
openstackgerritHarald Jensås proposed openstack/ironic master: Add documentation for baremetal mech  https://review.openstack.org/53735322:13
*** dmellado has quit IRC22:17
*** dmellado has joined #openstack-ironic22:20
*** threestrands_ has joined #openstack-ironic22:21
*** cdearborn has quit IRC22:23
*** threestrands has quit IRC22:23
TheJuliahjensas: I've replied with some release note wording suggestions, otherwise looks good to me22:29
hjensasTheJulia: thanks, looking.22:29
TheJuliaAwesome, stepping away for a little while22:29
*** jlabarre has quit IRC22:33
*** jlabarre has joined #openstack-ironic22:33
*** namnh has joined #openstack-ironic22:35
*** jappleii__ has joined #openstack-ironic22:35
*** jappleii__ has quit IRC22:36
*** jappleii__ has joined #openstack-ironic22:37
*** jcoufal has quit IRC22:37
*** threestrands_ has quit IRC22:37
*** jlabarre has quit IRC22:39
*** namnh has quit IRC22:40
*** bfournie has quit IRC22:40
*** jlabarre has joined #openstack-ironic22:41
*** rcernin has joined #openstack-ironic22:46
*** jlabarre has quit IRC22:47
*** jlabarre has joined #openstack-ironic22:47
openstackgerritHarald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports.  https://review.openstack.org/53604022:58
openstackgerritHarald Jensås proposed openstack/ironic master: Add documentation for baremetal mech  https://review.openstack.org/53735322:58
*** pmannidi has joined #openstack-ironic23:01
*** charcol-v2 has joined #openstack-ironic23:02
*** weshay|ruck is now known as weshay|ruck|afk23:07
*** yangyapeng has joined #openstack-ironic23:07
*** rbudden has quit IRC23:08
*** Goneri has quit IRC23:09
*** namnh has joined #openstack-ironic23:11
*** yangyapeng has quit IRC23:12
*** namnh has quit IRC23:15
*** rbudden has joined #openstack-ironic23:16
*** bfournie has joined #openstack-ironic23:16
*** yangyapeng has joined #openstack-ironic23:16
*** Sukhdev_ has quit IRC23:16
*** namnh has joined #openstack-ironic23:19
*** bnemec has quit IRC23:20
*** bnemec has joined #openstack-ironic23:20
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Update docs and generate config file example  https://review.openstack.org/53901023:20
*** yangyapeng has quit IRC23:21
*** namnh has quit IRC23:23
*** namnh has joined #openstack-ironic23:24
*** yangyapeng has joined #openstack-ironic23:29
*** yangyapeng has quit IRC23:33
*** namnh has quit IRC23:38
*** cjloader has quit IRC23:38
*** namnh has joined #openstack-ironic23:42
*** logan- has quit IRC23:45
*** jlvillal has quit IRC23:45
*** mkrai has quit IRC23:45
*** honza has quit IRC23:45
*** jistr has quit IRC23:45
*** dtantsur|afk has quit IRC23:45
*** r-daneel has quit IRC23:45
*** mgoddard has quit IRC23:48
*** yangyapeng has joined #openstack-ironic23:49
*** jlvillal has joined #openstack-ironic23:51
*** logan- has joined #openstack-ironic23:51
*** mkrai has joined #openstack-ironic23:51
*** honza has joined #openstack-ironic23:51
*** jistr has joined #openstack-ironic23:51
*** dtantsur|afk has joined #openstack-ironic23:51
*** rpioso is now known as rpioso|afk23:52
*** yangyapeng has quit IRC23:53
openstackgerritMerged openstack/ironic-inspector master: Use the 'ironic' queue for the gate  https://review.openstack.org/53800123:55
openstackgerritMerged openstack/ironic-lib master: Zuul: Remove project name  https://review.openstack.org/53849123:55
*** mgoddard has joined #openstack-ironic23:56
*** sapd_ has quit IRC23:57
*** namnh has quit IRC23:58
*** namnh has joined #openstack-ironic23:59

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