Thursday, 2015-05-28

*** naohirot has joined #openstack-ironic00:02
*** Sukhdev has quit IRC00:04
*** Sukhdev has joined #openstack-ironic00:08
Sukhdevdevananda, jroll: Ping00:09
jrollhey Sukhdev00:09
Sukhdevdevananda, jroll: I just sent you an email - please have a look00:09
*** mtanino has quit IRC00:09
jrollSukhdev: will do, ty00:10
openstackgerritHaomeng,Wang proposed openstack/bifrost: supports redhat yum installation  https://review.openstack.org/18529400:26
*** Sukhdev has quit IRC00:28
*** alexpilotti has joined #openstack-ironic00:34
*** eghobo has quit IRC00:59
*** dprince has quit IRC00:59
*** blakec has joined #openstack-ironic01:05
*** igordcard_ has quit IRC01:12
*** igordcard_ has joined #openstack-ironic01:12
*** romcheg has quit IRC01:20
*** romcheg has joined #openstack-ironic01:21
*** blakec has quit IRC01:24
*** romcheg has quit IRC01:35
*** romcheg has joined #openstack-ironic01:36
*** alexpilotti has quit IRC01:42
*** kan_ has joined #openstack-ironic01:46
*** blakec has joined #openstack-ironic01:59
*** blakec1 has joined #openstack-ironic02:14
*** spandhe has quit IRC02:15
*** blakec has quit IRC02:17
*** blakec1 has quit IRC02:34
openstackgerritbaiyuan proposed openstack/ironic: Add documentation for getting a node's console  https://review.openstack.org/17664202:35
*** ijw has joined #openstack-ironic02:36
*** ramineni has joined #openstack-ironic02:37
*** MattMan has joined #openstack-ironic02:38
*** ijw_ has quit IRC02:39
*** Sukhdev has joined #openstack-ironic02:46
*** rwsu has quit IRC02:46
*** natorious is now known as zz_natorious02:46
*** takadayuiko has joined #openstack-ironic03:07
*** achanda has joined #openstack-ironic03:22
*** ijw_ has joined #openstack-ironic03:37
*** ijw_ has quit IRC03:39
*** ijw has quit IRC03:40
*** eghobo has joined #openstack-ironic03:56
*** achanda has quit IRC04:01
*** spandhe has joined #openstack-ironic04:01
*** spandhe_ has joined #openstack-ironic04:05
*** jaison has joined #openstack-ironic04:05
*** spandhe has quit IRC04:06
*** spandhe_ is now known as spandhe04:06
*** Haomeng|2 has joined #openstack-ironic04:45
*** Sukhdev has quit IRC04:45
*** Haomeng has quit IRC04:48
*** openstackgerrit has quit IRC04:50
*** openstackgerrit has joined #openstack-ironic04:51
*** romcheg1 has joined #openstack-ironic04:57
*** romcheg has quit IRC04:58
*** pradipta has joined #openstack-ironic05:05
*** jaison has quit IRC05:18
*** dkehn__ has joined #openstack-ironic05:24
*** dkehn has quit IRC05:27
*** achanda has joined #openstack-ironic05:38
*** harlowja_ has quit IRC05:40
*** spandhe has quit IRC05:58
*** rameshg87 has joined #openstack-ironic05:59
*** gridinv_ has quit IRC06:05
openstackgerritsandhya proposed openstack/ironic-specs: Ironic Chassis Discovery  https://review.openstack.org/18628506:10
*** yuanying has quit IRC06:11
*** radek__ has joined #openstack-ironic06:15
rameshg87good morning ironic06:16
takadayuikorameshg87: good morning :)06:17
rameshg87takadayuiko: o/06:17
zhenguorameshg87: morning :)06:21
rameshg87zhenguo: o/06:25
*** ukalifon1 has joined #openstack-ironic06:25
openstackgerritsandhya proposed openstack/ironic-specs: Ironic Chassis Discovery  https://review.openstack.org/18628506:28
openstackgerritsandhya proposed openstack/ironic-specs: Ironic Chassis Discovery  https://review.openstack.org/18628506:29
*** saripurigopi has joined #openstack-ironic06:29
*** yuanying has joined #openstack-ironic06:36
*** dlpartain has joined #openstack-ironic06:39
*** dlpartain has left #openstack-ironic06:39
openstackgerritsandhya proposed openstack/ironic-specs: Ironic Chassis Discovery  https://review.openstack.org/18628506:40
*** pas-ha has joined #openstack-ironic06:51
saripurigopiI'm seeing this error with py34 tests, not sure how to resolve it. 2015-05-14 23:41:14.400 | TypeError: catching classes that do not inherit from BaseException is not allowed06:54
*** yuanying has quit IRC06:59
*** yuanying_ has joined #openstack-ironic07:01
*** yog__ has joined #openstack-ironic07:04
*** Nisha has joined #openstack-ironic07:11
openstackgerritZhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning  https://review.openstack.org/18001407:22
*** ifarkas has joined #openstack-ironic07:29
Haomeng|2saripurigopi: can you show your code, looks like have to inhert from baseexception  in py34, else raise type error07:32
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add generic RAID configuration spec for liberty  https://review.openstack.org/17321407:46
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add inband RAID configuration spec for liberty  https://review.openstack.org/17321807:46
openstackgerritShivanand Tendulker proposed openstack/ironic: Update iLO documentation for UEFI secure boot  https://review.openstack.org/17858207:48
naohirotrameshg87: hello07:51
rameshg87naohirot: hi07:52
naohirotrameshg87: Hi, have recovered from the summit?07:53
naohirotrameshg87: have you recovered from the summit?07:53
rameshg87naohirot: almost :)07:53
naohirotrameshg87: good07:53
rameshg87naohirot: just joined work today07:53
naohirotrameshg87: I have a basic question regarding power driver.07:54
rameshg87sure07:54
naohirotrameshg87: currently Ironic reboot is hard reboot.07:54
*** jistr has joined #openstack-ironic07:54
naohirotrameshg87: If customer typed "nova reboot", he/she expect it is soft reboot.07:54
naohirotrameshg87: Have you discussed this inconsistency in Ironic ever?07:55
rameshg87naohirot: ah no07:56
rameshg87naohirot: how does it work for vms ?07:56
rameshg87naohirot: does it issue a soft reboot ?07:56
rameshg87naohirot: i think the main question is what happens if the soft reboot gets stuck (instance never reboots)07:56
naohirotrameshg87: In nova cli, there is a option --hard, "nova reboot --hard" means hard reboot.07:57
rameshg87oh okay07:57
naohirotrameshg87: and in Nova cli, no option means soft reboot.07:57
rameshg87naohirot: i always wished we do a soft power off during a reboot, wait for bare metal to be powered off polling the state07:57
rameshg87naohirot: wait for x seconds07:58
rameshg87naohirot: hard power off baremetal if it doesn't power off in x seconds07:58
rameshg87naohirot: and then power on the node07:58
naohirotnaohirot: I got a bug report from iRMC customer regarding this inconsistency.07:58
rameshg87in that way we should be able to reliably detect if instance ever rebooted07:58
rameshg87oh okay07:58
*** romcheg1 has quit IRC07:59
rameshg87naohirot: wdyt ?07:59
naohirotrameshg87: And iRMC has graceful shutdown scci API, and I'm thinking to implement it in vendor paththrough07:59
rameshg87naohirot: i think it was discussed before08:00
naohirotrameshg87: what does "wdyt" stand for?08:00
rameshg87what do you think - wdyt08:00
rameshg87:)08:00
naohirotrameshg87: :)08:00
naohirotrameshg87: then what was the conclusion at that time?08:00
rameshg87naohirot: yeah, may be try pursuing to make this a standard08:01
rameshg87naohirot: instead of a vendor passthru08:01
rameshg87there is enough interest i guess08:01
rameshg87i am +1 on introducing soft reboot in ironic :)08:01
naohirotrameshg87: does ilo have gracefull shutdown API too?08:01
rameshg87yes08:01
naohirotrameshg87: How about NMI API?08:02
rameshg87NMI ?08:02
naohirotrameshg87: which stands for Non Markable Interrupt,08:03
rameshg87naohirot: no idea08:03
naohirotrameshg87: out customer also would like to issue this API, which allow us to take kernel dump.08:03
rameshg87oh okay, got it08:04
naohirotrameshg87: in order to investigate kernel problem.08:04
rameshg87i need to check. at least haven't seen relating to nmi08:04
rameshg87i mean for ilo08:04
rameshg87*in ilo08:04
*** chenglch has joined #openstack-ironic08:04
naohirotrameshg87: When customer typed "nova reboot --hard",08:05
naohirotrameshg87: is the option "--hard" passed to ironic API server?08:06
rameshg87naohirot: i think no08:06
naohirotrameshg87: if you already knew it.08:06
rameshg87naohirot: it's upto ironic virt driver to pass that option08:07
naohirotrameshg87: Okay.08:07
rameshg87i think it doesn't today08:07
*** eghobo has quit IRC08:08
naohirotrameshg87: I'd like to fix this bug for irmc customer first through vendor API and then like to standardize it.08:09
naohirotrameshg87: do you see any problem regarding this approach, I agree to standardize it, but it takes time.08:10
rameshg87naohirot: sure. your wish :)08:10
rameshg87naohirot: i guess vendor passthrus dont require spec approval08:10
rameshg87naohirot: so you push code directly08:10
naohirotrameshg87: Okay, thanks.08:11
rameshg87naohirot: but disadvantage of vendor passthru is ironic api can be invoked only with admin credentials08:11
rameshg87naohirot: i.e tenants can't still invoke a soft reboot08:11
rameshg87naohirot: tenants don't even see ironic api08:11
rameshg87naohirot: is that acceptable for you ?08:12
naohirotrameshg87: really, I have checked policy.json,08:12
naohirotrameshg87: I thought all apis regardless vendor or not need admin credentials.08:13
rameshg87naohirot: yes, that is a problem right ?08:13
dtantsurMorning folks08:13
rameshg87naohirot: all users might not have admin rights08:13
rameshg87dtantsur: o/08:13
naohirotrameshg87: Yeah, kind of.08:14
naohirotdtantsur: good morning08:14
rameshg87naohirot: so soft power off through api a better option with proper integration with nova :)08:14
*** edcragg has joined #openstack-ironic08:15
naohirotrameshg87: are you thinking a scenario that tenant admin provides tenant user with power on off privilege?08:16
rameshg87naohirot: not exactly08:16
rameshg87naohirot: currently i believe vendor passthru apis are invokable only with admin rights08:17
rameshg87naohirot: every user might not be able to do this08:17
rameshg87(unless as you mentioned we tweak policy.json)08:17
naohirotrameshg87: I see08:18
rameshg87naohirot: need to go away. brb08:18
*** rameshg87 is now known as rameshg87-away08:18
naohirotrameshg87: Okay, thanks08:18
*** romcheg has joined #openstack-ironic08:24
saripurigopiHaomeng|2 : here is the review link https://review.openstack.org/#/c/159734/21/ironic/tests/drivers/ucs/test_power.py https://review.openstack.org/#/c/159734/21/ironic/common/exception.py08:28
*** dguerri`away is now known as dguerri08:30
*** leopoldj has joined #openstack-ironic08:31
*** Haomeng has joined #openstack-ironic08:31
*** Haomeng|2 has quit IRC08:34
*** rameshg87-away is now known as rameshg8708:38
*** ndipanov has joined #openstack-ironic08:40
*** lucasagomes has joined #openstack-ironic08:41
openstackgerritchenglch proposed openstack/ironic-specs: Add petitboot iscsi deploy driver support  https://review.openstack.org/17764108:46
*** derekh has joined #openstack-ironic09:00
*** itamarl has joined #openstack-ironic09:03
*** e0ne has joined #openstack-ironic09:22
*** e0ne is now known as e0ne_09:30
*** dtantsur is now known as dtantsur|brb09:34
*** e0ne_ has quit IRC09:35
*** itamarl has quit IRC09:37
sambettsMorning all o/09:37
takadayuikosambetts: o/09:45
lucasagomestakadayuiko, sambetts morning09:50
takadayuikolucasagomes: hi :D09:50
lucasagomeshello there09:51
*** naohirot has quit IRC09:55
*** e0ne has joined #openstack-ironic09:57
Haomengsaripurigopi: can you try to change this line - https://github.com/openstack/ironic/blob/master/ironic/common/exception.py#L53 to class IronicException(BaseException):09:58
rameshg87lucasagomes: hi there09:59
lucasagomesrameshg87, yo, hi09:59
rameshg87lucasagomes: i will try to write up a spec for wol power driver09:59
Haomengsaripurigopi: looks like from py3.x, have to derive from BaseException, then we can try/raise the exception I think09:59
rameshg87lucasagomes: i was wondering if you want to take that up09:59
saripurigopiHaomeng: sure. let me try.09:59
Haomengsaripurigopi: but odd, why the existing code works with py34 gate09:59
lucasagomesrameshg87, oh, that would be helpful09:59
lucasagomesrameshg87, did you test the code?09:59
rameshg87lucasagomes: i *should* get the hardware for trying it out in a week09:59
rameshg87:)09:59
lucasagomesack09:59
lucasagomesrameshg87, if I get some free time before I will start a spec for that10:00
Haomengsaripurigopi: good luck:)10:00
lucasagomesshould be fairly simple10:00
saripurigopiHaomeng: thought the same, I thought something wrong with my patch.10:00
rameshg87lucasagomes: yeah whoever gets to it first :)10:00
Haomengsaripurigopi: yes, take carefuly and good luck:)10:00
rameshg87and lucasagomes thanks for the ipa soft reboot patch10:00
saripurigopiHaomeng: :-)10:01
rameshg87i will try out and confirm it solves the uefi localboot issue10:01
lucasagomesrameshg87, oh no problem... Can you test it? I still need to open a bug for that10:01
Haomengsaripurigopi: :)10:01
lucasagomesrameshg87, cool10:01
*** lucasagomes has quit IRC10:03
*** lucasagomes has joined #openstack-ironic10:03
openstackgerritGopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers  https://review.openstack.org/15973410:11
saripurigopido we need to submit spec to add vendor apis?10:12
*** pradipta has quit IRC10:13
*** thrash|g0ne is now known as thrash10:18
*** ifarkas has quit IRC10:21
*** Nisha has quit IRC10:29
*** cdearborn has joined #openstack-ironic10:33
*** dtantsur|brb is now known as dtantsur10:34
*** kbyrne has quit IRC10:34
lucasagomessaripurigopi, vendor passthru stuff? nop10:35
takadayuikoI'm confused about ironic tests in tempest :( https://etherpad.openstack.org/p/liberty-ironic-functional-testing10:36
saripurigopilucasagomes: okay10:36
takadayuikowe have to leave away from tempest, right?10:37
dtantsurtakadayuiko, functional test != tempest10:37
dtantsurtakadayuiko, tempest is integration tests10:37
dtantsurboth will stay as far as I understand10:37
takadayuikodtantsur: oh, hi10:37
takadayuikodtantsur: Heh??10:37
dtantsurand g'evening :)10:37
dtantsuradam_g might know better10:38
dtantsuradam_g, around?10:38
* dtantsur digs into tempest10:39
*** e0ne is now known as e0ne_10:39
takadayuikodtantsur: what is functional test? At first, API tests in tempest will leave away from tempest. Is it correct?10:39
dtantsurtakadayuiko, functional test is a test of project in isolation from other services. we move it from tempest for sure.10:40
dtantsurtakadayuiko, integration test is test of common scenarios with other services; I'm not sure it leaves tempest10:40
*** kbyrne has joined #openstack-ironic10:41
dtantsurtakadayuiko, so this https://github.com/openstack/tempest/blob/master/tempest/scenario/test_baremetal_basic_ops.py seems to be an integration test10:42
dtantsurtakadayuiko, and these https://github.com/openstack/tempest/tree/master/tempest/api/baremetal are functional ones10:42
zhenguohi all, why node cannot be deleted in power on state?10:44
dtantsurzhenguo, hi. usually because something may go on on the node10:44
*** e0ne_ has quit IRC10:45
takadayuikodtantsur, tests in https://github.com/openstack/tempest/tree/master/tempest/api/baremetal will be moved into ironic/tests/functional, right?10:45
dtantsurI think so10:45
takadayuikook, thanks10:46
*** early has quit IRC10:46
zhenguodtantsur: but deleted from ironic seems doesn't affect that, it can still keep on.10:46
openstackgerritGopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers  https://review.openstack.org/15973410:47
*** Haomeng|2 has joined #openstack-ironic10:48
dtantsurzhenguo, I think it's rather a precautions in case some Ironic-related processes are going on the node10:49
dtantsurmaybe we should rather judge by provision state, not sure10:49
dtantsurdiscussable10:49
*** Haomeng has quit IRC10:52
zhenguodtantsur: I just find the original patch about this: https://review.openstack.org/#/c/76258/10:52
zhenguodtantsur: Lucas said the main reason is because otherwise these machines that are not being used might be on for long time10:52
*** e0ne has joined #openstack-ironic10:53
zhenguolucasagomes: are you around?10:53
*** early has joined #openstack-ironic10:53
lucasagomeszhenguo, hi yes10:54
*** ramineni has quit IRC10:54
lucasagomesoh that's old stuff heh10:54
lucasagomesI'm not sure I agree with that anymore10:54
lucasagomesbefore we had an assumption that deployments should start only when the node was powered off10:54
lucasagomesbut that has changed, now we don't care whether it's power on or off we reboot it anyway10:55
dtantsurit's still makes some sense to me to have deleted node in some clean state10:55
dtantsurat least including it being powered off10:55
dtantsurbut I don't insist :)10:55
lucasagomesdtantsur, it doesn't actually matter now with our state machine10:55
lucasagomesit will pass throught cleaning10:55
lucasagomesso it will check consistency there10:56
dtantsurlucasagomes, cleaning on deletion?10:56
dtantsurI don't think we do it (probably we should)10:56
lucasagomesnot on the delete, but on unprovisioning and provisioning10:56
dtantsuryep and we're talking about deletion :)10:56
lucasagomes(which we call delete)10:56
lucasagomesbut we are talking about delete from the db node-delete10:57
lucasagomesyeah names are hard heh10:57
openstackgerritGopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers  https://review.openstack.org/15973410:57
dtantsurexactly, which brings back my point "have deleted node in some clean state", i.e. if user wants to throw away the machine, he/she might want to at least make sure it's powered off10:58
dtantsurcompletely not sure here :)10:58
dtantsurbut maybe we should even allow cleaning before deletion10:58
dtantsurso that people wipe the hard driver before throwing it away10:58
dtantsurlucasagomes, ^10:59
lucasagomesdtantsur, we do cleaning after unprovisioning...10:59
dtantsurright10:59
lucasagomesbut I think we are talking about deletion from the DB only right?10:59
dtantsuryes10:59
lucasagomesmeaning that we can call node-delete for a node that was just enrolled10:59
lucasagomesbut never provisioned10:59
lucasagomesso we don't need to clean it11:00
dtantsuralso valid11:00
lucasagomeswe don't even know if we have credentials for that etc11:00
dtantsurok as we clean after unprovision, it no longer makes sense to me :)11:00
lucasagomesexactly11:00
dtantsurthen we should probably check provision state on node-delete, not power state11:00
lucasagomesyeah we do as part of unprovisioning11:00
lucasagomesand it's automatically triggered11:01
lucasagomesdtantsur, I think we do check provision_state too, we can't delete a node which is active and so on11:01
lucasagomesso it should pass throught cleaning to become available again and then you can delete from the db11:01
dtantsurright11:02
lucasagomesdtantsur, a-ha actually we don't... this need to be fixed11:03
lucasagomeswe only check if the node is associated with an instance (instance_uuid)11:03
dtantsurdefinitely11:03
lucasagomeswhich kinda means it's provisioned, BUT11:03
lucasagomesfor the standalone case it's not true11:04
lucasagomeswe don't need to associate with an instance to provision something11:04
* lucasagomes puts a patch up11:04
*** kan_ has quit IRC11:06
*** saripurigopi has quit IRC11:08
*** e0ne is now known as e0ne_11:11
*** e0ne_ is now known as e0ne11:12
dtantsurlucasagomes, ++ for patch11:13
TheJuliagood morning11:13
openstackgerritchenglch proposed openstack/ironic: Ironic should use its own context  https://review.openstack.org/18471411:14
dtantsurTheJulia, morning11:14
*** chenglch has quit IRC11:14
*** e0ne is now known as e0ne_11:20
TheJulia /win 111:24
TheJuliadoh11:24
*** e0ne_ has quit IRC11:25
*** dprince has joined #openstack-ironic11:31
*** e0ne has joined #openstack-ironic11:34
*** rameshg87 has quit IRC11:37
*** openstackgerrit has quit IRC11:39
*** openstackgerrit has joined #openstack-ironic11:39
*** takadayuiko has quit IRC11:44
*** early has quit IRC11:46
*** mjturek1 has joined #openstack-ironic11:49
*** early has joined #openstack-ironic11:49
TheJuliaHaomeng|2: Out of curiosity, Is revision 185294 to address bug 1455334 ?11:50
openstackbug 1455334 in bifrost "[biforst] install ironic failed" [Undecided,New] https://launchpad.net/bugs/145533411:50
*** saripurigopi_ has joined #openstack-ironic11:57
*** achanda has quit IRC11:57
sambettsTheJulia: Morning :)11:58
TheJuliagood morning sambetts11:59
*** saripurigopi has joined #openstack-ironic12:01
*** saripurigopi_ has quit IRC12:02
*** romainh has joined #openstack-ironic12:04
openstackgerritHaomeng,Wang proposed openstack/bifrost: supports redhat yum installation  https://review.openstack.org/18529412:09
*** dlpartain has joined #openstack-ironic12:15
*** dlpartain has left #openstack-ironic12:15
openstackgerritPeter Martini proposed openstack/bifrost: Add CONTRIBUTING.rst to doc/source/dev  https://review.openstack.org/18600512:20
openstackgerritImre Farkas proposed stackforge/ironic-discoverd: Do not require local_gb by ramdisk  https://review.openstack.org/18603312:20
Haomeng|2TheJulia: no, https://bugs.launchpad.net/bifrost/+bug/1455334 is fixed by Deva's patch, let me check out the number, and for 185294, it supports Redhat rpm installation12:21
openstackLaunchpad bug 1455334 in bifrost "[biforst] install ironic failed" [Undecided,New]12:21
openstackgerritPeter Martini proposed openstack/bifrost: Redirect error echoes to STDERR  https://review.openstack.org/18582312:25
*** trown|outttypeww is now known as trown12:25
Haomeng|2TheJulia: shoud be this fix - https://review.openstack.org/#/c/183190/12:27
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove conditional that prevent node delete based on power state  https://review.openstack.org/18636112:28
*** ifarkas has joined #openstack-ironic12:31
Haomeng|2TheJulia: and I have updated launch bug -145533412:31
*** e0ne is now known as e0ne_12:32
*** romainh has quit IRC12:33
*** romainh has joined #openstack-ironic12:35
Haomeng|2TheJulia: One idea, how do you think if we have plan to integrate ironic discoverd into biforst, then we have standalone ironic+discoverd which can cover more functions12:36
* dtantsur whispers: it's called ironic-inspector now :)12:38
TheJuliaHaomeng|2: I don't presently have a plan aside from reading the code, but I do agree it sounds intruiging.  Sadly I was just too burnt out from OpenPlague on Friday to have an attention span suitable to learn about the inspector. :)12:39
* TheJulia wonders if there is a function called gadget in the inspector... since referencing the inspector gadget cartoon just seems like something that should be done.12:40
* TheJulia clearly has woken up silly12:41
Haomeng|2TheJulia: ok, yes, it is called ironic-inspector now:)12:41
openstackgerritImre Farkas proposed stackforge/ironic-discoverd: Fake missing local_gb for root_device_hint plugin  https://review.openstack.org/18589612:41
Haomeng|2TheJulia: and I have anoher idea, but not sure if it is easy to implement, can we integrate biforost as a dib element, and if we can run dib with 'biforst' to build a standalone iso/usb drive which can be run directly:) Just a draft idea, but not sure...12:42
*** e0ne_ has quit IRC12:43
*** saripurigopi has quit IRC12:45
Haomeng|2TheJulia: that is my dream, a *live* standalone ironic to deploy baremetal directly without any installation and configurations:)12:45
TheJuliaHaomeng|2: I share this dream :)12:46
Haomeng|2TheJulia: :)12:46
TheJuliaHaomeng|2: interesting idea on the dib element, perhaps once we have the dream :)12:46
Haomeng|2TheJulia: :)12:46
*** e0ne has joined #openstack-ironic12:47
Haomeng|2TheJulia: now ironic-inspector does not support to work with standalone ironic, I wil try to enable12:48
Haomeng|2go to sleep, nice day Ironic:)12:49
TheJulia:)12:49
lucasagomesHaomeng|2, night12:50
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Rename discoverd -> inspector  https://review.openstack.org/18549912:50
lucasagomesdtantsur, w00t12:51
dtantsur:)12:51
dtantsurTheJulia, I strongly believe lucasagomes will draw a mascot for inspector: bear with gadget hands ;)12:52
TheJuliaI fully support this idea12:53
*** romcheg1 has joined #openstack-ironic12:53
*** romcheg has quit IRC12:54
*** yog__ has quit IRC12:55
*** yog__ has joined #openstack-ironic12:56
martiniI changed my doc patch to move CONTRIBUTING.rst to doc/source/dev/ and now gate-bifrost failed :-(13:00
TheJulia:(13:01
TheJuliawow, it was almost done, and we hit the timeout I set on the job13:03
lucasagomes:-D13:03
lucasagomestalking about drawing mascots etc13:03
lucasagomesI got a surprise for all you guys... I want to create an official website for ironic13:03
lucasagomesI got the domain already13:03
lucasagomesironic.rocks13:03
lucasagomestype that in ur browsers :-)13:03
TheJulialucasagomes: very nice13:04
dtantsuroh man that's gorgeous!13:05
* dtantsur uses major version bump due to rename to drop all the old stuff13:07
*** rloo has joined #openstack-ironic13:07
lucasagomesyeah now it just redirects to the wiki page13:07
lucasagomes(I got it yesterday so... didn't have time to create anything yet)13:09
lucasagomesand whoever wants to help give me a shout cause I know nothing about html css etc...13:09
lucasagomeshtml5*13:09
dtantsuroh html...13:10
dtantsurI usually start with taking Bootstrap :)13:10
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/ironic-lib: Merge lastet disk partitioning code from ironic  https://review.openstack.org/16216213:11
lucasagomesdtantsur, you mean this thing http://getbootstrap.com ?13:13
dtantsuryep13:13
dtantsurlucasagomes, just take one if the example and adapt13:13
lucasagomesdtantsur, nice, thanks! I will take a look after work13:13
lucasagomesyeah sounds good13:14
*** dlpartain has joined #openstack-ironic13:14
lucasagomeslooks neat, works with multiple devices (resizing and stuff)13:14
*** dlpartain has left #openstack-ironic13:14
martiniTheJulia: so, what should I do now?13:16
TheJuliamartini: recheck the job13:18
TheJuliaI suspect I'll need to bump the timeout a little13:19
martiniWhat's the proper way to do that?  (Should I wait a bit then?)13:19
TheJuliaJust leave a comment with "recheck" on the review. Waiting for a little while would be a good idea, this is the peak time for jobs to run.13:21
cdearbornMorning Ironic!13:25
cdearbornI found the issue that was causing the tons of object not iterable errors when running the py34 unit tests13:25
cdearbornIt turns out there is a bug in mock in py34 on at least Fedora 2113:26
lucasagomes:-(13:27
* lucasagomes is still stuck on F2013:27
rloomorning and hi everyone. ironic.rocks :-)13:28
lucasagomesrloo, morning :-)13:28
cdearbornwhere code like x.side_effect = xxx.SomeException() will cause the not iterable error13:29
*** jamielennox is now known as jamielennox|away13:29
cdearbornI hacked a fix into my local copy of mock.py & can run all unit tests with no problems13:29
lucasagomescdearborn, is it something that was fixed on the last versions of mock?13:29
cdearbornhmmmm - not sure - I'll poke around13:31
*** ChuckC has quit IRC13:32
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove bogus conditional from node_update  https://review.openstack.org/18638613:33
lucasagomescdearborn, ack, otherwise we may have some problems once other distros starts packing newer versions of this lib13:33
cdearbornyeah - it looks like they added the capability to pass in an iterable as the side_effect, but when they did that, they broke the ability to pass in an exception13:34
lucasagomesright13:34
lucasagomesyeah maybe side_effect should always be an interable and we have been using it wrongly13:34
lucasagomes(that's when bugs turns into features :-) )13:35
rloolucasagomes: fyi, if you had any wsme changes you wanted in, looks like they might do a new release on Monday (https://bugs.launchpad.net/wsme/+bug/1459469)13:35
openstackLaunchpad bug 1459469 in WSME "wsme is not correctly requiring simplegeneric" [Undecided,New]13:35
sambettsthe mock docs say you can pass an exception, and even their example is a KeyError13:35
lucasagomesrloo, oh yeah. We def should make a release of wsme soon13:36
cdearbornyup - saw that13:37
cdearbornand you can definitely do that in prior versions of mock13:37
*** bnemec has quit IRC13:37
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Rename discoverd -> inspector  https://review.openstack.org/18549913:37
dtantsurthe biggest change I've ever done ^^^13:37
lucasagomesrloo, so apparently that problem of wsme not pulling the dependencies correctly has been fixed in master13:38
TheJuliadtantsur: impressive :)13:38
rloolucasagomes: right, which is why they are going to do a release on Monday.13:38
rloolucasagomes: so if there is anything else you wanted in that upcoming release and can get it in this week... :-)13:39
lucasagomesyeah13:39
lucasagomesI've been reviewing and approving stuff in wsme13:39
rloolucasagomes: ahh, good. thx for doing that!13:39
lucasagomeshe things I had to fix that impacted on the ipxe is already merged upstream13:40
lucasagomesthe*13:40
lucasagomesI hope we will do release more often from now on13:40
*** achanda has joined #openstack-ironic13:40
rloolucasagomes: looks like you might just need to ask hellman if you want a release?13:40
lucasagomesrloo, yeah, currently I think only hellmann can tag a release for WSME13:41
lucasagomesso we just need to ping him when we think it's a good time13:41
rloolucasagomes: here's hoping we don't need any more changes in wsme after this release :-)13:42
*** bnemec has joined #openstack-ironic13:42
lucasagomesoh we do... for the sake of wsme itself13:42
lucasagomesthere's still quite few things that needs improving13:43
lucasagomeswe got few good new contributors since that email thread13:43
lucasagomesso it's speeding up things a lot13:43
lucasagomesand with all this talk about wsme, someone can take a look at https://review.openstack.org/#/c/177726/ please?13:45
rloolucasagomes: will try to look today. (thinking of reviewing specs but am procrastinating...)13:49
lucasagomesheh thanks13:49
jrollmorning everyone :)13:50
dtantsurmorning jroll, rloo13:51
lucasagomesjroll, yo morning13:51
cdearborndid some poking, and the bug is in py3.4.3 and 3.5.0.b113:54
*** mdbooth has quit IRC13:54
*** mdbooth has joined #openstack-ironic13:55
rloohiya jroll, dtantsur13:56
lucasagomescdearborn, right... so we better fix out code to make side_effect always interable?13:58
lucasagomesit looks like they want to enforce it for newer versions of mock13:58
*** romcheg1 has quit IRC13:59
*** romcheg has joined #openstack-ironic13:59
cdearbornpulling down latest python source now14:00
cdearbornlucasagomes, where did you see that?  according to the docs, it's ok to set side_effect to an ex...14:01
lucasagomescdearborn, I didn't that was a question... maybe I misread ur comments14:02
cdearbornlucasagomes, ah ok14:06
rloocdearborn: thx for looking into that.14:06
cdearbornso it's broken in the latest py source too14:06
rloocdearborn: sounds similar to some bug that jlvillal worked around: http://bugs.python.org/issue1782614:06
cdearbornrloo, np14:06
rloocdearborn: if this helps, this is the (one of anyway) patches that jlvillal added: https://review.openstack.org/#/c/169109/14:08
jrolldevananda: can you review https://review.openstack.org/#/c/185171/3 this morning? wanted to update it sometime today14:08
*** achanda_ has joined #openstack-ironic14:13
NobodyCamgood morning Ironic14:14
sambettscdearborn: I made note yesterday that the only difference between my devstack install (ubuntu 14.04) and my dev machine (Debian Jessie) was that Jessie is running 3.4.2 and ubuntu is still on 3.4.014:15
sambettsMorning NobodyCam14:16
openstackgerritShivanand Tendulker proposed openstack/ironic: Update iLO documentation for UEFI secure boot  https://review.openstack.org/17858214:16
NobodyCammorning sambetts14:16
lucasagomesNobodyCam, morning14:16
NobodyCammorning lucasagomes :)14:16
dtantsurNobodyCam, morning14:16
bcornecmorning everybody here ;-)14:16
dtantsurmorning you too :)14:17
sambettso/ bcornec14:17
*** achanda has quit IRC14:17
NobodyCammorning dtantsur bcornec :)14:17
cdearbornsambetts, it's easy enough to check for the bug - i can let you know how if you are interested14:19
cdearbornso changing the unit tests from this: "xxx.side_effect = exception" to this: "xxx.side_effect = [ exception ]" does hack around the bug14:21
sambettscdearborn: How do I check, I'm pretty sure its the same issue, I think I might end up install pyEnv so I can change my python install easily14:21
cdearbornin mock.py, go to the __get_side_effect method.  In that method, if you see this "if" statement, then it's busted "if sf is not None and not callable(sf) and not isinstance(sf, _MockIter):"14:23
*** mtanino has joined #openstack-ironic14:28
*** leopoldj has quit IRC14:30
jrollin case you missed it: https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg53815.html14:34
lucasagomesyuriyz, ping why was https://review.openstack.org/#/c/185885/ abandoned?14:37
rloojroll: do you think we should add that meeting/link to https://wiki.openstack.org/wiki/Meetings/Ironic or https://wiki.openstack.org/wiki/Ironic14:38
yuriyzhi lucasagomes Ironic there is new and better (IMO) solution https://review.openstack.org/#/c/186129/14:38
lucasagomesgotcha14:39
lucasagomesthanks for pointing out14:39
jrollrloo: probably :)14:39
rloothx jroll :D14:40
NobodyCammorning jroll rloo and yuriyz14:40
NobodyCam:)14:40
rloomorning NobodyCam, yuriyz and jroll14:41
jrollmorning *14:41
NobodyCam:)14:41
mordredNobodyCam: have I mentioned how much I enjoy the bearmetal thing ?14:42
NobodyCammorning mordred :) thing == tee shirt?14:43
NobodyCamhehe14:43
BadCubhiya folks14:43
* NobodyCam notes its always morning in Ironic-land-de-a14:44
jrollrloo: updated :P14:45
persiahttp://www.total-knowledge.com/~ilya/mips/ugt.html14:45
mordredNobodyCam: I've seen more references to it14:45
mordredNobodyCam: https://blueprints.launchpad.net/python-openstacksdk/+spec/high-level-bearmetal14:45
mordredNobodyCam: so it's catching on :)14:45
mordredNobodyCam: I think you should run with it14:45
*** zz_jgrimm is now known as jgrimm14:46
mordredand spell bare metal provisioning bear as often as you can :)14:46
jrollhehehe14:46
rlooha ha, no more nit'ing on 'bear metal'!14:46
jrollI love it14:46
jrollrloo: s/bare/bear/ -214:46
*** lilia_ has joined #openstack-ironic14:47
NobodyCam:)14:47
jrollpersia: +++++14:47
* NobodyCam look for his running shoes 14:47
jrollUGT is my new favorite tz14:47
NobodyCampersia: ++++ I love it14:47
NobodyCamyes!!!14:47
BadCublol14:48
*** afaranha has joined #openstack-ironic14:51
lilia_Hey guys, how are you? Hope everything is fine (: So, this is going to be a bit of a long message but I need to give you context haha ((:14:51
lilia_We're working in a new driver for OneView and, hopefully, some of you might be able to help us in an "issue" we just stumbled across. The thing is, right after we managed to complete the whole deployment process using our driver, we realized the node and the instance are assuming an Active state before the instance is actually accessible.14:52
lilia_This happens after the deploy kernel and ramdisk images are downloaded and the machine starts a reboot, but no user image is completely deployed. After 2 minutes or so, when the reboot process is finished and an user image is deployed, we then have SSH access.14:52
*** Sukhdev has joined #openstack-ironic14:52
jrolllilia_: right, so that's expected behavior14:52
lilia_Is this somehow misleading in your opinion? We thought it might be, but not sure if this is a regular behavior or something to worry about.14:52
jrollit sucks, but14:53
jrollwe also don't want to force users to run code in their image, so there's really no way to deal with it14:53
jlvillalcdearborn, sambetts: Are you seeing the same mock errors for Python 3.4 on the python-ironicclient also?14:55
* jlvillal wonders is it a bug in the new Python 3.4 or are we going to have to update all of our mock test code...14:55
lilia_jroll: Oh I see.. We really thought it might be something we couldn't have control over. That's fine then, thanks!!14:57
jrolllilia_: cool, you're welcome :)14:58
jrolllilia_: also looking forward to seeing a patch for that driver ;D14:58
NobodyCamlilia_: jroll: ++14:58
*** zz_natorious is now known as natorious14:59
*** e0ne is now known as e0ne_15:00
*** e0ne_ is now known as e0ne15:00
lilia_jroll: NobodyCam +++++++++++++++15:00
lilia_(((:15:00
lilia_We're working in the spec right now, so you're going to see  it really soon (: (fingers crossed!!)15:01
*** spandhe has joined #openstack-ironic15:02
* BadCub heads off to meeting15:02
jroll\o/15:02
*** spandhe_ has joined #openstack-ironic15:05
*** mtanino has quit IRC15:06
*** spandhe has quit IRC15:06
*** spandhe_ is now known as spandhe15:06
lucasagomespersia, +1!15:07
*** mtanino has joined #openstack-ironic15:07
*** mtanino has quit IRC15:07
*** mtanino has joined #openstack-ironic15:08
jrollhttps://conference.hitb.org/hitbsecconf2015ams/wp-content/uploads/2015/02/D1T1-Xeno-Kovah-and-Corey-Kallenberg-How-Many-Million-BIOSes-Would-You-Like-to-Infect.pdf15:09
jrollholy s%^&%15:09
jrollwtb open firmware15:09
jroll^ devananda you'll dig these slides15:09
*** wuhg has quit IRC15:09
lucasagomeslilia_, that's the driver that the folks in recife brazil are working on?15:09
devanandamorning, all15:13
NobodyCammorning (UGT) devananda15:13
BadCubmorning devananda15:13
devanandajroll: open firmware exists for some platforms. buy different hardware15:14
jrolldevananda: I don't want this for me. I want it for the world15:14
dtantsurdevananda, morning!15:15
dtantsurdevananda, do you have folks to punch to make https://review.openstack.org/#/c/185442 marge faster?15:16
dtantsurwill block the renaming work in project-config, I guess15:16
lucasagomesdevananda, morning15:17
devanandajroll: hah. demo of defeating tails proves what I suspected last year. I should not laugh ... this is bad sh*t15:17
jrolldevananda: yeah. laughter is a form of grieving for some people.15:18
devanandadtantsur: nope. that will take about a week.15:18
devanandadtantsur: TC meets and approves those on tuesdays. needs to be proposed before the previous friday, but this week was basically omg-were-tired because of summit15:18
dtantsurI see15:19
devanandadtantsur: it should get put on https://wiki.openstack.org/wiki/Meetings/TechnicalCommittee by ttx, though I dont see it yet15:19
* devananda pokes him15:19
NobodyCamjroll: ya holy _______15:20
lucasagomesjroll, http://mjg59.dreamwidth.org/35110.html15:22
lucasagomesSMM is ugly... but it's not all bad15:22
lucasagomesit's useful for some stuff15:22
dtantsurdevananda, thanks!15:22
* NobodyCam wounders if he can do his job from a 6502 based system15:22
dtantsurdevananda, do I need to be present?15:22
devanandadtantsur: in case there are questions, it wouldn't hurt15:25
dtantsurack15:25
dtantsur20:00 UTC omg15:25
devanandadtantsur: see -dev for discussion, but yea, this should be an easy merge because it's housekeeping15:26
dtantsurok I'll try to be there15:27
*** ukalifon1 has quit IRC15:37
jnielsen55question.  I'm using PXE for deploys. Do I need to tell dnsmasq where to find the tftp server or is that taken care of by Ironic?15:39
devanandadtantsur: though, re: inspector client, I wonder if it's reasonable to use ironicclient? I haven't thought about it much, just want your initial reaction15:39
dtantsurhmmm15:40
dtantsurmy initial thought was separate service -> separate client15:40
devanandajnielsen55: depends - are you using neutron to provide dhcp or not?15:41
devanandadtantsur: same project, same client? heh15:41
dtantsurwell...15:42
jnielsen55yes, dhcp is supplied by neutron15:42
devanandadtantsur: what would happen if I ran inspector on the same host as ironic-api15:42
dtantsurwhat's the problem with it?15:42
devanandajnielsen55: then ironic+neutron should take care of dhcp settings15:42
devanandadtantsur: i dont know - is there a problem with that? just wondering if it makes sense15:43
jnielsen55very nice, setup going well so far. Thanks for the info.15:43
dtantsurdevananda, I'm testing everything running on one host usually :)15:43
dtantsurdevananda, one problem about using one client is that inspector client is much thinner: 2 functions using requests lib and a shell using openstackclient15:44
dtantsurI'll probably have to write essentially more code for it to look alike usual openstack client (not that it's bad...)15:45
devanandadtantsur: here's my thoughts: I think it would make operators' workflow better if there was one client and one way to talk to "ironic" - whether for discovery or enrollment or what ever  - but considering that discoverd may or may not be run on the same host as ironic-api, i'm not sure how we can do that15:47
dtantsuroh yeah, that will give us some headache. we'll also explain people that they need to provide one more endpoint15:47
devanandaI'm also not sure how we would expose (if there were a single client) that discoverd was or was not present15:48
dtantsur* s/discoverd/inspector/ ;)15:48
devanandaour headaches aside, what's best for operators here?15:48
devananda*yes :)15:48
dtantsurfor operators it's best to switch to openstackclient IMO15:48
dtantsurand then we all will have one shell15:48
devanandafair enough15:48
dtantsurspeaking of which, should we start discussion this API? patches are already up15:49
devanandayep15:49
devanandawe should try to do that this cycle15:49
*** natorious is now known as zz_natorious15:49
lucasagomesdtantsur, zhenguo https://review.openstack.org/#/c/186361/15:49
lucasagomesanswer that re FAIL states15:50
lucasagomeslemme know WDYT before I update the patch15:50
dtantsurdevananda, so I would say: leave ironic_inspector_client separate, rely on openstackclient plugins for unification. wdyt?15:50
dtantsurlucasagomes, sounds fine to me15:51
lucasagomesdtantsur, ack, I will update the exception bits15:51
lucasagomesthanks for the review15:51
devanandadtantsur: seems ok15:52
dtantsurdevananda, and last (hopefully) question: can I propose such a governance change for not-yet-existing repository?15:53
devanandayes15:53
devanandain the commit msg, call out what you're proposing15:53
dtantsurcool, thanks!15:54
devanandathat'll be all the "ok" that infra needs when you ththen peopose the change to create the actual repo15:54
*** chlong has quit IRC15:54
devananda*propose15:54
*** jistr has quit IRC15:56
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove conditional that prevent node delete based on power state  https://review.openstack.org/18636115:57
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove bogus conditional from node_update  https://review.openstack.org/18638615:57
*** mtanino_ has joined #openstack-ironic16:02
*** mtanino has quit IRC16:03
*** zz_natorious is now known as natorious16:04
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove deprecated pxe_deploy_{kernel, ramdisk}  https://review.openstack.org/18593716:05
*** absubram has joined #openstack-ironic16:05
*** chlong has joined #openstack-ironic16:07
*** chuckC_ has joined #openstack-ironic16:09
sambettsThe IRC bots make me feel bad for pushing a whole bunch of patchs all at once, I've just split a huge one into 10 separate ones, and doing git review is going to spam the chat :/16:10
NobodyCamsambetts: better then reviewing a hugh patch :)16:10
*** dtantsur is now known as dtantsur|afk16:12
sambettsNobodyCam: I guess thats true :)16:13
jlvillaljroll: On https://review.openstack.org/#/c/130877/10/nova/tests/unit/cells/test_cells_state_manager.py   Since I don't really understand I will probably just blindly do what is suggested.16:13
jlvillaljroll: Unless you object.16:13
openstackgerritSam Betts proposed openstack/ironic: Remove E12[3-8] from flake8 ignore list  https://review.openstack.org/18602116:14
openstackgerritSam Betts proposed openstack/ironic: Fix flake8 E12[3-8] for ironic/tests/drivers  https://review.openstack.org/18645016:14
openstackgerritSam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in non-driver ironic/tests  https://review.openstack.org/18645116:14
openstackgerritSam Betts proposed openstack/ironic: Fix flake8 E12[3-8] for ironic/tests/api  https://review.openstack.org/18645216:14
openstackgerritSam Betts proposed openstack/ironic: Fixes flake8 E12[3-8] for ironic/tests/conductor  https://review.openstack.org/18645316:14
openstackgerritSam Betts proposed openstack/ironic: Fixes flake8 E12[3-8] for ironic/drivers  https://review.openstack.org/18645416:14
openstackgerritSam Betts proposed openstack/ironic: Fixes flake8 E12[3-8] in ironic/conductor  https://review.openstack.org/18645516:14
openstackgerritSam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in ironic/common  https://review.openstack.org/18645616:14
openstackgerritSam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in ironic /cmd /db and /dhcp  https://review.openstack.org/18645716:14
openstackgerritSam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in ironic/api  https://review.openstack.org/18645816:14
sambettshaha sorry guys :/16:15
NobodyCamsambetts: don't be sorry for submitting patches16:16
NobodyCambe happy!16:16
*** rwsu has joined #openstack-ironic16:16
*** ijw has joined #openstack-ironic16:18
sambettsI hope I broke them down enough :/ a couple of them are still biggish16:19
*** derekh has quit IRC16:21
jlvillalsambetts: Are you abandoning the other patch?16:21
sambettsjlvillal: No, I reused that patch as the one at the top of the tree, that finally removes the rule from tox.ini16:22
*** romcheg has quit IRC16:22
lucasagomesfolks I will call it a day16:22
jlvillallucasagomes: Good night!16:22
sambettslucasagomes: Cya later o/16:22
lucasagomescame to the office and gotta get the train back home yet16:22
lucasagomessee y'all16:22
jlvillalsambetts: Okay, thanks.16:22
lucasagomesg'night16:22
*** lucasagomes has quit IRC16:23
*** eghobo has joined #openstack-ironic16:23
jlvillaljroll: Never mind.  I added new patch.16:23
*** spandhe has quit IRC16:23
cineramahi ironic people16:24
jlvillalsambetts: Yeah that first patch (drivers) is big :)16:24
NobodyCammorning cinerama16:24
jlvillalsambetts: But still reviewable16:24
sambettsjlvillal: the rest are much smaller and managable :)16:25
*** ndipanov has quit IRC16:26
NobodyCamnight lucas16:26
jlvillalsambetts: Thanks for doing this.  I would like the code to be more PEP8 compliant.16:26
jlvillalsambetts: Feel free to tackle Nova next :D   They exclude I think every E12[1-9]16:27
sambettsjlvillal: man that would be insane ...16:28
jlvillalsambetts: We (Ironic) had it the same when I first started.  So it can be done :)16:28
BadCubmorning cinerama :)16:28
sambettsjlvillal: For not having it enforced by flake8 I was surprise how compliant most of the code was16:30
jlvillalsambetts: I think people have been doing comments like "nit: indentation"   It is better to have the computer say that.16:31
jlvillalsambetts: On a somewhat related note.  Have you seen 'yapf'?  https://github.com/google/yapf   I am not suggesting we run it on the code.  But it is kind of cool16:33
rlooI've been noticing quite a few 'recheck' comments. Are people looking to see why a test fails?16:34
rlooeg http://logs.openstack.org/82/178582/6/check/check-tempest-dsvm-ironic-agent_ssh/821d090/console.html16:35
rlooI saw that yesterday in another patch, and it seems related to https://bugs.launchpad.net/openstack-ci/+bug/139309916:35
openstackLaunchpad bug 1393099 in OpenStack-Gate "test_baremetal_server_ops fails with timeout waiting for provision_state to change" [Undecided,Confirmed]16:35
rlooI can put a note about it in our etherpad, but don't know if it'll help. People need to look to see why tests are failing.16:36
sambettsjlvillal: I hadn't seen that, but I've been using autopep8 for my own code for a while, it looks pretty cool, I'll have to try it out16:36
devanandarloo: i think I posted a patch to raise that timeout ....16:38
rloodevananda: did it get merged yet?16:39
*** saripurigopi has joined #openstack-ironic16:40
jlvillalsambetts: And on another note.  Welcome to Ironic!  I don't think I have seen you on here before this week.  If you have been, sorry for not noticing.16:41
rloojlvillal: sam was at the summit last week. Sorry, should have introduced you to him :-)16:41
jlvillalrloo: Maybe I would recognize.  I vaguely remember someone from Cisco.  But people became a blur....16:42
rloojlvillal: haha. yeah, I know what you mean. you're all blurry to me too :D16:42
ijwsambetts is naturally blurry16:42
sambettsThanks ijw :-P16:43
cineramai *feel* blurry...still have the dregs of summit plague16:43
rloook, i added a note to the etherpad about rechecking. am thinking that if I notice people checking for test failures, maybe I'll give them brownie points and review one of their patches ;)16:43
*** trown is now known as trown|lunch16:43
*** yog__ has quit IRC16:44
jlvillalrloo: I do look at the log.  I will admit when I see the timeout error I do a 'recheck' as I didn't now how to fix the underlying issue.16:45
jlvillals/now/know/16:45
jlvillalrloo: Also I have done recheck on other people's patches when different tests fail.  One time it is test A, and then next time test B.  I do a recheck and then usually it works.16:45
rloojlvillal: you aren't expected to know how to fix it. the recheck thingy is part of the 'elastic recheck' process16:46
rloojlvillal: http://status.openstack.org/elastic-recheck/16:46
sambettsjust adding recheck bug <bug number> helps :)16:46
jrollthe timeout error is typically "pxe and/or callback" failed16:47
jlvillalrloo: Thanks16:47
jrollso you can dig into console logs, conductor logs16:47
jlvillaljroll: thanks for the info.16:50
*** ijw_ has joined #openstack-ironic16:51
*** Sukhdev has quit IRC16:52
*** pradipta has joined #openstack-ironic16:53
sambettsThis one is weird as, http://logs.openstack.org/37/185937/2/check/gate-ironic-python27/1204cf2/console.html it seems like all the UTs passed no worries, then something goes wrong :/16:54
*** achanda_ has quit IRC16:54
*** ijw has quit IRC16:54
jlvillalsambetts: I thought I would never get through https://review.openstack.org/#/c/186450/1   But I did.  Looks good to me.16:54
sambettsjlvillal: the others are much nicer :)16:55
*** jrist has quit IRC16:58
*** romcheg has joined #openstack-ironic17:01
*** harlowja has joined #openstack-ironic17:05
*** dguerri is now known as dguerri`away17:06
*** e0ne has quit IRC17:08
*** romainh has left #openstack-ironic17:11
*** jrist has joined #openstack-ironic17:11
cineramahey TheJulia, when you saw that segfault the other day what kind of system were you running? and how did you resolve?17:12
*** ifarkas has quit IRC17:14
sambettsRight, I'm heading off for the evening :) Night all17:17
NobodyCamnight sambetts17:17
jlvillalsambetts: Good night!17:22
*** dguerri`away is now known as dguerri17:22
TheJuliacinerama: I was on my base system Ubuntu 14.04.  I've not gotten back to it yet17:24
cineramasame17:24
*** saripurigopi has quit IRC17:26
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Refactor resource_fields.py  https://review.openstack.org/18198517:29
TheJuliabbiab17:29
*** lilia_ has quit IRC17:42
*** pas-ha has quit IRC17:46
*** romcheg has quit IRC17:49
*** trown|lunch is now known as trown17:49
cdearbornjlvillal, just got back from lunch - just tried running the py34 unit test on python-ironicclient, and they run successfully17:51
jlvillalcdearborn: But openstack/ironic fails?17:51
jlvillalwierd...17:51
jlvillalor weird17:52
*** achanda has joined #openstack-ironic18:00
cdearbornjlvillal, yeah - must not follow the same code path18:05
*** e0ne has joined #openstack-ironic18:06
*** jrist has quit IRC18:11
cineramaTheJulia: looks like it's the -I option that's triggering it18:12
jlvillalTheJulia: Is Bifrost check broken?  Or did this patch: https://review.openstack.org/#/c/185294/  Actually cause a valid failure?18:12
*** alexpilotti has joined #openstack-ironic18:14
devanandakrotscheck: hey, so at some point, I assume you're planning to propose your ui project to the openstack/ namespace -- any thoughts on when? (you probably told me at the summit and I've just forgotten)18:16
krotscheckdevananda: I haven't really sat down and drawn up the dependency chain for all the thigns I need to land to make that webclient packageable.18:18
krotscheckAnd it's not a small list.18:19
devanandajlvillal, TheJulia: 2015-05-28 12:39:24.074 | Build timed out (after 30 minutes). Marking the build as failed.18:19
devanandajlvillal: not a real failure. something was slow, it timed out18:19
krotscheckEspecially given that I personally do not consider ironic usable without glance.18:19
NobodyCamdevananda: yea I just started a recheck on that18:19
jlvillaldevananda: Thanks.18:19
devanandaTheJulia: In looking at the logs there, there are a lot of steps that could be pre-cached18:20
devanandaTheJulia: eg, we dont need to build a new ramdisk or download images every time we test bifrost18:20
*** e0ne has quit IRC18:21
devanandakrotscheck: I can supply Ironic with an HTTP ref to Amazon S3 and deploy from that18:21
devanandakrotscheck: i think we had a conversation about glance images in YVR, but i've forgotten what we said18:21
krotscheckdevananda: That's useful for someone who's willing to copy/paste those references for every deploy.18:21
krotscheckCopy/Paste does not scale to human usage.18:22
devanandakrotscheck: I'd have to copy/paste the glance image ref too18:22
krotscheckNot if glance provided the list.18:22
krotscheckWhich is what searchlight is all about.18:22
devanandaah. you'd present a drop-down choice, or something18:22
krotscheckOr some kind of an autocomplete, yes.18:22
devanandayah. that'd be cool too18:22
krotscheckSomething humans can use.18:22
* devananda can neither confirm nor deny whether he is a human18:23
krotscheckMind you, I _love_ that this means that I can start working on glance & searchlight UI's :)18:23
devanandahah18:23
krotscheckBut, well, extra work.18:23
*** jrist has joined #openstack-ironic18:23
devanandaso - that's cool - but also, my question wasn't when can we package it, but when do you want to move it to gerrit18:23
krotscheckPoint18:23
*** achanda has quit IRC18:23
krotscheckI want to move it to gerrit when a reasonable set of MVP features have been implemented.18:24
devanandathat sounds reasonable18:24
devanandaalso, can you do CI testing (of some sort) on it?18:24
krotscheckAnd if copy/paste image URI's are a thing, then hey, v1.18:24
krotscheckThat's a more complicated question.18:25
devanandamy bar for other projects has been: move to gerrit when a) it can be used by others, b) there is CI, c) you want others to start submitting patches18:25
krotscheckI can write tests that run headlessly and mock the ironic API, but that will bitrot on us.18:26
devanandanoting that some projects dont have CI at all ... the bar is a bit flexible ... but I'd like to have some degree of confidence that gerrit isn't going to land craziness18:26
devanandahm. functional test?18:26
krotscheckThe only way to avoid bitrot is to gate the builds on each other, which requires a real ironic api instance.18:26
NobodyCamjroll: have you put any thought into krotscheck's question about decoupling global requirements on the feature-based-releases spec?18:26
devanandafunctional doesn't require co-gating18:27
*** krtaylor has quit IRC18:27
*** edcragg has quit IRC18:27
devanandaif ironic changes its API in a way that breaks your JS UI -- omg we just broke all our users. pls punch us.18:28
krotscheckCan I punch you anyway?18:28
jrollNobodyCam: not too much, I want to talk about it with others18:28
jrollNobodyCam: initial thought for now is "deal with it"18:28
NobodyCamack and ++18:29
devanandakrotscheck: maybe18:29
krotscheckSo, how feasible is it to A) stand up a functioning ironic API that responsds to requests in the gate, and B) Stand up a _persistent_ ironic API in infra that we can use for UI draft builds?18:29
devanandaeeeeeh18:29
devananda(A) is trivial. use devstack. or bifrost.18:29
krotscheckI know that B) is currently being looked at by the translation team.18:29
devananda(B) is not something I've considered before and seems crazy18:29
devanandaoh?18:29
krotscheckYes, and it's a thing infra's looking at to enable, because that way a translator can see in-context translations.18:30
devananda(B) seems appropriate for a sandbox environment, eg. where it is wiped & recreated once a day or something18:30
krotscheckAlso, it'd be useful for the docs team to run their livedocs things against.18:30
devanandaconfigure it only with the "fake" driver18:31
krotscheckYep.18:31
devanandaonly difficulty will be the lack of any tenancy in ironic18:31
devanandaevery client will see all resources18:31
*** davideagnello has quit IRC18:36
krotscheckdevananda: That should be fine on a dev instance.18:38
*** gridinv_ has joined #openstack-ironic18:45
devanandaseems reasonable then18:45
devanandai'm about to send a "summit recap" email to the list - anyone want to check it for completeness // make sure there's not some giant elephant that I am leaving out?18:46
jrolldevananda: I'm about to lunch but can look when I'm back :)18:46
*** gridinv_ has quit IRC18:46
devanandajroll: k k. no rush18:46
cineramathat's a great idea, i saw some other ones<blockquote class="twitter-tweet" lang="en"><p lang="und" dir="ltr"><a href="http://t.co/xq6Ham3Jbe">pic.twitter.com/xq6Ham3Jbe</a></p>&mdash; Parcorama (@parcorama) <a href="https://twitter.com/parcorama/status/603990972444323842">May 28, 2015</a></blockquote>18:48
cinerama<script async src="//platform.twitter.com/widgets.js" charset="utf-8"></script> go out and meant to as18:48
cineramawhoops18:48
cineramaneed to get these cats off the keyboard18:49
devananda:p18:49
*** romcheg has joined #openstack-ironic18:56
devanandakrotscheck: given the mountain of work before you related to the _general_ task of making browser-based clients a thing in openstack18:56
*** Sukhdev has joined #openstack-ironic18:56
devanandakrotscheck: do you want a call-out in my ironic summit sumary email? I'm presently assuming "yes" until you say otherwise18:56
krotscheckdevananda: I have req's!18:57
krotscheckAlso, yes.18:57
devanandakrotscheck: !!! \o/18:57
devanandakrotscheck: fwiw, I do not18:57
krotschecksadpanda18:57
*** UForgotten has joined #openstack-ironic18:58
UForgottenHas anyone used ironic to deploy things other than openstack nodes?18:59
devanandaUForgotten: uh.... that's mostly what we use it for19:00
*** pradipta has quit IRC19:03
TheJuliajlvillal: My guess at the moment is I set the threshold way too low for how long the job is running, and devananda is right, the job is timing out :(19:09
TheJuliadevananda: for the record, I'm 95.5% sure your a human19:10
devanandaTheJulia: cache more, do less ?19:10
devanandaTheJulia: that's a high degree of accuracy in your lack of certainty :)19:10
TheJuliadevananda: yes19:10
TheJuliaon both items19:10
cinerama"Fascinating."19:15
*** alexpilotti has quit IRC19:23
TheJuliacinerama: Your on 14.04LTS right?19:31
cineramaTheJulia: yep19:31
cineramaTheJulia: so the thing that breaks it is including "-I"19:31
cineramaTheJulia: if i run it with --no-cache-dir i'm good19:31
cineramaTheJulia: it complains about ownership of ~/.cache/pip/http and exits 139 without that19:32
TheJuliacinerama: I think we could loose -I since that is actually a timesink19:32
cineramaTheJulia: i have one other experiment cooked up19:32
TheJuliacinerama: bug 145980119:32
openstackbug 1459801 in bifrost "Running bifrost on Ubuntu 14.04 LTS results in segmentation fault" [Undecided,New] https://launchpad.net/bugs/145980119:32
*** radek__ has quit IRC19:34
*** natorious is now known as zz_natorious19:35
*** radek__ has joined #openstack-ironic19:35
cineramaTheJulia: nice one. i will report my findings19:35
jrolldevananda: so, if you want me to review your summit recap, I'll need a link or something :P19:36
*** mriedem has joined #openstack-ironic19:37
mriedemthere is no ironic-stable-maint group in gerrit?19:37
jrollUForgotten: our customers use it to deploy lots of things that are probably not openstack :)19:37
jrollmriedem: I'm 99% sure there is19:37
jrollperhaps a different name? not sure. devananda would know.19:38
mriedemhttps://review.openstack.org/#/admin/groups/?filter=stable19:38
mriedemnot seeing it19:38
mriedemi wanted to rebase this https://review.openstack.org/#/c/175540/19:39
mriedemi don't get a rebase button though apparently even though i'm stable maint core, but ironic doesn't have tha19:39
mriedem*that19:39
jrollmriedem: ironic-milestone or ironic-release maybe? https://review.openstack.org/#/admin/groups/?filter=ironic19:39
UForgottenjroll: am trying to PoC it as a possible replacement for a baremetal provisioning system that we use currently. trying to get past the learning curve of openstack.19:40
jrollmilestone sounds right19:40
NobodyCammriedem: https://review.openstack.org/#/admin/groups/167,members19:40
NobodyCamI think thats is19:40
mriedemhmmm19:40
mriedemthat should include stable-maint-core like the other projects, e.g. https://review.openstack.org/#/admin/groups/540,members19:40
TheJuliaUForgotten: Well, you've come to the right place then :)19:40
mriedemadam_g: are you around?19:40
adam_gmriedem, hi19:41
mriedemadam_g: hey, is this ironic's stable maint core group? https://review.openstack.org/#/admin/groups/167,members19:41
NobodyCamhey adam_g :) \o19:41
cineramaTheJulia: how come we don't use the ansible pip module?19:42
adam_gmriedem, uhm, looks like its the one we setup a while ago but not sure why its named ironic-milestone?19:42
UForgottenTheJulia: I hope so.  Is anyone using ironic without openstack and have a nice gui to configure it?19:42
*** krtaylor has joined #openstack-ironic19:42
TheJuliacinerama: There was some reason originally, possibly because I was changing stuff really often.  I'm fine with changing that or -I.19:43
mriedemadam_g: it also doesn't include stable-maint-core19:43
cineramaTheJulia: yeah i thought there might be some reason like that it's broken etc19:43
adam_gmriedem, okay, it does now. not sure how to go about renaming that, or why it was in the first place19:44
mriedemadam_g: i think you can rename through a project-config or system-config patch19:44
mriedemi'll take a look19:44
*** spandhe has joined #openstack-ironic19:45
TheJuliaUForgotten: No nice gui to configure it yet, but if your comfortable with a command line its not that hard and we're good about answering questions.19:46
*** wshao has joined #openstack-ironic19:46
wshaohello, question on standalone use of ironic, any guide on setting it up?19:47
devanandaUForgotten: overview I gave last week, along with a demo of using ironic outside of openstack around 21:00 - https://www.youtube.com/watch?v=C9-o1gLHHWo19:47
devanandawshao: ^ :)19:48
devanandagithub.com/openstack/bifrost19:48
devanandaTheJulia: you're going to need an autoresponder :p19:48
TheJuliadevananda: Yeah... but the autoresponder would lack the human touch :)19:48
devanandajroll: oh right. you can't just read my mind?19:49
devanandajroll: you have email19:49
jrolldevananda: cool, but now you've distracted me with the release guide thing :P19:51
jroller, release model19:51
*** eghobo has quit IRC19:51
*** penick has joined #openstack-ironic19:52
TheJuliadevananda: people reading your mind would be weird, I would be afraid we would begin saying "resistance is futile" ;)19:52
*** zz_natorious is now known as natorious19:53
devanandaTheJulia: it is :)19:53
*** mriedem has left #openstack-ironic19:53
*** natorious is now known as zz_natorious19:54
*** romainh has joined #openstack-ironic19:58
*** romainh has left #openstack-ironic19:59
*** eghobo has joined #openstack-ironic20:00
jrolldevananda: looks fine, I had to read quickly though20:02
* NobodyCam tries to figure out how to post "read at your own risk" signs around his mind20:02
devanandajroll: cool. mostly looking for "did I forget to mentoin $FOO"20:04
jrolldevananda: yeah, not that I noticed... didn't reference against anything else though20:04
jrollyou certainly covered the stuff I care about :)20:05
devanandagood enough20:05
TheJuliaNobodyCam: Once you figure out the protocol for posting those, I'll need to post a small forest around mine20:05
*** davideagnello has joined #openstack-ironic20:10
*** achanda has joined #openstack-ironic20:13
*** ijw has joined #openstack-ironic20:15
jlvillaljroll: Woot!  Got one +2 :)  https://review.openstack.org/#/c/130877/20:17
*** ijw_ has quit IRC20:17
jrolljlvillal: my production environment is the only validation I need :)20:17
jlvillal:)20:18
*** achanda has quit IRC20:20
dtantsur|afkdevananda, once you have some time, could you re-ack https://review.openstack.org/#/c/185442 please?20:20
* dtantsur|afk disappears again :)20:20
* jlvillal wonders what time it is where dtantsur|afk is???20:20
dtantsur|afk22:2120:21
dtantsur|afkgoing to have some pizza with cider20:21
jlvillaldtantsur|afk: Come back tomorrow ;)20:21
harlowjadevananda u forgot https://review.openstack.org/#/c/141961/ in your state-machine section :-P20:21
harlowja*of your recent ML (summit highlights) email20:22
harlowjaha20:22
*** eghobo_ has joined #openstack-ironic20:24
*** wshao has quit IRC20:26
*** eghobo has quit IRC20:27
openstackgerritMerged openstack/ironic: Get admin auth token for Glance client in image_service  https://review.openstack.org/18612920:42
*** Sukhdev has quit IRC20:56
devanandajroll: re the SMM attacks, coreboot to the rescue?20:56
UForgottenis it ok to run a newer ironic package with a JUNO release of openstack?20:58
UForgottenor do ubuntu 12 / precise deb packages exist somewhere?21:00
rlooUForgotten: no, I think it'll break the nova driver in the juno release. Or will it. sigh. I forgot.21:01
devanandanewer ironic should be backwards compatibile with juno nova. HOWEVER it will not expose all features21:02
devananda*all the features added in kilo21:02
rloothx devananda! (And I just looked at our release notes and it mentions that.)21:03
UForgottenok, so where can I look for a precise-compatible deb package ?21:03
devanandaI doubt that an official one exists. zigo?21:04
*** ijw_ has joined #openstack-ironic21:06
*** dguerri is now known as dguerri`away21:06
zigodevananda: I don't have some.21:06
zigoUForgotten: Why using such an outdated distro?21:06
zigoIt's going to be soon EOL.21:06
jrolldevananda: I guess/hope so, however I'm not smart enough to know21:06
UForgottenzigo, its complicated :)21:06
UForgottenI guess the rub is that we're not really sure what components we'd need from openstack in order to run ironic the way we want to use it, so my group was hesitant to run it standalone21:08
*** dguerri`away is now known as dguerri21:08
*** ijw has quit IRC21:09
*** dguerri is now known as dguerri`away21:09
devanandaUForgotten: that seems orthogonal to "is there a Precise package for ironic"21:10
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Report/Configure LAN alert destinations  https://review.openstack.org/18511221:12
krotscheckhttps://etherpad.openstack.org/p/ironic-webclient-use-cases21:12
krotscheck^^ where I'm coming up with a list of things to do.21:12
*** jgrimm is now known as zz_jgrimm21:13
UForgottendevananda: not following21:14
devanandaUForgotten: maybe start by describing your use case :)21:14
openstackgerritChristopher Dearborn proposed openstack/ironic: Add additional required RPMs to dev instructions  https://review.openstack.org/18599321:16
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Report/Configure LAN alert destinations  https://review.openstack.org/18511221:16
*** dguerri`away is now known as dguerri21:17
*** dguerri is now known as dguerri`away21:18
*** alexpilotti has joined #openstack-ironic21:21
NobodyCambrb21:22
*** trown is now known as trown|outttypeww21:22
devanandakrotscheck: pausing a moment to try fixing my monitor (again)21:26
krotscheckdevananda: No worries21:26
krotscheckOther people should comment!21:26
*** Ng has quit IRC21:28
devanandai am sad now. this hdmi -> dp connector doesn't work21:30
*** mtanino_ has quit IRC21:31
krotscheckDo we know of anyone who's currently deploying Ironic, who'd be willing to be a guineapig for UX things?21:31
mrdaMorning!21:31
*** jamielennox|away is now known as jamielennox21:34
*** ijw_ has quit IRC21:35
*** ijw has joined #openstack-ironic21:36
*** openstackgerrit has quit IRC21:36
*** openstackgerrit has joined #openstack-ironic21:37
krotscheckmrda: Is that you just saying good morning, or responding to my question?21:37
* TheJulia suspects it is just mrda saying good morning :)21:39
*** Ng has joined #openstack-ironic21:39
mrdakrotscheck: :)21:39
mrdakrotscheck: Just cooincidence21:39
* krotscheck plays the sad trombone.21:40
mrdaSorry, I'm pre-coffee and hence not much help until that is rectified21:40
jrolldevananda: this release plan has great success on the list so far21:41
* jroll is excite21:41
jrollkrotscheck: we'd probably be willing to poke at it, I would at least21:41
jrollkrotscheck: when the CORS thing lands, can I set it for multiple domains or just one? I'd personally probably set to localhost and play around21:42
jrollkrotscheck: that said, read only mode is a hard requirement for me to run it in prod21:43
devanandai now thik i have a bad connector AND a broken desktop21:44
jroll:|21:45
jrolltoo bad you don't know any companies that might have extra hardware21:45
devanandakrotscheck: i think the 'pad lost my colors21:47
devanandajroll: it's not a desktop I use often enough for this to have an impact. also, i dont know why i have any computers that i can't fit in a carry on21:48
devanandabut it is frustrating to turn something on and have it not. turn. on.21:48
jrollindeed.21:48
jrolllike my phone all weekend.21:48
devananda:-/21:49
jlvillaljroll: It gave you more time to enjoy Portland! :)21:49
jrolljlvillal: it did indeed, I didn't mind it21:49
jrollactually went out without my phone for once21:49
krotscheckjroll: Multiple. Everything in [cors] is the default, anything added in additional [cors.FOO] blocks is an override21:49
jrollkrotscheck: sweet. I would play.21:49
krotscheckjroll: Read only mode?21:49
jlvillaljroll: As a note, Voodoo Doughnuts is over-hyped.  But I am telling you that after you were here :)21:49
krotscheckjlvillal: Yes. Go to Blue Star instead.21:50
jlvillalkrotscheck: Yep! :)21:50
jrolljlvillal: still delicious. I didn't go because 100-person line, though.21:50
* jlvillal Thinks Krispy Kreme has better tasting donuts than Voodoo. 21:50
jrollkrotscheck: yes, read only21:50
jrollthough now that I think about it... only allow GET in cors config21:51
jrolland that's done21:51
jlvillaljroll: krotscheck is correct.  Blue Star has good donuts :)21:51
jrollbut it would be cool if we failed gracefully on that :)21:51
krotscheckjroll: Well, the client is basically stateless.21:51
krotscheckjroll: So Ironic would have to tell the client whether it's allowed to do anything.21:52
jrollkrotscheck: right, but I don't want folks modifying data or hitting deploy or whatever21:52
jrolland we don't have good policy support21:52
*** dguerri`away is now known as dguerri21:53
krotscheckI understand your concerns, however determining whether to run in readonly or not readonly mode isn't really the client's job. It must defer that to something else, which could be ironic.21:53
krotscheckafk for a sec, sick of standing21:53
*** alexpilotti has quit IRC21:55
*** dguerri is now known as dguerri`away21:56
krotscheckback21:56
*** ijw_ has joined #openstack-ironic21:56
jrollkrotscheck: sure, but then we'll need to do work in ironic before I can run it21:59
jrollI could still test locally though21:59
*** ijw has quit IRC22:00
krotscheckjroll: I'm a little confused on how what you're saying is more or less risky than, say, someone getting their hands on ironic-pythonclient and accidentally futzing with your metal that way.22:00
krotscheckIt's not the UI's fault that the API exposes those things. It's not the CLI's fault either.22:01
jrollkrotscheck: accidentally clicking a thing is way easier. I'm more concerned with accidents than malicious users22:01
jrollit's just a rule we've made, that our production dashboard is read-only22:01
jrollto avoid accidents/stupidity/etc22:01
jrollthat hasn't prevented all accidents, but I'm sure it's prevented some22:02
krotscheckSounds to me like you need to run chaosmonkey22:02
jrolldeleting a customer instance would be real real bad.22:02
jrolljust sayin.22:02
jrollbut like I said, I'm happy to play, just not run it in production until we have some proper policy type stuff22:03
krotscheckAight22:03
devanandajroll: ++ for policy stuff22:05
JayFheh; our customers wouldn't like us running chaosmonkey against their instances22:06
JayFI don't think even I could sell that as a managed services featuer22:06
MadasiI'd love to run it against the control plane though22:07
jroll^^22:07
jrollone day.22:07
* TheJulia has fond memories of letting a chaosmonkey just knock things over22:07
Madasior make a chaos monkey gate for gerrit22:07
krotscheckAlright, any last requests for the webclient MVP?22:09
krotscheckhttps://etherpad.openstack.org/p/ironic-webclient-use-cases22:09
* mrda would like to see a chaos monkey cehck job22:10
krotscheckWho is purple?22:15
jrollnot I22:15
TheJuliaI am22:16
krotscheckGotcha22:16
* TheJulia thanks chrome for forgetting who she is22:16
jrollkrotscheck: this reminds me. I need keystone to make this work for me. :/22:16
krotscheckChassis, according to devananda, isn't really used at this moment. Is it reasonable to put it into the "Here's the project we're introducing to gerrit" piece?22:16
krotscheckjroll: I'm wary of feature creep. Also, of additional dependencies to things that are not ironic.22:17
openstackgerritJulia Kreger proposed openstack/bifrost: Preserve logs in logs/ folder  https://review.openstack.org/18658122:18
krotscheckjroll: My suspicion is that it'll become the Horizon team's job to bring the reusable components from this webclient into Horizon, and use that implementation of keystone.22:18
krotscheckRight now, I'm restricting the project to "Only things that ironic does."22:19
krotscheckNext iteration we can start talking to "Things that ironic needs to be usable"22:20
openstackgerritJulia Kreger proposed openstack/bifrost: Make Ansible modules 1.9 compatible  https://review.openstack.org/18658222:20
krotscheckTheJulia: Does anyone actually use chassis?22:20
TheJuliakrotscheck: afaik no, so I'll rip that out22:20
TheJuliaand gone22:21
devanandakeystone is expressly not something that ironic requires (to be used when standalone)22:21
devanandaat some point, I expect someone may deploy ironic+neutron (and nothing else)22:21
devanandaor ironic+swift22:21
devanandabut to krotscheck's point, yes, I imagine any UI integration becomes a question for horizon22:22
*** errr has joined #openstack-ironic22:28
krotscheckI'm guessing the "Ironic Appliance" is going to be a raspberry Pi running ironic, glance, searchlight, and some kind of a generic auth adapter.22:28
jrollkrotscheck: sure, I get it, just letting you know :)22:29
* devananda dives into expense reports22:32
krotscheckOooh, what if that ironic applicane contained a bunch of containers instead.22:32
krotscheckContainers that contained, say, different parts of openstack!22:32
krotscheckAnd then we'd plug it in!22:32
krotscheckAnd it would install OpenStack!22:32
krotscheckThis project needs a name!22:32
krotscheckLet's call it....22:33
krotscheckTriple-I!22:33
TheJuliakrotscheck: Have you had too much coffee?22:33
JayF-.- /o\22:33
krotscheckTheJulia: Define.... "too much"22:34
TheJuliakrotscheck: It being the late afternoon and one has only had coffee to drink during that day. ?22:37
*** absubram has quit IRC22:37
jrollTheJulia: wait, are you saying that would be a problem?22:39
TheJuliait can be :)22:40
devanandakrotscheck: Ironic In Ironic ? ...22:42
devanandakrotscheck: I second TheJulia 's questoin - perhaps you've had too much coffee :)22:42
jrollhmm22:42
*** barra204 has joined #openstack-ironic22:43
krotscheckClearly, I haven't had enough.22:45
* krotscheck goes to fix that.22:45
TheJuliaHmm22:46
* jroll chugs his coffee22:46
*** chlong has quit IRC22:52
*** eghobo_ has quit IRC22:57
*** rloo has quit IRC22:59
*** barra204 has quit IRC23:01
*** Haomeng has joined #openstack-ironic23:03
*** krtaylor has quit IRC23:05
*** Haomeng|2 has quit IRC23:06
*** Sukhdev has joined #openstack-ironic23:07
*** romcheg has quit IRC23:08
*** barra204 has joined #openstack-ironic23:16
*** hemna is now known as hemnafk23:18
jlvillalShould something like this have a spec associated with it?  https://review.openstack.org/#/c/171672/23:30
*** dprince has quit IRC23:37
*** eghobo has joined #openstack-ironic23:37
*** eghobo has quit IRC23:45
*** eghobo has joined #openstack-ironic23:46
NobodyCamI'm actually ok without a spec for that.23:47
*** krtaylor has joined #openstack-ironic23:51
NobodyCamhumm need to think on the set / unset commands23:54
NobodyCammay get confusing setting node and port [chassis too] properties23:55
jrollI think there was an openstack-wide spec on it or something23:56
jrollI wouldn't mind one to hash out what the "UI" looks like, but fine without one too23:56
*** radek__ has quit IRC23:59

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