| NobodyCam | humm now /me wants to add a "random" drinking event to the Liberty-ironic-design-summit-ideas etherpad | 00:00 |
|---|---|---|
| BadCub_Hiding | NobodyCam: ++++++++++++++ | 00:01 |
| *** hemna is now known as hemnafk | 00:07 | |
| *** penick has quit IRC | 00:08 | |
| *** mtanino has quit IRC | 00:15 | |
| *** achanda has quit IRC | 00:18 | |
| jroll | comstud: so we actually put in some fixes for this in shutdown and whatnot, we don't get them in real life any more, but why hack around it in python | 00:18 |
| *** achanda has joined #openstack-ironic | 00:19 | |
| jroll | wtf openstackgerrit | 00:19 |
| jroll | 23:38:30 openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Make node locking pluggable https://review.openstack.org/179962 | 00:19 |
| jroll | 23:38:31 openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: Add etcd task manager https://review.openstack.org/179965 | 00:19 |
| * jroll has to do everything himself, just uploaded new versinos | 00:19 | |
| * jroll done for the day | 00:20 | |
| *** achanda has quit IRC | 00:23 | |
| *** rloo has quit IRC | 00:25 | |
| *** saripurigopi has joined #openstack-ironic | 00:34 | |
| *** Sukhdev has quit IRC | 00:36 | |
| *** saripurigopi has quit IRC | 00:41 | |
| -openstackstatus- NOTICE: Restarted gerrit due to stuck stream-events connections. Events since 23:49 were missed and changes uploaded since then will need to be rechecked. | 00:42 | |
| *** cui has joined #openstack-ironic | 00:45 | |
| *** cui has quit IRC | 00:48 | |
| *** cui has joined #openstack-ironic | 00:49 | |
| *** saripurigopi has joined #openstack-ironic | 00:53 | |
| *** igordcard has quit IRC | 00:56 | |
| *** alexpilotti has quit IRC | 01:08 | |
| *** saripurigopi has quit IRC | 01:13 | |
| *** ijw has quit IRC | 01:18 | |
| *** natorious is now known as zz_natorious | 01:23 | |
| *** alexpilotti has joined #openstack-ironic | 01:30 | |
| *** Marga_ has quit IRC | 01:36 | |
| *** spandhe has quit IRC | 01:43 | |
| *** spandhe has joined #openstack-ironic | 01:45 | |
| *** derekh_ has joined #openstack-ironic | 01:46 | |
| *** spandhe has quit IRC | 01:47 | |
| *** penick has joined #openstack-ironic | 01:48 | |
| *** alexpilotti has quit IRC | 01:48 | |
| *** derekh has quit IRC | 01:50 | |
| *** penick_ has joined #openstack-ironic | 01:51 | |
| *** r-daneel has quit IRC | 01:52 | |
| *** penick has quit IRC | 01:52 | |
| *** penick_ is now known as penick | 01:52 | |
| *** cui has quit IRC | 01:56 | |
| *** kan_ has joined #openstack-ironic | 01:59 | |
| *** zhenguo has joined #openstack-ironic | 02:01 | |
| openstackgerrit | Merged openstack/ironic: Return HTTP 400 for invalid sort_key https://review.openstack.org/179159 | 02:38 |
| *** achanda has joined #openstack-ironic | 02:40 | |
| *** penick has quit IRC | 02:41 | |
| *** spandhe has joined #openstack-ironic | 02:48 | |
| *** clett has joined #openstack-ironic | 02:54 | |
| *** lazy_prince has joined #openstack-ironic | 03:35 | |
| *** krtaylor has quit IRC | 03:37 | |
| *** lazy_prince has quit IRC | 03:38 | |
| *** lazy_prince has joined #openstack-ironic | 03:39 | |
| *** rameshg87 has joined #openstack-ironic | 03:40 | |
| rameshg87 | good morning ironic | 03:40 |
| openstackgerrit | sonu proposed openstack/python-ironicclient: Added unit tests for command-line shell https://review.openstack.org/180396 | 03:41 |
| *** links has joined #openstack-ironic | 03:41 | |
| *** Marga_ has joined #openstack-ironic | 03:49 | |
| *** Marga_ has quit IRC | 03:50 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on https://review.openstack.org/177642 | 03:51 |
| *** Marga_ has joined #openstack-ironic | 03:51 | |
| *** coolsvap|afk is now known as coolsvap | 03:53 | |
| *** krtaylor has joined #openstack-ironic | 03:55 | |
| *** ramineni has joined #openstack-ironic | 03:57 | |
| openstackgerrit | Merged openstack/ironic: Use oslo_log lib https://review.openstack.org/157602 | 04:16 |
| *** saripurigopi has joined #openstack-ironic | 04:26 | |
| *** Marga_ has quit IRC | 05:01 | |
| *** dlpartain has joined #openstack-ironic | 05:02 | |
| *** yog_ has joined #openstack-ironic | 05:05 | |
| saripurigopi | What is 'out of tree' ironic driver? | 05:09 |
| *** spandhe_ has joined #openstack-ironic | 05:29 | |
| *** spandhe has quit IRC | 05:30 | |
| *** spandhe_ is now known as spandhe | 05:30 | |
| *** subscope_ has joined #openstack-ironic | 05:35 | |
| *** yog_ has quit IRC | 05:40 | |
| *** early has quit IRC | 05:49 | |
| mrda | saripurigopi: you mean the Nova Ironic driver that sits in Nova's tree? | 05:54 |
| *** early has joined #openstack-ironic | 05:59 | |
| *** dlpartain has left #openstack-ironic | 06:01 | |
| *** subscope_ has quit IRC | 06:02 | |
| *** jcoufal has joined #openstack-ironic | 06:06 | |
| rameshg87 | mrda: i think saripurigopi meant about ironic drivers that are not upstream yet | 06:06 |
| rameshg87 | saripurigopi: is that what you meant ? | 06:07 |
| mrda | oh, thanks rameshg87 | 06:09 |
| *** ukalifon1 has joined #openstack-ironic | 06:14 | |
| *** kalpase has joined #openstack-ironic | 06:18 | |
| *** kalpase has quit IRC | 06:24 | |
| *** cui has joined #openstack-ironic | 06:30 | |
| *** cui has quit IRC | 06:38 | |
| *** alex_xu has quit IRC | 06:38 | |
| *** cui has joined #openstack-ironic | 06:39 | |
| *** tteggel has quit IRC | 06:41 | |
| *** bkero has quit IRC | 06:41 | |
| *** tteggel has joined #openstack-ironic | 06:41 | |
| *** alex_xu has joined #openstack-ironic | 06:42 | |
| saripurigopi | rameshg87: yes, I've heard it in conversation. and pxe_ucs driver was mentioned as 'out of tree' driver. Not sure what is 'out of tree' driver. so wanted to confirm. | 06:42 |
| *** bkero has joined #openstack-ironic | 06:43 | |
| *** kalpase has joined #openstack-ironic | 06:45 | |
| *** athomas has joined #openstack-ironic | 06:45 | |
| *** viktors|afk is now known as viktors | 06:48 | |
| *** jcoufal_ has joined #openstack-ironic | 06:49 | |
| *** cui has quit IRC | 06:52 | |
| *** cui has joined #openstack-ironic | 06:52 | |
| *** jcoufal has quit IRC | 06:52 | |
| rameshg87 | saripurigopi: sorry i was away | 06:53 |
| rameshg87 | saripurigopi: yeah, so in ironic you can have the driver within the tree in upstream | 06:54 |
| rameshg87 | saripurigopi: or you can have your own module which implements all the required interfaces and mentions the stevedore entry points | 06:54 |
| rameshg87 | saripurigopi: https://github.com/openstack/ironic/blob/master/setup.cfg#L35-L67 | 06:55 |
| rameshg87 | saripurigopi: stevedore entry point namespace is ironic.drivers | 06:55 |
| rameshg87 | saripurigopi: when ironic conductor loads it will scan through all modules in the system with an entry point in this namepace | 06:55 |
| rameshg87 | saripurigopi: and hence will pick up the out of the tree driver as well | 06:55 |
| saripurigopi | rameshg87: thanks, it is clear now. | 06:57 |
| *** Haomeng has joined #openstack-ironic | 07:04 | |
| *** kalpase has quit IRC | 07:06 | |
| *** cui has quit IRC | 07:07 | |
| *** yog_ has joined #openstack-ironic | 07:07 | |
| *** achanda has quit IRC | 07:07 | |
| *** cui has joined #openstack-ironic | 07:08 | |
| *** achanda has joined #openstack-ironic | 07:11 | |
| *** achanda has quit IRC | 07:11 | |
| *** afaranha has quit IRC | 07:18 | |
| *** cui has quit IRC | 07:22 | |
| *** cui has joined #openstack-ironic | 07:22 | |
| *** dlpartain has joined #openstack-ironic | 07:23 | |
| *** dlpartain has left #openstack-ironic | 07:30 | |
| *** ifarkas has joined #openstack-ironic | 07:38 | |
| *** kalpase has joined #openstack-ironic | 07:45 | |
| *** romcheg has joined #openstack-ironic | 07:46 | |
| *** jistr has joined #openstack-ironic | 07:59 | |
| openstackgerrit | Merged openstack/ironic: Include graphviz in install prerequisites https://review.openstack.org/179930 | 08:01 |
| openstackgerrit | Merged openstack/ironic: Link to config reference in our docs https://review.openstack.org/179892 | 08:01 |
| *** romcheg has quit IRC | 08:02 | |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Bare Metal Trust Using Intel TXT https://review.openstack.org/133902 | 08:02 |
| kejiahu | morning Ironic | 08:03 |
| Haomeng | kejiahu: morning:) | 08:04 |
| kejiahu | Haomeng, morning :) | 08:04 |
| Haomeng | kejiahu: :) | 08:04 |
| *** ndipanov has quit IRC | 08:06 | |
| *** ndipanov has joined #openstack-ironic | 08:08 | |
| *** lucasagomes has joined #openstack-ironic | 08:08 | |
| *** lsmola has joined #openstack-ironic | 08:11 | |
| lucasagomes | morning, gate is broken?! | 08:12 |
| lucasagomes | 2015-05-06 02:04:06.501 | ERROR: openstack 500 Internal Server Error: Failed to upload image a80cfc6e-e4b9-48b5-8e33-51a440b25cb5 (HTTP 500) hmmmm | 08:14 |
| *** kalpase has quit IRC | 08:14 | |
| *** edcragg has joined #openstack-ironic | 08:19 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Override PXE kernel cmdline via Glance property https://review.openstack.org/178229 | 08:19 |
| *** e0ne has joined #openstack-ironic | 08:22 | |
| rameshg87 | lucasagomes: i think i missed to run tox on https://review.openstack.org/#/c/167929/ | 08:27 |
| rameshg87 | lucasagomes: morning btw :) | 08:27 |
| lucasagomes | rameshg87, :-D | 08:27 |
| lucasagomes | morning | 08:27 |
| Haomeng | lucasagomes: yes, a lot of gate error | 08:31 |
| lucasagomes | :-( | 08:31 |
| Haomeng | lucasagomes: :) | 08:31 |
| Haomeng | lucasagomes, rameshg87, morning:) | 08:32 |
| rameshg87 | morning Haomeng | 08:33 |
| Haomeng | rameshg87: :) | 08:33 |
| *** Marga_ has joined #openstack-ironic | 08:37 | |
| Haomeng | ramineni: hi | 08:41 |
| ramineni | Haomeng: hi , | 08:41 |
| Haomeng | ramineni: what OS type you are using, redhat? | 08:41 |
| Haomeng | ramineni: or ubuntu? | 08:41 |
| ramineni | Haomeng: ubuntu 14.04 | 08:42 |
| Haomeng | ramineni: yes, we can take devestack ironic scripts as reference | 08:42 |
| ramineni | Haomeng: tftp works fine until i specify map-file | 08:42 |
| Haomeng | ramineni: yes, let me shou you the devstack ironic scripts | 08:42 |
| *** kalpase has joined #openstack-ironic | 08:42 | |
| ramineni | Haomeng: ok | 08:43 |
| Haomeng | ramineni: this one - https://github.com/openstack-dev/devstack/blob/2ebcd3817f6b9f18ddb408026ac07f48d0bea801/tools/ironic/templates/tftpd-xinetd.template | 08:43 |
| *** e0ne has quit IRC | 08:43 | |
| Haomeng | ramineni: we can run tftp client to verify your tftp servier | 08:43 |
| ramineni | Haomeng: I have the same, still doesnt work :( | 08:44 |
| Haomeng | ramineni: tftp <YOUR_TFTPSERVER_IP> | 08:44 |
| Haomeng | ramineni: get pxelinux.0 to see if it is working | 08:44 |
| ramineni | Haomeng: yes, i'm doing that, it times out | 08:44 |
| Haomeng | ramineni: if it is map issue, should return 'can not find file' | 08:44 |
| Haomeng | ramineni: if timeout, should be network issue I think | 08:44 |
| Haomeng | ramineni: can you check firewall? | 08:45 |
| ramineni | Haomeng: oh, let me check | 08:45 |
| Haomeng | ramineni: make sure the udp 69 port is opened for outcoming | 08:45 |
| ramineni | Haomeng: but, that works if i remove the argument , in xinetd/tftp file | 08:45 |
| Haomeng | ramineni: and check if 69 port is opened - netstat -an|grep 69|grep udp | 08:46 |
| Haomeng | ramineni: if it works without map, just go ahead to remove it:) | 08:46 |
| *** kalpase has quit IRC | 08:47 | |
| *** kalpase1 has joined #openstack-ironic | 08:47 | |
| *** kalpase1 has quit IRC | 08:48 | |
| ramineni | Haomeng: ok, | 08:48 |
| ramineni | Haomeng: btw its opened | 08:49 |
| openstackgerrit | Victor Sergeyev proposed openstack/ironic: Run tests in py34 environment https://review.openstack.org/156192 | 08:51 |
| Haomeng | ramineni: will be back for a while:) | 08:54 |
| Haomeng | ramineni: :) | 08:54 |
| *** lsmola has quit IRC | 08:58 | |
| *** clett has quit IRC | 08:58 | |
| *** mrda has quit IRC | 08:58 | |
| *** rsFF has quit IRC | 08:58 | |
| *** lsmola has joined #openstack-ironic | 09:00 | |
| *** clett has joined #openstack-ironic | 09:00 | |
| *** mrda has joined #openstack-ironic | 09:00 | |
| *** rsFF has joined #openstack-ironic | 09:00 | |
| *** yog_ has quit IRC | 09:09 | |
| ramineni | lucasagomes: hi | 09:09 |
| lucasagomes | ramineni, hi there | 09:13 |
| *** Bertie_ has joined #openstack-ironic | 09:13 | |
| ramineni | lucasagomes: wanted to know if you have ever tested ipxe with pxe_ilo driver? | 09:13 |
| lucasagomes | ramineni, no, the only time I ever tested pxe_ilo was a couple of days ago where they have a problem using pxe_ipmitool with some ilo gen 7 machines | 09:14 |
| lucasagomes | but I don't have access to those anymore :-( | 09:14 |
| ramineni | lucasagomes: oohok :( | 09:14 |
| lucasagomes | ramineni, did you find more problems with it ? | 09:14 |
| lucasagomes | IIRC it was a tftp problem you had before right? | 09:14 |
| ramineni | lucasagomes: i got it wrked by removing the map-file , but im wondering if there is any extra configuration needed for pxe_ilo driver to support ipxe | 09:16 |
| lucasagomes | ramineni, right... What's the problem you're seeing now? | 09:16 |
| mrda | lucasagomes: thanks for answering the nova ironic driver email. | 09:20 |
| lucasagomes | mrda, oh no problem, yeah we perhaps need to elect someone to keep an eye on those bugs | 09:20 |
| lucasagomes | or idk, just call more attention to it | 09:21 |
| mrda | It escaped my attention, f'sure | 09:21 |
| lucasagomes | no worries | 09:22 |
| rameshg87 | lucasagomes: when you get some time, please have a look at comments on https://review.openstack.org/#/c/173214/ | 09:24 |
| rameshg87 | the raid configuration spec | 09:24 |
| lucasagomes | rameshg87, yeah I said I would do that yesterday but I didn't have much time | 09:25 |
| lucasagomes | I will look today fo sure | 09:25 |
| rameshg87 | sure thanks :) | 09:25 |
| *** e0ne has joined #openstack-ironic | 09:25 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove instance_uuid as part of tear_down https://review.openstack.org/180459 | 09:31 |
| ramineni | lucasagomes: it gets booted and it shows ipxe initializing devices .. tftp://10.10.1.11/undionly.kpxe ...ok, but keeps on looping over same and finally stops at 'No more netwrok devcies ' | 09:32 |
| ramineni | lucasagomes: dropping to shell, can we debug something? | 09:32 |
| lucasagomes | ramineni, is neutron configured? | 09:33 |
| lucasagomes | I mean, with the ipxe tag | 09:34 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove instance_uuid as part of tear_down https://review.openstack.org/180459 | 09:34 |
| ramineni | ya , configured according to installation guide | 09:34 |
| * ramineni checks again | 09:34 | |
| lucasagomes | ramineni, right hmmm | 09:36 |
| lucasagomes | odd... apparently it's not getting the dhcp option 175 from the undionly.kpxe and keep chainbooting all the time | 09:36 |
| ramineni | lucasagomes: my bad , i think i removed while debugging for something yesterday, added now, restarted deploy again | 09:38 |
| ramineni | lucasagomes: thanks | 09:38 |
| lucasagomes | ramineni, ack... you gotta restart the neutron dhcp agent too | 09:38 |
| lucasagomes | so the config gets applied | 09:38 |
| ramineni | lucasagomes: yes, did that :) | 09:39 |
| lucasagomes | ok | 09:39 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-python-agent: Use mkdtemp for mounting virtual media device https://review.openstack.org/180001 | 09:39 |
| *** cui has quit IRC | 09:48 | |
| *** athomas has quit IRC | 09:48 | |
| *** Marga_ has quit IRC | 09:50 | |
| *** athomas has joined #openstack-ironic | 09:53 | |
| ramineni | lucasagomes: Booted succesfully :) | 09:55 |
| ramineni | lucasagomes, Haomeng: thanks for the help :) | 09:56 |
| *** romcheg has joined #openstack-ironic | 09:57 | |
| *** e0ne is now known as e0ne_ | 09:59 | |
| lucasagomes | ramineni, yay! awesome | 10:00 |
| lucasagomes | ramineni, so the guide is correct right? | 10:00 |
| lucasagomes | (just making sure) | 10:00 |
| ramineni | lucasagomes: :) , yes, but unfortumately specifying map-file in tftp config didnt wrk for me .. but could be my env issue | 10:02 |
| ramineni | lucasagomes: but , yes, install-guide has every detail of configuration :) | 10:02 |
| lucasagomes | ramineni, right... hmm odd I dunno much about tftp and mapping to actually be able to point what would cause that problem you saw | 10:03 |
| lucasagomes | but yeah, thanks for confirming the guide works | 10:03 |
| ramineni | lucasagomes: :) | 10:04 |
| *** e0ne_ is now known as e0ne | 10:05 | |
| ramineni | lucasagomes: btw, could you please have a look at https://review.openstack.org/#/c/178925/3, when you get some time | 10:05 |
| lucasagomes | ramineni, sure | 10:05 |
| ramineni | lucasagomes: gr8, tnx | 10:06 |
| *** zhenguo has quit IRC | 10:07 | |
| rameshg87 | lucasagomes: hi | 10:19 |
| rameshg87 | lucasagomes: regarding https://review.openstack.org/#/c/178229/8/ironic/drivers/modules/pxe_config.template | 10:19 |
| rameshg87 | lucasagomes: do you think it's good if we merged pxe_options.kernel_cmdline and pxe_options.pxe_append_params | 10:19 |
| rameshg87 | lucasagomes: i mean overlayed pxe_options.kernel_cmdline on top of pxe_options.pxe_append_params, so that any overrides over normal pxe_append_params may also be specified | 10:20 |
| rameshg87 | lucasagomes: without that if someone specified same option on both pxe_options.kernel_cmdline and pxe_options.pxe_append_params, 2 options would end up finally for the kernel (which i don't know which one the linux kernel will pick up) | 10:20 |
| lucasagomes | rameshg87, hi hmm we could do that yes | 10:33 |
| rameshg87 | lucasagomes: and if we do that we might get one advantage | 10:34 |
| rameshg87 | lucasagomes: currently the use case of kernel_cmdline is limited to only when you have a "root=" argument | 10:34 |
| rameshg87 | lucasagomes: i.e only if i have an alternate mechanism for root | 10:34 |
| lucasagomes | rameshg87, well not only, if u use ostree you also need to specify it along with the root | 10:34 |
| rameshg87 | lucasagomes: but i could have use-cases to use kernel_cmdline other than "root=", right ? | 10:35 |
| rameshg87 | lucasagomes: like if the image needs foo=bar to be added to kernel command line (where foo != root) | 10:35 |
| lucasagomes | rameshg87, yes, sure... whatever the image itself requires for booting | 10:36 |
| rameshg87 | lucasagomes: in those cases, if i don't have "root=" in kernel_cmdline, ironic will not populate "root=UUID=xxxx" for me | 10:36 |
| rameshg87 | lucasagomes: hence my booting will fail, right ? | 10:36 |
| rameshg87 | lucasagomes: so i think we should skip the block uuid stuff only if we have "root=" in the kernel_cmdline. wdyt ? | 10:37 |
| rameshg87 | i.e only if an alternate mechanism for finding root was specified in the kernel_cmdline | 10:38 |
| lucasagomes | rameshg87, right I don't know if root is actually mandatory | 10:38 |
| lucasagomes | but for the use cases I have at least it is | 10:38 |
| *** coolsvap is now known as coolsvap|afk | 10:38 | |
| * lucasagomes research | 10:39 | |
| * rameshg87 has seen root= in linux kernel argument from day 1 | 10:39 | |
| rameshg87 | lucasagomes: unless rootfstype=ramfs i guess :) | 10:39 |
| lucasagomes | rameshg87, there are many things going on in the booting world | 10:41 |
| lucasagomes | like http://www.freedesktop.org/wiki/Specifications/DiscoverablePartitionsSpec/ | 10:41 |
| lucasagomes | actually a minimal fedora installation (without gui etc) you can boot only having the /usr | 10:42 |
| lucasagomes | no /etc or /var at all | 10:42 |
| lucasagomes | so no fstab anything like that | 10:42 |
| lucasagomes | it's growing pretty crazy :-) this useful if you do things like factory reset on linux systems | 10:42 |
| rameshg87 | lucasagomes: yeah, so /usr is in initrd ? | 10:43 |
| lucasagomes | yeah all the OS binaries are under /usr | 10:43 |
| rameshg87 | lucasagomes: but since we have deployed the image, we will need to boot from that, no ? | 10:43 |
| lucasagomes | sure | 10:43 |
| lucasagomes | I just trying to find out whether root is actually mandatory | 10:43 |
| lucasagomes | IIRC you can omit it | 10:43 |
| lucasagomes | so that's why I don't want to have ironic to be overcomplex and rely on having a specific kernel cmdline | 10:44 |
| lucasagomes | some images may not actually require it | 10:44 |
| rameshg87 | lucasagomes: hmm.. okay | 10:45 |
| rameshg87 | lucasagomes: my only concern was we wouldn't be able to use kernel_cmdline if i had *only* non-root related stuff in kernel_cmdline | 10:45 |
| lucasagomes | rameshg87, but then you can include the root= in the kernel_cmdline too no? | 10:46 |
| lucasagomes | I mean the root= parameter is part of the image, even when Ironic deploys it it looks at the image filesystem UUID to set root | 10:46 |
| rameshg87 | lucasagomes: yeah but only if i had a label, right ? | 10:46 |
| rameshg87 | lucasagomes: oh find out root uuid myself and then do | 10:46 |
| rameshg87 | ? | 10:46 |
| lucasagomes | yeah | 10:47 |
| rameshg87 | oh yeah, that can be done :) | 10:47 |
| lucasagomes | it's a blkid command.. if u know the rest of the parameters need to boot that specific images | 10:47 |
| lucasagomes | I don't see why u can't get the fs uuid | 10:47 |
| rameshg87 | uncompress the image, run blkid and then get it, right ? | 10:47 |
| lucasagomes | yup | 10:47 |
| rameshg87 | okay. make sense. thanks :) | 10:47 |
| lucasagomes | or if it's raw, just mount it and blkid it | 10:47 |
| lucasagomes | I mean mount in a loop | 10:47 |
| rameshg87 | yeah | 10:47 |
| *** Bertie_ has quit IRC | 10:48 | |
| rameshg87 | lucasagomes: and what about overlaying pxe_options.kernel_cmdline on top of pxe_append_params | 10:48 |
| rameshg87 | ? | 10:48 |
| rameshg87 | does that make sense to mention overrides ? | 10:49 |
| lucasagomes | rameshg87, btw, in the http://www.freedesktop.org/wiki/Specifications/DiscoverablePartitionsSpec/ see "Doesn't this break multi-boot scenarios" | 10:49 |
| lucasagomes | root is optional | 10:49 |
| rameshg87 | i don't know if there will ba use-case | 10:49 |
| lucasagomes | if u use discoverable partitions | 10:49 |
| lucasagomes | rameshg87, I think it actually does for the pxe_append_params | 10:49 |
| lucasagomes | cause if both are mentioned and pxe_append_params is global. Perhaps the image one should overwrite it | 10:49 |
| lucasagomes | rameshg87, I will add a new patch set merging both | 10:50 |
| *** romcheg has quit IRC | 10:50 | |
| *** romcheg has joined #openstack-ironic | 10:50 | |
| rameshg87 | okay, i am just going through it .. pls give me some 5 mins | 10:50 |
| rameshg87 | will post comments ... | 10:50 |
| lucasagomes | rameshg87, np, thanks for reviewing it! | 10:50 |
| *** Bertie_ has joined #openstack-ironic | 11:00 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Improve root partition size check in deploy_partition_image https://review.openstack.org/179043 | 11:06 |
| openstackgerrit | Merged openstack/python-ironicclient: Disable invalid sort key in list command https://review.openstack.org/176219 | 11:06 |
| * rameshg87 goes home | 11:10 | |
| *** rameshg87 has quit IRC | 11:10 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Delete neutron ports when the node cleaning fails https://review.openstack.org/178570 | 11:25 |
| lucasagomes | omg github is offline | 11:35 |
| lucasagomes | time for lunch then... | 11:36 |
| openstackgerrit | Merged openstack/ironic: Adds docstrings to some functions in ironic/conductor/manager.py https://review.openstack.org/179910 | 11:39 |
| *** saripurigopi has quit IRC | 11:41 | |
| *** romcheg has quit IRC | 11:41 | |
| *** arif-ali has quit IRC | 11:45 | |
| *** romcheg has joined #openstack-ironic | 11:49 | |
| *** dprince has joined #openstack-ironic | 11:51 | |
| *** dlpartain has joined #openstack-ironic | 11:54 | |
| *** alexpilotti has joined #openstack-ironic | 11:57 | |
| *** dlpartain has quit IRC | 12:05 | |
| *** dlpartain1 has joined #openstack-ironic | 12:05 | |
| *** lucasagomes is now known as lucas-hungry | 12:11 | |
| *** dlpartain1 has left #openstack-ironic | 12:21 | |
| *** e0ne is now known as e0ne_ | 12:22 | |
| *** e0ne_ is now known as e0ne | 12:29 | |
| *** afaranha has joined #openstack-ironic | 12:30 | |
| *** jjohnson2 has joined #openstack-ironic | 12:35 | |
| *** rameshg87 has joined #openstack-ironic | 12:36 | |
| rameshg87 | devananda: hi | 12:36 |
| *** dlpartain has joined #openstack-ironic | 12:36 | |
| *** dlpartain has quit IRC | 12:37 | |
| openstackgerrit | Alessandro Pilotti proposed openstack/ironic: Adds OCS Power and Management interfaces https://review.openstack.org/176390 | 12:37 |
| alexpilotti | rameshg87 NobodyCam could you guys please +a again the OCS patch? https://review.openstack.org/#/c/176390/ | 12:38 |
| rameshg87 | alexpilotti: ack | 12:39 |
| alexpilotti | rameshg87: after swearing at gerrit all night I got the additional surprise that solo_log was needed :-) | 12:39 |
| rameshg87 | alexpilotti: i think it was just merged yesterday or something, right ? | 12:40 |
| alexpilotti | the olso_log one? | 12:40 |
| rameshg87 | yeah | 12:41 |
| alexpilotti | yep: https://review.openstack.org/#/c/157602/ | 12:41 |
| alexpilotti | it says 7.16 AM | 12:41 |
| alexpilotti | it’s localized, now is 3.42PM here | 12:42 |
| openstackgerrit | Bruno Cornec proposed openstack/ironic: Specify environment variables needed for a standalone usage https://review.openstack.org/180002 | 12:42 |
| * bcornec Trying to understand why the build is failing now | 12:43 | |
| alexpilotti | so some 8.30 hours ago, while the OCS one was fighting with gerrit and zuul :-) | 12:43 |
| rameshg87 | alexpilotti: yeah :) | 12:44 |
| *** Bertie_ has quit IRC | 12:46 | |
| *** BertieF has joined #openstack-ironic | 12:49 | |
| *** saripurigopi has joined #openstack-ironic | 12:51 | |
| *** openstackgerrit has quit IRC | 12:53 | |
| *** openstackgerrit has joined #openstack-ironic | 12:53 | |
| *** links has quit IRC | 12:54 | |
| jroll | morning all | 13:03 |
| jroll | lucas-hungry: +1 on specs solving problems, -1 on making me do more work :) | 13:03 |
| *** thrash|g0ne is now known as thrash | 13:03 | |
| *** saripurigopi has quit IRC | 13:04 | |
| *** trown|outttypeww is now known as trown | 13:05 | |
| *** lucas-hungry is now known as lucasagomes | 13:06 | |
| lucasagomes | jroll, :-D cool | 13:07 |
| *** edcragg has quit IRC | 13:07 | |
| lucasagomes | jroll, it's all right, you would have to implement a new plugin soon or later right? | 13:07 |
| *** ijw has joined #openstack-ironic | 13:07 | |
| *** kkoski has joined #openstack-ironic | 13:07 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Add new boot interface in Ironic https://review.openstack.org/168698 | 13:07 |
| *** ijw has quit IRC | 13:08 | |
| *** ijw has joined #openstack-ironic | 13:08 | |
| *** edcragg has joined #openstack-ironic | 13:09 | |
| lucasagomes | jroll, and morning :-) | 13:09 |
| jroll | heh morning :) | 13:10 |
| jroll | yeah, that's the goal | 13:10 |
| rameshg87 | anyone feels a new node.zap_info might be good idea ? | 13:10 |
| *** saripurigopi has joined #openstack-ironic | 13:11 | |
| rameshg87 | to store all the input for zapping | 13:11 |
| jroll | lucasagomes: though now I went to get a reference for etcd's lock module and it appears it might be deprecated :/ | 13:12 |
| lucasagomes | jroll, :-/ | 13:12 |
| lucasagomes | jroll, what about tooz? | 13:12 |
| lucasagomes | jroll, https://github.com/openstack/tooz | 13:12 |
| jroll | lucasagomes: yeah, it's probably fine, though I really like etcd's simplicity | 13:12 |
| *** jjohnson2 has quit IRC | 13:12 | |
| jroll | have you read up on raft at all? | 13:12 |
| lucasagomes | jroll, fair enuff, I dunno much etcd or even tooz to weight in | 13:13 |
| lucasagomes | jroll, nop | 13:13 |
| jroll | it's a consensus protocol built for understandability, the whitepaper is a good read if you have an hour or so | 13:13 |
| lucasagomes | oh nice, I will take a look | 13:13 |
| jroll | and etcd 1) implements the raft protocol, and 2) is just a single binary to deploy | 13:13 |
| lucasagomes | https://ramcloud.stanford.edu/wiki/download/attachments/11370504/raft.pdf ? | 13:14 |
| jroll | here's a good start, also links to the paper https://raftconsensus.github.io/ | 13:14 |
| jroll | yep | 13:14 |
| lucasagomes | jroll, cool, will def take a look | 13:14 |
| lucasagomes | thanks! | 13:14 |
| jroll | np :) | 13:14 |
| *** e0ne is now known as e0ne_ | 13:19 | |
| *** e0ne_ is now known as e0ne | 13:21 | |
| *** rameshg87 is now known as rameshg87-away | 13:23 | |
| *** spandhe has quit IRC | 13:27 | |
| *** raildo_away is now known as raildo | 13:28 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on https://review.openstack.org/177642 | 13:33 |
| *** BadCub_Hiding is now known as BadCub | 13:37 | |
| BadCub | morning folks! | 13:37 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: WIP: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 13:37 |
| *** openstackgerrit has quit IRC | 13:38 | |
| *** openstackgerrit has joined #openstack-ironic | 13:38 | |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers https://review.openstack.org/159734 | 13:39 |
| *** zz_natorious is now known as natorious | 13:42 | |
| *** lifeless has quit IRC | 13:42 | |
| NobodyCam | good morning Ironic | 13:45 |
| NobodyCam | morning BadCub | 13:45 |
| BadCub | morning NobodyCam | 13:46 |
| openstackgerrit | Kan proposed openstack/python-ironicclient: Disable meaningless sort keys in list command https://review.openstack.org/177578 | 13:51 |
| *** athomas_ has joined #openstack-ironic | 13:51 | |
| *** athomas has quit IRC | 13:52 | |
| *** athomas_ has quit IRC | 13:53 | |
| *** athomas has joined #openstack-ironic | 13:54 | |
| *** rameshg87-away is now known as rameshg87 | 14:00 | |
| rameshg87 | JoshNang: hi | 14:01 |
| NobodyCam | morning rameshg87 | 14:02 |
| rameshg87 | morning NobodyCam | 14:02 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC: automatic RAID configuration https://review.openstack.org/176319 | 14:02 |
| rameshg87 | ifarkas: hi | 14:04 |
| ifarkas | rameshg87, o/ | 14:04 |
| rameshg87 | ifarkas: let me know what you think about comments on https://review.openstack.org/#/c/173214/ | 14:04 |
| ifarkas | rameshg87, ok, I will review it! | 14:05 |
| rameshg87 | ifarkas: rloo raised some concerns on the initial proposal - the main concern is that it is deviating from the *usual* way of how things are done in ironic | 14:05 |
| ifarkas | rameshg87, you mean the driver_internal_info vs driver_info thing? | 14:06 |
| rameshg87 | ifarkas: yeah | 14:06 |
| rameshg87 | ifarkas: mainly if we switch from driver_internal_info to driver_info | 14:06 |
| rameshg87 | ifarkas: and raid being an interface should have a validate() method which validates like all other interfaces in ironic - reading the input information from the db | 14:07 |
| rameshg87 | ifarkas: and a look at the zapping spec - https://github.com/openstack/ironic-specs/blob/master/specs/kilo-archive/implement-zapping-states.rst | 14:07 |
| rameshg87 | ifarkas: it points out that JoshNang had left space for the input for zap tasks (raid configuration) to be stored in the API itself | 14:08 |
| *** rloo has joined #openstack-ironic | 14:09 | |
| rameshg87 | ifarkas: totally it seems like we need to revamp the raid spec again :) | 14:09 |
| *** yuriyz has quit IRC | 14:09 | |
| *** shakamunyi has joined #openstack-ironic | 14:09 | |
| ifarkas | rameshg87, hehe, it will never be accepted :-) | 14:09 |
| ifarkas | rameshg87, as for the the actual place of the data: driver_info isn't really suitable for this. In this case I would go back to one of the very first proposal which is node.extra. BUT: I don't feel the actual place of the information matters that much, so I am fine either way | 14:10 |
| *** lifeless has joined #openstack-ironic | 14:10 | |
| rameshg87 | ifarkas: now i have a question on the drac implementation | 14:10 |
| rameshg87 | ifarkas: so are you still relying on the root vs non-root to be called as two separate zap tasks ? | 14:11 |
| ifarkas | rameshg87, yeah | 14:11 |
| rameshg87 | okay | 14:11 |
| ifarkas | rameshg87, drac has no way to expose root device, so it needs to use discoverd for introspection | 14:11 |
| *** yuriyz has joined #openstack-ironic | 14:11 | |
| rameshg87 | ifarkas: okay | 14:11 |
| *** kan_ has quit IRC | 14:12 | |
| rameshg87 | ifarkas: i am thinking the current raid configuration should be populated in node.properties.raid_config. are you fine with that ? | 14:12 |
| rameshg87 | ifarkas: that seems to be in place that after the raid is applied, it becomes a property of the node | 14:13 |
| ifarkas | rameshg87, yep, I prefer that one to driver_info | 14:13 |
| *** shakamunyi has quit IRC | 14:13 | |
| rameshg87 | ifarkas: okay and what about target_raid_config ? | 14:13 |
| rameshg87 | ifarkas: one option is node.driver_info.target_raid_config | 14:14 |
| ifarkas | rameshg87, I would go for node.properties.target_raid_config | 14:14 |
| ifarkas | rameshg87, but driver_info is also an option | 14:14 |
| *** edcragg has quit IRC | 14:15 | |
| rameshg87 | ifarkas: okay | 14:15 |
| ifarkas | rameshg87, as for validation: the reason why we have separate validate call in ironic because it many cases it's an async call. but not in case of raid configuration where the input is validated against a static schema and could be done synchronously as well | 14:16 |
| *** lazy_prince has quit IRC | 14:16 | |
| ifarkas | rameshg87, so I vote for keeping it as you proposed. I will comment on the patch | 14:16 |
| rameshg87 | ifarkas: yeah but in my opinion it might become confusing for the user | 14:16 |
| rameshg87 | ifarkas: thinking of raid as an interface | 14:17 |
| *** edcragg has joined #openstack-ironic | 14:17 | |
| rameshg87 | ifarkas: some interfaces validate the input on the API call itself | 14:17 |
| rameshg87 | ifarkas: some interfaces accept the input and have a separate API for validation (/nodes/<>/validate) | 14:17 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 14:18 |
| rameshg87 | ifarkas: it becomes confusing right ? | 14:18 |
| ifarkas | rameshg87, yeah, that's right | 14:18 |
| ifarkas | rameshg87, then the raid interface could do both :-) | 14:18 |
| rameshg87 | ifarkas: and some day we might have nova drill down a raid configuration to ironic. who knows :) | 14:19 |
| rameshg87 | ifarkas: yeah and then switch over to node.properties.xxxx | 14:20 |
| ifarkas | rameshg87, +1 | 14:20 |
| rameshg87 | ifarkas: do you still see value in having separate PUT /nodes/<>/raid/config and GET /nodes/<>/raid/config if contents are in node.properties ? | 14:21 |
| rameshg87 | ifarkas: user could do PATCH /nodes/<> and GET /nodes/<>/detail to get these detail. we shouldn't have duplicate api also, right ? | 14:22 |
| ifarkas | rameshg87, not really. the less api call we have the better | 14:22 |
| rameshg87 | ifarkas: yeah, so i will put down those 3 APIs | 14:22 |
| rameshg87 | ifarkas: and switch over to node.properties.xxxx | 14:22 |
| ifarkas | rameshg87, great! thanks! | 14:23 |
| rameshg87 | ifarkas: and have validate() read information from node.properties.target_raid_config and then do validation | 14:23 |
| * rameshg87 goes and updates the spec | 14:23 | |
| * NobodyCam gets ready for a call.. | 14:23 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 14:24 |
| NobodyCam | congratz to discoverd and bifrost! | 14:27 |
| *** e0ne is now known as e0ne_ | 14:27 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 14:28 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Specify environment variables needed for a standalone usage https://review.openstack.org/180002 | 14:28 |
| *** jlvillal has quit IRC | 14:28 | |
| *** jlvillal has joined #openstack-ironic | 14:29 | |
| *** shakamunyi has joined #openstack-ironic | 14:35 | |
| *** Marga_ has joined #openstack-ironic | 14:36 | |
| rloo | hello and good morning to one and all, including NobodyCam, rameshg87, ifarkas :) | 14:36 |
| NobodyCam | morning rloo :) | 14:36 |
| *** shakamunyi has quit IRC | 14:36 | |
| rloo | rameshg87: hi, sorry, I forgot to comment on your RAID spec yesterday, but I see that you and ifarkas have been discussing, so I'll wait til your next revision | 14:36 |
| ifarkas | morning all! | 14:36 |
| *** shakamunyi has joined #openstack-ironic | 14:36 | |
| rloo | NobodyCam: congrats to discoverd and bifrost cuz ... ?? | 14:36 |
| *** e0ne_ is now known as e0ne | 14:37 | |
| *** kkoski has quit IRC | 14:37 | |
| rameshg87 | rloo: about to roll a new version :) | 14:37 |
| rameshg87 | rloo: morning | 14:37 |
| rloo | rameshg87: ok | 14:38 |
| NobodyCam | rloo: https://review.openstack.org/#/c/178067 & https://review.openstack.org/#/c/178068 | 14:39 |
| rloo | NobodyCam: nice! | 14:39 |
| NobodyCam | :) | 14:39 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Add generic RAID configuration spec for liberty https://review.openstack.org/173214 | 14:40 |
| rameshg87 | vow !! | 14:41 |
| *** lazy_prince has joined #openstack-ironic | 14:41 | |
| rameshg87 | rloo: ifarkas: ^^^ | 14:42 |
| ifarkas | rameshg87, thanks, will have a look | 14:42 |
| rloo | rameshg87: now it is my turn, what does 'vow' mean? | 14:43 |
| rameshg87 | rloo: i meant for acceptance of discoverd and bifrost to "openstack/" :) | 14:43 |
| rameshg87 | rloo: that wasn't for my spec anyway :D | 14:44 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 14:44 |
| rloo | rameshg87: ah :) | 14:45 |
| *** kkoski has joined #openstack-ironic | 14:45 | |
| *** ijw has quit IRC | 14:47 | |
| *** Sukhdev has joined #openstack-ironic | 14:53 | |
| *** tiagogomes_ has quit IRC | 15:00 | |
| *** raildo has quit IRC | 15:00 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on https://review.openstack.org/177642 | 15:05 |
| *** natorious is now known as zz_natorious | 15:08 | |
| *** Sukhdev has quit IRC | 15:10 | |
| *** dlpartain has joined #openstack-ironic | 15:11 | |
| *** zz_natorious has quit IRC | 15:21 | |
| openstackgerrit | Merged stackforge/pyghmi: Implement event log retrieval from BMCs https://review.openstack.org/179188 | 15:22 |
| *** dlpartain has left #openstack-ironic | 15:24 | |
| *** r-daneel has joined #openstack-ironic | 15:24 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Improve root partition size check in deploy_partition_image https://review.openstack.org/179043 | 15:26 |
| *** lazy_prince has quit IRC | 15:26 | |
| * rameshg87 leaves for the day | 15:26 | |
| rameshg87 | good night ironic | 15:26 |
| *** rameshg87 has quit IRC | 15:27 | |
| *** lazy_prince has joined #openstack-ironic | 15:27 | |
| NobodyCam | night rameshg87 ... | 15:27 |
| NobodyCam | dang he's quick | 15:27 |
| NobodyCam | lol | 15:27 |
| *** krtaylor has quit IRC | 15:31 | |
| *** zz_natorious has joined #openstack-ironic | 15:32 | |
| *** zz_natorious is now known as natorious | 15:32 | |
| *** gridinv has joined #openstack-ironic | 15:34 | |
| *** openstackgerrit has quit IRC | 15:37 | |
| *** openstackgerrit has joined #openstack-ironic | 15:37 | |
| *** ukalifon1 has quit IRC | 15:40 | |
| *** natorious has quit IRC | 15:40 | |
| *** zz_natorious has joined #openstack-ironic | 15:41 | |
| *** zz_natorious is now known as natorious | 15:41 | |
| openstackgerrit | Kan proposed openstack/python-ironicclient: Disable meaningless sort keys in list command https://review.openstack.org/177578 | 15:43 |
| *** Sukhdev has joined #openstack-ironic | 15:43 | |
| *** lazy_prince has quit IRC | 15:44 | |
| openstackgerrit | Kan proposed openstack/python-ironicclient: Disable meaningless sort keys in list command https://review.openstack.org/177578 | 15:45 |
| *** saripurigopi has quit IRC | 15:47 | |
| *** gridinv has quit IRC | 15:47 | |
| *** gridinv has joined #openstack-ironic | 15:48 | |
| *** 16WAAW5JF has joined #openstack-ironic | 15:48 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 15:51 |
| *** kkoski has quit IRC | 15:53 | |
| *** kkoski has joined #openstack-ironic | 15:53 | |
| *** viktors is now known as viktors|afk | 15:54 | |
| *** 16WAAW5JF is now known as tiagogomes | 15:56 | |
| *** romcheg has quit IRC | 15:57 | |
| *** dlpartain has joined #openstack-ironic | 15:57 | |
| *** dlpartain has quit IRC | 15:58 | |
| *** shakamunyi has quit IRC | 15:58 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/177231 | 16:03 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/176831 | 16:03 |
| *** ifarkas has quit IRC | 16:05 | |
| *** gridinv has quit IRC | 16:08 | |
| *** tab_ has joined #openstack-ironic | 16:20 | |
| NobodyCam | rloo: was 180002 previously approved? | 16:20 |
| rloo | NobodyCam: Maybe I should clarify that. Lucas and I had +2'd it, and i was waiting for jenkins to +A. | 16:22 |
| NobodyCam | ahh ack!!!! | 16:22 |
| *** tab_ has quit IRC | 16:22 | |
| NobodyCam | :-p | 16:22 |
| rloo | NobodyCam: so no, it hadn't been approved. but also, there was no change since then except the commit -- due to bruno trying to figure out how to 'fix' jenkins. | 16:22 |
| NobodyCam | I would have +a'd | 16:22 |
| rloo | i can clarify. will add a comment. and mention that you would have +a'd it :-) | 16:23 |
| NobodyCam | :) | 16:23 |
| NobodyCam | na... its all good | 16:23 |
| NobodyCam | :) | 16:23 |
| rloo | just trying to get these patches done with quicky. | 16:23 |
| NobodyCam | yes ++++++ | 16:23 |
| *** saripurigopi has joined #openstack-ironic | 16:23 | |
| rloo | cuz there seems to be a 'backlog' starting to happen... | 16:24 |
| NobodyCam | ya zuul seems to be getting slower and slower :-p | 16:24 |
| rloo | NobodyCam: there were some odd Jenkins failures yesterday. I think with some other project, that affected ours. Anyway, am hoping it has been fixed. haven't yet seen those Jenkins failures today. | 16:27 |
| *** jcoufal_ has quit IRC | 16:27 | |
| BadCub | zuul was completely down late last night from what I heard | 16:28 |
| NobodyCam | nor I .. but I haven't really looked this morning ... calls and things | 16:28 |
| *** jistr has quit IRC | 16:29 | |
| *** ramineni has quit IRC | 16:33 | |
| *** krtaylor has joined #openstack-ironic | 16:35 | |
| *** saripurigopi has quit IRC | 16:36 | |
| NobodyCam | rloo: do you know if this was the error yesterday? http://logs.openstack.org/34/159734/18/check/gate-ironic-python27/3e735fc/console.html#_2015-05-06_13_51_07_848 | 16:39 |
| *** saripurigopi has joined #openstack-ironic | 16:42 | |
| *** tab_ has joined #openstack-ironic | 16:42 | |
| *** e0ne has quit IRC | 16:43 | |
| rloo | NobodyCam: it had something to do with swift. it could have been that error, I scrolled quickly. I thought some package was missing though. | 16:45 |
| saripurigopi | is something changed pep8 test failed with this error "2015-05-06 13:51:18.989 | ./ironic/drivers/modules/ucs/helper.py:27:1: H302 import only modules.'from ironic.openstack.common import log as logging' does not import a module" | 16:45 |
| rloo | NobodyCam: we just merged a patch that changes our logging, using oslo_log instead. | 16:46 |
| rloo | NobodyCam: that error is complaining about logging. i wonder if someone rebased and didn't update their code. | 16:46 |
| jroll | saripurigopi: ^ I think rloo meant that for you | 16:48 |
| saripurigopi | jroll,rloo: will check that. | 16:48 |
| jroll | saripurigopi: from oslo_log import log as logging | 16:49 |
| saripurigopi | jroll: (y) | 16:49 |
| rloo | jroll, saripurigopi: actually I meant it for NobodyCam, didn't see saripurigopi's comment, but yeah | 16:49 |
| NobodyCam | rloo: yep I have a comments ready to post about the logging errors on that patch | 16:49 |
| jlvillal | Simple review here: https://review.openstack.org/179933 :) | 16:49 |
| jroll | rloo: right on :) | 16:49 |
| *** shakamunyi has joined #openstack-ironic | 16:49 | |
| jroll | jlvillal: bam | 16:49 |
| jlvillal | jroll: Wow! You fast! :) | 16:50 |
| jlvillal | jroll: And thanks | 16:50 |
| jroll | it was easy :P | 16:50 |
| jlvillal | :) | 16:50 |
| NobodyCam | morning jroll jlvillal and saripurigopi | 16:50 |
| jroll | ohai NobodyCam | 16:50 |
| jlvillal | NobodyCam: Thanks, and good morning to you too. | 16:51 |
| saripurigopi | Morning NobodyCam | 16:51 |
| *** spandhe has joined #openstack-ironic | 16:51 | |
| *** dumbhead has joined #openstack-ironic | 16:51 | |
| NobodyCam | :) | 16:52 |
| openstackgerrit | Merged openstack/ironic-specs: Fix minor grammar issues in msft-ocs-power-driver.rst https://review.openstack.org/179933 | 16:52 |
| *** Nisha has joined #openstack-ironic | 16:54 | |
| lucasagomes | folks I will call it a day | 16:54 |
| lucasagomes | havea good night everyone | 16:54 |
| *** lucasagomes is now known as lucas-dinner | 16:54 | |
| NobodyCam | night lucas-dinner | 16:54 |
| jlvillal | NobodyCam: BadCub: did you see email thread about Ironic bugs in Nova? I did a quick parse of the scroll back and didn't see any discussion, but I could easily have missed it. | 16:58 |
| jroll | jlvillal: lucas replied to it | 16:58 |
| *** igordcard_ has quit IRC | 16:59 | |
| jlvillal | jroll: I saw that and then there was a reply back about a liason. | 16:59 |
| *** derekh_ has quit IRC | 17:01 | |
| jlvillal | jroll: NobodyCam: I would be willing to volunteer to be the liason, if desired. But thought I would check first to see if someone else should do it. | 17:02 |
| jroll | ¯\_(ツ)_/¯ | 17:02 |
| jroll | I don't have a strong opinion | 17:02 |
| jroll | I've submitted patches for a few nova bugs around ironic, and they tend to bitrot to death because nobody reviews them | 17:03 |
| jlvillal | :( | 17:03 |
| jroll | I literally pointed two cores at this patch yesterday and got nothing (I know it's a hard review and all but) https://review.openstack.org/#/c/176395/ | 17:03 |
| jlvillal | jroll: Hopefully the new PTL does something about getting more cores. | 17:05 |
| jroll | hopefully. | 17:05 |
| jroll | they just added one | 17:05 |
| *** shakamunyi has quit IRC | 17:05 | |
| *** Nisha has quit IRC | 17:07 | |
| BadCub | jlvillal: no, I missed that :( | 17:08 |
| *** athomas has quit IRC | 17:10 | |
| *** Marga_ has quit IRC | 17:11 | |
| jlvillal | jroll: As a note on that patch. Might want to update the function docstring. It currently says 'injected_files' is unused. I think your patch changes that. | 17:14 |
| jlvillal | Though it also says the same thing for admin_password which is also being used. | 17:15 |
| jroll | this is why I hate docstrings :) | 17:16 |
| *** igordcard_ has joined #openstack-ironic | 17:17 | |
| *** achanda has joined #openstack-ironic | 17:17 | |
| jlvillal | jroll: Any objection to me adding comment. I don't want to derail the patch though... So I'm torn. I could do a follow on patch to fix both based on your current patch? | 17:17 |
| jroll | jlvillal: +1 for a followup patch | 17:17 |
| jlvillal | jroll: Will do | 17:18 |
| jroll | thanks | 17:18 |
| *** tiagogomes has quit IRC | 17:18 | |
| *** shakamunyi has joined #openstack-ironic | 17:20 | |
| jlvillal | jroll: https://review.openstack.org/#/c/180632/ | 17:23 |
| jroll | cool, ty | 17:24 |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers https://review.openstack.org/159734 | 17:25 |
| NobodyCam | jroll: have you seen this yet: https://review.openstack.org/#/c/179962 | 17:30 |
| jroll | NobodyCam: my own patch? yes, I've seen it | 17:30 |
| *** saripurigopi has quit IRC | 17:31 | |
| NobodyCam | lol the pep8 errors? | 17:31 |
| jroll | yeah | 17:32 |
| jroll | I saw it | 17:32 |
| jroll | so many things to do :P | 17:32 |
| NobodyCam | :) okay just checking :) | 17:32 |
| *** pelix has quit IRC | 17:38 | |
| *** alexpilotti has quit IRC | 17:41 | |
| *** Sukhdev has quit IRC | 17:43 | |
| *** Marga_ has joined #openstack-ironic | 17:46 | |
| *** Sukhdev has joined #openstack-ironic | 17:47 | |
| rloo | hey, NobodyCam, jroll, jlvillal, anyone else. I was just looking at a spec that mentions changes to REST API. We don't have any section to capture changes to python-ironicclient. What do you think about eg changing 'REST API impact' to 'REST API & CLI impact'? | 17:51 |
| *** e0ne has joined #openstack-ironic | 17:51 | |
| rloo | or a separate 'CLI impact' section. | 17:51 |
| JayF | I would call it "client impact" in a separate section if we were to do it | 17:51 |
| jroll | rloo: I think CLI is noted in the user impact section or something | 17:51 |
| rloo | jroll: yeah, i just looked at the descriptions in the template. So I don't think it is visible enough. | 17:53 |
| *** alexpilotti has joined #openstack-ironic | 17:53 | |
| rloo | jroll: Other end user impact mentions the clieht and horizon or other projects. | 17:53 |
| jroll | rloo: sure, just pointing out where it is today | 17:54 |
| rloo | jroll: yeah. would like it more explicit though, as eg a checklist to know if the spec is done/coded/completed. | 17:54 |
| jroll | rloo: sure, I don't have a strong opinion personally | 17:55 |
| rloo | jroll: ok, i'll propose something and wait to get shot down :-) | 17:55 |
| jroll | heh | 17:56 |
| *** saripurigopi has joined #openstack-ironic | 18:01 | |
| NobodyCam | rloo: ya I think that would be a good change | 18:01 |
| rloo | NobodyCam: ok. I don't know anything about the openstackclient stuff -- guess we can add that later if we need/want. | 18:03 |
| openstackgerrit | Merged openstack/ironic: Specify environment variables needed for a standalone usage https://review.openstack.org/180002 | 18:07 |
| *** Nisha has joined #openstack-ironic | 18:09 | |
| openstackgerrit | Merged openstack/ironic: Run tests in py34 environment https://review.openstack.org/156192 | 18:09 |
| *** Marga_ has quit IRC | 18:10 | |
| *** Marga_ has joined #openstack-ironic | 18:11 | |
| *** Sukhdev has quit IRC | 18:14 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Add a 'Client (CLI) impact' section to the spec https://review.openstack.org/180662 | 18:14 |
| *** e0ne_ has joined #openstack-ironic | 18:15 | |
| *** e0ne has quit IRC | 18:15 | |
| *** e0ne has joined #openstack-ironic | 18:17 | |
| *** e0ne_ has quit IRC | 18:20 | |
| jlvillal | Should the gate now turn on testing of 'py34' for ironic? If so, how does that get done? | 18:27 |
| *** mtanino has joined #openstack-ironic | 18:32 | |
| NobodyCam | any IPA folk have a free minute to take a glance at https://review.openstack.org/#/c/173218 ... has two +2's | 18:33 |
| *** romcheg has joined #openstack-ironic | 18:33 | |
| rloo | yeah, bcornec: finally, 180002 merged. Now on to bigger and better things :-) | 18:35 |
| *** Marga_ has quit IRC | 18:35 | |
| *** Marga_ has joined #openstack-ironic | 18:36 | |
| *** spandhe has quit IRC | 18:47 | |
| *** openstackgerrit has quit IRC | 18:47 | |
| *** lsmola has quit IRC | 18:47 | |
| *** clett has quit IRC | 18:47 | |
| *** mrda has quit IRC | 18:48 | |
| *** rsFF has quit IRC | 18:48 | |
| *** e0ne has quit IRC | 18:48 | |
| *** Nisha has quit IRC | 18:48 | |
| *** morganfainberg has quit IRC | 18:48 | |
| *** vipul has quit IRC | 18:48 | |
| *** devananda has quit IRC | 18:48 | |
| *** kkoski has quit IRC | 18:48 | |
| *** early has quit IRC | 18:48 | |
| *** raginbajin has quit IRC | 18:48 | |
| *** e0ne has joined #openstack-ironic | 18:49 | |
| *** Nisha has joined #openstack-ironic | 18:49 | |
| *** kkoski has joined #openstack-ironic | 18:49 | |
| *** early has joined #openstack-ironic | 18:49 | |
| *** devananda has joined #openstack-ironic | 18:49 | |
| *** vipul has joined #openstack-ironic | 18:49 | |
| *** morganfainberg has joined #openstack-ironic | 18:49 | |
| *** raginbajin has joined #openstack-ironic | 18:49 | |
| *** spandhe has joined #openstack-ironic | 18:49 | |
| *** openstackgerrit has joined #openstack-ironic | 18:49 | |
| *** lsmola has joined #openstack-ironic | 18:49 | |
| *** clett has joined #openstack-ironic | 18:49 | |
| *** mrda has joined #openstack-ironic | 18:49 | |
| *** rsFF has joined #openstack-ironic | 18:49 | |
| openstackgerrit | Merged openstack/ironic: Update config generator to use oslo released libs https://review.openstack.org/180046 | 18:50 |
| krotscheck | I'm trying to figure out the correct place in the ironic docs to put configuration options that are optional. Any suggestions? | 19:01 |
| *** Nisha has quit IRC | 19:01 | |
| mariojv | anyone know why sphinx is globally version-capped below 1.3? need to upgrade for https://review.openstack.org/#/c/180339/1 | 19:03 |
| openstackgerrit | Merged openstack/ironic: Adds OCS Power and Management interfaces https://review.openstack.org/176390 | 19:03 |
| mariojv | this is the line that i'd need to remove the version cap: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L249 | 19:03 |
| *** Sukhdev has joined #openstack-ironic | 19:03 | |
| mariojv | someone in #openstack-dev said it broke all man pages, but i wanted to verify with you all to see if that's still true. building docs and testing them now | 19:04 |
| jroll | mariojv: clarkb knows all :) | 19:07 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added CORS support middleware to Ironic. https://review.openstack.org/180680 | 19:07 |
| jroll | probably not true, but he certainly would know about that better than us | 19:07 |
| *** achanda has quit IRC | 19:07 | |
| mariojv | ty | 19:07 |
| krotscheck | dum de dum | 19:08 |
| krotscheck | Ignore me, just adding weird things to ironic for no reason whatsoever. | 19:08 |
| krotscheck | 0:) | 19:08 |
| NobodyCam | lol | 19:09 |
| jroll | krotscheck: :P | 19:09 |
| NobodyCam | morning krotscheck | 19:09 |
| krotscheck | Hey hey | 19:09 |
| jroll | was there ever a global decision thing about CORS? | 19:09 |
| jroll | I know some people were debating on the ML | 19:09 |
| krotscheck | jroll: The global spec patch is linked. It's still pending. | 19:09 |
| *** natorious is now known as zz_natorious | 19:09 | |
| *** BertieF has quit IRC | 19:09 | |
| krotscheck | jroll: There was an effort about 4 months ago that was abandoned. This is a new one. | 19:09 |
| jroll | krotscheck: aha, I see. | 19:10 |
| jroll | this is scary to me :/ | 19:10 |
| jroll | or at least allow: * | 19:10 |
| *** zz_natorious is now known as natorious | 19:10 | |
| krotscheck | jroll: Yep. This particular middleware puts that decision in the hands of the operators. | 19:11 |
| krotscheck | jroll: With global default values, individual domain overrides, multiple domain support, etc etc etc. | 19:11 |
| krotscheck | jroll: The configuration options are actually pretty fine-grained. | 19:11 |
| jroll | krotscheck: yeah, I just know lots of people will allow * without realizing it | 19:11 |
| *** ndipanov has quit IRC | 19:11 | |
| krotscheck | And the default is Nope, sorry. | 19:11 |
| openstackgerrit | Doug Hellmann proposed openstack/python-ironicclient: Drop use of 'oslo' namespace package https://review.openstack.org/180683 | 19:12 |
| jroll | right, seems mostly fine :) | 19:12 |
| krotscheck | jroll: Incidentally, do you have any suggestions on where in OpenStack's docs any additional security details should live? I've got no love on the mailing list. | 19:13 |
| jroll | krotscheck: no clue | 19:13 |
| jroll | sorry :( | 19:13 |
| krotscheck | jroll: Oh well. | 19:13 |
| jroll | I'd bug anne gentle or the docs channel in general | 19:13 |
| krotscheck | jroll: Good idea. | 19:13 |
| krotscheck | I already have a response from anne. | 19:13 |
| jroll | oh, cool | 19:13 |
| krotscheck | She wanted more specifics on where the security should live. | 19:14 |
| *** e0ne has quit IRC | 19:18 | |
| *** morganfainberg has quit IRC | 19:18 | |
| *** vipul has quit IRC | 19:18 | |
| *** devananda has quit IRC | 19:18 | |
| *** e0ne has joined #openstack-ironic | 19:19 | |
| *** devananda has joined #openstack-ironic | 19:19 | |
| *** vipul has joined #openstack-ironic | 19:19 | |
| *** morganfainberg has joined #openstack-ironic | 19:19 | |
| *** morganfainberg has quit IRC | 19:20 | |
| *** morganfainberg has joined #openstack-ironic | 19:21 | |
| *** Marga_ has quit IRC | 19:24 | |
| *** absubram has joined #openstack-ironic | 19:25 | |
| NobodyCam | yuriyz: if your around... I just added a comment to 177242, let me know what you think | 19:28 |
| *** coolsvap|afk has quit IRC | 19:28 | |
| NobodyCam | brb | 19:30 |
| *** coolsvap|afk has joined #openstack-ironic | 19:31 | |
| *** kkoski has quit IRC | 19:32 | |
| *** lucas-dinner has quit IRC | 19:42 | |
| *** BertieF has joined #openstack-ironic | 19:46 | |
| *** kkoski has joined #openstack-ironic | 19:46 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Rename description field and split data out https://review.openstack.org/180706 | 19:46 |
| rloo | hi NobodyCam. what's the difference between putting something in the agenda, in or outside of 'Open Discussion' section? | 19:52 |
| openstackgerrit | Merged stackforge/pyghmi: Rename description field and split data out https://review.openstack.org/180706 | 19:58 |
| NobodyCam | rloo: not sure what "outside" of open discussion means. let me open the agenda | 19:58 |
| jroll | rloo: AIUI open discussion isn't guaranteed to happen, time-wise | 19:59 |
| devananda | morning, all | 19:59 |
| NobodyCam | good morning devananda | 20:00 |
| NobodyCam | how things down under | 20:00 |
| jroll | ohai | 20:00 |
| rloo | NobodyCam: I just updated the agenda -- feel free to move/delete what I just added. | 20:00 |
| rloo | morning devananda | 20:00 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Decrease severity of 'Disabled' presence https://review.openstack.org/180717 | 20:00 |
| devananda | NobodyCam: warm | 20:01 |
| rloo | jroll: thx. | 20:01 |
| NobodyCam | rloo: those look like valid discussion items to /me | 20:02 |
| NobodyCam | :) | 20:02 |
| rloo | NobodyCam: you won't have to mention that the agenda is light :-) | 20:02 |
| NobodyCam | hehehe :) there is always summit planning to fill the time! | 20:03 |
| *** igordcard has joined #openstack-ironic | 20:03 | |
| *** absubram has quit IRC | 20:05 | |
| *** Haomeng|2 has joined #openstack-ironic | 20:07 | |
| openstackgerrit | Merged stackforge/pyghmi: Decrease severity of 'Disabled' presence https://review.openstack.org/180717 | 20:08 |
| *** Haomeng has quit IRC | 20:10 | |
| *** jcoufal has joined #openstack-ironic | 20:11 | |
| *** jcoufal has quit IRC | 20:11 | |
| *** absubram has joined #openstack-ironic | 20:15 | |
| *** Marga_ has joined #openstack-ironic | 20:25 | |
| openstackgerrit | Doug Hellmann proposed openstack/ironic: Drop use of 'oslo' namespace package https://review.openstack.org/178356 | 20:25 |
| *** hemnafk is now known as hemna | 20:26 | |
| *** e0ne has quit IRC | 20:26 | |
| *** Marga_ has quit IRC | 20:30 | |
| *** e0ne has joined #openstack-ironic | 20:34 | |
| *** achanda has joined #openstack-ironic | 20:38 | |
| *** natorious is now known as zz_natorious | 20:43 | |
| *** zz_natorious is now known as natorious | 20:45 | |
| *** achanda has quit IRC | 20:45 | |
| *** e0ne has quit IRC | 20:45 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Fix time correction code https://review.openstack.org/180737 | 20:46 |
| *** Marga_ has joined #openstack-ironic | 20:46 | |
| *** Marga_ has quit IRC | 20:52 | |
| openstackgerrit | Merged stackforge/pyghmi: Fix time correction code https://review.openstack.org/180737 | 20:52 |
| *** Marga_ has joined #openstack-ironic | 20:57 | |
| *** arif-ali has joined #openstack-ironic | 21:00 | |
| *** absubram has quit IRC | 21:01 | |
| *** Sukhdev has quit IRC | 21:03 | |
| *** trown is now known as trown|outttypeww | 21:04 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added CORS support middleware to Ironic https://review.openstack.org/180680 | 21:05 |
| *** BertieF has quit IRC | 21:12 | |
| *** maurosr has quit IRC | 21:21 | |
| *** jgrimm_ has quit IRC | 21:22 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/177231 | 21:22 |
| *** maurosr has joined #openstack-ironic | 21:22 | |
| *** jgrimm has joined #openstack-ironic | 21:23 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Fixes some docstring warnings https://review.openstack.org/180339 | 21:25 |
| rloo | jroll: I saw a comment from lucas about adding you as a reviewer for this, I wasn't able to add you: https://review.openstack.org/#/c/178925/ | 21:27 |
| *** david-lyle has quit IRC | 21:29 | |
| jroll | rloo: oh, my gerrit ID is really broken :( | 21:30 |
| * jroll looks | 21:30 | |
| * rloo thinks jroll broke it on purpose so he can't be singled out | 21:30 | |
| NobodyCam | brb | 21:31 |
| jroll | heh | 21:31 |
| jroll | I did not! | 21:31 |
| * BadCub 's gerrit access has been in an unexplained denial for quite some time | 21:32 | |
| *** DCWillia_ has joined #openstack-ironic | 21:37 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix Python 3.4 test failure https://review.openstack.org/180763 | 21:41 |
| jlvillal | rloo, jroll or anyone else. I just submitted a patch that makes 'tox -epy34' testing work. What steps can we take so that py34 is added to the gate? | 21:42 |
| rloo | jlvillal: I thought we had a patch for py34 added to the gate? | 21:43 |
| jlvillal | rloo: Oh great if we do. The fix for all the Python 3.4 errors was merged into Ironic. But one error snuck in. So I just pushed a patch for that. | 21:44 |
| jlvillal | rloo: And by error snuck in, I mean a different patch added code that didn't work with Python 3.4 testing. | 21:44 |
| rloo | jlvillal: i thought in that patch for py34 stuff, there was mention of a test. let me see... | 21:44 |
| * devananda gets up and goes for breakfast | 21:44 | |
| jlvillal | rloo: I can do: tox -epy34 | 21:45 |
| jroll | jlvillal: turning on py34 would be a patch to openstack-infra/project-config | 21:45 |
| jlvillal | rloo: If that is what you mean | 21:45 |
| jroll | I bet you can figure it out from there :) | 21:45 |
| devananda | ++ to gating on py34 unit tests as soon as they work | 21:45 |
| jlvillal | jroll: Thanks, let me look and see if anyone has already posted one. | 21:45 |
| jlvillal | If not I will work on adding it. | 21:45 |
| *** achanda has joined #openstack-ironic | 21:46 | |
| rloo | jlvillal: http://logs.openstack.org/92/156192/8/experimental/gate-ironic-python34/78ee894/ | 21:46 |
| *** alexpilotti has quit IRC | 21:46 | |
| jlvillal | rloo: thanks | 21:46 |
| rloo | jlvillal: i think i had a link to the patch somewhere. now where did i put it... | 21:47 |
| jlvillal | rloo: https://review.openstack.org/180599 | 21:48 |
| rloo | jlvillal: yeah, maybe that's it :-) | 21:48 |
| rloo | jlvillal: guess we/I should have run check experimental first | 21:49 |
| jlvillal | rloo: Do need this to land first: https://review.openstack.org/180763 | 21:49 |
| rloo | jlvillal: thought it had already been hooked in. | 21:49 |
| rloo | jlvillal: oh yeah. looking | 21:49 |
| *** achanda has quit IRC | 21:51 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Remove unneeded usage of '# noqa' https://review.openstack.org/180767 | 21:55 |
| *** thrash is now known as thrash|g0ne | 21:56 | |
| *** maurosr has quit IRC | 21:59 | |
| *** achanda has joined #openstack-ironic | 22:01 | |
| *** achanda has quit IRC | 22:02 | |
| *** achanda has joined #openstack-ironic | 22:03 | |
| *** maurosr has joined #openstack-ironic | 22:03 | |
| *** jgrimm has quit IRC | 22:07 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix Python 3.4 test failure https://review.openstack.org/180763 | 22:08 |
| jlvillal | rloo: Thanks for the review. | 22:08 |
| *** jgrimm has joined #openstack-ironic | 22:09 | |
| *** maurosr has quit IRC | 22:12 | |
| *** maurosr has joined #openstack-ironic | 22:14 | |
| *** tab_ has quit IRC | 22:17 | |
| *** dprince has quit IRC | 22:20 | |
| mrda | Morning | 22:25 |
| jlvillal | mrda: Good morning | 22:26 |
| *** DCWillia_ has quit IRC | 22:27 | |
| NobodyCam | morning mrda | 22:27 |
| *** DCWillia_ has joined #openstack-ironic | 22:28 | |
| mrda | o/ | 22:32 |
| *** DCWillia_ has quit IRC | 22:33 | |
| NobodyCam | did we ever figure out who Bob Hansen was | 22:41 |
| jroll | I think he's from cisco | 22:42 |
| jroll | dunno about irc nick though | 22:42 |
| *** DCWillia_ has joined #openstack-ironic | 22:44 | |
| NobodyCam | :) | 22:46 |
| *** DCWillia_ has quit IRC | 22:47 | |
| *** DCWillia_ has joined #openstack-ironic | 22:48 | |
| *** mgoddard1 has joined #openstack-ironic | 22:48 | |
| *** mgoddard has quit IRC | 22:49 | |
| *** DCWilliams_VA has joined #openstack-ironic | 22:51 | |
| *** DCWillia_ has quit IRC | 22:52 | |
| *** DCWilliams_VA has quit IRC | 22:56 | |
| jlvillal | jroll: Bob Hansen is from IBM | 22:56 |
| jlvillal | He emailed me to apologize | 22:56 |
| jroll | oh hm | 22:57 |
| jroll | maybe I'm thinking of something else | 22:57 |
| jroll | apologize for what, I have no context here | 22:57 |
| jlvillal | jroll: Oh his CI program voted -1 on patches. Saying that it failed to merge. | 22:57 |
| jlvillal | Which was not true. | 22:57 |
| jroll | oh | 22:58 |
| jroll | meh | 22:58 |
| jroll | which patches? | 22:58 |
| jroll | ironic? | 22:58 |
| jroll | we don't have any third-party voting CI | 22:58 |
| rloo | jroll: not clear it was for ironic. he affected patches from other projects too. | 23:02 |
| jroll | oh, hm | 23:03 |
| * jroll wonders why people care if it's sorted now | 23:03 | |
| rloo | what's sorted? | 23:03 |
| jroll | the whole broken CI thing? | 23:03 |
| jroll | maybe it isn't sorted, I haven't been paying attention | 23:04 |
| *** maurosr has quit IRC | 23:05 | |
| rloo | dunno, I haven't been paying attention either :) Also -- I don't think Hansen voted -1, I thought I only saw a comment about merge failures from him. | 23:06 |
| *** maurosr has joined #openstack-ironic | 23:08 | |
| *** rloo has quit IRC | 23:09 | |
| jlvillal | jroll: rloo is correct. No voting. Just messages saying: This change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset. | 23:12 |
| jlvillal | Sample here: https://review.openstack.org/#/c/179910/ | 23:13 |
| jroll | yeah, I'm long over it :) | 23:14 |
| jlvillal | :) | 23:15 |
| *** hemna is now known as hemnafk | 23:16 | |
| *** kkoski1 has joined #openstack-ironic | 23:22 | |
| *** kkoski has quit IRC | 23:22 | |
| *** kkoski1 has quit IRC | 23:30 | |
| *** natorious is now known as zz_natorious | 23:42 | |
| openstackgerrit | Merged openstack/python-ironicclient: Drop use of 'oslo' namespace package https://review.openstack.org/180683 | 23:44 |
| *** Marga_ has quit IRC | 23:45 | |
| NobodyCam | lol I just saw that on a patch I was reviewing and did know who he was... :-p | 23:45 |
| *** david-lyle has joined #openstack-ironic | 23:47 | |
| jlvillal | NobodyCam: If you are bored :) https://review.openstack.org/180763 This would enable setting up Python 3.4 as a test on the gate. | 23:49 |
| jlvillal | NobodyCam: Enable setting up means that now 'tox -epy34' works. So they could enable that as a test for the gate as the next step. | 23:50 |
| NobodyCam | jlvillal: patch lgtm. should I look in to the grenade failures before approving? | 23:54 |
| jlvillal | NobodyCam: That was the 'experimental' tests. | 23:54 |
| jlvillal | NobodyCam: I had to manually run those via 'check experimental' to see the Python 3.4 test result. | 23:55 |
| jlvillal | NobodyCam: Normally they don't run, I believe. | 23:55 |
| jlvillal | NobodyCam: So short answer, No :) | 23:55 |
| *** DCWillia_ has joined #openstack-ironic | 23:57 | |
| NobodyCam | I know adam_g was working on them, not sure what state they got to.. /me should look ... but this patch looks good | 23:57 |
| jlvillal | NobodyCam: Thanks. Time to go home to then join a 90 minute meeting :( | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!