Monday, 2018-02-05

*** cbt has quit IRC00:03
*** SerenaFeng has joined #openstack-ironic00:12
*** ijw has joined #openstack-ironic00:17
*** ijw has quit IRC00:18
*** yangyape_ has quit IRC00:33
*** threestrands has joined #openstack-ironic00:43
TheJuliapeitolm: yes, neutron00:52
TheJuliaultimately dnsmasq in most cases00:52
openstackgerritMerged openstack/ironic master: Correct link address  https://review.openstack.org/53888601:03
*** hshiina has joined #openstack-ironic01:11
*** phuongnh has joined #openstack-ironic01:12
*** zshi has quit IRC01:16
*** ijw has joined #openstack-ironic01:18
*** yangyapeng has joined #openstack-ironic01:20
*** ijw has quit IRC01:23
*** dpawar has joined #openstack-ironic01:25
*** dpawar has quit IRC01:25
*** dpawar has joined #openstack-ironic01:25
*** dpawar has quit IRC01:26
*** dpawar has joined #openstack-ironic01:27
*** zshi has joined #openstack-ironic01:33
*** zhangfei has joined #openstack-ironic01:40
*** namnh has joined #openstack-ironic01:44
*** SerenaFeng has quit IRC01:58
*** mtreinish has quit IRC01:58
*** mtreinish has joined #openstack-ironic01:59
*** threestrands has quit IRC02:01
*** threestrands has joined #openstack-ironic02:16
*** threestrands has quit IRC02:16
*** threestrands has joined #openstack-ironic02:16
*** ijw has joined #openstack-ironic02:20
*** liuzz has joined #openstack-ironic02:22
*** ijw has quit IRC02:31
*** charcol-v2 has quit IRC03:21
*** zhenguo has joined #openstack-ironic03:21
*** hshiina is now known as hshiina|afk03:35
*** hshiina|afk has quit IRC03:35
*** gcb has joined #openstack-ironic03:37
*** charcol-v2 has joined #openstack-ironic04:00
*** threestrands has quit IRC04:07
*** threestrands has joined #openstack-ironic04:19
*** threestrands has quit IRC04:19
*** threestrands has joined #openstack-ironic04:19
*** threestrands has quit IRC04:36
*** threestrands has joined #openstack-ironic04:37
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix for proper deployment when heystone enabled and minimal OS  https://review.openstack.org/54071304:59
*** threestrands has quit IRC05:07
openstackgerritShivanand Tendulker proposed openstack/ironic-python-agent master: Add documentation for rescue mode  https://review.openstack.org/43162205:09
openstackgerritMerged openstack/ironic master: Add option to specify mac adress in devstack/.../create-node.sh  https://review.openstack.org/53948805:25
*** zhenguo has quit IRC05:31
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix for proper deployment when heystone enabled and minimal OS  https://review.openstack.org/54071305:39
*** threestrands has joined #openstack-ironic05:42
*** links has joined #openstack-ironic05:46
*** threestrands has quit IRC05:47
*** threestrands has joined #openstack-ironic05:47
*** links has quit IRC05:49
*** links has joined #openstack-ironic05:51
*** jaganathan has joined #openstack-ironic05:56
*** links has quit IRC05:59
*** links has joined #openstack-ironic06:00
*** threestrands_ has joined #openstack-ironic06:04
*** threestrands_ has quit IRC06:04
*** threestrands_ has joined #openstack-ironic06:04
*** threestrands has quit IRC06:06
*** links has quit IRC06:14
*** links has joined #openstack-ironic06:16
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Introducing an api_workers option to inspector  https://review.openstack.org/54072206:18
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata  https://review.openstack.org/54026406:23
*** tuanla____ has joined #openstack-ironic06:29
*** e0ne has joined #openstack-ironic06:38
*** e0ne has quit IRC06:45
*** dpawar_ has joined #openstack-ironic06:51
*** mjura has joined #openstack-ironic06:53
*** dpawar has quit IRC06:54
*** dpawar_ has quit IRC06:55
*** rbartal has joined #openstack-ironic07:00
*** marios has joined #openstack-ironic07:00
*** pcaruana has joined #openstack-ironic07:04
*** e0ne has joined #openstack-ironic07:09
*** dpawar has joined #openstack-ironic07:10
*** e0ne_ has joined #openstack-ironic07:13
*** e0ne has quit IRC07:14
*** e0ne_ has quit IRC07:17
*** rcernin has quit IRC07:18
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix for proper deployment when heystone enabled and minimal OS  https://review.openstack.org/54071307:21
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add validate_rescue() method to network interface  https://review.openstack.org/54044507:22
openstackgerritShivanand Tendulker proposed openstack/ironic master: Implements rescue interface for 'ilo' hardware type  https://review.openstack.org/45301207:22
*** makowals has joined #openstack-ironic07:27
*** makowals has quit IRC07:28
*** vsaienk0 has joined #openstack-ironic07:29
*** e0ne has joined #openstack-ironic07:29
*** e0ne has quit IRC07:29
olivierbourdon38concerning dib, I have had some issues on my Ubuntu Xenial x64 server. When I add "-lf" to the umount done at https://github.com/openstack/diskimage-builder/blob/master/diskimage_builder/block_device/level3/mount.py#L112 it works great. As I did not find something related to this, does this ring a bell ?07:30
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Add creation of openrc file when keystone enabled  https://review.openstack.org/54009607:40
*** AlexeyAbashkin has joined #openstack-ironic07:41
*** d0ugal has quit IRC07:42
openstackgerritDao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode standalone tests  https://review.openstack.org/53811907:43
*** AlexeyAbashkin has quit IRC07:45
*** AlexeyAbashkin has joined #openstack-ironic07:50
olivierbourdon38could also anyone shed some light on the errors triggered by https://review.openstack.org/#/c/540713/ in the CIs please?07:52
patchbotpatch 540713 - bifrost - Fix for proper deployment when heystone enabled an...07:52
olivierbourdon38for the OPNFV centos7 case, seems to me like the error is outside of the scope of the patch. For the others, I don't understand what is wrong. Thanx07:53
olivierbourdon38for the OPNFV centos7 case, seems to me like the error is outside of the scope of the patch. For the others, I don't understand what is wrong. Thanks07:53
olivierbourdon38furthermore, one other patch seems also to have now the same issues passing the tests and it was succeeding few days ago https://review.openstack.org/#/c/54009607:58
patchbotpatch 540096 - bifrost - Add creation of openrc file when keystone enabled07:58
*** rbartal_ has joined #openstack-ironic08:04
*** rbartal has quit IRC08:06
*** d0ugal has joined #openstack-ironic08:06
*** hjensas has quit IRC08:11
*** hjensas has joined #openstack-ironic08:12
*** hoonetorg has quit IRC08:15
*** d0ugal has quit IRC08:26
*** hoonetorg has joined #openstack-ironic08:28
*** tesseract has joined #openstack-ironic08:28
*** e0ne has joined #openstack-ironic08:43
*** d0ugal has joined #openstack-ironic08:47
openstackgerritMerged openstack/ironic master: correct referenced url in comments  https://review.openstack.org/53471008:49
*** zhangfei has quit IRC08:50
*** threestrands_ has quit IRC08:54
*** zhangfei has joined #openstack-ironic09:14
*** derekh has joined #openstack-ironic09:20
*** mgoddard_ has joined #openstack-ironic09:30
openstackgerritzhangzhaoshan proposed openstack/bifrost master: Fix some spelling errors  https://review.openstack.org/54079209:33
*** stendulker has joined #openstack-ironic09:34
openstackgerritMallikarjuna Reddy Kolagatla proposed openstack/ironic master: Update boot_mode for node cleaning as per driver info  https://review.openstack.org/54079309:36
*** frti has joined #openstack-ironic09:39
*** vsaienk0 has quit IRC09:45
openstackgerritAnkit Kumar proposed openstack/ironic-tempest-plugin master: Test cases for iLO drivers in standalone ironic  https://review.openstack.org/53565209:57
*** vsaienk0 has joined #openstack-ironic09:58
*** dtantsur|afk is now known as dtantsur10:05
openstackgerritMerged openstack/sushy master: Zuul: Remove project name  https://review.openstack.org/53871910:10
openstackgerritMerged openstack/sushy stable/pike: Zuul: Remove project name  https://review.openstack.org/53872410:10
dtantsurTheJulia: when you have a minute: https://review.openstack.org/#/c/540445/10:15
patchbotpatch 540445 - ironic - Add validate_rescue() method to network interface10:15
stendulkerGood morning dtantsur :)10:16
dtantsurTheJulia: also ideas are welcome on how to avoid dummy interface implementations like https://review.openstack.org/#/c/453012/10:16
patchbotpatch 453012 - ironic - Implements rescue interface for 'ilo' hardware type10:16
dtantsurand morning stendulker :)10:16
stendulkerI saw you comment on https://review.openstack.org/#/c/453012/10:16
patchbotpatch 453012 - ironic - Implements rescue interface for 'ilo' hardware type10:16
stendulkerIt could simplify, but i did not boot.get_rescue_properties()10:17
dtantsuryeah, maybe it's a bad idea10:17
stendulkerok10:17
stendulkerI think we can implement boot.validate_rescue()10:18
dtantsurmaybe we should do validate_rescue, and move the new property to the boot interface as well with a comment "required for rescue"10:18
dtantsurI don't like introducing validate_rescue all over the place, but I like having empty interfaces only for validation even less10:18
stendulkerso you mean rescue.get_properties() would return boot.RESCUE_PROPERTIES?10:19
*** phuongnh has quit IRC10:20
stendulkeryes, we will have validate_rescue() jsut as in network interface.10:20
dtantsurrescue.get_properties, mmmm...10:21
dtantsurstendulker: I think the confusion here is that ilo_rescue_iso is not a rescue property, but a boot property10:22
dtantsuras weird as it may sound10:22
dtantsurbecause it's never used in the rescue interface10:22
stendulkerThats true, buit it comes into play only when rescue operation is invoked10:23
dtantsurstendulker: so I think we should only do the same as with network: add boot.validate_rescue and call it from generic agent rescue validate10:23
dtantsuror even from conductor rescue methods, I'm not decided10:23
dtantsurmeh, the driver composition is hard :(10:24
stendulkerOr we can have boot.get_rescue_properties(task) that would return rescue.RESCUE_PROPERTIES10:24
stendulkerThough it does not solve much...10:24
stendulkerdtantsur: Ok will just implement boot.validate_rescue()10:25
dtantsuryeah, let's give it a try and see how it looks10:25
stendulkerThank you. Will post the patch shortly.10:25
dtantsurthanks!10:25
*** rbartal_ has quit IRC10:25
*** amarao has joined #openstack-ironic10:31
openstackgerritJacek Tomasiak proposed openstack/virtualbmc master: Remove devices/*/boot when setting bootdev  https://review.openstack.org/54036810:31
*** namnh has quit IRC10:33
pas-hamorning ironic10:42
pas-hahave a question re uwsgi support patch https://review.openstack.org/#/c/50706710:42
patchbotpatch 507067 - ironic - [devstack] Switch ironic to uWSGI10:42
pas-hathere is a general var in devstack called $ENABLE_HTTPD_MOD_WSGI_SERVICES, and it defaults to True10:43
pas-hawe were ignoring it until now, and most of our jobs it seems were running w/o wsgi by default10:43
pas-hashould we start honoring it and run most of the jobs under (u)wsgi instead&10:44
pas-hagrepping our playbooks, it seems only the standalone job is using wsgi currently10:45
* pas-ha inclined to leave only something exotic like postgres job under eventlet api, and grenade for the time being, and make the wsgi api the default10:46
*** lucas-afk is now known as lucasagomes10:52
*** tuanla____ has quit IRC10:53
*** yangyapeng has quit IRC10:56
*** yangyapeng has joined #openstack-ironic10:57
dtantsurpas-ha: "should we start honoring it" I think yes10:58
dtantsurI think the goal is to deploy services via wsgi10:58
dtantsurI know tripleo has switched to mod_wsgi for most of the services already10:58
pas-hadtantsur: is triple0 using devstack or puppet/ansible? should we continue support for mod_wsgi in our devstack plugin, or just support uwsgi (as the patch does)?11:00
dtantsurpas-ha: tripleo is using puppet. ++ to only supporting one wsgi engine11:00
*** sambetts|afk is now known as sambetts11:00
pas-haok, cool11:01
*** SerenaFeng has joined #openstack-ironic11:07
*** SerenaFeng has quit IRC11:09
*** SerenaFeng has joined #openstack-ironic11:10
openstackgerritMark Goddard proposed openstack/bifrost master: Allow missing ironic policy.json sample  https://review.openstack.org/54061711:14
*** stendulker_ has joined #openstack-ironic11:16
openstackgerritSam Betts proposed openstack/ironic master: Use zuul.override_checkout  instead of custom branch_override var  https://review.openstack.org/54043211:18
openstackgerritSam Betts proposed openstack/ironic stable/pike: Use zuul.override_checkout  instead of custom branch_override var  https://review.openstack.org/54082911:19
openstackgerritSam Betts proposed openstack/ironic stable/ocata: Use zuul.override_checkout  instead of custom branch_override var  https://review.openstack.org/54083011:19
openstackgerritSam Betts proposed openstack/ironic-tempest-plugin master: Switch to override-checkout for stable branch jobs  https://review.openstack.org/54035511:20
*** stendulker has quit IRC11:20
*** stendulker_ has quit IRC11:21
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: [devstack] Switch ironic to uWSGI  https://review.openstack.org/50706711:40
openstackgerritIlya Etingof proposed openstack/ironic master: Add boot mode support to ManagementInterface  https://review.openstack.org/52677311:44
*** SerenaFeng has quit IRC11:47
*** zshi has quit IRC11:52
*** vsaienk0 has quit IRC11:53
*** vsaienk0 has joined #openstack-ironic11:55
*** pcaruana has quit IRC12:02
openstackgerritMark Goddard proposed openstack/bifrost master: Support storage of introspection data in Nginx  https://review.openstack.org/46000512:05
openstackgerritIlya Etingof proposed openstack/ironic master: Added redfish boot mode management  https://review.openstack.org/52831412:11
*** nicodemos has joined #openstack-ironic12:12
*** SerenaFeng has joined #openstack-ironic12:12
*** SerenaFeng has quit IRC12:16
*** dtantsur is now known as dtantsur|brb12:17
*** yangyapeng has quit IRC12:29
*** gcb has quit IRC12:30
*** vsaienk0 has quit IRC12:31
*** yangyapeng has joined #openstack-ironic12:33
*** yangyapeng has quit IRC12:33
*** bfournie has quit IRC12:40
*** bfournie has joined #openstack-ironic12:40
*** bfournie has quit IRC12:45
*** stendulker has joined #openstack-ironic12:47
*** cbt has joined #openstack-ironic12:51
cbthello, do you guys know if ironic supports FC Volumes? I read that I can use iSCSI but couldn't find anything about FC12:52
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM experiment re-using devstack-tempest job  https://review.openstack.org/52016712:54
stendulkercbt: ipxe based implementation supports iscsi. Fujitsu's irmc driver supports FC.12:56
olivierbourdon38is it possible to get https://review.openstack.org/#/c/540617 integrated ASAP as without this Zuul CI reports error for all other patches12:59
patchbotpatch 540617 - bifrost - Allow missing ironic policy.json sample12:59
olivierbourdon38thanks12:59
*** rbartal_ has joined #openstack-ironic13:02
*** stendulker has quit IRC13:02
*** jlabarre has joined #openstack-ironic13:05
*** cbt has quit IRC13:16
*** bfournie has joined #openstack-ironic13:21
*** hw_wutianwei has quit IRC13:33
*** hw_wutianwei1 has joined #openstack-ironic13:33
*** yangyapeng has joined #openstack-ironic13:35
*** hw_wutianwei1 is now known as hw_wutianwei13:35
*** yangyapeng has quit IRC13:39
*** jcoufal has joined #openstack-ironic13:50
*** vsaienk0 has joined #openstack-ironic13:51
*** rloo has joined #openstack-ironic13:52
*** d0ugal has quit IRC13:55
openstackgerritzhengyong proposed openstack/ironic master: Create pxe config over IPv6 network  https://review.openstack.org/54064913:57
sambettsdtantsur|brb: looks like this latest patch + the dependant patches are now checking out the right branches and jobs but it seems like where we weren't loading the jobs on the right branches to begin with there are breakages13:58
*** baha has joined #openstack-ironic13:59
openstackgerritSam Betts proposed openstack/ironic-tempest-plugin master: Switch to override-checkout for stable branch jobs  https://review.openstack.org/54035513:59
*** vsaienk0 has quit IRC14:01
*** cdearborn has joined #openstack-ironic14:02
openstackgerritzhengyong proposed openstack/ironic master: Create pxe config over IPv6 network  https://review.openstack.org/54064914:02
*** vsaienk0 has joined #openstack-ironic14:02
*** ricardoas-away has quit IRC14:02
*** cdearborn has quit IRC14:03
*** SerenaFeng has joined #openstack-ironic14:04
*** dtantsur|brb is now known as dtantsur14:07
dtantsursambetts: oh14:07
sambettsyeah :( probably didn't backport some fixes to the CI we should have done or something14:08
*** mjturek has joined #openstack-ironic14:10
*** SerenaFeng has quit IRC14:11
openstackgerritMerged openstack/ironic master: Remove unused code in unittest  https://review.openstack.org/54019414:14
*** zshi has joined #openstack-ironic14:17
TheJuliaGood morning everyone14:20
dtantsurmorning TheJulia14:20
openstackgerritzhengyong proposed openstack/ironic master: Create pxe config over IPv6 network  https://review.openstack.org/54064914:20
openstackgerritRuby Loo proposed openstack/ironic master: Fix handling of 'timeout' parameter to power methods  https://review.openstack.org/54015014:23
rlooGood Monday morning everyone, TheJulia, dtantsur, sambetts14:24
*** SerenaFeng has joined #openstack-ironic14:24
sambettsHey rloo14:24
dtantsurmorning rloo14:24
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM experiment re-using devstack-tempest job  https://review.openstack.org/52016714:26
*** rbudden has joined #openstack-ironic14:27
*** d0ugal has joined #openstack-ironic14:29
*** SerenaFeng has quit IRC14:30
*** lucasagomes is now known as lucas-hungry14:30
*** Goneri has joined #openstack-ironic14:34
*** ricardoas has joined #openstack-ironic14:46
*** gcb has joined #openstack-ironic14:46
rloodtantsur, TheJulia: wrt whiteboard, week's priorities & subteam reports. I cleaned up/deleted the FFE notes we had for traits, rescue, and some of the others. would you deal with classic drivers deprecation & routed network support --maybe move whatever is left/status to the subteam reports part?14:58
rlooTheJulia: also, if you know the status of CI/testing for rescue, maybe you can update status of that in the whiteboard. otherwise, i should try to ping stendulker this week...15:00
TheJuliarloo: Did you delete all of the FFE notes for rescue?15:00
rlooTheJulia: yes and no. i moved the relevant ones to the rescue section.15:01
TheJuliaack15:01
TheJuliarloo: shouldn't be a problem then15:01
rlooTheJulia: take a look at what it sez. there is stuff about CI and there are CI patches. I don't know if CI is ready/working, or if people are still trying to get it to work...15:01
rlooTheJulia: which isn't a big deal for this week. i was just curious about status. and i guess even if client hasn't landed, before we release, woudl be good to know if rescue works :)15:02
dtantsurrloo: I will, thanks15:02
TheJuliarloo: ci is effectively blocked at the moment, I offered up a possible solution lacking osc, but haven't seen any revisions to the patch. I'll look and update the section15:02
TheJuliaonce I finish with the two reviews I'm looking at15:03
rlooTheJulia: ok, this isn't a high priority, so do the other stuff first. thx!15:03
*** r-daneel has joined #openstack-ironic15:03
*** SerenaFeng has joined #openstack-ironic15:04
rloopas-ha: hi! wrt ansible deploy interface, is the doc patch the only outstanding thing? https://review.openstack.org/#/c/525501/15:09
patchbotpatch 525501 - ironic - Add docs for ansible deploy interface15:09
openstackgerritMark Goddard proposed openstack/bifrost master: Support storage of introspection data in Nginx  https://review.openstack.org/46000515:09
*** dpawar has quit IRC15:13
* TheJulia finally reads the backlog and ponders15:14
rloowe're doing a lousy job of updating our subteam status reports.15:14
rloojlvillal: wrt 'Split away the tempest plugin', that is all done, right? around L409 on etherpad15:14
*** dpawar has joined #openstack-ironic15:14
rloosambetts: if you get a chance, please update whiteboard, cisco UCS status (around L450). Or maybe that is still the current status...15:16
*** zshi has quit IRC15:17
sambettsrloo: thanks for the reminder, I've updated15:18
rloothx sambetts!15:18
TheJuliamgoddard: replied to 540617, I feel like the copy should also be removed, but works for me. Maybe get a NobodyCam to review it once he is coffeed15:18
*** dpawar has quit IRC15:18
TheJuliaif any ironic-core wants to review a quick bifrost patch, 54061715:20
TheJuliahmm... need to remember to talk to cbt more when they reappear15:20
*** lucas-hungry is now known as lucasagomes15:22
*** zhangfei has quit IRC15:25
mgoddard_TheJulia: do you think 540617 should just remove those two tasks?15:27
mgoddard_(well the original copy and the new stat)15:28
TheJuliamgoddard_: I _think_ so, but it clearly is not breaking right now15:30
*** vsaienk0 has quit IRC15:31
TheJuliafrom not breaking standpoint, I'm happy for it to land15:31
mgoddard_TheJulia: ok, I'll rustle up a follow up change15:31
*** SerenaFeng has quit IRC15:31
NobodyCamGood morning folks. It's coffee make'n time.15:32
*** yangyapeng has joined #openstack-ironic15:34
dtantsurmorning NobodyCam15:37
* dtantsur has "omg this damned embassy needs a copy of my rent agreement, I hope at least not a TRANSLATED copy.." time15:37
openstackgerritMark Goddard proposed openstack/bifrost master: Don't copy ironic's policy.json sample  https://review.openstack.org/54089315:37
* TheJulia goes and grabs more coffee15:38
sambettsdtantsur: wow...15:38
*** yangyapeng has quit IRC15:38
NobodyCamGood morning dtantsur15:39
TheJuliadtantsur: ugh :(15:39
*** lbragstad_ is now known as lbragstad15:43
*** gcb has quit IRC15:47
*** jcoufal has quit IRC15:50
*** amarao has quit IRC15:53
*** rbartal_ has quit IRC15:54
*** tduek has joined #openstack-ironic15:57
*** ijw has joined #openstack-ironic15:58
*** jcoufal has joined #openstack-ironic15:59
*** SerenaFeng has joined #openstack-ironic16:01
*** vsaienk0 has joined #openstack-ironic16:02
pas-harloo: for Queens I will say 'yes', ansible-deploy docs are all that left16:03
rloopas-ha: thx, i will update the whiteboard. we should try to land that this week...16:03
tduekHello, I have a question about Bifrost. What does a standalone ironic installation means? What features of the “full” ironic installation am I missing when using only Bifrost?  Thanks16:04
sambettstduek: standalone means just ironic, without any other openstack services e.g. nova/neutron/keystone16:04
pas-hatduek: ironic is capable of working in stadalone mode without dependencies on other OpenStack components16:04
pas-hain fact you can even mix-and-match those, like ironic+keystone only, w/o neutron|glance etc16:05
pas-haafaik bifrost already allows to deploy ironic+keystone, and there's some WiP for adding neutron as well16:06
tduekThat I understand. What I don't understand is what key features am I missing? Does using standalone means that I'll have to manage my images elsewhere? Or manage my network configuraion in some other way?16:07
pas-hatduek: yes, all that16:07
pas-haimages are stored in a static HTTP16:07
pas-haand no network management ('none' network interface in ironic, with bifrost setting up a static dnsmasq for you)16:08
TheJuliapas-ha: I rebased it all friday \o/16:08
pas-haand no authentication16:08
pas-haTheJulia: you mean the neutron thing? :)16:09
TheJuliapas-ha: yeah16:09
tduekOK. Now I understand. just wanted to be sure on that. Thank you16:09
TheJuliaI needs it for some ansible module stuffs16:09
pas-hatduek: basically the main use case for bifrost (at least from its beginnings) was like "you have a bunch of HW nodes and a VM somewhere nearby. use bifrost's ansible playbooks to setup ironic on this VM and deploy all your HW nodes with the same image via ironic"16:11
dtantsurhjensas: could you please clean up https://etherpad.openstack.org/p/IronicWhiteBoard line 300 and on?16:11
tduekpas-ha: so for production grade system to manage all my HW machine with different images and different network configurations, bifrost will not be the right solution. right?16:13
pas-hatduek: that depends.. but ideally IMO you'd need a more complete OpenStack installation, with Glance and Neutron16:14
pas-hasmth TrippleO like :)16:15
TheJuliarloo: removed my -2 from https://review.openstack.org/#/c/524118/16:16
patchbotpatch 524118 - ironic - devstack: add support for rescue mode16:16
pas-haand of course if you need notion of tenants and tenant isolation, Keystone + Neutron is required16:17
rlooTheJulia: ah, so it is worth reviewing now? (not sure if we have time, but...)16:17
TheJuliarloo: well, it doesn't expect there is a rescue interface on the node using python-ironicclient anymore... so in theory we can move forward16:17
rlooTheJulia: ok thx16:18
TheJuliathe job change after also looks good16:18
TheJuliaI have a -1 on the devstack change atm16:18
* TheJulia updates whiteboard16:18
*** tduek has quit IRC16:20
*** mrch has joined #openstack-ironic16:21
*** jcoufal has quit IRC16:21
TheJulialooks like standalone tests are also proposed, but have not looked at them16:22
rloothat bifrost question/use cases is useful ^^ if anyone wants to add more docs to bifrost :)16:22
jrollrloo | we're doing a lousy job of updating our subteam status reports. <- I suspect this is related to having so many topics there, we really need to keep it focused and minimal next cycle. or maybe group things into categories?16:25
jrollmorning everyone16:25
* jroll is on sunnyvale time this week16:25
*** jcoufal has joined #openstack-ironic16:26
dtantsurrloo, TheJulia, can we drop the whole FFE section on line 133? we're past Feb 2nd, and now we have two sources of truth on the whiteboard..16:27
*** baha has quit IRC16:30
TheJuliarloo: I think we can drop the FFE section16:31
TheJuliaerr, dtantsur ^^^16:31
TheJuliajroll: +++++++++16:31
dtantsurwell, it's gone :)16:33
*** AlexeyAbashkin has quit IRC16:34
*** AlexeyAbashkin has joined #openstack-ironic16:34
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/54057516:35
*** links has quit IRC16:37
NobodyCamTheJulia: so 540617 is just until queen's is cut?16:38
*** rpioso|afk is now known as rpioso16:38
TheJuliaNobodyCam: moving forward16:39
rloodtantsur: thx for dropping it!16:40
dtantsurfolks, status check before the meeting: are you aware of anything that must merge into queens and is NOT backportable?16:41
dtantsurTheJulia, rloo, and everyone else ^^^16:41
rloojroll: that is partly it. Although the subteam folks are responsible for keeping the status up to date, we're spreading the work load but...16:41
rloodtantsur: have you or anyone else gone through the bugs?16:41
dtantsurI only scanned stagnating bug fix reviews16:42
dtantsurrevived a few, have another one to revive16:42
dtantsurdid not go through the whole list16:42
TheJuliaI'm not aware of anything off-hand, although I think we have a couple doc patches that could use landing16:43
rloodtantsur: ok, let us know if there any HIGHs or whatever that we should address16:43
TheJuliadtantsur:  https://review.openstack.org/54092216:43
patchbotpatch 540922 - releases - Releasing ironic-ui 3.1.0 for Queens16:43
rloodtantsur: there is this: https://review.openstack.org/#/c/540150/. given the change, i'd prefer it got into the release, vs backporting it...16:43
patchbotpatch 540150 - ironic - Fix handling of 'timeout' parameter to power methods16:43
*** mjura has quit IRC16:43
dtantsurTheJulia: thanks! this includes the translation update, right?16:44
TheJuliayup, the one that quite literally just merged16:44
dtantsurrloo: good point. ditto about https://review.openstack.org/54044516:44
patchbotpatch 540445 - ironic - Add validate_rescue() method to network interface16:44
rlooTheJulia: thx, i was going to ask you about ironic-ui :)16:44
rloodtantsur: oh, i didn't see that one. on my list...16:44
rloodtantsur: oh, you guys approved it already :)16:44
dtantsuroh indeed :D16:45
*** e0ne has quit IRC16:48
*** stendulker has joined #openstack-ironic16:50
*** hjensas_mobile has joined #openstack-ironic16:51
openstackgerritSam Betts proposed openstack/ironic master: Add detail=[True, False] query string to API list endpoints  https://review.openstack.org/43149716:53
jlvillalrloo, I think the tempest work is done. I don't know of any outstanding issues.16:54
rloojlvillal: thx, you were responsible so if you say it is done, it is done :)16:55
jlvillaljroll, Say hello to vdrok, I think he is on Sunnyvale time to :)16:55
*** hshiina has joined #openstack-ironic16:55
sambettsany one seen this error before? http://logs.openstack.org/55/540355/6/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ocata/4d60687/logs/screen-ir-cond.txt.gz?level=WARNING#_2018-02-05_14_38_56_26616:56
*** marios has quit IRC16:56
sambettsI'm trying to debug way the ironic-tempest-plugin gate is still broken16:56
sambettseven after fixing the branches16:56
rloojlvillal: sorry, just want to check. plugin code was NOT removed from stable branches? (L380)16:57
jrolljlvillal: I waved through the window, hopefully the right direction16:57
dtantsurhjensas, sambetts, TheJulia, are we ready to release networking-baremetal for queens?16:57
dtantsurmgoddard_, pas-ha, dtto ^^ re n-g-s16:58
jlvillalrloo, Correct. Stable branches were left untouched.16:58
jrollsambetts: did we support tls agent->ironic in ocata?16:58
dtantsurno ^^^16:58
rloojlvillal: thx, am updating tense :)16:58
dtantsursambetts: we need to disable TLS proxy on Ocata16:58
dtantsurwe had hacks in place for that in our jobs16:58
openstackgerritmelissaml proposed openstack/sushy master: fix error url  https://review.openstack.org/54093216:58
TheJuliadtantsur: afaik, yes, but I haven't looked at either yet today16:59
dtantsurreminder: the meeting is in 1 minute16:59
sambettsdtantsur: got it, /me updates my patches16:59
dtantsur#startmeeting ironic17:00
openstackMeeting started Mon Feb  5 17:00:03 2018 UTC and is due to finish in 60 minutes.  The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
*** openstack changes topic to " (Meeting topic: ironic)"17:00
openstackThe meeting name has been set to 'ironic'17:00
jroll\o17:00
dtantsuro/17:00
rlooo/17:00
mgoddard_o/17:00
stendulkero/17:00
bfournieo/17:00
dtantsur#link https://wiki.openstack.org/wiki/Meetings/Ironic17:00
etingofo/17:00
hshiinao/17:00
dtantsurhi (again) everyone! :) this might be the last time I chair this meeting, heh17:00
jlvillalo/17:00
TheJuliadtantsur: networking-baremetal could use two more things to land most likely, minor patches17:00
TheJuliao/17:00
rpiosoo/17:01
* jroll makes dtantsur the default substitute17:01
dtantsur#topic Announcements / Reminder17:01
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"17:01
dtantsur#info Feature freeze is upon us17:01
dtantsurplease be very considerate when approving anything that looks like a feature17:01
dtantsurparticularly anything that modifies API/RPC17:02
rlooptl election ends Feb 14. dtantsur has at least one more meeting, if not more :)17:02
*** baha has joined #openstack-ironic17:02
dtantsurrloo: if we have elections, that is ;)17:02
mjtureko/17:02
dtantsur#info All projects that have not created stable/queens will do it by Thu17:02
dtantsur(all our projects, I mean)17:02
dtantsurwe will be releasing projects earlier too, when we find appropriate17:03
dtantsur#info The multinode grenade job is broken and needs YOU to help debugging it17:03
* jlvillal wonders who the candidate(s) are for PTL as he doesn't see any emails in quick search17:03
rloojlvillal: none yet for ironic.17:03
dtantsurjlvillal: TheJulia is, I am not, that's all I know :)(17:03
TheJuliajlvillal: I submitted my candidacy to the elections repo last friday, but have not sent an email17:03
rlooahhh, a candidate emerges... :)17:04
openstackgerritSam Betts proposed openstack/ironic stable/ocata: Use zuul.override_checkout  instead of custom branch_override var  https://review.openstack.org/54083017:04
jlvillalAh :)17:04
jrollhttps://github.com/openstack/election/tree/master/candidates/rocky/Ironic fyi17:04
jroll:)17:04
rloothx jroll!17:04
dtantsur#info The PTL nominations are accepted till Wed17:04
rloodtantsur is relieved...17:04
dtantsurkinda ;)17:04
ricardoaso/17:04
*** frti has quit IRC17:04
openstackgerritSam Betts proposed openstack/ironic stable/pike: Use zuul.override_checkout  instead of custom branch_override var  https://review.openstack.org/54082917:05
*** AlexeyAbashkin has quit IRC17:05
dtantsur#link https://etherpad.openstack.org/p/ironic-rocky-ptg PTG planning continues17:05
dtantsuranything else I should mention?17:05
* dtantsur is still struggling with a visa, sigh17:06
rloodtantsur: wrt ptg planning. are there any dates we need to be aware of. eg, deadline for topics? or will we go over them before ptg?17:06
rloodtantsur: oh, wait. maybe i should wait for our new leader...17:06
dtantsurrloo: on the etherpad I asked topic to be submitted by Feb 16th17:06
*** crushil has joined #openstack-ironic17:06
dtantsurbut yes, the PTG planning will be in hands of the new victim :D17:06
TheJuliaThe new leader can sort and break things into logical chunks17:06
rloos/victim/lucky duck/17:06
TheJulias/leader/victim/ :)17:06
dtantsurlet's agree on 'lucky victim' :D17:07
TheJulia++17:07
* jroll confirms victim is the right word17:07
rloolucky bear?17:07
TheJuliawait, that should be willing lucky victim17:07
dtantsurlol, ok. ready to move on?17:07
TheJulia++17:07
dtantsur#topic Review action items from previous meeting17:08
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"17:08
dtantsur#link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-29-17.00.html17:08
dtantsuranything to report?17:08
TheJuliaI blocked the rescue patches at the logical point17:09
dtantsurmjturek: anything new from the bug triaging?17:09
*** trown is now known as trown|lunch17:09
TheJuliathey have since been revised for standalone testing to not use the OSC commands, I have not had a chance to review them since noticing that this morning.17:09
rloothx TheJulia17:10
mjturekhey dtantsur17:10
mjturekI've been cleaning up the wishlist17:10
dtantsuryeah, I've noticed it shrinking, thanks!17:10
mjturekI have a list of bugs I've moved to Opinion, so if there's something you disagree with let me know!17:10
dtantsur#link https://etherpad.openstack.org/p/ironic-bug-triage17:11
TheJuliamjturek: is there any easy way to pull that list?17:11
dtantsurTheJulia: it is there ^^^17:11
TheJuliaoh, nevermind :)17:11
openstackgerritMerged openstack/ironic master: Add validate_rescue() method to network interface  https://review.openstack.org/54044517:11
TheJuliathanks!17:11
mjturek:) thanks17:12
dtantsur#topic Review subteam status reports (capped at ten minutes)17:12
*** openstack changes topic to "Review subteam status reports (capped at ten minutes) (Meeting topic: ironic)"17:12
dtantsur#link https://etherpad.openstack.org/p/IronicWhiteBoard line 16117:12
pas-hadtantsur: re n-g-s, I think we have two more patches to land, one is actual bug, another is initial release note17:12
dtantsurI haven't found a new place for my bug dashboard, still running it locally.. :(17:12
rloodtantsur: what's the critical ironic bug?17:12
dtantsurrloo: IIRC multinode grenade?17:13
rloodtantsur: is it still critical?17:13
dtantsur#link https://bugs.launchpad.net/ironic/+bug/174413917:13
openstackLaunchpad bug 1744139 in Ironic "Grenade job fails due to systemd stopping n-cpu" [Critical,In progress] - Assigned to Julia Kreger (juliaashleykreger)17:13
dtantsurrloo: yes. we have to re-enable the job asap17:13
rloodtantsur: okay17:13
dtantsuror we're failing the rolling upgrade tag requirement17:13
pas-hayes, that17:13
TheJuliathe cause has likely shifted since we originally opened that bug17:14
rlooTheJulia, dtantsur: wrt api version negotiation. is the TODO still valid, or is that done? or what is left...17:14
dtantsurpas-ha: mind putting the patches after line 124?17:14
rlooL20417:14
pas-hadtantsur: ok17:14
TheJuliarloo: I think so yes17:14
TheJuliarloo: just we are on hold until rocky+ptg17:15
*** d0ugal has quit IRC17:15
rlooTheJulia: ok, good.17:15
rloodtantsur: wrt classic driver deprecation. anything left that we need to do for queens?17:15
dtantsurrloo: we need docs and CI updates, but they can be backported17:15
dtantsurno feature work to my best knowledge17:16
*** SerenaFeng has quit IRC17:16
rloodtantsur: ok, so all the TODOs need to be done & backported?17:16
dtantsuryep17:16
jrollnice17:16
rloodtantsur: good thing you'll have time when you aren't ptl ;)17:16
dtantsuroh yeah17:16
dtantsurmaybe even refarch...17:16
TheJuliawhat is this time word?17:16
*** AlexeyAbashkin has joined #openstack-ironic17:17
jrollrefarch ++17:17
rlooTheJulia: don't worry, it won't apply to you :D17:17
TheJuliaIf I buy coffee, can we just get refarch banged out while we're in ireland?17:17
TheJuliarloo: lol17:17
rlooso routed network support... lots of stuff there. what is the 2-line synopsis?17:17
dtantsurTheJulia: let's do it!17:17
sambettsTheJulia: ++17:17
rlooplease add refarch to ptg topics17:18
dtantsurrloo: it should be 1-2 patches left, I'm waiting for someone to clean up the status17:18
TheJuliarloo: mostly merged, one or two minor things before we can release17:18
sambettsyeah thanks so much to hjensas for getting that done BTW17:18
rloothe minor things can go in this week or was it part of ffe that ended last Fri?17:18
hjensas_mobileRouted networks, I think we need to push the last patch making the distributed thing. There is one or two patches to fix nits.17:19
TheJuliarloo: this week is fine, the ring patch would be good to land this cycle if we can17:19
dtantsurrloo: nothing that looks like a feature. corrections to the FFE are fine though.17:19
dtantsurthe border is really thin, I know17:19
hjensas_mobileUnless I find time, via magic, and can get jroll's comments done by wednesday ...17:19
rloohjensas_mobile: if you can, could you update the whiteboard to make it clear. or maybe someone can add those patches to weekly priorities?17:19
hjensas_mobileRloo, i can once back home.17:20
rlooi am not sure i like the ring patch landing so late, but if reviewers are confident...17:20
rloothx hjensas_mobile17:20
TheJuliaI'm fairly confident and it sets the service off being n+117:20
*** AlexeyAbashkin has quit IRC17:21
dtantsurlet's ask it this way: what do we lose if we DON'T land it?17:21
rloowrt python 3.5... what's left to do?17:21
dtantsurrloo: IPA17:21
rloodtantsur: oh. ok, so we have to carry that to rocky.17:21
TheJuliadtantsur: we complicate installs potentially by implementing a required single point of failure17:22
jrollTheJulia: hjensas_mobile: I'll check the responses on the patch after this meeting and we can decide how risky it is (not sure if julia has seen my comments)17:22
dtantsurTheJulia: yeah, this sounds like a correction, not a new feature17:22
TheJuliajroll: I have not17:22
dtantsurrloo: yep :(17:22
hjensas_mobileOne can run two or three of the agent as is.17:22
openstackgerritMerged openstack/bifrost master: Allow missing ironic policy.json sample  https://review.openstack.org/54061717:22
rloodtantsur: that's ok, we need stuff to do in Rocky. ha ha.17:23
TheJuliadtantsur: more so, tax form correction of sorts :)17:23
dtantsurrloo: yeah, what a relief: we won't end up without a backlog :D17:23
hjensas_mobileBut they would all do heartbeat for all ironic nodes. So neutron would be a bit hammered.17:23
*** d0ugal has joined #openstack-ironic17:23
dtantsurTheJulia: don't remind me, I have to do taxes for 2 countries :)17:23
dtantsuranyway, are we done with the statuses?17:24
rloo+done17:24
TheJuliarloo: re: py35 full stack testing as well, but swift it looks like might be next cycle17:24
rloonice that we have some cisco CI working, thx sambetts :)17:25
dtantsur#topic Deciding on priorities for the coming week17:26
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"17:26
rloodocs & bugs...17:26
dtantsurand CI, yes17:27
rloodtantsur: ansible doc: https://review.openstack.org/#/c/525501/17:27
patchbotpatch 525501 - ironic - Add docs for ansible deploy interface17:27
dtantsuryep, what else?17:27
TheJulialooks good to me17:27
rloodo we care about uwsgi? https://review.openstack.org/#/c/525501/17:28
patchbotpatch 525501 - ironic - Add docs for ansible deploy interface17:28
TheJuliayeah, we should likely try to land ansible docs17:28
TheJuliarloo: wrong link?17:28
rloooops, i mean https://review.openstack.org/#/c/50706717:28
patchbotpatch 507067 - ironic - [devstack] Switch ironic to uWSGI17:28
*** yangyapeng has joined #openstack-ironic17:28
dtantsurrloo: it's a goal => yes17:28
dtantsurthough.. maybe not as a priority17:28
dtantsurdunno17:28
TheJuliait would be nice, but we already have one job running uwsgi, so to me, at least in my mind, it doesn't have to land in the next 3.5 days17:29
dtantsuryep17:29
dtantsurso, these five items, how does it look?17:29
TheJuliaworks for me17:30
jrollmakes sense to me17:30
rloo5 items that are not that easy. do we have people working on them?17:30
rlooI like 'fix as many bugs as possible' :)17:30
TheJuliaall of us :) ?17:30
dtantsurI'm still on classic drivers deprecation17:30
rloodtantsur: should we try to do a release on Wed?17:30
dtantsurrloo: yes17:30
dtantsurlet's not wait till the last moment17:31
TheJuliaagreed17:31
rloodtantsur: ok, so factor in the patches needed that are related to a release. like our favorite reno edits.17:31
dtantsurreno \o/17:31
rloodtantsur: HOW MANY releases? ugh.17:31
dtantsuryes, I'll take a look tomorrow, if nobody beats me to it17:32
rloosometimes it isn't good to have so many projects...17:32
dtantsuranyone looking into the multinode CI? I can, but I don't know it really well17:32
*** yangyapeng has quit IRC17:32
TheJuliaI can start looking into it again since we now have logs17:32
jrollI could try to help, but I'm fairly stacked up with irl meetings this week17:32
* rloo wonders what 'irl'is17:33
jroll"in real life"17:33
dtantsurin WUT?17:33
rlooha ha17:33
NobodyCamLOL17:33
TheJulialol17:33
dtantsurokay, moving on? :D17:33
dtantsur#topic Appointing a bug triaging lead for the coming week17:34
*** openstack changes topic to "Appointing a bug triaging lead for the coming week (Meeting topic: ironic)"17:34
dtantsurI have a feeling that it is everyone this time17:34
dtantsuragreed?17:34
rlooyup17:34
dtantsur#action everyone: please triage new bugs and try to fix old ones this week17:35
dtantsurnext, I wanted to discuss what to fix and work on this week17:35
TheJuliaagreed17:35
dtantsurbut I feel like we've done it?17:35
TheJuliaI think so yes17:35
dtantsuranyway, let's do it this way:17:35
dtantsur#topic Open Discussion17:36
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"17:36
jrollquick dumb question: RFEs get tagged confirmed until they're looked at, right?17:36
dtantsurjroll: until approved rather, though we usually don't mark them triaged17:36
jroll"looked at" meaning approved or spec requested17:36
dtantsurmostly because they tend to be In Progress by that time17:36
jrolldtantsur: I can leave it new if that's what we prefer17:36
dtantsurnope, not New17:36
* jroll put one up but not working on it very soon17:36
jrollheh, ok. confirmed it is17:37
dtantsurConfirmed is the right one for something that looks potentially acceptable17:37
jrollthanks17:37
rloohere's our documentation: https://docs.openstack.org/ironic/latest/contributor/code-contribution-guide.html#feature-submission-process17:37
rloobut don't know if we really follow it17:37
dtantsuroh, FYI:17:37
dtantsur#info https://review.rdoproject.org/r/#/c/11795/17:37
* dtantsur just got pinged about it17:37
jrollah, it's triaged. thanks rloo, I should've looked17:37
jrolldtantsur: we all get the opportunity to break RDO ci now? awesome :P17:38
rloodtantsur: cool17:38
dtantsurjroll: we have long had it :D now we can check if we do in advance17:38
rloowe should document those somewhere...17:38
TheJuliasweet, more things to break!17:39
TheJuliarloo: that is a good point, ironic does get tested in several places outside direct view, might be good to enumerate where those are17:39
jrolldtantsur: do those run on upstream patches, then?17:39
dtantsurjroll: it's like experimental, you need to trigger it17:39
dtantsurI never tried :)17:39
jrolldtantsur: right, but I trigger it on review.o.o, right? not review.rdo.org17:40
dtantsurbut RDO CI works like a 3rd party CI17:40
dtantsuryes17:40
jrollok cool17:40
jrollsounds like a win \o/17:40
*** crushil has quit IRC17:40
dtantsuryep :)17:40
dtantsuranything else we should talk about?17:41
TheJuliaCoffee?17:41
dtantsur\o/17:41
dtantsurnot at 7pm probably though :(17:41
TheJuliaDeploying Crickets as a Service?17:41
jrollcoffee++17:41
dtantsurokay, thanks all17:41
NobodyCam:)17:41
TheJuliaThanks dtantsur!17:41
dtantsur#endmeeting17:41
*** 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:41
jrollthanks dtantsur :)17:41
openstackMeeting ended Mon Feb  5 17:41:52 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:41
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-05-17.00.html17:41
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-05-17.00.txt17:41
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-05-17.00.log.html17:41
*** amarao has joined #openstack-ironic17:44
*** amarao has quit IRC17:44
*** sambetts is now known as sambetts|afk17:45
sambetts|afkNight all o/17:45
rloogood night sambetts|afk17:45
sambetts|afknight rloo17:45
*** hjensas_mobile has quit IRC17:48
*** hjensas_mobile has joined #openstack-ironic17:48
*** hshiina has quit IRC17:50
*** devananda has joined #openstack-ironic17:52
* dtantsur goes as well17:52
*** dtantsur is now known as dtantsur|afk17:53
*** e0ne has joined #openstack-ironic17:53
dtantsur|afko/17:53
rlootschuss dtantsur|afk17:58
openstackgerritShivanand Tendulker proposed openstack/ironic master: Implements rescue interface for 'ilo' hardware type  https://review.openstack.org/45301217:58
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add validate_rescue() method to boot interface  https://review.openstack.org/54095417:58
*** derekh has quit IRC18:00
*** yangyapeng has joined #openstack-ironic18:05
*** dsneddon has joined #openstack-ironic18:06
*** mgoddard_ has quit IRC18:07
*** dprince has joined #openstack-ironic18:07
*** yangyapeng has quit IRC18:10
*** trown|lunch is now known as trown18:12
*** mrch has quit IRC18:12
*** d0ugal has quit IRC18:13
jlvillalTheJulia, rloo: If you have a chance to review this: https://review.openstack.org/#/c/540031/18:13
patchbotpatch 540031 - ironic-lib - Use six.wraps() for Metrics so decorated methods c...18:13
TheJuliajlvillal: I'll try in the next couple days, since we've already released ironic-lib for the cycle18:13
jlvillalTheJulia, I wonder if we will want to backport that to stable.18:14
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Add creation of openrc file when keystone enabled  https://review.openstack.org/54009618:15
TheJuliajlvillal: filing that into brain patch review scheduler18:17
TheJuliaIt only presently returns "take lunch" when I query it though :)18:18
jlvillalheh18:18
*** d0ugal has joined #openstack-ironic18:26
*** vsaienk0 has quit IRC18:31
rloothx for reminding me jlvillal. +A'd.18:32
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add validate_rescue() method to boot interface  https://review.openstack.org/54095418:36
TheJuliarloo: well that makes it easy :)  If you want to click hte backport button, I'll happily approve it.18:47
rlooTheJulia: I'm not sure we want to backport it. I'm not sure what we do with ironic-lib.18:47
rlooTheJulia: I'll leave it for you and dtantsur|afk to decide. I mean, what do we do with it if we backport it? release a new backported version? then what?18:48
*** mgoddard_ has joined #openstack-ironic18:49
TheJuliaup the micro version, but it is also not a breaking critical bug so we're frozen on releasing a new stable rev atm18:49
*** hjensas_mobile has quit IRC18:49
*** stendulker has quit IRC18:54
* TheJulia adds an evening gathering to the ptg planning etherpad18:58
*** d0ugal has quit IRC18:59
rlooTheJulia: wrt microversion for ironic-lib, let's wait/discuss with dmitry. if we're going to backport and then use it in stable/queens etc, i don't see why we'd want https://review.openstack.org/#/c/540150/19:01
patchbotpatch 540150 - ironic - Fix handling of 'timeout' parameter to power methods19:01
rlooTheJulia: wrt evening gathering -- wondering if we could ask a favour of lucasagomes to help out with that :)19:01
lucasagomesrloo, o/19:02
JayFI hear the whiskey is good.19:02
lucasagomesofc, I was talking to dtantsur|afk about it19:02
JayFYou should try some of that ;)19:02
rloolucasagomes is like a genie!19:02
rlooand JayF :)19:02
lucasagomesrloo, it's around ~25 ppl right ?19:02
lucasagomes(please include myself, I will go :D)19:02
rloolucasagomes: hard to know. i'd say max. i'm guessing there will be fewer folks than in denver ptg. TheJulia, any idea?19:03
lucasagomeshttp://www.galwaybaybrewery.com/againstthegrain/ < Is a great place with good beer and food19:03
rlooat denver ptg we had 30ish? and lucasagomes, of course, i expect to see you there !19:03
lucasagomesthe have an upstairs area that can be reserved for big groups19:03
*** harlowja has joined #openstack-ironic19:03
TheJuliarloo: yeah, I'm only willing largely because nothing is user exposed with the change19:04
rloolucasagomes: sounds great. TheJulia, what do you think? oh, i guess we need to know which evening.19:04
openstackgerritMerged openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/53705219:05
lucasagomesrloo, yeah, maybe we can send an email with a doodle so people can pick the day ?19:05
TheJuliaWe haven't had as many people indicate they will be at this PTG :(19:05
TheJuliaa doodle would be really good and we could then raise visibility19:06
lucasagomes++19:06
rlooTheJulia, lucasagomes: I'm guessing by now, most folks will know if they are going or not (apart from visa issues...) so yes, doodle would be good.19:06
rloowe always have stragglers, that's a given ;)19:06
lucasagomesrloo, TheJulia http://www.oneillspubdublin.com/ is another good place for large groups19:06
lucasagomesoneils is closer to the city centre too (it's just beside temple bar)19:07
rloolucasagomes: we're at some crokey? park. so needs to be easy to get to from there.19:07
TheJuliarloo: yeah, as of last week, it was mentioned in the TC meeting that the PTG was approaching being sold out. I suspect some people just haven't indicated attendence on our etherpad, so it would be good to get the evening gathering doodle out there becasue that will help us gague during ptg meeting attendance as well19:07
jrollfwiw, "sold out" was something like 250 or 300 people19:08
TheJuliajroll: 35019:09
jrollah yeah19:09
lucasagomesrloo, right... Yeah these pubs are around ~20min walk from croke park. I can check something closer as well19:09
lucasagomesI dunno much that area19:09
lucasagomesbut there are buses and train as well19:09
* jroll adds penick to the attendees on the pad19:10
* penick perks up19:10
penickdid someone say whisky19:10
penicki'm in19:10
jrollI signed you up, don't worry19:10
rlooquick, hide the wh*****19:10
penickPhew, thanks!19:10
rloolucasagomes: for me 20 min walk is ok. dunno about others though. i suspect it is fine. i'd rather walk a little more to get to a better place :)19:11
lucasagomesrloo, yeah I agree19:11
rlooTheJulia: how many people went to denver ptg, do you know? (just curious)19:11
rlooTheJulia, lucasagomes, others: who wants to volunteer to do google-doc for ptg get-together?19:13
rloough, not google-doc, doodle doc!19:14
rlooI could do it too if no one else wants to do this fun thing :)19:14
lucasagomesrloo, I can help u... We just need to pick a date right ? Or do we want to vote on the venue as well ?19:16
rloolucasagomes: nope, just the date. no voting on location, that's too hard.19:16
lucasagomesack19:17
TheJuliaI'm fairly good with a 20 minute walk personally19:17
rloolucasagomes: i know there are people with food allergies, etc. but i'm assuming if we are going with a pub, there is something offered that folks can eat19:17
TheJuliarloo: I think denver was close to 35019:17
rloootherwise, too hard to pleae everyone19:17
rlooTheJulia: ahh, ok, so similar size. nice.19:18
rlooTheJulia: but where are all the ironic contributors then... we will have to figure out a way to get more of the attendees to contribute ;)19:18
lucasagomesrloo, I think so too, they have a menu as well: http://www.galwaybaybrewery.com/menu/galway-bay-brewery-dublin-evening.pdf19:19
rloolucasagomes: whoa, 'scotch egg'... :)19:20
lucasagomeslol yeah it's pub food... We can try to find a place with something more sophisticated if people wants as well19:21
TheJuliarloo: I was thinking about hiring the hypnotoad to assist us with that effort (hopes you've watched futurama and thus get the reference)19:21
TheJuliaLets figure out a day first, and go from there19:21
rloolucasagomes: no, let's stick with pub food. when in ireland, ... :)19:21
rlooTheJulia: i will ask my family what 'hypnotoad is', they all know except me!19:22
*** d0ugal has joined #openstack-ironic19:22
jlvillalrloo, Thanks for the +A :)19:22
TheJuliarloo: :)19:23
rloojlvillal: you're welcome. you earned it :)19:23
TheJuliaAnyway, going back to the non-lunch laptop19:23
jlvillalrloo, heh :)19:23
lucasagomesrloo, TheJulia once we have the date lemme know so we can book it (or derekh can do as well, since I'm in brazil atm)19:24
rloolucasagomes: will do. thx! enjoy brazil!19:24
*** dprince has quit IRC19:27
TheJuliais anyone putting up a doodle poll for the date?19:29
*** e0ne has quit IRC19:31
* TheJulia pokes gerrit19:35
*** tesseract has quit IRC19:36
*** mgoddard_ has quit IRC19:36
*** AlexeyAbashkin has joined #openstack-ironic19:38
*** lucasagomes is now known as lucas-afk19:39
*** ijw has quit IRC19:41
rlooTheJulia: i was going to if no one else does. but not going to do it until later today or tomorrow.19:41
TheJuliarloo: I think that works19:42
rlooTheJulia: ok, ActionItem for me then :)19:43
jrollok, I'm +1 on this thing now: https://review.openstack.org/#/c/524709/19:44
patchbotpatch 524709 - networking-baremetal - Make the agent distributed using hashring and noti...19:44
openstackgerritJulia Kreger proposed openstack/ironic master: Don't try to lock for vif detach  https://review.openstack.org/53444119:46
openstackgerritMerged openstack/ironic-inspector master: Remove empty files  https://review.openstack.org/53885319:47
*** AlexeyAbashkin has quit IRC19:47
openstackgerritMerged openstack/ironic-inspector master: Imported Translations from Zanata  https://review.openstack.org/54026419:47
openstackgerritMerged openstack/ironic-lib master: Use six.wraps() for Metrics so decorated methods can be inspected  https://review.openstack.org/54003119:47
rloojlvillal: did you see this? hoping you know more about it... https://review.openstack.org/#/c/540355/19:48
patchbotpatch 540355 - ironic-tempest-plugin - Switch to override-checkout for stable branch jobs19:48
jlvillalrloo, I have glanced at it but have not yet dug into it. It doesn't appear to be working yet.19:51
rloojlvillal: yeah, just looking but not sure i want to go further w/o talking to sambetts|afk. that patch has a dependency, so am wondering if the idea is for the stable/ocata patch to land first. not sure.19:52
rloojlvillal: am also not sure what is broken w/o this.19:52
jrollrloo: this is meant to fix the broken jobs on ironic-tempest-plugin master branch CI19:53
jrollrloo: and I believe, but am not 100% sure, that all three of the patches in the depends-on need to land first, because ironic-tempest-plugin is branchless19:54
rloojroll: OH. when did that start breaking.19:54
rloojroll: i mean jlvillal19:54
TheJuliahjensas: looks like the hash ring change needs to be rebased.19:54
jrollrloo: last week sometime, it's in the whiteboard19:55
hjensasTheJulia: yes, I am trying to get my devstack for it to address the questions on stopping as well.19:55
jlvillalrloo, Uh, I'm not sure when exactly it started breaking. But I noticed it last week and put it into the whiteboard.19:55
jrollthursday?19:55
TheJuliahjensas: awesome19:55
jlvillalTheJulia, rloo: Might be of slight interest to you: https://review.openstack.org/54098619:56
patchbotpatch 540986 - openstack-infra/project-config - gerritbot: Add queens and rocky to ironic IRC noti...19:56
rloojroll, jlvillal: hmm. what caused it to break all of a sudden. is that override_checkout vs branch_override behaviour new since Thurs?19:56
jlvillalI noticed when I pushed a patch for stable/queens it did not appear here19:56
jlvillalrloo, That is the attempted fix for the issue.19:57
jlvillalUnsure on what caused it to break.19:57
* jroll finds the cause in scrollback19:57
rloojlvillal: thx for the link. isn't there some regex we can use, stable/*. seems d*** to have to specify them all19:58
jrollrloo: long version: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2018-02-02.log.html#t2018-02-02T11:38:59  short version: probably broken by https://review.openstack.org/#/c/536875/5/playbooks/legacy/tempest-dsvm-ironic-inspector/run.yaml19:59
patchbotpatch 536875 - ironic - Switch the CI to hardware types (MERGED)19:59
openstackgerritJulia Kreger proposed openstack/bifrost master: Fix typo in playbook markdown readme  https://review.openstack.org/54098819:59
jlvillalrloo, I agree, seems foolish to have to update it each time :(19:59
* jroll bbiab20:00
rloojroll: thx. those lovely hardware types. sigh.20:00
*** frti has joined #openstack-ironic20:01
*** yangyapeng has joined #openstack-ironic20:04
*** yangyapeng has quit IRC20:08
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications  https://review.openstack.org/52470920:10
openstackgerritMerged openstack/networking-baremetal master: Fix nits in networking-baremetal docs  https://review.openstack.org/54041320:11
*** Goneri has quit IRC20:11
*** ijw has joined #openstack-ironic20:12
TheJuliaI really need to fix my caching proxy....20:13
*** e0ne has joined #openstack-ironic20:15
*** mgoddard_ has joined #openstack-ironic20:19
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Fix bind_port error  https://review.openstack.org/54029520:27
openstackgerritJulia Kreger proposed openstack/networking-generic-switch master: Fix bind_port error  https://review.openstack.org/54029520:28
mgoddard_TheJulia: erm ^^^20:29
mgoddard_I think you won :)20:29
*** hw_wutianwei has quit IRC20:31
*** hw_wutianwei has joined #openstack-ironic20:32
TheJulialol20:33
*** mgoddard_ has quit IRC20:35
openstackgerritMerged openstack/bifrost master: Deprecate Classic Drivers  https://review.openstack.org/54015320:40
*** jcoufal has quit IRC20:48
*** yangyapeng has joined #openstack-ironic20:54
*** yangyapeng has quit IRC20:58
*** ijw has quit IRC21:04
*** ijw has joined #openstack-ironic21:06
TheJuliaanyone finding anything re: grenade?21:10
openstackgerritMerged openstack/networking-baremetal master: Node state configuration - add log_agent_heartbeat  https://review.openstack.org/53960021:16
openstackgerritJames E. Blair proposed openstack/bifrost master: Zuul: Remove project name  https://review.openstack.org/54101521:18
*** rcernin has joined #openstack-ironic21:26
*** e0ne has quit IRC21:36
*** e0ne has joined #openstack-ironic21:41
*** Goneri has joined #openstack-ironic21:42
*** yangyapeng has joined #openstack-ironic21:44
*** snasir has joined #openstack-ironic21:48
*** yangyapeng has quit IRC21:48
snasirHi,  can the ironic baremetal console be configured using novnc, or does it have to be shell-in-a box ?21:50
*** e0ne has quit IRC21:54
openstackgerritMerged openstack/python-ironicclient master: Replace use of functools.wraps() with six.wraps()  https://review.openstack.org/54019321:57
openstackgerritMerged openstack/python-ironicclient master: Use 'with' method rather than having to call close  https://review.openstack.org/53944221:57
rloosnasir: shell-in-a-box. not novnc. https://docs.openstack.org/ironic/latest/admin/console.html22:09
rlooTheJulia: nope (wrt grenade)22:14
snasirrloo: I don't need to integrate ironic GUI with horizon to access the baremetal web console right ?22:15
rloosnasir: oh. that's a good question. are you using ironic standalone (not with nova?)22:16
rloosnasir: i think that if you are using with nova, then you don't need ironic gui. if you are doing standalone, i suspect you do need ironic-ui.22:16
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications  https://review.openstack.org/52470922:17
snasirrloo: No i am using ironic with nova, not standalone, so initially i assumed that i could access the console using novnc just like for Nova VM instances, but it didnt work22:20
rloosnasir: with nova, there is serial console, socat.22:22
rloosnasir: it is this thing but i don't know if/where else there is documentation: https://blueprints.launchpad.net/nova/+spec/ironic-serial-console-support22:22
*** yangyapeng has joined #openstack-ironic22:23
snasirrloo: so socat is what i need to configure to access the baremetal console using nova-instance console in horizon, not shell-in-a box ? Sorry  for the confusion22:23
rloosnasir: yup22:24
snasirThanks22:26
*** yangyapeng has quit IRC22:28
*** early has quit IRC22:31
*** baha has quit IRC22:31
*** frti has quit IRC22:32
*** jlabarre has quit IRC22:33
*** early has joined #openstack-ironic22:33
*** etingof has quit IRC22:35
*** etingof has joined #openstack-ironic22:37
TheJuliaso this is interesting: http://logs.openstack.org/32/540432/2/check/ironic-grenade-dsvm-multinode-multitenant/199b2f4/logs/old/testr_results.html.gz is trying to connect to 172.24.5.7 but the address I traced back internally is 10.1.0.11. FIP request goes in... except I've thus far.. not seen any iptables commands executed22:57
TheJuliahttp://logs.openstack.org/32/540432/2/check/ironic-grenade-dsvm-multinode-multitenant/199b2f4/logs/screen-q-l3.txt.gz#_Feb_05_12_22_28_06319422:57
*** charcol-v2 has quit IRC23:05
*** ijw has quit IRC23:12
*** yangyapeng has joined #openstack-ironic23:13
*** bfournie has quit IRC23:15
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications  https://review.openstack.org/52470923:16
*** yangyapeng has quit IRC23:17
*** threestrands has joined #openstack-ironic23:19
*** threestrands has quit IRC23:19
*** threestrands has joined #openstack-ironic23:19
*** ijw has joined #openstack-ironic23:32
*** ijw has quit IRC23:37
*** MattMan has quit IRC23:37
*** MattMan has joined #openstack-ironic23:37
openstackgerritMerged openstack/networking-generic-switch master: Fix bind_port error  https://review.openstack.org/54029523:53
openstackgerritMerged openstack/ironic-inspector stable/pike: Remove use of construct lib FieldError exception  https://review.openstack.org/54061923:53
*** ijw has joined #openstack-ironic23:55
*** charcol-v2 has joined #openstack-ironic23:55
*** rpioso is now known as rpioso|afk23:58

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