*** naohirot has joined #openstack-ironic | 00:02 | |
*** Sukhdev has quit IRC | 00:04 | |
*** Sukhdev has joined #openstack-ironic | 00:08 | |
Sukhdev | devananda, jroll: Ping | 00:09 |
---|---|---|
jroll | hey Sukhdev | 00:09 |
Sukhdev | devananda, jroll: I just sent you an email - please have a look | 00:09 |
*** mtanino has quit IRC | 00:09 | |
jroll | Sukhdev: will do, ty | 00:10 |
openstackgerrit | Haomeng,Wang proposed openstack/bifrost: supports redhat yum installation https://review.openstack.org/185294 | 00:26 |
*** Sukhdev has quit IRC | 00:28 | |
*** alexpilotti has joined #openstack-ironic | 00:34 | |
*** eghobo has quit IRC | 00:59 | |
*** dprince has quit IRC | 00:59 | |
*** blakec has joined #openstack-ironic | 01:05 | |
*** igordcard_ has quit IRC | 01:12 | |
*** igordcard_ has joined #openstack-ironic | 01:12 | |
*** romcheg has quit IRC | 01:20 | |
*** romcheg has joined #openstack-ironic | 01:21 | |
*** blakec has quit IRC | 01:24 | |
*** romcheg has quit IRC | 01:35 | |
*** romcheg has joined #openstack-ironic | 01:36 | |
*** alexpilotti has quit IRC | 01:42 | |
*** kan_ has joined #openstack-ironic | 01:46 | |
*** blakec has joined #openstack-ironic | 01:59 | |
*** blakec1 has joined #openstack-ironic | 02:14 | |
*** spandhe has quit IRC | 02:15 | |
*** blakec has quit IRC | 02:17 | |
*** blakec1 has quit IRC | 02:34 | |
openstackgerrit | baiyuan proposed openstack/ironic: Add documentation for getting a node's console https://review.openstack.org/176642 | 02:35 |
*** ijw has joined #openstack-ironic | 02:36 | |
*** ramineni has joined #openstack-ironic | 02:37 | |
*** MattMan has joined #openstack-ironic | 02:38 | |
*** ijw_ has quit IRC | 02:39 | |
*** Sukhdev has joined #openstack-ironic | 02:46 | |
*** rwsu has quit IRC | 02:46 | |
*** natorious is now known as zz_natorious | 02:46 | |
*** takadayuiko has joined #openstack-ironic | 03:07 | |
*** achanda has joined #openstack-ironic | 03:22 | |
*** ijw_ has joined #openstack-ironic | 03:37 | |
*** ijw_ has quit IRC | 03:39 | |
*** ijw has quit IRC | 03:40 | |
*** eghobo has joined #openstack-ironic | 03:56 | |
*** achanda has quit IRC | 04:01 | |
*** spandhe has joined #openstack-ironic | 04:01 | |
*** spandhe_ has joined #openstack-ironic | 04:05 | |
*** jaison has joined #openstack-ironic | 04:05 | |
*** spandhe has quit IRC | 04:06 | |
*** spandhe_ is now known as spandhe | 04:06 | |
*** Haomeng|2 has joined #openstack-ironic | 04:45 | |
*** Sukhdev has quit IRC | 04:45 | |
*** Haomeng has quit IRC | 04:48 | |
*** openstackgerrit has quit IRC | 04:50 | |
*** openstackgerrit has joined #openstack-ironic | 04:51 | |
*** romcheg1 has joined #openstack-ironic | 04:57 | |
*** romcheg has quit IRC | 04:58 | |
*** pradipta has joined #openstack-ironic | 05:05 | |
*** jaison has quit IRC | 05:18 | |
*** dkehn__ has joined #openstack-ironic | 05:24 | |
*** dkehn has quit IRC | 05:27 | |
*** achanda has joined #openstack-ironic | 05:38 | |
*** harlowja_ has quit IRC | 05:40 | |
*** spandhe has quit IRC | 05:58 | |
*** rameshg87 has joined #openstack-ironic | 05:59 | |
*** gridinv_ has quit IRC | 06:05 | |
openstackgerrit | sandhya proposed openstack/ironic-specs: Ironic Chassis Discovery https://review.openstack.org/186285 | 06:10 |
*** yuanying has quit IRC | 06:11 | |
*** radek__ has joined #openstack-ironic | 06:15 | |
rameshg87 | good morning ironic | 06:16 |
takadayuiko | rameshg87: good morning :) | 06:17 |
rameshg87 | takadayuiko: o/ | 06:17 |
zhenguo | rameshg87: morning :) | 06:21 |
rameshg87 | zhenguo: o/ | 06:25 |
*** ukalifon1 has joined #openstack-ironic | 06:25 | |
openstackgerrit | sandhya proposed openstack/ironic-specs: Ironic Chassis Discovery https://review.openstack.org/186285 | 06:28 |
openstackgerrit | sandhya proposed openstack/ironic-specs: Ironic Chassis Discovery https://review.openstack.org/186285 | 06:29 |
*** saripurigopi has joined #openstack-ironic | 06:29 | |
*** yuanying has joined #openstack-ironic | 06:36 | |
*** dlpartain has joined #openstack-ironic | 06:39 | |
*** dlpartain has left #openstack-ironic | 06:39 | |
openstackgerrit | sandhya proposed openstack/ironic-specs: Ironic Chassis Discovery https://review.openstack.org/186285 | 06:40 |
*** pas-ha has joined #openstack-ironic | 06:51 | |
saripurigopi | I'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 allowed | 06:54 |
*** yuanying has quit IRC | 06:59 | |
*** yuanying_ has joined #openstack-ironic | 07:01 | |
*** yog__ has joined #openstack-ironic | 07:04 | |
*** Nisha has joined #openstack-ironic | 07:11 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 07:22 |
*** ifarkas has joined #openstack-ironic | 07:29 | |
Haomeng|2 | saripurigopi: can you show your code, looks like have to inhert from baseexception in py34, else raise type error | 07:32 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Add generic RAID configuration spec for liberty https://review.openstack.org/173214 | 07:46 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Add inband RAID configuration spec for liberty https://review.openstack.org/173218 | 07:46 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Update iLO documentation for UEFI secure boot https://review.openstack.org/178582 | 07:48 |
naohirot | rameshg87: hello | 07:51 |
rameshg87 | naohirot: hi | 07:52 |
naohirot | rameshg87: Hi, have recovered from the summit? | 07:53 |
naohirot | rameshg87: have you recovered from the summit? | 07:53 |
rameshg87 | naohirot: almost :) | 07:53 |
naohirot | rameshg87: good | 07:53 |
rameshg87 | naohirot: just joined work today | 07:53 |
naohirot | rameshg87: I have a basic question regarding power driver. | 07:54 |
rameshg87 | sure | 07:54 |
naohirot | rameshg87: currently Ironic reboot is hard reboot. | 07:54 |
*** jistr has joined #openstack-ironic | 07:54 | |
naohirot | rameshg87: If customer typed "nova reboot", he/she expect it is soft reboot. | 07:54 |
naohirot | rameshg87: Have you discussed this inconsistency in Ironic ever? | 07:55 |
rameshg87 | naohirot: ah no | 07:56 |
rameshg87 | naohirot: how does it work for vms ? | 07:56 |
rameshg87 | naohirot: does it issue a soft reboot ? | 07:56 |
rameshg87 | naohirot: i think the main question is what happens if the soft reboot gets stuck (instance never reboots) | 07:56 |
naohirot | rameshg87: In nova cli, there is a option --hard, "nova reboot --hard" means hard reboot. | 07:57 |
rameshg87 | oh okay | 07:57 |
naohirot | rameshg87: and in Nova cli, no option means soft reboot. | 07:57 |
rameshg87 | naohirot: i always wished we do a soft power off during a reboot, wait for bare metal to be powered off polling the state | 07:57 |
rameshg87 | naohirot: wait for x seconds | 07:58 |
rameshg87 | naohirot: hard power off baremetal if it doesn't power off in x seconds | 07:58 |
rameshg87 | naohirot: and then power on the node | 07:58 |
naohirot | naohirot: I got a bug report from iRMC customer regarding this inconsistency. | 07:58 |
rameshg87 | in that way we should be able to reliably detect if instance ever rebooted | 07:58 |
rameshg87 | oh okay | 07:58 |
*** romcheg1 has quit IRC | 07:59 | |
rameshg87 | naohirot: wdyt ? | 07:59 |
naohirot | rameshg87: And iRMC has graceful shutdown scci API, and I'm thinking to implement it in vendor paththrough | 07:59 |
rameshg87 | naohirot: i think it was discussed before | 08:00 |
naohirot | rameshg87: what does "wdyt" stand for? | 08:00 |
rameshg87 | what do you think - wdyt | 08:00 |
rameshg87 | :) | 08:00 |
naohirot | rameshg87: :) | 08:00 |
naohirot | rameshg87: then what was the conclusion at that time? | 08:00 |
rameshg87 | naohirot: yeah, may be try pursuing to make this a standard | 08:01 |
rameshg87 | naohirot: instead of a vendor passthru | 08:01 |
rameshg87 | there is enough interest i guess | 08:01 |
rameshg87 | i am +1 on introducing soft reboot in ironic :) | 08:01 |
naohirot | rameshg87: does ilo have gracefull shutdown API too? | 08:01 |
rameshg87 | yes | 08:01 |
naohirot | rameshg87: How about NMI API? | 08:02 |
rameshg87 | NMI ? | 08:02 |
naohirot | rameshg87: which stands for Non Markable Interrupt, | 08:03 |
rameshg87 | naohirot: no idea | 08:03 |
naohirot | rameshg87: out customer also would like to issue this API, which allow us to take kernel dump. | 08:03 |
rameshg87 | oh okay, got it | 08:04 |
naohirot | rameshg87: in order to investigate kernel problem. | 08:04 |
rameshg87 | i need to check. at least haven't seen relating to nmi | 08:04 |
rameshg87 | i mean for ilo | 08:04 |
rameshg87 | *in ilo | 08:04 |
*** chenglch has joined #openstack-ironic | 08:04 | |
naohirot | rameshg87: When customer typed "nova reboot --hard", | 08:05 |
naohirot | rameshg87: is the option "--hard" passed to ironic API server? | 08:06 |
rameshg87 | naohirot: i think no | 08:06 |
naohirot | rameshg87: if you already knew it. | 08:06 |
rameshg87 | naohirot: it's upto ironic virt driver to pass that option | 08:07 |
naohirot | rameshg87: Okay. | 08:07 |
rameshg87 | i think it doesn't today | 08:07 |
*** eghobo has quit IRC | 08:08 | |
naohirot | rameshg87: I'd like to fix this bug for irmc customer first through vendor API and then like to standardize it. | 08:09 |
naohirot | rameshg87: do you see any problem regarding this approach, I agree to standardize it, but it takes time. | 08:10 |
rameshg87 | naohirot: sure. your wish :) | 08:10 |
rameshg87 | naohirot: i guess vendor passthrus dont require spec approval | 08:10 |
rameshg87 | naohirot: so you push code directly | 08:10 |
naohirot | rameshg87: Okay, thanks. | 08:11 |
rameshg87 | naohirot: but disadvantage of vendor passthru is ironic api can be invoked only with admin credentials | 08:11 |
rameshg87 | naohirot: i.e tenants can't still invoke a soft reboot | 08:11 |
rameshg87 | naohirot: tenants don't even see ironic api | 08:11 |
rameshg87 | naohirot: is that acceptable for you ? | 08:12 |
naohirot | rameshg87: really, I have checked policy.json, | 08:12 |
naohirot | rameshg87: I thought all apis regardless vendor or not need admin credentials. | 08:13 |
rameshg87 | naohirot: yes, that is a problem right ? | 08:13 |
dtantsur | Morning folks | 08:13 |
rameshg87 | naohirot: all users might not have admin rights | 08:13 |
rameshg87 | dtantsur: o/ | 08:13 |
naohirot | rameshg87: Yeah, kind of. | 08:14 |
naohirot | dtantsur: good morning | 08:14 |
rameshg87 | naohirot: so soft power off through api a better option with proper integration with nova :) | 08:14 |
*** edcragg has joined #openstack-ironic | 08:15 | |
naohirot | rameshg87: are you thinking a scenario that tenant admin provides tenant user with power on off privilege? | 08:16 |
rameshg87 | naohirot: not exactly | 08:16 |
rameshg87 | naohirot: currently i believe vendor passthru apis are invokable only with admin rights | 08:17 |
rameshg87 | naohirot: every user might not be able to do this | 08:17 |
rameshg87 | (unless as you mentioned we tweak policy.json) | 08:17 |
naohirot | rameshg87: I see | 08:18 |
rameshg87 | naohirot: need to go away. brb | 08:18 |
*** rameshg87 is now known as rameshg87-away | 08:18 | |
naohirot | rameshg87: Okay, thanks | 08:18 |
*** romcheg has joined #openstack-ironic | 08:24 | |
saripurigopi | Haomeng|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.py | 08:28 |
*** dguerri`away is now known as dguerri | 08:30 | |
*** leopoldj has joined #openstack-ironic | 08:31 | |
*** Haomeng has joined #openstack-ironic | 08:31 | |
*** Haomeng|2 has quit IRC | 08:34 | |
*** rameshg87-away is now known as rameshg87 | 08:38 | |
*** ndipanov has joined #openstack-ironic | 08:40 | |
*** lucasagomes has joined #openstack-ironic | 08:41 | |
openstackgerrit | chenglch proposed openstack/ironic-specs: Add petitboot iscsi deploy driver support https://review.openstack.org/177641 | 08:46 |
*** derekh has joined #openstack-ironic | 09:00 | |
*** itamarl has joined #openstack-ironic | 09:03 | |
*** e0ne has joined #openstack-ironic | 09:22 | |
*** e0ne is now known as e0ne_ | 09:30 | |
*** dtantsur is now known as dtantsur|brb | 09:34 | |
*** e0ne_ has quit IRC | 09:35 | |
*** itamarl has quit IRC | 09:37 | |
sambetts | Morning all o/ | 09:37 |
takadayuiko | sambetts: o/ | 09:45 |
lucasagomes | takadayuiko, sambetts morning | 09:50 |
takadayuiko | lucasagomes: hi :D | 09:50 |
lucasagomes | hello there | 09:51 |
*** naohirot has quit IRC | 09:55 | |
*** e0ne has joined #openstack-ironic | 09:57 | |
Haomeng | saripurigopi: 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 |
rameshg87 | lucasagomes: hi there | 09:59 |
lucasagomes | rameshg87, yo, hi | 09:59 |
rameshg87 | lucasagomes: i will try to write up a spec for wol power driver | 09:59 |
Haomeng | saripurigopi: looks like from py3.x, have to derive from BaseException, then we can try/raise the exception I think | 09:59 |
rameshg87 | lucasagomes: i was wondering if you want to take that up | 09:59 |
saripurigopi | Haomeng: sure. let me try. | 09:59 |
Haomeng | saripurigopi: but odd, why the existing code works with py34 gate | 09:59 |
lucasagomes | rameshg87, oh, that would be helpful | 09:59 |
lucasagomes | rameshg87, did you test the code? | 09:59 |
rameshg87 | lucasagomes: i *should* get the hardware for trying it out in a week | 09:59 |
rameshg87 | :) | 09:59 |
lucasagomes | ack | 09:59 |
lucasagomes | rameshg87, if I get some free time before I will start a spec for that | 10:00 |
Haomeng | saripurigopi: good luck:) | 10:00 |
lucasagomes | should be fairly simple | 10:00 |
saripurigopi | Haomeng: thought the same, I thought something wrong with my patch. | 10:00 |
rameshg87 | lucasagomes: yeah whoever gets to it first :) | 10:00 |
Haomeng | saripurigopi: yes, take carefuly and good luck:) | 10:00 |
rameshg87 | and lucasagomes thanks for the ipa soft reboot patch | 10:00 |
saripurigopi | Haomeng: :-) | 10:01 |
rameshg87 | i will try out and confirm it solves the uefi localboot issue | 10:01 |
lucasagomes | rameshg87, oh no problem... Can you test it? I still need to open a bug for that | 10:01 |
Haomeng | saripurigopi: :) | 10:01 |
lucasagomes | rameshg87, cool | 10:01 |
*** lucasagomes has quit IRC | 10:03 | |
*** lucasagomes has joined #openstack-ironic | 10:03 | |
openstackgerrit | Gopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers https://review.openstack.org/159734 | 10:11 |
saripurigopi | do we need to submit spec to add vendor apis? | 10:12 |
*** pradipta has quit IRC | 10:13 | |
*** thrash|g0ne is now known as thrash | 10:18 | |
*** ifarkas has quit IRC | 10:21 | |
*** Nisha has quit IRC | 10:29 | |
*** cdearborn has joined #openstack-ironic | 10:33 | |
*** dtantsur|brb is now known as dtantsur | 10:34 | |
*** kbyrne has quit IRC | 10:34 | |
lucasagomes | saripurigopi, vendor passthru stuff? nop | 10:35 |
takadayuiko | I'm confused about ironic tests in tempest :( https://etherpad.openstack.org/p/liberty-ironic-functional-testing | 10:36 |
saripurigopi | lucasagomes: okay | 10:36 |
takadayuiko | we have to leave away from tempest, right? | 10:37 |
dtantsur | takadayuiko, functional test != tempest | 10:37 |
dtantsur | takadayuiko, tempest is integration tests | 10:37 |
dtantsur | both will stay as far as I understand | 10:37 |
takadayuiko | dtantsur: oh, hi | 10:37 |
takadayuiko | dtantsur: Heh?? | 10:37 |
dtantsur | and g'evening :) | 10:37 |
dtantsur | adam_g might know better | 10:38 |
dtantsur | adam_g, around? | 10:38 |
* dtantsur digs into tempest | 10:39 | |
*** e0ne is now known as e0ne_ | 10:39 | |
takadayuiko | dtantsur: what is functional test? At first, API tests in tempest will leave away from tempest. Is it correct? | 10:39 |
dtantsur | takadayuiko, functional test is a test of project in isolation from other services. we move it from tempest for sure. | 10:40 |
dtantsur | takadayuiko, integration test is test of common scenarios with other services; I'm not sure it leaves tempest | 10:40 |
*** kbyrne has joined #openstack-ironic | 10:41 | |
dtantsur | takadayuiko, so this https://github.com/openstack/tempest/blob/master/tempest/scenario/test_baremetal_basic_ops.py seems to be an integration test | 10:42 |
dtantsur | takadayuiko, and these https://github.com/openstack/tempest/tree/master/tempest/api/baremetal are functional ones | 10:42 |
zhenguo | hi all, why node cannot be deleted in power on state? | 10:44 |
dtantsur | zhenguo, hi. usually because something may go on on the node | 10:44 |
*** e0ne_ has quit IRC | 10:45 | |
takadayuiko | dtantsur, tests in https://github.com/openstack/tempest/tree/master/tempest/api/baremetal will be moved into ironic/tests/functional, right? | 10:45 |
dtantsur | I think so | 10:45 |
takadayuiko | ok, thanks | 10:46 |
*** early has quit IRC | 10:46 | |
zhenguo | dtantsur: but deleted from ironic seems doesn't affect that, it can still keep on. | 10:46 |
openstackgerrit | Gopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers https://review.openstack.org/159734 | 10:47 |
*** Haomeng|2 has joined #openstack-ironic | 10:48 | |
dtantsur | zhenguo, I think it's rather a precautions in case some Ironic-related processes are going on the node | 10:49 |
dtantsur | maybe we should rather judge by provision state, not sure | 10:49 |
dtantsur | discussable | 10:49 |
*** Haomeng has quit IRC | 10:52 | |
zhenguo | dtantsur: I just find the original patch about this: https://review.openstack.org/#/c/76258/ | 10:52 |
zhenguo | dtantsur: Lucas said the main reason is because otherwise these machines that are not being used might be on for long time | 10:52 |
*** e0ne has joined #openstack-ironic | 10:53 | |
zhenguo | lucasagomes: are you around? | 10:53 |
*** early has joined #openstack-ironic | 10:53 | |
lucasagomes | zhenguo, hi yes | 10:54 |
*** ramineni has quit IRC | 10:54 | |
lucasagomes | oh that's old stuff heh | 10:54 |
lucasagomes | I'm not sure I agree with that anymore | 10:54 |
lucasagomes | before we had an assumption that deployments should start only when the node was powered off | 10:54 |
lucasagomes | but that has changed, now we don't care whether it's power on or off we reboot it anyway | 10:55 |
dtantsur | it's still makes some sense to me to have deleted node in some clean state | 10:55 |
dtantsur | at least including it being powered off | 10:55 |
dtantsur | but I don't insist :) | 10:55 |
lucasagomes | dtantsur, it doesn't actually matter now with our state machine | 10:55 |
lucasagomes | it will pass throught cleaning | 10:55 |
lucasagomes | so it will check consistency there | 10:56 |
dtantsur | lucasagomes, cleaning on deletion? | 10:56 |
dtantsur | I don't think we do it (probably we should) | 10:56 |
lucasagomes | not on the delete, but on unprovisioning and provisioning | 10:56 |
dtantsur | yep and we're talking about deletion :) | 10:56 |
lucasagomes | (which we call delete) | 10:56 |
lucasagomes | but we are talking about delete from the db node-delete | 10:57 |
lucasagomes | yeah names are hard heh | 10:57 |
openstackgerrit | Gopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers https://review.openstack.org/159734 | 10:57 |
dtantsur | exactly, 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 off | 10:58 |
dtantsur | completely not sure here :) | 10:58 |
dtantsur | but maybe we should even allow cleaning before deletion | 10:58 |
dtantsur | so that people wipe the hard driver before throwing it away | 10:58 |
dtantsur | lucasagomes, ^ | 10:59 |
lucasagomes | dtantsur, we do cleaning after unprovisioning... | 10:59 |
dtantsur | right | 10:59 |
lucasagomes | but I think we are talking about deletion from the DB only right? | 10:59 |
dtantsur | yes | 10:59 |
lucasagomes | meaning that we can call node-delete for a node that was just enrolled | 10:59 |
lucasagomes | but never provisioned | 10:59 |
lucasagomes | so we don't need to clean it | 11:00 |
dtantsur | also valid | 11:00 |
lucasagomes | we don't even know if we have credentials for that etc | 11:00 |
dtantsur | ok as we clean after unprovision, it no longer makes sense to me :) | 11:00 |
lucasagomes | exactly | 11:00 |
dtantsur | then we should probably check provision state on node-delete, not power state | 11:00 |
lucasagomes | yeah we do as part of unprovisioning | 11:00 |
lucasagomes | and it's automatically triggered | 11:01 |
lucasagomes | dtantsur, I think we do check provision_state too, we can't delete a node which is active and so on | 11:01 |
lucasagomes | so it should pass throught cleaning to become available again and then you can delete from the db | 11:01 |
dtantsur | right | 11:02 |
lucasagomes | dtantsur, a-ha actually we don't... this need to be fixed | 11:03 |
lucasagomes | we only check if the node is associated with an instance (instance_uuid) | 11:03 |
dtantsur | definitely | 11:03 |
lucasagomes | which kinda means it's provisioned, BUT | 11:03 |
lucasagomes | for the standalone case it's not true | 11:04 |
lucasagomes | we don't need to associate with an instance to provision something | 11:04 |
* lucasagomes puts a patch up | 11:04 | |
*** kan_ has quit IRC | 11:06 | |
*** saripurigopi has quit IRC | 11:08 | |
*** e0ne is now known as e0ne_ | 11:11 | |
*** e0ne_ is now known as e0ne | 11:12 | |
dtantsur | lucasagomes, ++ for patch | 11:13 |
TheJulia | good morning | 11:13 |
openstackgerrit | chenglch proposed openstack/ironic: Ironic should use its own context https://review.openstack.org/184714 | 11:14 |
dtantsur | TheJulia, morning | 11:14 |
*** chenglch has quit IRC | 11:14 | |
*** e0ne is now known as e0ne_ | 11:20 | |
TheJulia | /win 1 | 11:24 |
TheJulia | doh | 11:24 |
*** e0ne_ has quit IRC | 11:25 | |
*** dprince has joined #openstack-ironic | 11:31 | |
*** e0ne has joined #openstack-ironic | 11:34 | |
*** rameshg87 has quit IRC | 11:37 | |
*** openstackgerrit has quit IRC | 11:39 | |
*** openstackgerrit has joined #openstack-ironic | 11:39 | |
*** takadayuiko has quit IRC | 11:44 | |
*** early has quit IRC | 11:46 | |
*** mjturek1 has joined #openstack-ironic | 11:49 | |
*** early has joined #openstack-ironic | 11:49 | |
TheJulia | Haomeng|2: Out of curiosity, Is revision 185294 to address bug 1455334 ? | 11:50 |
openstack | bug 1455334 in bifrost "[biforst] install ironic failed" [Undecided,New] https://launchpad.net/bugs/1455334 | 11:50 |
*** saripurigopi_ has joined #openstack-ironic | 11:57 | |
*** achanda has quit IRC | 11:57 | |
sambetts | TheJulia: Morning :) | 11:58 |
TheJulia | good morning sambetts | 11:59 |
*** saripurigopi has joined #openstack-ironic | 12:01 | |
*** saripurigopi_ has quit IRC | 12:02 | |
*** romainh has joined #openstack-ironic | 12:04 | |
openstackgerrit | Haomeng,Wang proposed openstack/bifrost: supports redhat yum installation https://review.openstack.org/185294 | 12:09 |
*** dlpartain has joined #openstack-ironic | 12:15 | |
*** dlpartain has left #openstack-ironic | 12:15 | |
openstackgerrit | Peter Martini proposed openstack/bifrost: Add CONTRIBUTING.rst to doc/source/dev https://review.openstack.org/186005 | 12:20 |
openstackgerrit | Imre Farkas proposed stackforge/ironic-discoverd: Do not require local_gb by ramdisk https://review.openstack.org/186033 | 12:20 |
Haomeng|2 | TheJulia: 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 installation | 12:21 |
openstack | Launchpad bug 1455334 in bifrost "[biforst] install ironic failed" [Undecided,New] | 12:21 |
openstackgerrit | Peter Martini proposed openstack/bifrost: Redirect error echoes to STDERR https://review.openstack.org/185823 | 12:25 |
*** trown|outttypeww is now known as trown | 12:25 | |
Haomeng|2 | TheJulia: shoud be this fix - https://review.openstack.org/#/c/183190/ | 12:27 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove conditional that prevent node delete based on power state https://review.openstack.org/186361 | 12:28 |
*** ifarkas has joined #openstack-ironic | 12:31 | |
Haomeng|2 | TheJulia: and I have updated launch bug -1455334 | 12:31 |
*** e0ne is now known as e0ne_ | 12:32 | |
*** romainh has quit IRC | 12:33 | |
*** romainh has joined #openstack-ironic | 12:35 | |
Haomeng|2 | TheJulia: 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 functions | 12:36 |
* dtantsur whispers: it's called ironic-inspector now :) | 12:38 | |
TheJulia | Haomeng|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 silly | 12:41 | |
Haomeng|2 | TheJulia: ok, yes, it is called ironic-inspector now:) | 12:41 |
openstackgerrit | Imre Farkas proposed stackforge/ironic-discoverd: Fake missing local_gb for root_device_hint plugin https://review.openstack.org/185896 | 12:41 |
Haomeng|2 | TheJulia: 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 IRC | 12:43 | |
*** saripurigopi has quit IRC | 12:45 | |
Haomeng|2 | TheJulia: that is my dream, a *live* standalone ironic to deploy baremetal directly without any installation and configurations:) | 12:45 |
TheJulia | Haomeng|2: I share this dream :) | 12:46 |
Haomeng|2 | TheJulia: :) | 12:46 |
TheJulia | Haomeng|2: interesting idea on the dib element, perhaps once we have the dream :) | 12:46 |
Haomeng|2 | TheJulia: :) | 12:46 |
*** e0ne has joined #openstack-ironic | 12:47 | |
Haomeng|2 | TheJulia: now ironic-inspector does not support to work with standalone ironic, I wil try to enable | 12:48 |
Haomeng|2 | go to sleep, nice day Ironic:) | 12:49 |
TheJulia | :) | 12:49 |
lucasagomes | Haomeng|2, night | 12:50 |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Rename discoverd -> inspector https://review.openstack.org/185499 | 12:50 |
lucasagomes | dtantsur, w00t | 12:51 |
dtantsur | :) | 12:51 |
dtantsur | TheJulia, I strongly believe lucasagomes will draw a mascot for inspector: bear with gadget hands ;) | 12:52 |
TheJulia | I fully support this idea | 12:53 |
*** romcheg1 has joined #openstack-ironic | 12:53 | |
*** romcheg has quit IRC | 12:54 | |
*** yog__ has quit IRC | 12:55 | |
*** yog__ has joined #openstack-ironic | 12:56 | |
martini | I changed my doc patch to move CONTRIBUTING.rst to doc/source/dev/ and now gate-bifrost failed :-( | 13:00 |
TheJulia | :( | 13:01 |
TheJulia | wow, it was almost done, and we hit the timeout I set on the job | 13:03 |
lucasagomes | :-D | 13:03 |
lucasagomes | talking about drawing mascots etc | 13:03 |
lucasagomes | I got a surprise for all you guys... I want to create an official website for ironic | 13:03 |
lucasagomes | I got the domain already | 13:03 |
lucasagomes | ironic.rocks | 13:03 |
lucasagomes | type that in ur browsers :-) | 13:03 |
TheJulia | lucasagomes: very nice | 13:04 |
dtantsur | oh man that's gorgeous! | 13:05 |
* dtantsur uses major version bump due to rename to drop all the old stuff | 13:07 | |
*** rloo has joined #openstack-ironic | 13:07 | |
lucasagomes | yeah now it just redirects to the wiki page | 13:07 |
lucasagomes | (I got it yesterday so... didn't have time to create anything yet) | 13:09 |
lucasagomes | and whoever wants to help give me a shout cause I know nothing about html css etc... | 13:09 |
lucasagomes | html5* | 13:09 |
dtantsur | oh html... | 13:10 |
dtantsur | I usually start with taking Bootstrap :) | 13:10 |
openstackgerrit | Syed Ismail Faizan Barmawer proposed openstack/ironic-lib: Merge lastet disk partitioning code from ironic https://review.openstack.org/162162 | 13:11 |
lucasagomes | dtantsur, you mean this thing http://getbootstrap.com ? | 13:13 |
dtantsur | yep | 13:13 |
dtantsur | lucasagomes, just take one if the example and adapt | 13:13 |
lucasagomes | dtantsur, nice, thanks! I will take a look after work | 13:13 |
lucasagomes | yeah sounds good | 13:14 |
*** dlpartain has joined #openstack-ironic | 13:14 | |
lucasagomes | looks neat, works with multiple devices (resizing and stuff) | 13:14 |
*** dlpartain has left #openstack-ironic | 13:14 | |
martini | TheJulia: so, what should I do now? | 13:16 |
TheJulia | martini: recheck the job | 13:18 |
TheJulia | I suspect I'll need to bump the timeout a little | 13:19 |
martini | What's the proper way to do that? (Should I wait a bit then?) | 13:19 |
TheJulia | Just 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 |
cdearborn | Morning Ironic! | 13:25 |
cdearborn | I found the issue that was causing the tons of object not iterable errors when running the py34 unit tests | 13:25 |
cdearborn | It turns out there is a bug in mock in py34 on at least Fedora 21 | 13:26 |
lucasagomes | :-( | 13:27 |
* lucasagomes is still stuck on F20 | 13:27 | |
rloo | morning and hi everyone. ironic.rocks :-) | 13:28 |
lucasagomes | rloo, morning :-) | 13:28 |
cdearborn | where code like x.side_effect = xxx.SomeException() will cause the not iterable error | 13:29 |
*** jamielennox is now known as jamielennox|away | 13:29 | |
cdearborn | I hacked a fix into my local copy of mock.py & can run all unit tests with no problems | 13:29 |
lucasagomes | cdearborn, is it something that was fixed on the last versions of mock? | 13:29 |
cdearborn | hmmmm - not sure - I'll poke around | 13:31 |
*** ChuckC has quit IRC | 13:32 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove bogus conditional from node_update https://review.openstack.org/186386 | 13:33 |
lucasagomes | cdearborn, ack, otherwise we may have some problems once other distros starts packing newer versions of this lib | 13:33 |
cdearborn | yeah - 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 exception | 13:34 |
lucasagomes | right | 13:34 |
lucasagomes | yeah maybe side_effect should always be an interable and we have been using it wrongly | 13:34 |
lucasagomes | (that's when bugs turns into features :-) ) | 13:35 |
rloo | lucasagomes: 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 |
openstack | Launchpad bug 1459469 in WSME "wsme is not correctly requiring simplegeneric" [Undecided,New] | 13:35 |
sambetts | the mock docs say you can pass an exception, and even their example is a KeyError | 13:35 |
lucasagomes | rloo, oh yeah. We def should make a release of wsme soon | 13:36 |
cdearborn | yup - saw that | 13:37 |
cdearborn | and you can definitely do that in prior versions of mock | 13:37 |
*** bnemec has quit IRC | 13:37 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Rename discoverd -> inspector https://review.openstack.org/185499 | 13:37 |
dtantsur | the biggest change I've ever done ^^^ | 13:37 |
lucasagomes | rloo, so apparently that problem of wsme not pulling the dependencies correctly has been fixed in master | 13:38 |
TheJulia | dtantsur: impressive :) | 13:38 |
rloo | lucasagomes: right, which is why they are going to do a release on Monday. | 13:38 |
rloo | lucasagomes: so if there is anything else you wanted in that upcoming release and can get it in this week... :-) | 13:39 |
lucasagomes | yeah | 13:39 |
lucasagomes | I've been reviewing and approving stuff in wsme | 13:39 |
rloo | lucasagomes: ahh, good. thx for doing that! | 13:39 |
lucasagomes | he things I had to fix that impacted on the ipxe is already merged upstream | 13:40 |
lucasagomes | the* | 13:40 |
lucasagomes | I hope we will do release more often from now on | 13:40 |
*** achanda has joined #openstack-ironic | 13:40 | |
rloo | lucasagomes: looks like you might just need to ask hellman if you want a release? | 13:40 |
lucasagomes | rloo, yeah, currently I think only hellmann can tag a release for WSME | 13:41 |
lucasagomes | so we just need to ping him when we think it's a good time | 13:41 |
rloo | lucasagomes: here's hoping we don't need any more changes in wsme after this release :-) | 13:42 |
*** bnemec has joined #openstack-ironic | 13:42 | |
lucasagomes | oh we do... for the sake of wsme itself | 13:42 |
lucasagomes | there's still quite few things that needs improving | 13:43 |
lucasagomes | we got few good new contributors since that email thread | 13:43 |
lucasagomes | so it's speeding up things a lot | 13:43 |
lucasagomes | and with all this talk about wsme, someone can take a look at https://review.openstack.org/#/c/177726/ please? | 13:45 |
rloo | lucasagomes: will try to look today. (thinking of reviewing specs but am procrastinating...) | 13:49 |
lucasagomes | heh thanks | 13:49 |
jroll | morning everyone :) | 13:50 |
dtantsur | morning jroll, rloo | 13:51 |
lucasagomes | jroll, yo morning | 13:51 |
cdearborn | did some poking, and the bug is in py3.4.3 and 3.5.0.b1 | 13:54 |
*** mdbooth has quit IRC | 13:54 | |
*** mdbooth has joined #openstack-ironic | 13:55 | |
rloo | hiya jroll, dtantsur | 13:56 |
lucasagomes | cdearborn, right... so we better fix out code to make side_effect always interable? | 13:58 |
lucasagomes | it looks like they want to enforce it for newer versions of mock | 13:58 |
*** romcheg1 has quit IRC | 13:59 | |
*** romcheg has joined #openstack-ironic | 13:59 | |
cdearborn | pulling down latest python source now | 14:00 |
cdearborn | lucasagomes, where did you see that? according to the docs, it's ok to set side_effect to an ex... | 14:01 |
lucasagomes | cdearborn, I didn't that was a question... maybe I misread ur comments | 14:02 |
cdearborn | lucasagomes, ah ok | 14:06 |
rloo | cdearborn: thx for looking into that. | 14:06 |
cdearborn | so it's broken in the latest py source too | 14:06 |
rloo | cdearborn: sounds similar to some bug that jlvillal worked around: http://bugs.python.org/issue17826 | 14:06 |
cdearborn | rloo, np | 14:06 |
rloo | cdearborn: if this helps, this is the (one of anyway) patches that jlvillal added: https://review.openstack.org/#/c/169109/ | 14:08 |
jroll | devananda: can you review https://review.openstack.org/#/c/185171/3 this morning? wanted to update it sometime today | 14:08 |
*** achanda_ has joined #openstack-ironic | 14:13 | |
NobodyCam | good morning Ironic | 14:14 |
sambetts | cdearborn: 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.0 | 14:15 |
sambetts | Morning NobodyCam | 14:16 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Update iLO documentation for UEFI secure boot https://review.openstack.org/178582 | 14:16 |
NobodyCam | morning sambetts | 14:16 |
lucasagomes | NobodyCam, morning | 14:16 |
NobodyCam | morning lucasagomes :) | 14:16 |
dtantsur | NobodyCam, morning | 14:16 |
bcornec | morning everybody here ;-) | 14:16 |
dtantsur | morning you too :) | 14:17 |
sambetts | o/ bcornec | 14:17 |
*** achanda has quit IRC | 14:17 | |
NobodyCam | morning dtantsur bcornec :) | 14:17 |
cdearborn | sambetts, it's easy enough to check for the bug - i can let you know how if you are interested | 14:19 |
cdearborn | so changing the unit tests from this: "xxx.side_effect = exception" to this: "xxx.side_effect = [ exception ]" does hack around the bug | 14:21 |
sambetts | cdearborn: 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 easily | 14:21 |
cdearborn | in 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-ironic | 14:28 | |
*** leopoldj has quit IRC | 14:30 | |
jroll | in case you missed it: https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg53815.html | 14:34 |
lucasagomes | yuriyz, ping why was https://review.openstack.org/#/c/185885/ abandoned? | 14:37 |
rloo | jroll: do you think we should add that meeting/link to https://wiki.openstack.org/wiki/Meetings/Ironic or https://wiki.openstack.org/wiki/Ironic | 14:38 |
yuriyz | hi lucasagomes Ironic there is new and better (IMO) solution https://review.openstack.org/#/c/186129/ | 14:38 |
lucasagomes | gotcha | 14:39 |
lucasagomes | thanks for pointing out | 14:39 |
jroll | rloo: probably :) | 14:39 |
rloo | thx jroll :D | 14:40 |
NobodyCam | morning jroll rloo and yuriyz | 14:40 |
NobodyCam | :) | 14:40 |
rloo | morning NobodyCam, yuriyz and jroll | 14:41 |
jroll | morning * | 14:41 |
NobodyCam | :) | 14:41 |
mordred | NobodyCam: have I mentioned how much I enjoy the bearmetal thing ? | 14:42 |
NobodyCam | morning mordred :) thing == tee shirt? | 14:43 |
NobodyCam | hehe | 14:43 |
BadCub | hiya folks | 14:43 |
* NobodyCam notes its always morning in Ironic-land-de-a | 14:44 | |
jroll | rloo: updated :P | 14:45 |
persia | http://www.total-knowledge.com/~ilya/mips/ugt.html | 14:45 |
mordred | NobodyCam: I've seen more references to it | 14:45 |
mordred | NobodyCam: https://blueprints.launchpad.net/python-openstacksdk/+spec/high-level-bearmetal | 14:45 |
mordred | NobodyCam: so it's catching on :) | 14:45 |
mordred | NobodyCam: I think you should run with it | 14:45 |
*** zz_jgrimm is now known as jgrimm | 14:46 | |
mordred | and spell bare metal provisioning bear as often as you can :) | 14:46 |
jroll | hehehe | 14:46 |
rloo | ha ha, no more nit'ing on 'bear metal'! | 14:46 |
jroll | I love it | 14:46 |
jroll | rloo: s/bare/bear/ -2 | 14:46 |
*** lilia_ has joined #openstack-ironic | 14:47 | |
NobodyCam | :) | 14:47 |
jroll | persia: +++++ | 14:47 |
* NobodyCam look for his running shoes | 14:47 | |
jroll | UGT is my new favorite tz | 14:47 |
NobodyCam | persia: ++++ I love it | 14:47 |
NobodyCam | yes!!! | 14:47 |
BadCub | lol | 14:48 |
*** afaranha has joined #openstack-ironic | 14: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-ironic | 14:52 | |
jroll | lilia_: right, so that's expected behavior | 14: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 |
jroll | it sucks, but | 14:53 |
jroll | we also don't want to force users to run code in their image, so there's really no way to deal with it | 14:53 |
jlvillal | cdearborn, 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 |
jroll | lilia_: cool, you're welcome :) | 14:58 |
jroll | lilia_: also looking forward to seeing a patch for that driver ;D | 14:58 |
NobodyCam | lilia_: jroll: ++ | 14:58 |
*** zz_natorious is now known as natorious | 14:59 | |
*** e0ne is now known as e0ne_ | 15:00 | |
*** e0ne_ is now known as e0ne | 15: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-ironic | 15:02 | |
* BadCub heads off to meeting | 15:02 | |
jroll | \o/ | 15:02 |
*** spandhe_ has joined #openstack-ironic | 15:05 | |
*** mtanino has quit IRC | 15:06 | |
*** spandhe has quit IRC | 15:06 | |
*** spandhe_ is now known as spandhe | 15:06 | |
lucasagomes | persia, +1! | 15:07 |
*** mtanino has joined #openstack-ironic | 15:07 | |
*** mtanino has quit IRC | 15:07 | |
*** mtanino has joined #openstack-ironic | 15:08 | |
jroll | https://conference.hitb.org/hitbsecconf2015ams/wp-content/uploads/2015/02/D1T1-Xeno-Kovah-and-Corey-Kallenberg-How-Many-Million-BIOSes-Would-You-Like-to-Infect.pdf | 15:09 |
jroll | holy s%^&% | 15:09 |
jroll | wtb open firmware | 15:09 |
jroll | ^ devananda you'll dig these slides | 15:09 |
*** wuhg has quit IRC | 15:09 | |
lucasagomes | lilia_, that's the driver that the folks in recife brazil are working on? | 15:09 |
devananda | morning, all | 15:13 |
NobodyCam | morning (UGT) devananda | 15:13 |
BadCub | morning devananda | 15:13 |
devananda | jroll: open firmware exists for some platforms. buy different hardware | 15:14 |
jroll | devananda: I don't want this for me. I want it for the world | 15:14 |
dtantsur | devananda, morning! | 15:15 |
dtantsur | devananda, do you have folks to punch to make https://review.openstack.org/#/c/185442 marge faster? | 15:16 |
dtantsur | will block the renaming work in project-config, I guess | 15:16 |
lucasagomes | devananda, morning | 15:17 |
devananda | jroll: hah. demo of defeating tails proves what I suspected last year. I should not laugh ... this is bad sh*t | 15:17 |
jroll | devananda: yeah. laughter is a form of grieving for some people. | 15:18 |
devananda | dtantsur: nope. that will take about a week. | 15:18 |
devananda | dtantsur: 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 summit | 15:18 |
dtantsur | I see | 15:19 |
devananda | dtantsur: it should get put on https://wiki.openstack.org/wiki/Meetings/TechnicalCommittee by ttx, though I dont see it yet | 15:19 |
* devananda pokes him | 15:19 | |
NobodyCam | jroll: ya holy _______ | 15:20 |
lucasagomes | jroll, http://mjg59.dreamwidth.org/35110.html | 15:22 |
lucasagomes | SMM is ugly... but it's not all bad | 15:22 |
lucasagomes | it's useful for some stuff | 15:22 |
dtantsur | devananda, thanks! | 15:22 |
* NobodyCam wounders if he can do his job from a 6502 based system | 15:22 | |
dtantsur | devananda, do I need to be present? | 15:22 |
devananda | dtantsur: in case there are questions, it wouldn't hurt | 15:25 |
dtantsur | ack | 15:25 |
dtantsur | 20:00 UTC omg | 15:25 |
devananda | dtantsur: see -dev for discussion, but yea, this should be an easy merge because it's housekeeping | 15:26 |
dtantsur | ok I'll try to be there | 15:27 |
*** ukalifon1 has quit IRC | 15:37 | |
jnielsen55 | question. 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 |
devananda | dtantsur: though, re: inspector client, I wonder if it's reasonable to use ironicclient? I haven't thought about it much, just want your initial reaction | 15:39 |
dtantsur | hmmm | 15:40 |
dtantsur | my initial thought was separate service -> separate client | 15:40 |
devananda | jnielsen55: depends - are you using neutron to provide dhcp or not? | 15:41 |
devananda | dtantsur: same project, same client? heh | 15:41 |
dtantsur | well... | 15:42 |
jnielsen55 | yes, dhcp is supplied by neutron | 15:42 |
devananda | dtantsur: what would happen if I ran inspector on the same host as ironic-api | 15:42 |
dtantsur | what's the problem with it? | 15:42 |
devananda | jnielsen55: then ironic+neutron should take care of dhcp settings | 15:42 |
devananda | dtantsur: i dont know - is there a problem with that? just wondering if it makes sense | 15:43 |
jnielsen55 | very nice, setup going well so far. Thanks for the info. | 15:43 |
dtantsur | devananda, I'm testing everything running on one host usually :) | 15:43 |
dtantsur | devananda, one problem about using one client is that inspector client is much thinner: 2 functions using requests lib and a shell using openstackclient | 15:44 |
dtantsur | I'll probably have to write essentially more code for it to look alike usual openstack client (not that it's bad...) | 15:45 |
devananda | dtantsur: 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 that | 15:47 |
dtantsur | oh yeah, that will give us some headache. we'll also explain people that they need to provide one more endpoint | 15:47 |
devananda | I'm also not sure how we would expose (if there were a single client) that discoverd was or was not present | 15:48 |
dtantsur | * s/discoverd/inspector/ ;) | 15:48 |
devananda | our headaches aside, what's best for operators here? | 15:48 |
devananda | *yes :) | 15:48 |
dtantsur | for operators it's best to switch to openstackclient IMO | 15:48 |
dtantsur | and then we all will have one shell | 15:48 |
devananda | fair enough | 15:48 |
dtantsur | speaking of which, should we start discussion this API? patches are already up | 15:49 |
devananda | yep | 15:49 |
devananda | we should try to do that this cycle | 15:49 |
*** natorious is now known as zz_natorious | 15:49 | |
lucasagomes | dtantsur, zhenguo https://review.openstack.org/#/c/186361/ | 15:49 |
lucasagomes | answer that re FAIL states | 15:50 |
lucasagomes | lemme know WDYT before I update the patch | 15:50 |
dtantsur | devananda, so I would say: leave ironic_inspector_client separate, rely on openstackclient plugins for unification. wdyt? | 15:50 |
dtantsur | lucasagomes, sounds fine to me | 15:51 |
lucasagomes | dtantsur, ack, I will update the exception bits | 15:51 |
lucasagomes | thanks for the review | 15:51 |
devananda | dtantsur: seems ok | 15:52 |
dtantsur | devananda, and last (hopefully) question: can I propose such a governance change for not-yet-existing repository? | 15:53 |
devananda | yes | 15:53 |
devananda | in the commit msg, call out what you're proposing | 15:53 |
dtantsur | cool, thanks! | 15:54 |
devananda | that'll be all the "ok" that infra needs when you ththen peopose the change to create the actual repo | 15:54 |
*** chlong has quit IRC | 15:54 | |
devananda | *propose | 15:54 |
*** jistr has quit IRC | 15:56 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove conditional that prevent node delete based on power state https://review.openstack.org/186361 | 15:57 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove bogus conditional from node_update https://review.openstack.org/186386 | 15:57 |
*** mtanino_ has joined #openstack-ironic | 16:02 | |
*** mtanino has quit IRC | 16:03 | |
*** zz_natorious is now known as natorious | 16:04 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove deprecated pxe_deploy_{kernel, ramdisk} https://review.openstack.org/185937 | 16:05 |
*** absubram has joined #openstack-ironic | 16:05 | |
*** chlong has joined #openstack-ironic | 16:07 | |
*** chuckC_ has joined #openstack-ironic | 16:09 | |
sambetts | The 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 |
NobodyCam | sambetts: better then reviewing a hugh patch :) | 16:10 |
*** dtantsur is now known as dtantsur|afk | 16:12 | |
sambetts | NobodyCam: I guess thats true :) | 16:13 |
jlvillal | jroll: 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 |
jlvillal | jroll: Unless you object. | 16:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Remove E12[3-8] from flake8 ignore list https://review.openstack.org/186021 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E12[3-8] for ironic/tests/drivers https://review.openstack.org/186450 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in non-driver ironic/tests https://review.openstack.org/186451 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E12[3-8] for ironic/tests/api https://review.openstack.org/186452 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E12[3-8] for ironic/tests/conductor https://review.openstack.org/186453 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E12[3-8] for ironic/drivers https://review.openstack.org/186454 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E12[3-8] in ironic/conductor https://review.openstack.org/186455 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in ironic/common https://review.openstack.org/186456 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in ironic /cmd /db and /dhcp https://review.openstack.org/186457 | 16:14 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E12[3-8] in ironic/api https://review.openstack.org/186458 | 16:14 |
sambetts | haha sorry guys :/ | 16:15 |
NobodyCam | sambetts: don't be sorry for submitting patches | 16:16 |
NobodyCam | be happy! | 16:16 |
*** rwsu has joined #openstack-ironic | 16:16 | |
*** ijw has joined #openstack-ironic | 16:18 | |
sambetts | I hope I broke them down enough :/ a couple of them are still biggish | 16:19 |
*** derekh has quit IRC | 16:21 | |
jlvillal | sambetts: Are you abandoning the other patch? | 16:21 |
sambetts | jlvillal: No, I reused that patch as the one at the top of the tree, that finally removes the rule from tox.ini | 16:22 |
*** romcheg has quit IRC | 16:22 | |
lucasagomes | folks I will call it a day | 16:22 |
jlvillal | lucasagomes: Good night! | 16:22 |
sambetts | lucasagomes: Cya later o/ | 16:22 |
lucasagomes | came to the office and gotta get the train back home yet | 16:22 |
lucasagomes | see y'all | 16:22 |
jlvillal | sambetts: Okay, thanks. | 16:22 |
lucasagomes | g'night | 16:22 |
*** lucasagomes has quit IRC | 16:23 | |
*** eghobo has joined #openstack-ironic | 16:23 | |
jlvillal | jroll: Never mind. I added new patch. | 16:23 |
*** spandhe has quit IRC | 16:23 | |
cinerama | hi ironic people | 16:24 |
jlvillal | sambetts: Yeah that first patch (drivers) is big :) | 16:24 |
NobodyCam | morning cinerama | 16:24 |
jlvillal | sambetts: But still reviewable | 16:24 |
sambetts | jlvillal: the rest are much smaller and managable :) | 16:25 |
*** ndipanov has quit IRC | 16:26 | |
NobodyCam | night lucas | 16:26 |
jlvillal | sambetts: Thanks for doing this. I would like the code to be more PEP8 compliant. | 16:26 |
jlvillal | sambetts: Feel free to tackle Nova next :D They exclude I think every E12[1-9] | 16:27 |
sambetts | jlvillal: man that would be insane ... | 16:28 |
jlvillal | sambetts: We (Ironic) had it the same when I first started. So it can be done :) | 16:28 |
BadCub | morning cinerama :) | 16:28 |
sambetts | jlvillal: For not having it enforced by flake8 I was surprise how compliant most of the code was | 16:30 |
jlvillal | sambetts: I think people have been doing comments like "nit: indentation" It is better to have the computer say that. | 16:31 |
jlvillal | sambetts: 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 cool | 16:33 |
rloo | I've been noticing quite a few 'recheck' comments. Are people looking to see why a test fails? | 16:34 |
rloo | eg http://logs.openstack.org/82/178582/6/check/check-tempest-dsvm-ironic-agent_ssh/821d090/console.html | 16:35 |
rloo | I saw that yesterday in another patch, and it seems related to https://bugs.launchpad.net/openstack-ci/+bug/1393099 | 16:35 |
openstack | Launchpad bug 1393099 in OpenStack-Gate "test_baremetal_server_ops fails with timeout waiting for provision_state to change" [Undecided,Confirmed] | 16:35 |
rloo | I 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 |
sambetts | jlvillal: 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 out | 16:36 |
devananda | rloo: i think I posted a patch to raise that timeout .... | 16:38 |
rloo | devananda: did it get merged yet? | 16:39 |
*** saripurigopi has joined #openstack-ironic | 16:40 | |
jlvillal | sambetts: 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 |
rloo | jlvillal: sam was at the summit last week. Sorry, should have introduced you to him :-) | 16:41 |
jlvillal | rloo: Maybe I would recognize. I vaguely remember someone from Cisco. But people became a blur.... | 16:42 |
rloo | jlvillal: haha. yeah, I know what you mean. you're all blurry to me too :D | 16:42 |
ijw | sambetts is naturally blurry | 16:42 |
sambetts | Thanks ijw :-P | 16:43 |
cinerama | i *feel* blurry...still have the dregs of summit plague | 16:43 |
rloo | ok, 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|lunch | 16:43 | |
*** yog__ has quit IRC | 16:44 | |
jlvillal | rloo: 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 |
jlvillal | s/now/know/ | 16:45 |
jlvillal | rloo: 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 |
rloo | jlvillal: you aren't expected to know how to fix it. the recheck thingy is part of the 'elastic recheck' process | 16:46 |
rloo | jlvillal: http://status.openstack.org/elastic-recheck/ | 16:46 |
sambetts | just adding recheck bug <bug number> helps :) | 16:46 |
jroll | the timeout error is typically "pxe and/or callback" failed | 16:47 |
jlvillal | rloo: Thanks | 16:47 |
jroll | so you can dig into console logs, conductor logs | 16:47 |
jlvillal | jroll: thanks for the info. | 16:50 |
*** ijw_ has joined #openstack-ironic | 16:51 | |
*** Sukhdev has quit IRC | 16:52 | |
*** pradipta has joined #openstack-ironic | 16:53 | |
sambetts | This 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 IRC | 16:54 | |
*** ijw has quit IRC | 16:54 | |
jlvillal | sambetts: I thought I would never get through https://review.openstack.org/#/c/186450/1 But I did. Looks good to me. | 16:54 |
sambetts | jlvillal: the others are much nicer :) | 16:55 |
*** jrist has quit IRC | 16:58 | |
*** romcheg has joined #openstack-ironic | 17:01 | |
*** harlowja has joined #openstack-ironic | 17:05 | |
*** dguerri is now known as dguerri`away | 17:06 | |
*** e0ne has quit IRC | 17:08 | |
*** romainh has left #openstack-ironic | 17:11 | |
*** jrist has joined #openstack-ironic | 17:11 | |
cinerama | hey 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 IRC | 17:14 | |
sambetts | Right, I'm heading off for the evening :) Night all | 17:17 |
NobodyCam | night sambetts | 17:17 |
jlvillal | sambetts: Good night! | 17:22 |
*** dguerri`away is now known as dguerri | 17:22 | |
TheJulia | cinerama: I was on my base system Ubuntu 14.04. I've not gotten back to it yet | 17:24 |
cinerama | same | 17:24 |
*** saripurigopi has quit IRC | 17:26 | |
openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient: Refactor resource_fields.py https://review.openstack.org/181985 | 17:29 |
TheJulia | bbiab | 17:29 |
*** lilia_ has quit IRC | 17:42 | |
*** pas-ha has quit IRC | 17:46 | |
*** romcheg has quit IRC | 17:49 | |
*** trown|lunch is now known as trown | 17:49 | |
cdearborn | jlvillal, just got back from lunch - just tried running the py34 unit test on python-ironicclient, and they run successfully | 17:51 |
jlvillal | cdearborn: But openstack/ironic fails? | 17:51 |
jlvillal | wierd... | 17:51 |
jlvillal | or weird | 17:52 |
*** achanda has joined #openstack-ironic | 18:00 | |
cdearborn | jlvillal, yeah - must not follow the same code path | 18:05 |
*** e0ne has joined #openstack-ironic | 18:06 | |
*** jrist has quit IRC | 18:11 | |
cinerama | TheJulia: looks like it's the -I option that's triggering it | 18:12 |
jlvillal | TheJulia: 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-ironic | 18:14 | |
devananda | krotscheck: 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 |
krotscheck | devananda: 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 |
krotscheck | And it's not a small list. | 18:19 |
devananda | jlvillal, TheJulia: 2015-05-28 12:39:24.074 | Build timed out (after 30 minutes). Marking the build as failed. | 18:19 |
devananda | jlvillal: not a real failure. something was slow, it timed out | 18:19 |
krotscheck | Especially given that I personally do not consider ironic usable without glance. | 18:19 |
NobodyCam | devananda: yea I just started a recheck on that | 18:19 |
jlvillal | devananda: Thanks. | 18:19 |
devananda | TheJulia: In looking at the logs there, there are a lot of steps that could be pre-cached | 18:20 |
devananda | TheJulia: eg, we dont need to build a new ramdisk or download images every time we test bifrost | 18:20 |
*** e0ne has quit IRC | 18:21 | |
devananda | krotscheck: I can supply Ironic with an HTTP ref to Amazon S3 and deploy from that | 18:21 |
devananda | krotscheck: i think we had a conversation about glance images in YVR, but i've forgotten what we said | 18:21 |
krotscheck | devananda: That's useful for someone who's willing to copy/paste those references for every deploy. | 18:21 |
krotscheck | Copy/Paste does not scale to human usage. | 18:22 |
devananda | krotscheck: I'd have to copy/paste the glance image ref too | 18:22 |
krotscheck | Not if glance provided the list. | 18:22 |
krotscheck | Which is what searchlight is all about. | 18:22 |
devananda | ah. you'd present a drop-down choice, or something | 18:22 |
krotscheck | Or some kind of an autocomplete, yes. | 18:22 |
devananda | yah. that'd be cool too | 18:22 |
krotscheck | Something humans can use. | 18:22 |
* devananda can neither confirm nor deny whether he is a human | 18:23 | |
krotscheck | Mind you, I _love_ that this means that I can start working on glance & searchlight UI's :) | 18:23 |
devananda | hah | 18:23 |
krotscheck | But, well, extra work. | 18:23 |
*** jrist has joined #openstack-ironic | 18:23 | |
devananda | so - that's cool - but also, my question wasn't when can we package it, but when do you want to move it to gerrit | 18:23 |
krotscheck | Point | 18:23 |
*** achanda has quit IRC | 18:23 | |
krotscheck | I want to move it to gerrit when a reasonable set of MVP features have been implemented. | 18:24 |
devananda | that sounds reasonable | 18:24 |
devananda | also, can you do CI testing (of some sort) on it? | 18:24 |
krotscheck | And if copy/paste image URI's are a thing, then hey, v1. | 18:24 |
krotscheck | That's a more complicated question. | 18:25 |
devananda | my 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 patches | 18:25 |
krotscheck | I can write tests that run headlessly and mock the ironic API, but that will bitrot on us. | 18:26 |
devananda | noting 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 craziness | 18:26 |
devananda | hm. functional test? | 18:26 |
krotscheck | The only way to avoid bitrot is to gate the builds on each other, which requires a real ironic api instance. | 18:26 |
NobodyCam | jroll: have you put any thought into krotscheck's question about decoupling global requirements on the feature-based-releases spec? | 18:26 |
devananda | functional doesn't require co-gating | 18:27 |
*** krtaylor has quit IRC | 18:27 | |
*** edcragg has quit IRC | 18:27 | |
devananda | if ironic changes its API in a way that breaks your JS UI -- omg we just broke all our users. pls punch us. | 18:28 |
krotscheck | Can I punch you anyway? | 18:28 |
jroll | NobodyCam: not too much, I want to talk about it with others | 18:28 |
jroll | NobodyCam: initial thought for now is "deal with it" | 18:28 |
NobodyCam | ack and ++ | 18:29 |
devananda | krotscheck: maybe | 18:29 |
krotscheck | So, 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 |
devananda | eeeeeh | 18:29 |
devananda | (A) is trivial. use devstack. or bifrost. | 18:29 |
krotscheck | I 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 crazy | 18:29 |
devananda | oh? | 18:29 |
krotscheck | Yes, 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 something | 18:30 |
krotscheck | Also, it'd be useful for the docs team to run their livedocs things against. | 18:30 |
devananda | configure it only with the "fake" driver | 18:31 |
krotscheck | Yep. | 18:31 |
devananda | only difficulty will be the lack of any tenancy in ironic | 18:31 |
devananda | every client will see all resources | 18:31 |
*** davideagnello has quit IRC | 18:36 | |
krotscheck | devananda: That should be fine on a dev instance. | 18:38 |
*** gridinv_ has joined #openstack-ironic | 18:45 | |
devananda | seems reasonable then | 18:45 |
devananda | i'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 |
jroll | devananda: I'm about to lunch but can look when I'm back :) | 18:46 |
*** gridinv_ has quit IRC | 18:46 | |
devananda | jroll: k k. no rush | 18:46 |
cinerama | that'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>— 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 as | 18:48 |
cinerama | whoops | 18:48 |
cinerama | need to get these cats off the keyboard | 18:49 |
devananda | :p | 18:49 |
*** romcheg has joined #openstack-ironic | 18:56 | |
devananda | krotscheck: given the mountain of work before you related to the _general_ task of making browser-based clients a thing in openstack | 18:56 |
*** Sukhdev has joined #openstack-ironic | 18:56 | |
devananda | krotscheck: do you want a call-out in my ironic summit sumary email? I'm presently assuming "yes" until you say otherwise | 18:56 |
krotscheck | devananda: I have req's! | 18:57 |
krotscheck | Also, yes. | 18:57 |
devananda | krotscheck: !!! \o/ | 18:57 |
devananda | krotscheck: fwiw, I do not | 18:57 |
krotscheck | sadpanda | 18:57 |
*** UForgotten has joined #openstack-ironic | 18:58 | |
UForgotten | Has anyone used ironic to deploy things other than openstack nodes? | 18:59 |
devananda | UForgotten: uh.... that's mostly what we use it for | 19:00 |
*** pradipta has quit IRC | 19:03 | |
TheJulia | jlvillal: 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 |
TheJulia | devananda: for the record, I'm 95.5% sure your a human | 19:10 |
devananda | TheJulia: cache more, do less ? | 19:10 |
devananda | TheJulia: that's a high degree of accuracy in your lack of certainty :) | 19:10 |
TheJulia | devananda: yes | 19:10 |
TheJulia | on both items | 19:10 |
cinerama | "Fascinating." | 19:15 |
*** alexpilotti has quit IRC | 19:23 | |
TheJulia | cinerama: Your on 14.04LTS right? | 19:31 |
cinerama | TheJulia: yep | 19:31 |
cinerama | TheJulia: so the thing that breaks it is including "-I" | 19:31 |
cinerama | TheJulia: if i run it with --no-cache-dir i'm good | 19:31 |
cinerama | TheJulia: it complains about ownership of ~/.cache/pip/http and exits 139 without that | 19:32 |
TheJulia | cinerama: I think we could loose -I since that is actually a timesink | 19:32 |
cinerama | TheJulia: i have one other experiment cooked up | 19:32 |
TheJulia | cinerama: bug 1459801 | 19:32 |
openstack | bug 1459801 in bifrost "Running bifrost on Ubuntu 14.04 LTS results in segmentation fault" [Undecided,New] https://launchpad.net/bugs/1459801 | 19:32 |
*** radek__ has quit IRC | 19:34 | |
*** natorious is now known as zz_natorious | 19:35 | |
*** radek__ has joined #openstack-ironic | 19:35 | |
cinerama | TheJulia: nice one. i will report my findings | 19:35 |
jroll | devananda: so, if you want me to review your summit recap, I'll need a link or something :P | 19:36 |
*** mriedem has joined #openstack-ironic | 19:37 | |
mriedem | there is no ironic-stable-maint group in gerrit? | 19:37 |
jroll | UForgotten: our customers use it to deploy lots of things that are probably not openstack :) | 19:37 |
jroll | mriedem: I'm 99% sure there is | 19:37 |
jroll | perhaps a different name? not sure. devananda would know. | 19:38 |
mriedem | https://review.openstack.org/#/admin/groups/?filter=stable | 19:38 |
mriedem | not seeing it | 19:38 |
mriedem | i wanted to rebase this https://review.openstack.org/#/c/175540/ | 19:39 |
mriedem | i don't get a rebase button though apparently even though i'm stable maint core, but ironic doesn't have tha | 19:39 |
mriedem | *that | 19:39 |
jroll | mriedem: ironic-milestone or ironic-release maybe? https://review.openstack.org/#/admin/groups/?filter=ironic | 19:39 |
UForgotten | jroll: 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 |
jroll | milestone sounds right | 19:40 |
NobodyCam | mriedem: https://review.openstack.org/#/admin/groups/167,members | 19:40 |
NobodyCam | I think thats is | 19:40 |
mriedem | hmmm | 19:40 |
mriedem | that should include stable-maint-core like the other projects, e.g. https://review.openstack.org/#/admin/groups/540,members | 19:40 |
TheJulia | UForgotten: Well, you've come to the right place then :) | 19:40 |
mriedem | adam_g: are you around? | 19:40 |
adam_g | mriedem, hi | 19:41 |
mriedem | adam_g: hey, is this ironic's stable maint core group? https://review.openstack.org/#/admin/groups/167,members | 19:41 |
NobodyCam | hey adam_g :) \o | 19:41 |
cinerama | TheJulia: how come we don't use the ansible pip module? | 19:42 |
adam_g | mriedem, uhm, looks like its the one we setup a while ago but not sure why its named ironic-milestone? | 19:42 |
UForgotten | TheJulia: I hope so. Is anyone using ironic without openstack and have a nice gui to configure it? | 19:42 |
*** krtaylor has joined #openstack-ironic | 19:42 | |
TheJulia | cinerama: There was some reason originally, possibly because I was changing stuff really often. I'm fine with changing that or -I. | 19:43 |
mriedem | adam_g: it also doesn't include stable-maint-core | 19:43 |
cinerama | TheJulia: yeah i thought there might be some reason like that it's broken etc | 19:43 |
adam_g | mriedem, okay, it does now. not sure how to go about renaming that, or why it was in the first place | 19:44 |
mriedem | adam_g: i think you can rename through a project-config or system-config patch | 19:44 |
mriedem | i'll take a look | 19:44 |
*** spandhe has joined #openstack-ironic | 19:45 | |
TheJulia | UForgotten: 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-ironic | 19:46 | |
wshao | hello, question on standalone use of ironic, any guide on setting it up? | 19:47 |
devananda | UForgotten: 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-o1gLHHWo | 19:47 |
devananda | wshao: ^ :) | 19:48 |
devananda | github.com/openstack/bifrost | 19:48 |
devananda | TheJulia: you're going to need an autoresponder :p | 19:48 |
TheJulia | devananda: Yeah... but the autoresponder would lack the human touch :) | 19:48 |
devananda | jroll: oh right. you can't just read my mind? | 19:49 |
devananda | jroll: you have email | 19:49 |
jroll | devananda: cool, but now you've distracted me with the release guide thing :P | 19:51 |
jroll | er, release model | 19:51 |
*** eghobo has quit IRC | 19:51 | |
*** penick has joined #openstack-ironic | 19:52 | |
TheJulia | devananda: 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 natorious | 19:53 | |
devananda | TheJulia: it is :) | 19:53 |
*** mriedem has left #openstack-ironic | 19:53 | |
*** natorious is now known as zz_natorious | 19:54 | |
*** romainh has joined #openstack-ironic | 19:58 | |
*** romainh has left #openstack-ironic | 19:59 | |
*** eghobo has joined #openstack-ironic | 20:00 | |
jroll | devananda: looks fine, I had to read quickly though | 20:02 |
* NobodyCam tries to figure out how to post "read at your own risk" signs around his mind | 20:02 | |
devananda | jroll: cool. mostly looking for "did I forget to mentoin $FOO" | 20:04 |
jroll | devananda: yeah, not that I noticed... didn't reference against anything else though | 20:04 |
jroll | you certainly covered the stuff I care about :) | 20:05 |
devananda | good enough | 20:05 |
TheJulia | NobodyCam: Once you figure out the protocol for posting those, I'll need to post a small forest around mine | 20:05 |
*** davideagnello has joined #openstack-ironic | 20:10 | |
*** achanda has joined #openstack-ironic | 20:13 | |
*** ijw has joined #openstack-ironic | 20:15 | |
jlvillal | jroll: Woot! Got one +2 :) https://review.openstack.org/#/c/130877/ | 20:17 |
*** ijw_ has quit IRC | 20:17 | |
jroll | jlvillal: my production environment is the only validation I need :) | 20:17 |
jlvillal | :) | 20:18 |
*** achanda has quit IRC | 20:20 | |
dtantsur|afk | devananda, 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|afk | 22:21 | 20:21 |
dtantsur|afk | going to have some pizza with cider | 20:21 |
jlvillal | dtantsur|afk: Come back tomorrow ;) | 20:21 |
harlowja | devananda u forgot https://review.openstack.org/#/c/141961/ in your state-machine section :-P | 20:21 |
harlowja | *of your recent ML (summit highlights) email | 20:22 |
harlowja | ha | 20:22 |
*** eghobo_ has joined #openstack-ironic | 20:24 | |
*** wshao has quit IRC | 20:26 | |
*** eghobo has quit IRC | 20:27 | |
openstackgerrit | Merged openstack/ironic: Get admin auth token for Glance client in image_service https://review.openstack.org/186129 | 20:42 |
*** Sukhdev has quit IRC | 20:56 | |
devananda | jroll: re the SMM attacks, coreboot to the rescue? | 20:56 |
UForgotten | is it ok to run a newer ironic package with a JUNO release of openstack? | 20:58 |
UForgotten | or do ubuntu 12 / precise deb packages exist somewhere? | 21:00 |
rloo | UForgotten: no, I think it'll break the nova driver in the juno release. Or will it. sigh. I forgot. | 21:01 |
devananda | newer ironic should be backwards compatibile with juno nova. HOWEVER it will not expose all features | 21:02 |
devananda | *all the features added in kilo | 21:02 |
rloo | thx devananda! (And I just looked at our release notes and it mentions that.) | 21:03 |
UForgotten | ok, so where can I look for a precise-compatible deb package ? | 21:03 |
devananda | I doubt that an official one exists. zigo? | 21:04 |
*** ijw_ has joined #openstack-ironic | 21:06 | |
*** dguerri is now known as dguerri`away | 21:06 | |
zigo | devananda: I don't have some. | 21:06 |
zigo | UForgotten: Why using such an outdated distro? | 21:06 |
zigo | It's going to be soon EOL. | 21:06 |
jroll | devananda: I guess/hope so, however I'm not smart enough to know | 21:06 |
UForgotten | zigo, its complicated :) | 21:06 |
UForgotten | I 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 standalone | 21:08 |
*** dguerri`away is now known as dguerri | 21:08 | |
*** ijw has quit IRC | 21:09 | |
*** dguerri is now known as dguerri`away | 21:09 | |
devananda | UForgotten: that seems orthogonal to "is there a Precise package for ironic" | 21:10 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Report/Configure LAN alert destinations https://review.openstack.org/185112 | 21:12 |
krotscheck | https://etherpad.openstack.org/p/ironic-webclient-use-cases | 21:12 |
krotscheck | ^^ where I'm coming up with a list of things to do. | 21:12 |
*** jgrimm is now known as zz_jgrimm | 21:13 | |
UForgotten | devananda: not following | 21:14 |
devananda | UForgotten: maybe start by describing your use case :) | 21:14 |
openstackgerrit | Christopher Dearborn proposed openstack/ironic: Add additional required RPMs to dev instructions https://review.openstack.org/185993 | 21:16 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Report/Configure LAN alert destinations https://review.openstack.org/185112 | 21:16 |
*** dguerri`away is now known as dguerri | 21:17 | |
*** dguerri is now known as dguerri`away | 21:18 | |
*** alexpilotti has joined #openstack-ironic | 21:21 | |
NobodyCam | brb | 21:22 |
*** trown is now known as trown|outttypeww | 21:22 | |
devananda | krotscheck: pausing a moment to try fixing my monitor (again) | 21:26 |
krotscheck | devananda: No worries | 21:26 |
krotscheck | Other people should comment! | 21:26 |
*** Ng has quit IRC | 21:28 | |
devananda | i am sad now. this hdmi -> dp connector doesn't work | 21:30 |
*** mtanino_ has quit IRC | 21:31 | |
krotscheck | Do we know of anyone who's currently deploying Ironic, who'd be willing to be a guineapig for UX things? | 21:31 |
mrda | Morning! | 21:31 |
*** jamielennox|away is now known as jamielennox | 21:34 | |
*** ijw_ has quit IRC | 21:35 | |
*** ijw has joined #openstack-ironic | 21:36 | |
*** openstackgerrit has quit IRC | 21:36 | |
*** openstackgerrit has joined #openstack-ironic | 21:37 | |
krotscheck | mrda: 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-ironic | 21:39 | |
mrda | krotscheck: :) | 21:39 |
mrda | krotscheck: Just cooincidence | 21:39 |
* krotscheck plays the sad trombone. | 21:40 | |
mrda | Sorry, I'm pre-coffee and hence not much help until that is rectified | 21:40 |
jroll | devananda: this release plan has great success on the list so far | 21:41 |
* jroll is excite | 21:41 | |
jroll | krotscheck: we'd probably be willing to poke at it, I would at least | 21:41 |
jroll | krotscheck: when the CORS thing lands, can I set it for multiple domains or just one? I'd personally probably set to localhost and play around | 21:42 |
jroll | krotscheck: that said, read only mode is a hard requirement for me to run it in prod | 21:43 |
devananda | i now thik i have a bad connector AND a broken desktop | 21:44 |
jroll | :| | 21:45 |
jroll | too bad you don't know any companies that might have extra hardware | 21:45 |
devananda | krotscheck: i think the 'pad lost my colors | 21:47 |
devananda | jroll: 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 on | 21:48 |
devananda | but it is frustrating to turn something on and have it not. turn. on. | 21:48 |
jroll | indeed. | 21:48 |
jroll | like my phone all weekend. | 21:48 |
devananda | :-/ | 21:49 |
jlvillal | jroll: It gave you more time to enjoy Portland! :) | 21:49 |
jroll | jlvillal: it did indeed, I didn't mind it | 21:49 |
jroll | actually went out without my phone for once | 21:49 |
krotscheck | jroll: Multiple. Everything in [cors] is the default, anything added in additional [cors.FOO] blocks is an override | 21:49 |
jroll | krotscheck: sweet. I would play. | 21:49 |
krotscheck | jroll: Read only mode? | 21:49 |
jlvillal | jroll: As a note, Voodoo Doughnuts is over-hyped. But I am telling you that after you were here :) | 21:49 |
krotscheck | jlvillal: Yes. Go to Blue Star instead. | 21:50 |
jlvillal | krotscheck: Yep! :) | 21:50 |
jroll | jlvillal: 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 | |
jroll | krotscheck: yes, read only | 21:50 |
jroll | though now that I think about it... only allow GET in cors config | 21:51 |
jroll | and that's done | 21:51 |
jlvillal | jroll: krotscheck is correct. Blue Star has good donuts :) | 21:51 |
jroll | but it would be cool if we failed gracefully on that :) | 21:51 |
krotscheck | jroll: Well, the client is basically stateless. | 21:51 |
krotscheck | jroll: So Ironic would have to tell the client whether it's allowed to do anything. | 21:52 |
jroll | krotscheck: right, but I don't want folks modifying data or hitting deploy or whatever | 21:52 |
jroll | and we don't have good policy support | 21:52 |
*** dguerri`away is now known as dguerri | 21:53 | |
krotscheck | I 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 |
krotscheck | afk for a sec, sick of standing | 21:53 |
*** alexpilotti has quit IRC | 21:55 | |
*** dguerri is now known as dguerri`away | 21:56 | |
krotscheck | back | 21:56 |
*** ijw_ has joined #openstack-ironic | 21:56 | |
jroll | krotscheck: sure, but then we'll need to do work in ironic before I can run it | 21:59 |
jroll | I could still test locally though | 21:59 |
*** ijw has quit IRC | 22:00 | |
krotscheck | jroll: 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 |
krotscheck | It's not the UI's fault that the API exposes those things. It's not the CLI's fault either. | 22:01 |
jroll | krotscheck: accidentally clicking a thing is way easier. I'm more concerned with accidents than malicious users | 22:01 |
jroll | it's just a rule we've made, that our production dashboard is read-only | 22:01 |
jroll | to avoid accidents/stupidity/etc | 22:01 |
jroll | that hasn't prevented all accidents, but I'm sure it's prevented some | 22:02 |
krotscheck | Sounds to me like you need to run chaosmonkey | 22:02 |
jroll | deleting a customer instance would be real real bad. | 22:02 |
jroll | just sayin. | 22:02 |
jroll | but like I said, I'm happy to play, just not run it in production until we have some proper policy type stuff | 22:03 |
krotscheck | Aight | 22:03 |
devananda | jroll: ++ for policy stuff | 22:05 |
JayF | heh; our customers wouldn't like us running chaosmonkey against their instances | 22:06 |
JayF | I don't think even I could sell that as a managed services featuer | 22:06 |
Madasi | I'd love to run it against the control plane though | 22:07 |
jroll | ^^ | 22:07 |
jroll | one day. | 22:07 |
* TheJulia has fond memories of letting a chaosmonkey just knock things over | 22:07 | |
Madasi | or make a chaos monkey gate for gerrit | 22:07 |
krotscheck | Alright, any last requests for the webclient MVP? | 22:09 |
krotscheck | https://etherpad.openstack.org/p/ironic-webclient-use-cases | 22:09 |
* mrda would like to see a chaos monkey cehck job | 22:10 | |
krotscheck | Who is purple? | 22:15 |
jroll | not I | 22:15 |
TheJulia | I am | 22:16 |
krotscheck | Gotcha | 22:16 |
* TheJulia thanks chrome for forgetting who she is | 22:16 | |
jroll | krotscheck: this reminds me. I need keystone to make this work for me. :/ | 22:16 |
krotscheck | Chassis, 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 |
krotscheck | jroll: I'm wary of feature creep. Also, of additional dependencies to things that are not ironic. | 22:17 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Preserve logs in logs/ folder https://review.openstack.org/186581 | 22:18 |
krotscheck | jroll: 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 |
krotscheck | Right now, I'm restricting the project to "Only things that ironic does." | 22:19 |
krotscheck | Next iteration we can start talking to "Things that ironic needs to be usable" | 22:20 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Make Ansible modules 1.9 compatible https://review.openstack.org/186582 | 22:20 |
krotscheck | TheJulia: Does anyone actually use chassis? | 22:20 |
TheJulia | krotscheck: afaik no, so I'll rip that out | 22:20 |
TheJulia | and gone | 22:21 |
devananda | keystone is expressly not something that ironic requires (to be used when standalone) | 22:21 |
devananda | at some point, I expect someone may deploy ironic+neutron (and nothing else) | 22:21 |
devananda | or ironic+swift | 22:21 |
devananda | but to krotscheck's point, yes, I imagine any UI integration becomes a question for horizon | 22:22 |
*** errr has joined #openstack-ironic | 22:28 | |
krotscheck | I'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 |
jroll | krotscheck: sure, I get it, just letting you know :) | 22:29 |
* devananda dives into expense reports | 22:32 | |
krotscheck | Oooh, what if that ironic applicane contained a bunch of containers instead. | 22:32 |
krotscheck | Containers that contained, say, different parts of openstack! | 22:32 |
krotscheck | And then we'd plug it in! | 22:32 |
krotscheck | And it would install OpenStack! | 22:32 |
krotscheck | This project needs a name! | 22:32 |
krotscheck | Let's call it.... | 22:33 |
krotscheck | Triple-I! | 22:33 |
TheJulia | krotscheck: Have you had too much coffee? | 22:33 |
JayF | -.- /o\ | 22:33 |
krotscheck | TheJulia: Define.... "too much" | 22:34 |
TheJulia | krotscheck: It being the late afternoon and one has only had coffee to drink during that day. ? | 22:37 |
*** absubram has quit IRC | 22:37 | |
jroll | TheJulia: wait, are you saying that would be a problem? | 22:39 |
TheJulia | it can be :) | 22:40 |
devananda | krotscheck: Ironic In Ironic ? ... | 22:42 |
devananda | krotscheck: I second TheJulia 's questoin - perhaps you've had too much coffee :) | 22:42 |
jroll | hmm | 22:42 |
*** barra204 has joined #openstack-ironic | 22:43 | |
krotscheck | Clearly, I haven't had enough. | 22:45 |
* krotscheck goes to fix that. | 22:45 | |
TheJulia | Hmm | 22:46 |
* jroll chugs his coffee | 22:46 | |
*** chlong has quit IRC | 22:52 | |
*** eghobo_ has quit IRC | 22:57 | |
*** rloo has quit IRC | 22:59 | |
*** barra204 has quit IRC | 23:01 | |
*** Haomeng has joined #openstack-ironic | 23:03 | |
*** krtaylor has quit IRC | 23:05 | |
*** Haomeng|2 has quit IRC | 23:06 | |
*** Sukhdev has joined #openstack-ironic | 23:07 | |
*** romcheg has quit IRC | 23:08 | |
*** barra204 has joined #openstack-ironic | 23:16 | |
*** hemna is now known as hemnafk | 23:18 | |
jlvillal | Should something like this have a spec associated with it? https://review.openstack.org/#/c/171672/ | 23:30 |
*** dprince has quit IRC | 23:37 | |
*** eghobo has joined #openstack-ironic | 23:37 | |
*** eghobo has quit IRC | 23:45 | |
*** eghobo has joined #openstack-ironic | 23:46 | |
NobodyCam | I'm actually ok without a spec for that. | 23:47 |
*** krtaylor has joined #openstack-ironic | 23:51 | |
NobodyCam | humm need to think on the set / unset commands | 23:54 |
NobodyCam | may get confusing setting node and port [chassis too] properties | 23:55 |
jroll | I think there was an openstack-wide spec on it or something | 23:56 |
jroll | I wouldn't mind one to hash out what the "UI" looks like, but fine without one too | 23:56 |
*** radek__ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!