| openstackgerrit | Merged openstack/ironic: Update doc for test database migrations https://review.openstack.org/169703 | 00:02 |
|---|---|---|
| openstackgerrit | Merged openstack/ironic: Sync with oslo.incubator https://review.openstack.org/165940 | 00:02 |
| openstackgerrit | Merged openstack/ironic: Set hash seed to 0 in gendocs environment https://review.openstack.org/171360 | 00:05 |
| openstackgerrit | Merged openstack/ironic: Fix bad quoting in quickstart guide https://review.openstack.org/171361 | 00:05 |
| *** mtanino has quit IRC | 00:17 | |
| openstackgerrit | Merged openstack/ironic: Fix VM stuck when deploying with pxe_ssh + local boot https://review.openstack.org/170117 | 00:50 |
| *** Marga_ has quit IRC | 00:53 | |
| *** Sukhdev has quit IRC | 00:58 | |
| *** stendulker has joined #openstack-ironic | 01:01 | |
| *** spandhe has quit IRC | 01:08 | |
| *** meghal has quit IRC | 01:17 | |
| *** meghal has joined #openstack-ironic | 01:20 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Add unit tests for main.py https://review.openstack.org/171454 | 01:25 |
| *** ijw has quit IRC | 01:28 | |
| *** alex_xu has joined #openstack-ironic | 01:37 | |
| *** chenglch has joined #openstack-ironic | 01:37 | |
| *** achanda has quit IRC | 01:38 | |
| *** spandhe has joined #openstack-ironic | 01:54 | |
| *** spandhe_ has joined #openstack-ironic | 01:59 | |
| *** eglute has quit IRC | 02:00 | |
| *** spandhe has quit IRC | 02:01 | |
| *** spandhe_ is now known as spandhe | 02:01 | |
| *** comstud has quit IRC | 02:01 | |
| *** mgagne has quit IRC | 02:02 | |
| *** adam_g has quit IRC | 02:02 | |
| *** rainya has quit IRC | 02:02 | |
| *** Isotopp has quit IRC | 02:02 | |
| *** jroll has quit IRC | 02:02 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add maintenance check before call do_node_deploy https://review.openstack.org/170149 | 02:02 |
| *** meghal has quit IRC | 02:03 | |
| *** Isotopp has joined #openstack-ironic | 02:03 | |
| *** comstud has joined #openstack-ironic | 02:07 | |
| *** jroll has joined #openstack-ironic | 02:07 | |
| *** adam_g has joined #openstack-ironic | 02:07 | |
| *** adam_g has quit IRC | 02:07 | |
| *** adam_g has joined #openstack-ironic | 02:07 | |
| *** eglute has joined #openstack-ironic | 02:07 | |
| *** mgagne has joined #openstack-ironic | 02:08 | |
| *** mgagne is now known as Guest39926 | 02:08 | |
| *** rainya has joined #openstack-ironic | 02:08 | |
| *** jroll has quit IRC | 02:08 | |
| *** jroll has joined #openstack-ironic | 02:08 | |
| *** stendulker has quit IRC | 02:11 | |
| *** harlowja is now known as harlowja_away | 02:22 | |
| *** achanda has joined #openstack-ironic | 02:39 | |
| *** ramineni has joined #openstack-ironic | 02:46 | |
| *** mdbooth has quit IRC | 03:00 | |
| *** kkoski has quit IRC | 03:04 | |
| *** mdbooth has joined #openstack-ironic | 03:05 | |
| *** kkoski has joined #openstack-ironic | 03:10 | |
| *** kkoski has quit IRC | 03:11 | |
| *** alex_xu has quit IRC | 03:20 | |
| *** alex_xu has joined #openstack-ironic | 03:21 | |
| *** yuanying has joined #openstack-ironic | 03:28 | |
| *** kkoski has joined #openstack-ironic | 03:28 | |
| *** stendulker has joined #openstack-ironic | 03:37 | |
| *** achanda has quit IRC | 03:43 | |
| *** coolsvap|afk is now known as coolsvap | 03:48 | |
| *** pradipta has joined #openstack-ironic | 03:55 | |
| ramineni | JoshNang: hi, there? | 04:00 |
| *** sandhya has joined #openstack-ironic | 04:01 | |
| ramineni | jroll: around? | 04:02 |
| *** kkoski has quit IRC | 04:03 | |
| *** naohirot has joined #openstack-ironic | 04:03 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Improve AMT driver doc https://review.openstack.org/169606 | 04:07 |
| *** rwsu has quit IRC | 04:07 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Updated Ironic Microversions Spec to fix minor errors. https://review.openstack.org/171217 | 04:16 |
| *** kalpase has joined #openstack-ironic | 04:18 | |
| *** meghal has joined #openstack-ironic | 04:18 | |
| *** meghal1 has joined #openstack-ironic | 04:25 | |
| *** meghal has quit IRC | 04:27 | |
| *** coolsvap is now known as coolsvap|afk | 04:41 | |
| *** achanda has joined #openstack-ironic | 04:44 | |
| *** Marga_ has joined #openstack-ironic | 04:45 | |
| *** Marga_ has quit IRC | 04:45 | |
| *** Marga_ has joined #openstack-ironic | 04:45 | |
| *** Marga_ has quit IRC | 04:45 | |
| *** Marga_ has joined #openstack-ironic | 04:46 | |
| *** saripurigopi has joined #openstack-ironic | 04:46 | |
| *** achanda has quit IRC | 04:49 | |
| *** coolsvap|afk is now known as coolsvap | 04:53 | |
| *** yuanying has quit IRC | 04:58 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix mocks not being stopped as intended https://review.openstack.org/171464 | 05:00 |
| *** lazy_prince has joined #openstack-ironic | 05:05 | |
| *** romcheg has joined #openstack-ironic | 05:26 | |
| *** achanda has joined #openstack-ironic | 05:26 | |
| *** spandhe has quit IRC | 05:32 | |
| *** ukalifon1 has joined #openstack-ironic | 05:34 | |
| *** spandhe has joined #openstack-ironic | 05:37 | |
| *** spandhe has quit IRC | 05:48 | |
| *** bradjones has quit IRC | 05:52 | |
| bigjools | am I on a hiding to nothing if I try to get federated logins working on icehouse? | 05:54 |
| bigjools | ECHAN | 05:54 |
| *** bradjones has joined #openstack-ironic | 05:57 | |
| *** bradjones has quit IRC | 05:57 | |
| *** bradjones has joined #openstack-ironic | 05:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Transifex https://review.openstack.org/169184 | 06:27 |
| *** rsalevsky has joined #openstack-ironic | 06:34 | |
| *** sandhya has quit IRC | 06:37 | |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: add nic_capacity to capabilities for inspection https://review.openstack.org/171483 | 06:44 |
| *** jcoufal has joined #openstack-ironic | 06:47 | |
| *** romcheg has quit IRC | 06:50 | |
| *** subscope has quit IRC | 07:04 | |
| *** foexle has joined #openstack-ironic | 07:07 | |
| *** athomas has joined #openstack-ironic | 07:15 | |
| *** achanda has quit IRC | 07:15 | |
| *** achanda has joined #openstack-ironic | 07:16 | |
| *** subscope has joined #openstack-ironic | 07:17 | |
| *** rameshg87 has joined #openstack-ironic | 07:19 | |
| *** pas-ha has joined #openstack-ironic | 07:20 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Use RIBCL for reset_ilo https://review.openstack.org/171499 | 07:20 |
| ramineni | rameshg87: ^^ | 07:21 |
| ramineni | stendulker: ^^ | 07:21 |
| stendulker | ramineni: hi | 07:22 |
| *** romcheg has joined #openstack-ironic | 07:26 | |
| *** Marga_ has quit IRC | 07:33 | |
| *** yog__ has joined #openstack-ironic | 07:39 | |
| *** achanda has quit IRC | 07:40 | |
| *** mrda is now known as mrda-away | 07:41 | |
| *** dtantsur|afk is now known as dtantsur | 07:44 | |
| dtantsur | Morning Ironic | 07:45 |
| rameshg87 | ramineni: ack :) | 07:52 |
| rameshg87 | dtantsur: o/ | 07:52 |
| dtantsur | o/ | 07:52 |
| *** rameshg87 is now known as rameshg87-lunch | 07:52 | |
| pshige | dtantsur: morning | 07:55 |
| dtantsur | o/ | 07:55 |
| *** jistr has joined #openstack-ironic | 08:02 | |
| *** lucasagomes has joined #openstack-ironic | 08:05 | |
| *** meghal1 has quit IRC | 08:10 | |
| *** bradjones has quit IRC | 08:14 | |
| *** cppforlife_ has quit IRC | 08:15 | |
| *** cppforlife_ has joined #openstack-ironic | 08:16 | |
| pshige | i can't imagine what tz Ironic people live... :) | 08:18 |
| tiagogomes | NobodyCam, yes, I saw that. It didn't help | 08:19 |
| *** tiagogomes has quit IRC | 08:19 | |
| *** tiagogomes_ has joined #openstack-ironic | 08:19 | |
| *** bradjones has joined #openstack-ironic | 08:20 | |
| *** bradjones has quit IRC | 08:21 | |
| *** bradjones has joined #openstack-ironic | 08:21 | |
| *** mikal has quit IRC | 08:22 | |
| *** mikal has joined #openstack-ironic | 08:23 | |
| lucasagomes | :-) morning all | 08:24 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Use RIBCL for reset_ilo https://review.openstack.org/171499 | 08:24 |
| *** edcragg has joined #openstack-ironic | 08:28 | |
| pshige | lucasagomes: morning | 08:33 |
| dtantsur | lucasagomes, hey o/ | 08:34 |
| *** rsalevsky has quit IRC | 08:35 | |
| lucasagomes | dtantsur, I'm trying to enable ipxe on the instack undercloud | 08:36 |
| lucasagomes | selinux is beating me hard | 08:36 |
| dtantsur | mmmm, selinux... | 08:36 |
| dtantsur | :D | 08:36 |
| dtantsur | lucasagomes, setenforce 0 is not an answer, is it? :D | 08:36 |
| *** oomichi has joined #openstack-ironic | 08:37 | |
| Haomeng | lucasagomes: morning:) | 08:37 |
| *** rsalevsky has joined #openstack-ironic | 08:37 | |
| lucasagomes | dtantsur, hah not really :-( | 08:37 |
| Haomeng | lucasagomes: how about "/usr/sbin/setenforce Permissive", same with setenforce 0? | 08:37 |
| lucasagomes | I wish it was | 08:37 |
| lucasagomes | Haomeng, I have to have it enabled :-) | 08:38 |
| lucasagomes | so I gotta deal with it somehow | 08:38 |
| dtantsur | Haomeng, morning, that won't work for us unfortunately | 08:38 |
| Haomeng | lucasagomes: ok, good luck:) | 08:38 |
| lucasagomes | lol thanks | 08:38 |
| Haomeng | dtantsur: got:) | 08:38 |
| dtantsur | lucasagomes, what's the actual problem? it does not like the HTTP port? | 08:38 |
| lucasagomes | yeah | 08:38 |
| lucasagomes | well, I've added a rule to the http dir I created | 08:38 |
| lucasagomes | then the new port | 08:39 |
| dtantsur | hmm, I remember for SSH there was a special option to add a non-standard port | 08:39 |
| lucasagomes | sudo semanage fcontext -a -t httpd_sys_content_t "/httpboot(/.*)?" | 08:39 |
| lucasagomes | sudo restorecon -Rv /httpboot/ | 08:39 |
| lucasagomes | sudo semanage port -a -t http_port_t -p tcp 8088 | 08:39 |
| dtantsur | aha | 08:39 |
| lucasagomes | 8088 is the port I was trying | 08:39 |
| lucasagomes | anyway, now I don't know what else I gotta do | 08:39 |
| lucasagomes | haha | 08:39 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: README: move out release notes and start troubleshooting guide https://review.openstack.org/171211 | 08:52 |
| *** pelix has joined #openstack-ironic | 08:54 | |
| yog__ | yog__, Hello everyone, while deploying whole disk is it required to install bootloader since its already a bootable disk | 09:07 |
| yog__ | diskimage-builder "deploy-ironic" element seems to do so when boot_option=local | 09:19 |
| lucasagomes | yog__, I think you should not install a bootloader if the fulldisk image + boot_option=local is specified | 09:22 |
| lucasagomes | but the DIB ramdisk seems broken now and will always try to install the bootloader | 09:22 |
| lucasagomes | rameshg87-lunch, and yuriyz where talking about this problem before | 09:22 |
| lucasagomes | were* | 09:23 |
| *** kejiahu has joined #openstack-ironic | 09:25 | |
| yog__ | lucasagomes, was wondering about the same ..thanks for clarifying..guess its knw issue then | 09:29 |
| *** pradipta has quit IRC | 09:30 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Pre-release documentation update https://review.openstack.org/171211 | 09:33 |
| lucasagomes | yog__, yeah... the fix should be simple. Somehow we need to inform the DIB ramdisk that it's a whole disk image | 09:33 |
| lucasagomes | (probably via kernel cmdline) | 09:33 |
| lucasagomes | and then we should just skip the bootloader installation | 09:33 |
| lucasagomes | if I find some time today and nobody does that I will put a fix up for it | 09:33 |
| *** degorenko has quit IRC | 09:38 | |
| *** rameshg87-lunch is now known as rameshg87 | 09:39 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Use RIBCL for reset_ilo https://review.openstack.org/171499 | 09:39 |
| * rameshg87 checks | 09:39 | |
| rameshg87 | yog__: lucasagomes: yes, it's broken now :( | 09:40 |
| lucasagomes | rameshg87, I'm looking at the code | 09:40 |
| lucasagomes | I will put a fix soonish for it | 09:40 |
| lucasagomes | rameshg87, unless ur already working on that | 09:41 |
| rameshg87 | lucasagomes: planning for rc1 ? | 09:41 |
| lucasagomes | yup | 09:41 |
| rameshg87 | lucasagomes: no, i am not working on it | 09:41 |
| rameshg87 | great :) | 09:41 |
| lucasagomes | :-D | 09:41 |
| rameshg87 | lucasagomes: but we need changes in both dib and ironic for it, right ? | 09:41 |
| lucasagomes | right, yeah we have to make ironic pass the info to the ramdisk | 09:41 |
| rameshg87 | okay | 09:41 |
| lucasagomes | and the ramdisk should consume it, and skip the bootloader installation | 09:41 |
| rameshg87 | okay | 09:42 |
| * lucasagomes should open a bug about it, will do | 09:43 | |
| *** mgoddard has quit IRC | 09:45 | |
| rameshg87 | lucasagomes: devananda left a comment in dib status check thing shouldn't have been planned for rc1 :D | 09:47 |
| lucasagomes | but that's because it breaks backward compat no? | 09:47 |
| rameshg87 | lucasagomes: hmm yeah | 09:48 |
| lucasagomes | but the full disk image + local boot is a bug | 09:48 |
| lucasagomes | since before that patch | 09:48 |
| lucasagomes | it was already broken, I mean, I think it should be fixed | 09:48 |
| rameshg87 | yeah, and it can be fixed without breaking backward-compat | 09:48 |
| lucasagomes | it's not backward compat or anything, it's just broken right now | 09:48 |
| lucasagomes | yeah | 09:48 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix type of value in error middleware response header https://review.openstack.org/171544 | 09:51 |
| *** stendulker has quit IRC | 09:52 | |
| *** naohirot has quit IRC | 09:58 | |
| *** mgoddard has joined #openstack-ironic | 09:58 | |
| *** jlvillal has quit IRC | 10:04 | |
| *** jlvillal has joined #openstack-ironic | 10:05 | |
| *** yog__ has quit IRC | 10:10 | |
| ramineni | lucasagomes: hi | 10:21 |
| lucasagomes | ramineni, hi | 10:21 |
| openstackgerrit | Merged openstack/ironic: boot_mode is overwritten in node properties https://review.openstack.org/167952 | 10:21 |
| ramineni | lucasagomes: regarding your comment, discussed the same with JayF yesterday also :) | 10:22 |
| lucasagomes | ramineni, right, that's just the docs, so nothing to do there really | 10:23 |
| lucasagomes | but a comment explaning would be nice :-) | 10:23 |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: add nic_capacity to capabilities for inspection https://review.openstack.org/171483 | 10:23 |
| lucasagomes | (that's why I voted +1, to wait for some clarification) | 10:23 |
| ramineni | lucasagomes: actually , at the time of review, decided to have seperate config options for each driver and not to mix and match to avoid confusion | 10:23 |
| lucasagomes | gotcha | 10:24 |
| ramineni | lucasagomes: and also, if ilo using different mechanism for L release , it will remain the same , without changing config options again | 10:24 |
| ramineni | lucasagomes: that was initial thinking , and it will be easier of operator to enable at one place | 10:26 |
| lucasagomes | ramineni, right, the only confusing part is that, since it's not under [agent] but under [ilo] it seems that clkeaning would work for all ramdisks | 10:26 |
| *** yog__ has joined #openstack-ironic | 10:26 | |
| lucasagomes | but it's only for the agent one | 10:27 |
| lucasagomes | ramineni, anyway, it's all good. Thanks for clarifying | 10:28 |
| ramineni | lucasagomes: right, this option is only for agent .. could have made more clear with documenting teh config option also :( | 10:32 |
| ramineni | lucasagomes: i meant in conf file | 10:32 |
| ramineni | lucasagomes: anyway, thanks .. will add the explanation in comment also :) | 10:34 |
| *** chenglch has quit IRC | 10:35 | |
| lucasagomes | ramineni, yeah, it's ok :-) | 10:35 |
| lucasagomes | thanks! | 10:35 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Bug Fixes https://review.openstack.org/171499 | 10:48 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Bug Fixes https://review.openstack.org/171499 | 10:54 |
| *** Haomeng|2 has joined #openstack-ironic | 10:56 | |
| *** Haomeng has quit IRC | 10:59 | |
| *** ramineni has quit IRC | 11:02 | |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: add nic_capacity to capabilities for inspection https://review.openstack.org/171483 | 11:02 |
| *** yog__ has quit IRC | 11:12 | |
| *** coolsvap is now known as coolsvap|afk | 11:15 | |
| *** degorenko has joined #openstack-ironic | 11:28 | |
| *** yog__ has joined #openstack-ironic | 11:28 | |
| *** dprince has joined #openstack-ironic | 11:29 | |
| openstackgerrit | Merged stackforge/proliantutils: Bug Fixes https://review.openstack.org/171499 | 11:37 |
| *** lucasagomes is now known as lucas-hungry | 12:00 | |
| *** rameshg87 has left #openstack-ironic | 12:08 | |
| saripurigopi | is there a way to do firmware and/or bios upgrade on Ironic nodes? | 12:14 |
| openstackgerrit | Merged openstack/ironic-specs: Updated Ironic Microversions Spec to fix minor errors. https://review.openstack.org/171217 | 12:23 |
| *** trown|outttypeww is now known as trown | 12:23 | |
| dtantsur | saripurigopi, not now at least | 12:24 |
| *** saripurigopi has quit IRC | 12:28 | |
| *** lazy_prince has quit IRC | 12:32 | |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: add nic_capacity to capabilities for inspection https://review.openstack.org/171483 | 12:42 |
| *** athomas has quit IRC | 12:50 | |
| *** rloo has joined #openstack-ironic | 12:56 | |
| *** athomas has joined #openstack-ironic | 12:57 | |
| *** lucas-hungry is now known as lucasagomes | 13:05 | |
| openstackgerrit | Shivanand Tendulker proposed stackforge/proliantutils: add nic_capacity to capabilities for inspection https://review.openstack.org/171483 | 13:07 |
| openstackgerrit | Jim Mankovich proposed openstack/ironic: Add the ability to send vendor specific sensor events, in addition to IPMI sensor events, to ceilometer. This change is for the L release and it is designed to enable support in ironic for sending sensor events to future metering systems. https://review.openstack.org/171612 | 13:07 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Fix node_uuid option is required for port-create https://review.openstack.org/171429 | 13:09 |
| *** kkoski has joined #openstack-ironic | 13:11 | |
| *** jmank has joined #openstack-ironic | 13:12 | |
| *** ndipanov has quit IRC | 13:18 | |
| *** ndipanov has joined #openstack-ironic | 13:18 | |
| *** jmanko has quit IRC | 13:21 | |
| *** jmank has quit IRC | 13:21 | |
| *** jmank has joined #openstack-ironic | 13:21 | |
| *** kkoski has quit IRC | 13:25 | |
| *** kkoski has joined #openstack-ironic | 13:28 | |
| romcheg | Hi guys! | 13:33 |
| romcheg | I remember, NobodyCam told me there is some kind of component on intel-based user-end systems (laptops, pcs, etc) that looks like a tiny bmc and supports some kind of remote commands | 13:35 |
| romcheg | Anyone can tell me the name of that? | 13:35 |
| yuriyz | hello Ironic romcheg | 13:37 |
| romcheg | Hi yuriyz? | 13:37 |
| yuriyz | is there Intel AMT? | 13:37 |
| openstackgerrit | Merged openstack/ironic: tests/db: Add autospec=True to mocks https://review.openstack.org/170911 | 13:38 |
| yuriyz | https://github.com/openstack/ironic/blob/master/doc/source/drivers/amt.rst | 13:39 |
| *** Marga_ has joined #openstack-ironic | 13:39 | |
| *** Marga_ has quit IRC | 13:40 | |
| *** Marga_ has joined #openstack-ironic | 13:40 | |
| *** edcragg has quit IRC | 13:48 | |
| *** jcoufal has quit IRC | 13:53 | |
| *** jistr has quit IRC | 13:54 | |
| *** jistr has joined #openstack-ironic | 13:54 | |
| dtantsur | lucasagomes, could you review discoverd docs update https://review.openstack.org/#/c/171211/ when you have some time please? | 13:57 |
| lucasagomes | dtantsur, will do | 13:57 |
| *** rameshg87 has joined #openstack-ironic | 13:58 | |
| *** saripurigopi has joined #openstack-ironic | 13:58 | |
| *** chlong has quit IRC | 13:58 | |
| *** edcragg has joined #openstack-ironic | 14:00 | |
| *** killer_prince has joined #openstack-ironic | 14:05 | |
| *** oomichi has quit IRC | 14:05 | |
| *** killer_prince is now known as lazy_prince | 14:05 | |
| NobodyCam | good morning from the man maken coffee | 14:07 |
| NobodyCam | morning romcheg yuriyz dtantsur lucasagomes | 14:07 |
| yuriyz | morning NobodyCam | 14:08 |
| NobodyCam | :) | 14:08 |
| lucasagomes | morning NobodyCam | 14:08 |
| NobodyCam | :) | 14:08 |
| *** tiagogomes_ has quit IRC | 14:08 | |
| pshige | morning NobodyCam | 14:08 |
| NobodyCam | morning pshige :) | 14:08 |
| *** tiagogomes_ has joined #openstack-ironic | 14:09 | |
| *** saripurigopi_ has joined #openstack-ironic | 14:10 | |
| *** jistr has quit IRC | 14:10 | |
| *** saripurigopi has quit IRC | 14:12 | |
| *** saripurigopi_ is now known as saripurigopi | 14:12 | |
| *** jistr has joined #openstack-ironic | 14:12 | |
| *** zz_jgrimm is now known as jgrimm | 14:13 | |
| jroll | morning all :) | 14:14 |
| NobodyCam | morning jroll | 14:14 |
| pshige | morning jroll | 14:14 |
| NobodyCam | worth the time it takes to read: http://www.forbes.com/sites/paulmiller/2015/04/08/in-defense-of-openstack/ | 14:17 |
| jroll | the thought of the day was totally worth the time | 14:19 |
| *** mtanino has joined #openstack-ironic | 14:20 | |
| * rameshg87 reads | 14:20 | |
| rameshg87 | morning all | 14:20 |
| NobodyCam | morning rameshg87 | 14:20 |
| rameshg87 | NobodyCam: o/ | 14:20 |
| jroll | NobodyCam: I wish this guy linked to the article he's rebutting | 14:21 |
| jroll | for the record: http://www.businessinsider.com/openstack-vc-investments-are-slowing-2015-4 | 14:22 |
| rloo | hello ironic'ers. Morning jroll, NobodyCam, pshige. Hi yuriyz, lucasagomes, rameshg87 :) | 14:23 |
| jroll | morning rloo :) | 14:23 |
| yuriyz | hi rloo | 14:23 |
| rameshg87 | hello rloo | 14:23 |
| lucasagomes | morning rloo jroll | 14:24 |
| *** Sukhdev has joined #openstack-ironic | 14:24 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Introduce openstackclient plugin https://review.openstack.org/171672 | 14:25 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Update proliantutils version required to '2.1.0' https://review.openstack.org/171673 | 14:26 |
| NobodyCam | morning rloo | 14:27 |
| devananda | morning, all | 14:29 |
| rloo | NobodyCam: interesting article. thx. | 14:29 |
| rloo | morning devananda | 14:29 |
| NobodyCam | m0rning devananda | 14:29 |
| yuriyz | morning devananda | 14:30 |
| rloo | hey NobodyCam, if you feel like +2'ing again: https://review.openstack.org/#/c/171429/ | 14:32 |
| NobodyCam | rloo: sure give me a minute here | 14:32 |
| rloo | no worries NobodyCam, drink your coffee first :-) | 14:33 |
| NobodyCam | you know me all to well lol | 14:33 |
| *** pradipta has joined #openstack-ironic | 14:34 | |
| dtantsur | morning, NobodyCam, jroll, rloo, yuriyz, romcheg, rameshg87 and others :) | 14:35 |
| rloo | hi dtantsur! | 14:35 |
| rameshg87 | dtantsur: o/ | 14:35 |
| romcheg | Hi dtantsur and others! | 14:35 |
| NobodyCam | :) | 14:36 |
| devananda | adam_g: around? | 14:39 |
| NobodyCam | rloo: +2 +a'd | 14:39 |
| rloo | thx NobodyCam | 14:40 |
| rloo | hi romcheg :) | 14:40 |
| romcheg | Good morning rloo :) | 14:40 |
| NobodyCam | oh wow: https://review.openstack.org/#/c/171672 but I'm thinking we should wait for L on this one? thoughts? | 14:41 |
| *** jistr has quit IRC | 14:45 | |
| yuriyz | and can fix for https://bugs.launchpad.net/ironic/+bug/1441014 be landed in the K? | 14:46 |
| openstack | Launchpad bug 1441014 in Ironic "Improper error handling when Ironic application runs under Apache2" [High,In progress] - Assigned to Yuriy Zveryanskyy (yzveryanskyy) | 14:46 |
| *** BadCub_Hiding is now known as BadCub | 14:47 | |
| NobodyCam | morning BadCub :) | 14:47 |
| BadCub | morning folks | 14:47 |
| rloo | NobodyCam: wrt 171672, that's the client and I'm not sure if we're going to have a stable thingy for it. But regardless, I think that's L. | 14:48 |
| NobodyCam | devananda: now that we are releasing a client, is there a freeze on the client code? | 14:49 |
| rloo | yuriyz: is that bug really a high? if so, maybe we should get it in for kilo. | 14:49 |
| NobodyCam | rloo: we are going to start releasing a clent | 14:49 |
| devananda | regarding the client, we need to cut a stable branch of that | 14:49 |
| devananda | I'm going to say that 0.5.x is that stable branch for kilo | 14:50 |
| yuriyz | rloo not sure | 14:50 |
| devananda | but we have some critical bugs in there right now | 14:50 |
| devananda | that we need to fix | 14:50 |
| devananda | all of our stable/juno jobs are broken right now because of the --node_uuid bug | 14:50 |
| rloo | NobodyCam, devananda: ok. Are the critical bugs listed in the etherpad (I haven't looked yet but was going to do that soon) | 14:50 |
| NobodyCam | just approved that patch | 14:50 |
| devananda | dunno, lemme see | 14:50 |
| rloo | devananda: the node_uuid fix got approved | 14:50 |
| *** jistr has joined #openstack-ironic | 14:51 | |
| devananda | great | 14:52 |
| rloo | devananda: https://review.openstack.org/#/c/171334/. It is under 'release candidate critical patches'. What does that mean? We need it for kilo? | 14:54 |
| BadCub | rloo: I listed it under there yesterday because that is kinda impportant patch | 14:55 |
| NobodyCam | any one know tempest cores they could get to look at https://review.openstack.org/#/c/166386 ? | 14:56 |
| devananda | rloo: yes. means it needs to be on the stable/kilo branch of the client, which will get tagged soon too, though we have more leeway if we need to wait until next week for that | 14:56 |
| devananda | NobodyCam: you could ask in #openstack-qa | 14:57 |
| rloo | devananda, BadCub: is 171334 something you want someone to look into? or devananda are you going to work on it? | 14:57 |
| devananda | rloo: I'm looking at mrda's feedback now | 14:57 |
| devananda | BadCub: any bugs / patches on the server that need my attention this morning? | 14:57 |
| BadCub | devananda: was trying to get someone to pick it up from him yesterday when we signed off for the day lol | 14:57 |
| rloo | BadCub, devananda: or should I look/review 168120? That seems to be the only code-related patch for ironic. except for the bug that has no patch yet. | 14:58 |
| BadCub | rloo: that would be awesomeness | 14:58 |
| rloo | BadCub: ok | 14:59 |
| BadCub | rloo: TY :-) | 14:59 |
| rloo | looks like we're pretty close to rc-1 candidate... | 14:59 |
| BadCub | BTW.. Morning too!!!! Looks like everyone got off to a running start | 14:59 |
| TheJulia | yes, good morning | 15:00 |
| NobodyCam | morning TheJulia | 15:00 |
| BadCub | we are close to rc1 :) Should be cutting tonight | 15:00 |
| jlvillal | Good morning Ironic | 15:06 |
| dtantsur | morning jlvillal, BadCub, TheJulia | 15:06 |
| *** Marga_ has quit IRC | 15:06 | |
| BadCub | heya TheJulia jlvillal dtantsur :) | 15:06 |
| NobodyCam | morning jlvillal :) | 15:07 |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: ILO: Add nic_capacity to capabilities https://review.openstack.org/171483 | 15:08 |
| openstackgerrit | Merged openstack/ironic: Add maintenance check before call do_node_deploy https://review.openstack.org/170149 | 15:09 |
| *** jcoufal has joined #openstack-ironic | 15:11 | |
| dtantsur | Wow, my Canada visa was refused >_< | 15:13 |
| BadCub | HUH? | 15:13 |
| NobodyCam | >.< what | 15:13 |
| * BadCub needs to check if anything other than passport is needed | 15:14 | |
| dtantsur | looks like I didn't prove that I have sufficient funds for the trip | 15:15 |
| *** jistr is now known as jistr|mtg | 15:16 | |
| * rameshg87 is still hunting for one and checks bank statements | 15:16 | |
| openstackgerrit | Devananda van der Veen proposed openstack/python-ironicclient: Client should fall back to the lower versions if necessary https://review.openstack.org/171334 | 15:17 |
| openstackgerrit | Merged openstack/ironic: Imported Translations from Transifex https://review.openstack.org/169184 | 15:17 |
| openstackgerrit | Merged openstack/ironic: Imported Translations from Transifex https://review.openstack.org/169184 | 15:17 |
| rloo | dtantsur: :-( Even if it is work related? | 15:18 |
| lucasagomes | dtantsur, :-( seriously? | 15:19 |
| lucasagomes | f* | 15:19 |
| dtantsur | rloo, looks like nobody cares... How should I prove funds if I have a company credit card and some moderate amount of cache?? I don't know... | 15:19 |
| dtantsur | lucasagomes, yep :( | 15:19 |
| dtantsur | our folks are looking into this issue, don't know what they can do | 15:19 |
| *** romcheg has quit IRC | 15:19 | |
| lucasagomes | :-/ | 15:20 |
| JayF | lucasagomes: fwiw I agree with you about the iLo agent priority setting being super confusing, and I would've voted against it had I seen the change :( | 15:20 |
| *** rpodolyaka1 has joined #openstack-ironic | 15:20 | |
| lucasagomes | JayF, right, the reason I didn't -1 that patch with my comment is because that's only the docs | 15:20 |
| lucasagomes | :/ | 15:20 |
| lucasagomes | the option is already added | 15:21 |
| rloo | dtantsur: so sorry. I would think that redhat could provide some 'official' letter or something, but who knows. | 15:21 |
| JayF | lucasagomes: that's a pretty serious layering violation, right? | 15:21 |
| lucasagomes | JayF, that's a pita actually to have 2 config options for the same thing. And it looks a bit misleading if not under [agent] because it looks like cleaning would work for both ramdisks | 15:22 |
| lucasagomes | which in reality it's not true, it works for agent only | 15:22 |
| *** lazy_prince has quit IRC | 15:24 | |
| JayF | lucasagomes: I 100000% agree, to the level I think it's a bug, but honestly I don't use the iLo driver and so was going to pick another battle | 15:25 |
| devananda | dtantsur: !!? :( | 15:25 |
| dtantsur | :( | 15:25 |
| lucasagomes | JayF, yeah, since it's self contained in the iLO driver it's not critical. But a bug would be good indeed | 15:26 |
| dtantsur | having 3rd world passport sucks | 15:26 |
| lucasagomes | JayF, maybe the fix would at least be able to use both configurations and we can remove the custom ilo one after 1 cycle | 15:26 |
| lucasagomes | we can log some warning messages saying it's deprecated or something (if set) | 15:26 |
| *** yog__ has quit IRC | 15:26 | |
| devananda | dtantsur: i would think redhat would help arrange that ... :-/ | 15:26 |
| BadCub | dtantsur: I think your HR folks can contact the local Canadian Consulate and try going through them direct | 15:27 |
| dtantsur | maybe, I'm contacting them now... | 15:28 |
| *** killer_prince has joined #openstack-ironic | 15:29 | |
| *** killer_prince is now known as lazy_prince | 15:29 | |
| *** kalpase has quit IRC | 15:30 | |
| *** dprince has quit IRC | 15:30 | |
| *** jistr|mtg is now known as jistr | 15:30 | |
| *** dprince has joined #openstack-ironic | 15:30 | |
| *** stendulker has joined #openstack-ironic | 15:30 | |
| dtantsur | Oh, something intriguing, openstackclient plugin for Ironic: https://review.openstack.org/#/c/171672/ | 15:31 |
| openstackgerrit | Merged stackforge/proliantutils: ILO: Add nic_capacity to capabilities https://review.openstack.org/171483 | 15:32 |
| NobodyCam | TheJulia: thought on a patch to https://github.com/juliakreger/bifrost/blob/master/roles/ironic-install/templates/ironic.conf.j2#L637 switching to something like if (! testing or cleaning) ??? | 15:32 |
| openstackgerrit | Merged openstack/python-ironicclient: Fix node_uuid option is required for port-create https://review.openstack.org/171429 | 15:32 |
| NobodyCam | brb | 15:33 |
| *** romcheg has joined #openstack-ironic | 15:34 | |
| *** stendulker_ has joined #openstack-ironic | 15:35 | |
| *** lazy_prince has quit IRC | 15:36 | |
| *** rwsu has joined #openstack-ironic | 15:37 | |
| *** stendulker has quit IRC | 15:38 | |
| *** saripurigopi has quit IRC | 15:38 | |
| TheJulia | NobodyCam: I think that would work | 15:38 |
| *** stendulker has joined #openstack-ironic | 15:39 | |
| *** romcheg1 has joined #openstack-ironic | 15:39 | |
| lucasagomes | JayF, man, just found out that the version of ipxe in rhel doesn't understand ${mac:hexraw} :-( | 15:40 |
| devananda | woot - one client fix in. let's get the other in now :) | 15:40 |
| *** romcheg has quit IRC | 15:40 | |
| *** saripurigopi has joined #openstack-ironic | 15:40 | |
| JayF | lucasagomes: I generally use one built on an internal builder | 15:40 |
| JayF | lucasagomes: otherwise we were having trouble getting the serial output to work | 15:40 |
| *** romcheg1 has quit IRC | 15:40 | |
| devananda | >> 171334 << needs reviews to unbreak tripleo and stable/juno tests of ironic | 15:41 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implement the rest of the args for baremetal list https://review.openstack.org/171699 | 15:41 |
| lucasagomes | JayF, :/ it does understand hexhyp as a type | 15:41 |
| lucasagomes | I was wondering if we could change it in Ironic | 15:41 |
| *** stendulker_ has quit IRC | 15:41 | |
| JayF | lucasagomes: I honestly don't know enough about ipxe scripting to know what those two specific things do | 15:42 |
| JayF | lucasagomes: but if you can get a config that works with older and newer, idk why we wouldn't merge it | 15:42 |
| lucasagomes | JayF, yeah, I was looking at hexraw it seems "new" 2013 heh | 15:42 |
| lucasagomes | seems it was first proposed as hexnone but they changed | 15:42 |
| lucasagomes | a mess | 15:42 |
| lucasagomes | hexhyp does work on both, I will put a patch up | 15:42 |
| *** romcheg1 has joined #openstack-ironic | 15:43 | |
| dtantsur | devananda, am I getting it right: in some cases on successful response we output error? if a user requested a version, but server does not support microversions? | 15:43 |
| lucasagomes | when we will cut rc1? | 15:43 |
| devananda | dtantsur: eh? hm, i dont think that's right | 15:44 |
| *** stendulker_ has joined #openstack-ironic | 15:44 | |
| dtantsur | devananda, lines 184-189 | 15:45 |
| dtantsur | (in https://review.openstack.org/#/c/171334/6/ironicclient/common/http.py ) | 15:45 |
| devananda | dtantsur: those are inside if not resp.getheader('X-OpenStack-Ironic-API-Maximum-Version', | 15:45 |
| devananda | dtantsur: if server didn't return any header, and user specifically said "use version X.Y", then yes, it errors | 15:46 |
| dtantsur | devananda, yep, so old server might have fulfilled the request, but we report an error? | 15:46 |
| devananda | ahh | 15:46 |
| devananda | you're correct | 15:46 |
| devananda | that is, however, how we described the behavior in the spec | 15:46 |
| BadCub | lucasagomes: the plan to cut rc1 is tonight, if I am not mistaken | 15:46 |
| devananda | http://specs.openstack.org/openstack/ironic-specs/specs/kilo/api-microversions.html#use-case-3b-new-client-communicating-with-a-old-ironic-user-specified | 15:46 |
| lucasagomes | BadCub, oh... probably I won't get it fixed for rc1 then :/ | 15:47 |
| lucasagomes | will put a patch up anyway | 15:47 |
| dtantsur | devananda, hmm... ok, can we at least change an error message to say "Request was fulfilled, but server does not support this version"? because that's the truth | 15:47 |
| *** stendulker has quit IRC | 15:47 | |
| lucasagomes | BadCub, thank you | 15:47 |
| devananda | sure | 15:47 |
| BadCub | lucasagomes: which patch are you putting up? | 15:47 |
| BadCub | and yw :) | 15:47 |
| *** romcheg1 has quit IRC | 15:47 | |
| devananda | dtantsur: actuallym "request was fulfilled" is inaccurate | 15:48 |
| lucasagomes | BadCub, the one changing the ipxe script to use hexhyp instead of hexraw which is not supported by older versions (rhel ipxe for e.g) | 15:48 |
| devananda | dtantsur: if the user specifically requests v1.6 (say, because they know they want to use cleaning and want to be sur ethe server supports it) | 15:48 |
| devananda | dtantsur: then the request was not fulfilled. the connection was established, but the server can't do what the client asked | 15:48 |
| BadCub | Okay, let me know when you throw it up. :-) | 15:49 |
| *** romcheg has joined #openstack-ironic | 15:49 | |
| devananda | dtantsur: this should only kick in if the user *explicitly* specified a version. if the user didn't, it should just fall back to 1.1 and work | 15:49 |
| dtantsur | devananda, can we check for error code then and change error message based on it? I'm a bit frightened by the possibility that the server e.g. silently ignored the config drive... | 15:49 |
| *** alex_xu has quit IRC | 15:50 | |
| *** alex_xu has joined #openstack-ironic | 15:50 | |
| devananda | heh | 15:51 |
| devananda | yea, well, the client is not yet smart enough to know that --config-drive only works with version > N | 15:51 |
| devananda | I have anothe rpatch which started to do that | 15:51 |
| devananda | but I haven't finished it yet, and it's going to take a lot more work | 15:51 |
| devananda | dtantsur: with 171334, if I don't specify a version but I do specify --config-drive: | 15:52 |
| dtantsur | devananda, I suggest if code >= 400: (current error message) else: (warn user that request might have fulfilled) | 15:52 |
| devananda | (and point at new server) it should work fine | 15:52 |
| devananda | (and point at old server) it should not error, but will ignore the config drive | 15:52 |
| devananda | dtantsur: I think I misunderstand something. where would you put that warning? | 15:53 |
| dtantsur | devananda, alternative error message for line #188 if code < 400 | 15:54 |
| * dtantsur brb | 15:54 | |
| *** killer_prince has joined #openstack-ironic | 15:55 | |
| devananda | dtantsur: OOOH. crap. you're right | 15:55 |
| *** killer_prince is now known as lazy_prince | 15:55 | |
| *** jistr has quit IRC | 15:56 | |
| devananda | dtantsur: what do you think of moving this whole check to an initial "GET /v1/" call that is done before the requested op? | 15:56 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Update proliantutils version required for K release https://review.openstack.org/171673 | 15:56 |
| rameshg87 | dtantsur: ^^^ | 15:56 |
| *** Marga_ has joined #openstack-ironic | 15:57 | |
| *** Marga_ has quit IRC | 16:00 | |
| *** Marga_ has joined #openstack-ironic | 16:00 | |
| *** igordcard has quit IRC | 16:00 | |
| *** igordcard has joined #openstack-ironic | 16:01 | |
| *** absubram has joined #openstack-ironic | 16:02 | |
| devananda | rameshg87: are you working on https://bugs.launchpad.net/ironic/+bug/1439663 ? | 16:02 |
| openstack | Launchpad bug 1439663 in Ironic "localboot with whole disk image using dib ramdisk tries to install bootloader" [High,Triaged] - Assigned to Ramakrishnan G (rameshg87) | 16:02 |
| rameshg87 | devananda: lucasagomes started working on it from today | 16:03 |
| rameshg87 | lucasagomes: ^^^ | 16:03 |
| devananda | there's a different bug assigned to lucas https://bugs.launchpad.net/ironic/+bug/1441556 | 16:04 |
| openstack | Launchpad bug 1441556 in Ironic "Deploying whole disk image with local boot fails" [Medium,Triaged] - Assigned to Lucas Alvares Gomes (lucasagomes) | 16:04 |
| devananda | are these the same thing? | 16:04 |
| rameshg87 | devananda: they are same | 16:04 |
| lucasagomes | Oh there was another bug already open to it? | 16:04 |
| rloo | hi TheJulia: let me know what you think of my comments. I could as easily +A 168120 if you want to followup with a fixup patch. | 16:04 |
| rameshg87 | devananda: i missed to tell lucasagomes that i had opened a bug | 16:04 |
| lucasagomes | devananda, problem is the rc1 is cut today right? | 16:04 |
| devananda | BadCub: ^ | 16:05 |
| rameshg87 | lucasagomes: it was me who filed it earlier, i forgot about it :( | 16:05 |
| BadCub | already on it | 16:05 |
| devananda | :) | 16:05 |
| devananda | lucasagomes: yes, rc1 is cut toda,y, if possible. how much work is this going to be? | 16:05 |
| devananda | oooh. this also requires a change in dib? :( | 16:06 |
| rameshg87 | BadCub: i am just marking 1439663 as a duplicate of 1441556 | 16:06 |
| devananda | ugh, this is going to be a pain, because we will need to version-pin our dependency on a new version of dib | 16:06 |
| lucasagomes | devananda, yes, we need to change DIB | 16:06 |
| lucasagomes | get it merged it there :/ | 16:06 |
| lucasagomes | devananda, :-( | 16:06 |
| lucasagomes | rc2 maybe? | 16:06 |
| devananda | anyone talking with the dib core team yet? | 16:06 |
| BadCub | rameshg87: okay, that works. | 16:07 |
| lucasagomes | I also want to fix the iPXE problem I just found with old versions of ipxe | 16:07 |
| devananda | no? | 16:08 |
| BadCub | rameshg87: Ae you going to work 1441556? | 16:08 |
| NobodyCam | TheJulia: ok another pull req up for bifrost | 16:08 |
| devananda | lucasagomes: is there even a bug open against DIB for this? | 16:08 |
| lucasagomes | devananda, I don't think so | 16:08 |
| lucasagomes | I will tag DIB on that bug | 16:09 |
| rameshg87 | BadCub: lucasagomes has started working on that | 16:09 |
| devananda | lucasagomes: thanks | 16:09 |
| BadCub | rameshg87: okay | 16:09 |
| BadCub | just wnated to check since you were working the dup of it | 16:09 |
| BadCub | wanted even | 16:09 |
| TheJulia | NobodyCam: Merci | 16:09 |
| rameshg87 | okay | 16:09 |
| NobodyCam | :) | 16:10 |
| devananda | lucasagomes: https://etherpad.openstack.org/p/VK3R5xy0MW | 16:10 |
| dtantsur | devananda, makes sense (actually I proposed it some time ago :) | 16:13 |
| lucasagomes | devananda, re backward compat | 16:14 |
| lucasagomes | it was broken before, so it never worked with full disk image + local boot | 16:14 |
| * BadCub runs to hunt breakfast | 16:14 | |
| lucasagomes | even before rameshg87 changes on the DIB ramdisk to have another call to the Ironic API when installing the bootloader | 16:15 |
| *** foexle has quit IRC | 16:15 | |
| *** rloo has quit IRC | 16:19 | |
| *** rloo_ has joined #openstack-ironic | 16:19 | |
| *** saripurigopi has quit IRC | 16:23 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Nit fixes for boot_mode being overwritten https://review.openstack.org/171722 | 16:24 |
| *** Sukhdev has quit IRC | 16:24 | |
| *** Sukhdev has joined #openstack-ironic | 16:25 | |
| *** davideagnello has joined #openstack-ironic | 16:26 | |
| *** ijw has joined #openstack-ironic | 16:27 | |
| *** stendulker has joined #openstack-ironic | 16:28 | |
| *** ijw has quit IRC | 16:29 | |
| *** ijw has joined #openstack-ironic | 16:29 | |
| *** stendulker_ has quit IRC | 16:30 | |
| jroll | BadCub: devananda: what do y'all need help with today? | 16:35 |
| adam_g | devananda, sorry, here now | 16:36 |
| *** stendulker has quit IRC | 16:37 | |
| *** Marga_ has quit IRC | 16:42 | |
| *** Marga_ has joined #openstack-ironic | 16:43 | |
| lucasagomes | jroll, https://bugs.launchpad.net/ironic/+bug/1441556 | 16:45 |
| openstack | Launchpad bug 1441556 in Ironic "Deploying whole disk image with local boot fails" [Medium,Triaged] - Assigned to Lucas Alvares Gomes (lucasagomes) | 16:45 |
| * lucasagomes is currently fixing the ipxe problem | 16:45 | |
| lucasagomes | but needs to fix that too :/ | 16:45 |
| jroll | ew | 16:45 |
| jroll | I'll take a stab at it | 16:46 |
| *** Marga_ has quit IRC | 16:46 | |
| *** Marga_ has joined #openstack-ironic | 16:47 | |
| devananda | jroll: some notes at https://etherpad.openstack.org/p/VK3R5xy0MW | 16:47 |
| devananda | found a way to fix the backwards incompat problem. see hte bottom of hte pad | 16:47 |
| jroll | oh neat | 16:48 |
| jroll | devananda: do you want help with writing patches or what? | 16:49 |
| devananda | jroll: yes pls | 16:49 |
| * jroll has no idea about dib | 16:49 | |
| devananda | lucasagomes: you're doing the ipxe stuff, yes? is there a bug on the RC1 page for that? | 16:49 |
| devananda | nm, i see it now | 16:50 |
| BadCub | devananda: I think bug is 1441710 | 16:50 |
| lucasagomes | devananda, yeah there's | 16:50 |
| devananda | omg the roofers are back ... i need to leave ... | 16:50 |
| lucasagomes | devananda, is it possible to cut the RC maybe tomorrow or friday? | 16:50 |
| NobodyCam | devananda: :( | 16:51 |
| devananda | lucasagomes: checking ... | 16:53 |
| * rameshg87 goes the sleep | 16:53 | |
| rameshg87 | good night folks | 16:53 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add support for 'baremetal show' command https://review.openstack.org/171739 | 16:54 |
| BadCub | g'night rameshg87 | 16:54 |
| *** rameshg87 has quit IRC | 16:54 | |
| NobodyCam | night rmeshg | 16:54 |
| devananda | anyone know if Brad ^ is in IRC? | 16:54 |
| *** absubram has quit IRC | 16:54 | |
| lucasagomes | devananda, I mean we can also have a rc2 right? in the worst case scenario | 16:54 |
| lucasagomes | since we have 3 known problems | 16:54 |
| devananda | lucasagomes: totally. but we shouldn't tag RC if we know there are problems | 16:55 |
| lucasagomes | devananda, yeah that's fair | 16:55 |
| devananda | lucasagomes: because after RC1, any fixes have to go through stable backport process | 16:55 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix exception handling in Glance image service https://review.openstack.org/171740 | 16:55 |
| devananda | so it's much more work fo rus | 16:55 |
| vdrok | hi everyone | 16:55 |
| NobodyCam | mornign vdrok | 16:55 |
| devananda | vdrok: hi! | 16:55 |
| vdrok | morning devananda , NobodyCam | 16:55 |
| lucasagomes | devananda, indeed yeah it would be good if we could postpone at least 1 day more | 16:55 |
| vdrok | this bug also worth attention I think - https://bugs.launchpad.net/ironic/+bug/1441727 | 16:55 |
| openstack | Launchpad bug 1441727 in Ironic "Glance exceptions not handled correctly by ironic's glance image service" [Medium,In progress] - Assigned to Vladyslav Drok (vdrok) | 16:55 |
| vdrok | just found it | 16:56 |
| *** Marga_ has quit IRC | 16:56 | |
| *** Marga_ has joined #openstack-ironic | 16:57 | |
| * BadCub looks at 1441727 | 16:57 | |
| *** spandhe has joined #openstack-ironic | 16:57 | |
| NobodyCam | vdrok: oh wow :/ | 16:59 |
| BadCub | vdrok: I have tagged ^ for rc1. NobodyCam are you reviewing the patch? | 17:00 |
| NobodyCam | I'm looking at it now | 17:01 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix exception handling in Glance image service https://review.openstack.org/171740 | 17:01 |
| BadCub | TY :) | 17:01 |
| *** achanda has joined #openstack-ironic | 17:04 | |
| devananda | lucasagomes: with https://review.openstack.org/#/c/171743/ in place, I still question the upgrade path. How can we check on the Ironic side that the deploy ramdisk will support this // that we should wait for one more callback? | 17:05 |
| *** athomas has quit IRC | 17:07 | |
| lucasagomes | devananda, we can't. Unless we do something else in the ramdisk... old and new versions of the ramdisk does call "pass_deploy_info" in the Ironic api | 17:07 |
| lucasagomes | if we add a flag indicating this is a new version of the ramdisk we can detect that | 17:07 |
| lucasagomes | like passing an extra parameter, if set means it's a new version so we wait for the call back | 17:08 |
| lucasagomes | if not passed we just do as before | 17:08 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Update ilo drivers documentation for inspection https://review.openstack.org/170065 | 17:08 |
| dtantsur | devananda, looks like Brad is from Red Hat, trying to find out his nick... | 17:10 |
| dtantsur | devananda, do you want to discuss that openstackclient plugin? | 17:10 |
| *** trown is now known as trown|lunch | 17:11 | |
| *** meghal has joined #openstack-ironic | 17:11 | |
| BadCub | devananda: , lucasagomes 171743 is tagged to 1439663 which was marked as a dup of 1441556. Which one of those bugs do we want to keep active for tracking? | 17:13 |
| lucasagomes | BadCub, let's keep 1441556 ? | 17:15 |
| lucasagomes | since 1439663 has been marked as duplicated already | 17:16 |
| BadCub | lucasagomes: I think that is best as well. 1439663 has already been closed as a dup | 17:16 |
| BadCub | hehehe | 17:16 |
| lucasagomes | yup | 17:16 |
| devananda | dtantsur: OSC support should probably wait for Liberty -- I was just wondering who that is on IRC, since he's posting patches and not, afaict, here | 17:16 |
| BadCub | wnat me to change the bug ID on teh commit? | 17:16 |
| devananda | lucasagomes: ++ to having newer ramdisk tell ironic "hey, i'm new, you should do something new too" | 17:17 |
| devananda | lucasagomes: otherwise I think we're going to break theupgrade path that some operators will take | 17:17 |
| dtantsur | devananda, I pinged him internally, but he does not answer. What do you mean by "wait for Liberty" wrt client? wait until stable/kilo branch? | 17:17 |
| lucasagomes | devananda, yeah, :-( | 17:17 |
| adam_g | devananda, fyi https://review.openstack.org/#/c/171620/ will start installing ironicclient on stable runs according to versions in that branch's g-r | 17:17 |
| lucasagomes | problem is it's a bit late here already | 17:17 |
| devananda | dtantsur: right. wait until we cut a stable branch of th eclent | 17:17 |
| lucasagomes | not sure if I will have time to fix it all for today :/ | 17:18 |
| * lucasagomes is testing the ipxe change | 17:18 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implement baremetal delete command https://review.openstack.org/171751 | 17:18 |
| devananda | adam_g: yah. awesome. | 17:18 |
| dtantsur | devananda, IIRC out folks want it in K, but I agree that we may want to postpone it upstream | 17:18 |
| lucasagomes | devananda, did you find out if we can tag the rc1 later? | 17:18 |
| *** thrash has joined #openstack-ironic | 17:19 | |
| thrash | devananda: you rang? :) | 17:19 |
| dtantsur | devananda, ^^^ | 17:19 |
| devananda | adam_g: global-requirements.txt:python-ironicclient>=0.2.1,<=0.3.3 | 17:19 |
| devananda | I think that's reasonable for stable/juno | 17:19 |
| adam_g | devananda, yeah | 17:19 |
| BadCub | lucasagomes: I updated the BUG ID on the commit for 171743 | 17:19 |
| devananda | lucasagomes: no response from ttx yet | 17:20 |
| lucasagomes | BadCub, thanks | 17:20 |
| lucasagomes | devananda, ack | 17:20 |
| BadCub | yw :) | 17:20 |
| devananda | thrash: ohhai! saw you posting changes for OSC support and wondered what your IRC nic was. | 17:20 |
| adam_g | devananda, i still think its crazy not to have it in nova's requirements.txt, to ensure consistency between what we test and what users get | 17:20 |
| NobodyCam | BadCub: 171740 +2'd | 17:20 |
| BadCub | NobodyCam: TY :) | 17:21 |
| devananda | thrash: after the RC jumble this week, I'd like to take a look at those changes and chat with you about OSC support, interfaces, etc | 17:21 |
| *** harlowja_away is now known as harlowja | 17:21 | |
| devananda | adam_g: agreed | 17:21 |
| thrash | devananda: sure thing. | 17:21 |
| dtantsur | g'night folks | 17:22 |
| *** dtantsur is now known as dtantsur|afk | 17:22 | |
| BadCub | g'night dtantsur|afk | 17:22 |
| NobodyCam | night dtantsur|afk | 17:22 |
| *** pelix has quit IRC | 17:24 | |
| NobodyCam | lucasagomes: have you testing this dib patch locally? | 17:25 |
| *** edcragg has quit IRC | 17:26 | |
| lucasagomes | NobodyCam, the greghaynes one? Not yet | 17:26 |
| *** Sukhdev has quit IRC | 17:26 | |
| lucasagomes | I'm testing something else :-( | 17:26 |
| * lucasagomes no time really | 17:26 | |
| BadCub | Any Cores available to review: https://review.openstack.org/#/c/171740/ has one +2 already | 17:27 |
| jroll | devananda: fyi, I'm not working on that bug quite yet, but shortly I should be free for most of the rest of the day and can hack on it | 17:27 |
| *** pas-ha has quit IRC | 17:28 | |
| *** Guest39926 is now known as mgagne | 17:29 | |
| *** mgagne has joined #openstack-ironic | 17:29 | |
| *** meghal has quit IRC | 17:29 | |
| *** meghal has joined #openstack-ironic | 17:29 | |
| *** jcoufal has quit IRC | 17:31 | |
| *** meghal has quit IRC | 17:33 | |
| *** meghal has joined #openstack-ironic | 17:34 | |
| devananda | jroll: that'd be very helpful | 17:36 |
| devananda | i'm going to get back to working on the client issues | 17:38 |
| rloo_ | BadCub: I'll take a look at 171740. | 17:38 |
| rloo_ | BadCub: wrt 168120, I can +A it too. It is mostly nits and I would have liked one of them to be addressed so I don't have to look at a followon patch, but I don't want to hold up rc1 cuz of it. so just let me know. | 17:40 |
| *** achanda has quit IRC | 17:40 | |
| * BadCub looks | 17:41 | |
| TheJulia | rloo_: I'm happy to change it, I doubt it would get the reviews today for it to land in rc1 | 17:42 |
| rloo_ | TheJulia: good point. I think Ramesh was the other one that +2 | 17:42 |
| devananda | dtantsur|afk: using --configdrive against older server results in: Unknown argument: "configdrive" (HTTP 400) | 17:43 |
| BadCub | TheJulia: would you be so kind? we can get it into rc1 if rloo_ 's nit is addressed | 17:43 |
| rloo_ | TheJulia: I'll +A. BadCub: if she updates, will need 2 +2's. | 17:43 |
| TheJulia | BadCub: k, going to update it | 17:43 |
| BadCub | rloo_: awesome!! :) You rock!! | 17:43 |
| devananda | dtantsur|afk: has nothing to do with APi version | 17:43 |
| TheJulia | fwiw, I'm going to rip out the _time bit since testing with it seems... near impossible | 17:43 |
| TheJulia | s/testing/reliably testing/ | 17:44 |
| rloo_ | oh, wait. TheJulia, should I postpone the +A then? | 17:44 |
| NobodyCam | devananda: got a quick second for 6 line dib patch? https://review.openstack.org/#/c/171743 | 17:44 |
| TheJulia | I was going to do a follow-up to rip it out as un-necessary if it landed today, if it didn't I was going to change it | 17:44 |
| devananda | NobodyCam: i have not testd that yet. have you? | 17:45 |
| rloo_ | TheJulia: let me land it first, since we want to do the rc-1 cut today and I don't know who else might +2 it after your change. | 17:45 |
| devananda | also, I think jroll was going to look at improving that further | 17:45 |
| BadCub | TheJulia: That is a good plan. Thank you rloo_ :) | 17:45 |
| devananda | making it so that the newer ramdisks actually pass an option to ironic, and a corresponding change in ironic so that it can detect when being used with an older ramdisk and not fail | 17:45 |
| *** rloo_ is now known as rloo | 17:46 | |
| devananda | NobodyCam: IOW, that change in DIB needs to grow slightly, and we need anohter change in ironic's pxe // iscsi driver to handle it properly. | 17:46 |
| * BadCub is in dire need of more coffee to increase brain capacity | 17:46 | |
| NobodyCam | devananda: I think my OoO test env is messed up / need to look into it. not sure if setting up devstack or bifrost took it out :( | 17:46 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add support for 'baremetal show' command https://review.openstack.org/171739 | 17:47 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Introduce openstackclient plugin https://review.openstack.org/171672 | 17:47 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implement the rest of the args for baremetal list https://review.openstack.org/171699 | 17:47 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implement baremetal delete command https://review.openstack.org/171751 | 17:47 |
| devananda | right. so let's NOT approve that dib change without testing it .... | 17:47 |
| * devananda ignores the interesting OSC changes | 17:47 | |
| devananda | thrash: are those a dependent series of patches? | 17:47 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Use hexhyp instead of hexraw iPXE type https://review.openstack.org/171759 | 17:48 |
| lucasagomes | I'm still finishing tests for ^ | 17:48 |
| thrash | devananda: yes | 17:49 |
| BadCub | I moved ^ down on the pad and indicated pending testing | 17:49 |
| BadCub | 171743 that is | 17:49 |
| thrash | devananda: I just did a rebase and amended the first in the line | 17:49 |
| *** thrash is now known as thrash|biab | 17:50 | |
| *** foexle has joined #openstack-ironic | 17:50 | |
| devananda | thrash|biab: cheers. putting a hold on the first patch for now, but pls don't stop :) | 17:51 |
| jroll | devananda: I was going to make what better now? | 17:52 |
| BadCub | devananda: what is the patch # you are putting hold on? I will list it on the L pad | 17:52 |
| devananda | jroll: heh | 17:52 |
| devananda | BadCub: the stack of patches proposed by Brad P. Crochet just now | 17:52 |
| *** achanda has joined #openstack-ironic | 17:53 | |
| BadCub | got em | 17:53 |
| *** ukalifon1 has quit IRC | 17:57 | |
| lucasagomes | 2015-04-08 17:56:59.316 | [ERROR] /opt/stack/devstack/lib/ironic:565 Timed out waiting for Nova hypervisor-stats count >= 1 | 17:58 |
| lucasagomes | :-( | 17:58 |
| * lucasagomes found adam_g fix https://review.openstack.org/#/c/171313 | 17:59 | |
| devananda | lucasagomes: randomquestion - did you see my PTL candidacy letter on openstack-dev? apparently some folks did not get it ... | 17:59 |
| lucasagomes | devananda, I did | 17:59 |
| devananda | k k | 17:59 |
| lucasagomes | found it interesting that you talked about discoverd and so on | 18:00 |
| openstackgerrit | Merged openstack/ironic: Fix mocks not being stopped as intended https://review.openstack.org/171464 | 18:00 |
| lucasagomes | things we added 6 months ago | 18:00 |
| devananda | lucasagomes: discoverd is new in kilo, so ... | 18:00 |
| lucasagomes | yup | 18:00 |
| BadCub | devananda: I still have yet to receive it :( | 18:01 |
| openstackgerrit | Merged openstack/ironic: Update proliantutils version required for K release https://review.openstack.org/171673 | 18:01 |
| lucasagomes | it was sent yesterday | 18:01 |
| JayF | It was in my inbox, but there was something weird about the threading | 18:01 |
| lucasagomes | http://osdir.com/ml/openstack-dev/2015-04/msg00459.html | 18:02 |
| devananda | lucasagomes: also, 6mo ago? afaict, we didn't add the inspect interface until 2015, and discoverd support landed a bit after that | 18:02 |
| devananda | lucasagomes: but anyway, time does fly :) | 18:02 |
| devananda | hrm. strange that some folks didn't get it then | 18:03 |
| lucasagomes | devananda, yeah, maybe I expressed badly. I mean 6 months ago == K cycle | 18:03 |
| lucasagomes | within it | 18:03 |
| NobodyCam | it was linked to the elections page | 18:03 |
| devananda | lucasagomes: ooh. right | 18:03 |
| * devananda wonders briefly if he just went through a time distortion and we're not longer in the K cycle | 18:04 | |
| * BadCub wonders if devananda has had enough of the blessed coffee lol | 18:05 | |
| * NobodyCam brb ... looking for lost coffee cup | 18:05 | |
| jroll | read that as "bro" rather than "brb" | 18:05 |
| lucasagomes | lol | 18:05 |
| * BadCub knows where the lost coffee cup is but isn't saying | 18:06 | |
| devananda | clearlyi have not | 18:06 |
| devananda | i'm enjoying my comfy chair too much, and should really go down the street for coffee | 18:06 |
| BadCub | devananda: coffee more important than chair lol | 18:07 |
| * devananda wants both! | 18:07 | |
| BadCub | lol | 18:07 |
| devananda | so i hate to say this, but implementing http://specs.openstack.org/openstack/ironic-specs/specs/kilo/api-microversions.html#use-case-3b-new-client-communicating-with-a-old-ironic-user-specified | 18:08 |
| devananda | is going to be a pain right now :( | 18:08 |
| NobodyCam | I recall reading somewhere that starbucks is starting a delivery service :-p | 18:08 |
| * jroll throws an aeropress at devananda | 18:09 | |
| devananda | jroll: ooh! thanks! I didnt have one of those yet | 18:09 |
| NobodyCam | lol :) | 18:09 |
| * BadCub now needs to go refill the sacred coffee cup | 18:09 | |
| jroll | heh | 18:10 |
| BadCub | devananda: I think you may be right on ^ | 18:12 |
| devananda | i'm really tempted to punt on it | 18:12 |
| devananda | the /right/ solution is going to require client caching the server's version headers | 18:12 |
| * jroll devstacks | 18:12 | |
| devananda | and doing an initial connection (eg, GET /v1/) before sending the actual request | 18:13 |
| devananda | ie, pretty big rewrites in our client lib | 18:13 |
| jroll | devananda: can you or someone else work on #2 here https://etherpad.openstack.org/p/VK3R5xy0MW | 18:13 |
| jroll | and I assume #1 is a pre-req | 18:13 |
| * BadCub reads the email in dev list | 18:13 | |
| devananda | dtantsur already pointed out that what I've proposed is the wrong solution | 18:13 |
| jroll | oh. | 18:13 |
| jroll | WEPL | 18:13 |
| jroll | welp, too | 18:13 |
| jroll | how can I help. | 18:14 |
| devananda | jroll: we've got two problems. one between dib and pxe/iscsi drivers. the other in the client. | 18:14 |
| jroll | oh, ok | 18:14 |
| devananda | jroll: i'd like to punt on the real fix for the client, and d something closer to what we hvae know, which isn't great, but shouldn't break anyone | 18:14 |
| devananda | i hope | 18:15 |
| jroll | yeah you're still on about the client, my fault | 18:15 |
| jroll | I'm hacking on dib/iscsi | 18:15 |
| jroll | ... if and when etherpad comes back. | 18:15 |
| devananda | k k | 18:15 |
| * jroll starts on option to install bootloader or not | 18:15 | |
| jroll | I'm curious if we can just use the IRONIC_LOCAL_BOOT option that exists today | 18:16 |
| jroll | or if we need a new one | 18:16 |
| jroll | lucasagomes: got any opinions on that? | 18:16 |
| jroll | the only reason DIB cares about localboot is to know whether to install the bootloader, right? | 18:16 |
| lucasagomes | jroll, yeah | 18:17 |
| lucasagomes | jroll, idk I would pass is_whole_disk_image via cmdline | 18:17 |
| lucasagomes | and then skip local boot installation if that's set | 18:17 |
| lucasagomes | [ERROR] /opt/stack/devstack/lib/ironic:702 server didn't become ssh-able! | 18:17 |
| jroll | lucasagomes: what if we just didn't pass local_boot to DIB for whole disk images? | 18:17 |
| lucasagomes | :-( | 18:17 |
| rloo | BadCub: +A'd https://review.openstack.org/171740 | 18:17 |
| BadCub | rloo: TY for the +A on 171740 :-) | 18:17 |
| jroll | lucasagomes: whole disk images and pxe/iscsi are the same from DIB perspective, just expose iscsi | 18:18 |
| BadCub | :) | 18:18 |
| rloo | BadCub: guess you're watching everything :D | 18:18 |
| BadCub | rloo: like a hawk lol | 18:18 |
| lucasagomes | jroll, yeah that works too, but we then can't expect the DIB ramdisk to ping ironic back | 18:18 |
| lucasagomes | saying the bootloader was installed | 18:19 |
| lucasagomes | we should skip that in Ironic if is_whole_disk_image is set | 18:19 |
| jroll | lucasagomes: sure | 18:20 |
| lucasagomes | and I will need a break | 18:20 |
| lucasagomes | it's late here :/ I haven't done anything yet | 18:20 |
| lucasagomes | was trying to test the ipxe thing but devstack seems broken :/ | 18:20 |
| devananda | :-/ | 18:21 |
| lucasagomes | take a break* | 18:21 |
| jroll | devananda: wdyt, just treat whole disk images the same as partition images wrt DIB | 18:22 |
| jroll | devananda: e.g. don't pass boot_mode: local | 18:22 |
| BadCub | devananda: any word yet on holding rc1 for a day or so? Or should I be planning on moving what is left to L? | 18:22 |
| devananda | BadCub: no answer yet | 18:22 |
| lucasagomes | :( | 18:22 |
| BadCub | devananda: kk.. I will keep everything frozen until the last minute then | 18:23 |
| BadCub | if we can;t hold out, then I will move what is left to L | 18:23 |
| devananda | jroll: "treat it the same" ? | 18:23 |
| devananda | BadCub: I suspect we can hold out a few days | 18:23 |
| lucasagomes | https://wiki.openstack.org/wiki/Kilo_Release_Schedule | 18:23 |
| lucasagomes | it looks like from 9-23 is the time we have to do the rc releases | 18:23 |
| jroll | devananda: don't tell DIB that a whole disk image is localboot. then it won't write the bootloader. then ironic doesn't wait for the ping back. | 18:24 |
| BadCub | that would be awesomeness. Would love to get our "final few" in :) | 18:24 |
| lucasagomes | so i suspect it doesn't necessarily means 9 right? | 18:24 |
| jroll | devananda: also, don't we chain-load pxe to disk, so it really isn't localboot anyway? or is that an option? | 18:24 |
| lucasagomes | or is it 9 rc1, 16 rc2, 23 rc3? | 18:24 |
| * lucasagomes is not sure... | 18:25 | |
| lucasagomes | anyway I will take a break | 18:25 |
| jroll | I need to trace this more before I'm 100% sure, but I think this should work | 18:25 |
| BadCub | lucasagomes: I suspect it is 9 = rc1, 16=rc2, 23=rc3 (at least that is how we did it downstream) | 18:25 |
| lucasagomes | I see | 18:25 |
| lucasagomes | :/ | 18:25 |
| devananda | jroll: ah, think i get the idea. something like: if partition_image and local_boot: pass local_boot to DIB & do extra wait; else don't pass local-boot and don't do extra wait | 18:26 |
| jroll | devananda: correct | 18:26 |
| BadCub | now go take break man!!! | 18:26 |
| *** lucasagomes is now known as lucas-dinner | 18:26 | |
| lucas-dinner | will do :D | 18:26 |
| lucas-dinner | bbl | 18:26 |
| * BadCub goes for quick smoke | 18:26 | |
| devananda | lucasagomes: we must cut an RC at some point. if we delay, it delays other projects and people. if we do it too early, it makes more work for us. | 18:27 |
| JayF | devananda: if you'll ping me when you cut the RC; I can make sure patches get up to open specs for liberty if we don't need/want any template changes | 18:27 |
| devananda | if we know about a critical bug and we're close to fixing it, it's worth delaying a day or two. if it's going to take longer, we should cut the RC so others can begin testing it, and just accept that we'll backport the fix and cut RC2 | 18:27 |
| BadCub | devananda: I agree completely on that. :) | 18:28 |
| devananda | JayF: cheers. feel free to put the patches up any time ... just dont approve them yet :) | 18:28 |
| JayF | devananda: cool, I know aweeks and JoshNang are going to get an early start on L specs and I was hoping to save them a rebase; that'll work :D | 18:28 |
| devananda | JayF: also, i haven't looked at the template recently, but i dont recall us wanting any specific changes | 18:28 |
| JayF | devananda: I don't forsee needing any either, unless we want to add specific stuff about microversions | 18:29 |
| devananda | good point | 18:29 |
| JayF | but imo that should be questions asked/answered in the existing API Impact section | 18:29 |
| devananda | perhaps in the APi changes section a note would be good | 18:29 |
| devananda | :) | 18:29 |
| BadCub | speaking of L specs. Would the Spec Cores be available to congregate next Wed or Thurs to start approving Specs? | 18:29 |
| JayF | I can add a note there, cool | 18:29 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Don't pass boot_option: local for whole disk images https://review.openstack.org/171767 | 18:29 |
| jroll | devananda: this probably doesn't work right this moment, but first stab ^ | 18:29 |
| jroll | not sure how it will (or if it does at all) interact with IPA ramdisk | 18:30 |
| JayF | badcub: I'm more or less out of upstream next week; I'm travelling to texas and won't be around much, maybe next time? | 18:30 |
| * jroll waits for devstack | 18:30 | |
| jroll | BadCub: I can be around for that, however I'd love to have more clarity on which specs you mean | 18:30 |
| *** trown|lunch is now known as trown | 18:31 | |
| BadCub | JayF: okay, coolness. | 18:31 |
| BadCub | jroll: I was hoping to pull y'all together to start looking at what we have and what we want/need etc. | 18:31 |
| jroll | BadCub: so talking about specs, not approving them :) | 18:31 |
| BadCub | jroll: yeah, kinda a focus session on what we got floating about, maybe list up those that look tastey | 18:33 |
| jroll | +1 | 18:33 |
| BadCub | :) | 18:34 |
| BadCub | now I am really gonna go grab a smoke lol | 18:34 |
| jroll | 2015-04-08 18:35:44.941 | [ERROR] /opt/stack/devstack/lib/ironic:565 Timed out waiting for Nova hypervisor-stats count >= 3 | 18:36 |
| jroll | grrrrrr | 18:36 |
| jroll | adam_g: ^ that's with your patch applied | 18:36 |
| openstackgerrit | Merged openstack/ironic: Add retry logic to _exec_ipmitool https://review.openstack.org/168120 | 18:38 |
| *** jcoufal has joined #openstack-ironic | 18:39 | |
| TheJulia | woot | 18:41 |
| BadCub | TheJulia: congrats ^^ :) | 18:41 |
| TheJulia | rloo: should have follow-up posted shortly | 18:41 |
| rloo | thx TheJulia | 18:42 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Follow-up to "Add retry logic to _exec_ipmitool" https://review.openstack.org/171771 | 18:47 |
| TheJulia | rloo: ^^ | 18:47 |
| *** jcoufal has quit IRC | 18:47 | |
| rloo | thx TheJulia | 18:48 |
| TheJulia | np | 18:48 |
| TheJulia | Thank you! | 18:48 |
| rloo | :) | 18:48 |
| devananda | neat | 18:49 |
| devananda | when trying to change a node's name (introduced in v1.5) the client can't distinguish between a v1.4 server, or the fact that i requested v1.4 API like an idiot | 18:50 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Don't pass boot_option: local for whole disk images https://review.openstack.org/171767 | 18:51 |
| TheJulia | ugh | 18:51 |
| TheJulia | I need to step away for a little while, I'll be back in a few hours | 18:54 |
| openstackgerrit | Merged openstack/ironic: Fix exception handling in Glance image service https://review.openstack.org/171740 | 18:54 |
| rloo | devananda: our client doesn't treat any user like an idiot | 18:56 |
| devananda | rloo: indeed. my brain is just melting (i still haven't gotten up to get more coffee) | 18:57 |
| *** trown is now known as trown|taxman | 18:57 | |
| * jroll takes away deva's chair | 18:58 | |
| rloo | jroll: devananda just fell down :-( | 18:58 |
| BadCub | jroll: is 171767 in conjunction to 171743 or replacement of ? | 18:58 |
| jroll | BadCub: complementary | 19:01 |
| BadCub | sweet | 19:01 |
| jroll | BadCub: 171743 fixes the case where ironic doesn't have the support for the pingback from DIB after bootloader install | 19:01 |
| jroll | and 171767 fixes the bug with whole disk images | 19:02 |
| jroll | still needs unit tests and testing, trying to get devstack up | 19:02 |
| NobodyCam | https://www.youtube.com/watch?v=nr5nFfWIqhQ | 19:02 |
| BadCub | I have put up a pad for Spec gathering next week: https://etherpad.openstack.org/p/IronicSpecProcess put some initial thoughts on it along with the specs that got dropped from K | 19:02 |
| devananda | oh! that's why this is failing! some versions of our server didn't return any headers on an error ! | 19:06 |
| devananda | arrrgh | 19:06 |
| devananda | client compat between commits during kilo is F**# | 19:06 |
| * devananda gives up | 19:06 | |
| jroll | :/ | 19:07 |
| jroll | I'm so glad we were horrible about rebasing this cycle | 19:07 |
| rloo | jroll: rebasing? | 19:08 |
| jroll | rloo: downstream, rebasing to pick up new upstream | 19:08 |
| rloo | jroll: oh. got it. | 19:09 |
| rloo | jroll: but if you HAD rebased more often, we'd have found out issues sooner, no? | 19:09 |
| jroll | rloo: possibly | 19:09 |
| *** Marga_ has quit IRC | 19:12 | |
| * jroll steps away for a bit | 19:13 | |
| * BadCub needs to run out for a short while. BBIAB | 19:13 | |
| devananda | righs, so when I check out the server version that introduced logical names, a newer client is going to fail to use that feature because the server isn't returning the version headers on any error | 19:16 |
| devananda | unless i explicilty set the version on the client | 19:16 |
| devananda | yea, we made this really really painful :( | 19:16 |
| *** igordcard_ has joined #openstack-ironic | 19:18 | |
| rloo | devananda: not sure I understand. by 'newer client', do you mean that it is using 1.6? | 19:18 |
| devananda | i'm working on the autonegotiation patch | 19:18 |
| devananda | certain versions of the server didn't return the X-OpenStck-Ironic-API-Versoin header on an error, but they *understood* the header | 19:19 |
| devananda | so the new client passes v1.6 by default, gets a 406 error, but can't determine what version to use ... because the server didn't return that header | 19:19 |
| devananda | so right now, i'm coding this to fall back to v1.1 (juno). however, juno never returned a 406 | 19:20 |
| rloo | ohhhh. cuz we incrementally implemented the micro versioning support while features were being added. | 19:20 |
| devananda | but do I really care that much about the client working around a bug in the server that existed for about a month, mid-cycle? | 19:20 |
| rloo | devananda: NO, we shouldn't care. | 19:21 |
| devananda | maybe not. | 19:21 |
| *** thrash|biab is now known as thrash | 19:21 | |
| rloo | devananda: well, I mean, we want to avoid bugs, but I think it is understandable that bugs will creep in. | 19:21 |
| rloo | devananda: what are the combinations that shouldn't break? pre-kilo-ironic+any-client (except 0.5.0); kilo-release-ironic+any-client? | 19:24 |
| devananda | well, any server + any client. but we're not perfect. so let's say: juno or later server + client 0.5.1 (which will be tagged when i fix this) | 19:25 |
| * NobodyCam wounders if AMT plans on adding persistent boot device support? | 19:25 | |
| thrash | devananda: I will keep going on that same branch. | 19:25 |
| rloo | devananda: well, 'later server' could include anything off of master from when kilo was opened. That's where you encountered your problem above. | 19:26 |
| devananda | right | 19:27 |
| rloo | devananda: can't we just say kilo-server? | 19:27 |
| rloo | devananda: I mean juno server + kilo server. and if it works for kilo-1-2-whatever, that'd be a bonus. | 19:28 |
| devananda | that's not ideal, but i'm not sure there's another choice, short of spending a lot more time trying to work around this | 19:28 |
| rloo | my concern with spending a lot more time: 1. more time needed; 2. will it really work for all cases; 3. how many people care (maybe dtantsur if we break discoverd) | 19:29 |
| rloo | and 4. to fix, will the code become ugly to maintain etc. | 19:30 |
| rloo | to put in perspective, is the bug critical, high, medium, ?? | 19:31 |
| * rloo just wants to move on. But then, she isn't the PTL so doesn't need to make the hard decisions and incur the wrath of ?? :) | 19:32 | |
| *** achanda has quit IRC | 19:37 | |
| *** mtanino has quit IRC | 19:40 | |
| *** Sukhdev has joined #openstack-ironic | 19:41 | |
| *** absubram has joined #openstack-ironic | 19:44 | |
| *** romcheg has left #openstack-ironic | 19:45 | |
| *** achanda has joined #openstack-ironic | 19:46 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Agent driver fails without Ironic-managed TFTP https://review.openstack.org/170984 | 19:50 |
| *** trown|taxman is now known as trown | 19:50 | |
| *** enikanorov has joined #openstack-ironic | 19:51 | |
| JoshNang | ^ I still need to step through the PXE driver and see how much of a change that would require (if we want to support the pxe driver without ironic managed tftp). A quick look through the iLO drivers didn't show anything that would cause issues. | 19:53 |
| jroll | nice | 19:53 |
| NobodyCam | JayF: was just looking at https://review.openstack.org/#/c/171372 and woundering if we should add anything about dirvers may have unique config options. (IE. agent_erase_devices_priority vs. clean_priority_erase_devices) | 19:54 |
| jroll | devananda: ^ we probably want to get that into rc1 | 19:54 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added vagrant VM for developer use https://review.openstack.org/167415 | 19:55 |
| *** ijw has quit IRC | 19:56 | |
| NobodyCam | JayF: never mind.. looks like 169162 addresses that | 19:56 |
| NobodyCam | :-p | 19:56 |
| *** pas-ha has joined #openstack-ironic | 20:00 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Fixup post-merge comments on cleaning document https://review.openstack.org/171372 | 20:04 |
| JayF | rloo: ^ thanks for the review; I'm punting on making all the states appear consistent since that should be done on all docs in one shot to make a reasonable standard | 20:04 |
| rloo | JayF: makes sense. just fix that typo! | 20:04 |
| *** achanda has quit IRC | 20:05 | |
| NobodyCam | brb | 20:07 |
| *** ndipanov has quit IRC | 20:09 | |
| *** bradjones_ has joined #openstack-ironic | 20:11 | |
| *** Marga_ has joined #openstack-ironic | 20:11 | |
| *** bradjones has quit IRC | 20:13 | |
| *** bradjones_ is now known as bradjones | 20:13 | |
| *** bradjones has joined #openstack-ironic | 20:13 | |
| * BadCub returns | 20:14 | |
| *** dttocs has joined #openstack-ironic | 20:16 | |
| devananda | BadCub: got the OK for monday | 20:18 |
| BadCub | devananda: awesomeness!!!! :) | 20:18 |
| *** meghal has quit IRC | 20:19 | |
| BadCub | that should give us plenty of time to land leftovers | 20:19 |
| adam_g | jroll, there are no x509 commands being run anymore? any idea whats causing it? got a devstack log? | 20:28 |
| *** mtanino has joined #openstack-ironic | 20:32 | |
| *** achanda has joined #openstack-ironic | 20:39 | |
| *** dttocs_ has joined #openstack-ironic | 20:42 | |
| pshige | morning | 20:43 |
| *** pas-ha has quit IRC | 20:46 | |
| *** dttocs has quit IRC | 20:47 | |
| pshige | devanand: I'd like to fix bug/137893 in your commit "Update installation service overview": https://review.openstack.org/#/c/166346/ as well, because my aim is nearly the same. | 20:47 |
| pshige | May I update your commit together with other people's comments? | 20:47 |
| pshige | This bug is to fix documentation inconsistency of cinder-integration between "Bare Metal Installation Guide" and the "Introduction to Ironic". | 20:47 |
| openstackgerrit | Devananda van der Veen proposed openstack/python-ironicclient: Client should fall back to the lower versions if necessary https://review.openstack.org/171334 | 20:49 |
| devananda | there! | 20:49 |
| * devananda walks away for a bit | 20:49 | |
| *** meghal has joined #openstack-ironic | 20:51 | |
| pshige | morning devananda | 20:51 |
| devananda | that needs reviews, btw. fixing critical issues with the client backwards compat... from my local testing, that rev should work with all the servers since stable/juno, but I didn't actually test every single one... | 20:51 |
| * devananda really goes afk for a bit | 20:51 | |
| *** Marga_ has quit IRC | 20:51 | |
| *** Marga_ has joined #openstack-ironic | 20:53 | |
| *** Marga_ has quit IRC | 20:54 | |
| *** Marga_ has joined #openstack-ironic | 20:54 | |
| *** dttocs_ has quit IRC | 20:55 | |
| *** meghal has quit IRC | 20:55 | |
| *** absubram has quit IRC | 20:57 | |
| jroll | adam_g: they were still being run | 21:02 |
| jroll | adam_g: I only grabbed the patch that bumps ironic retries to 180 | 21:03 |
| adam_g | jroll, oh, theres another | 21:03 |
| adam_g | https://review.openstack.org/#/c/171406/ | 21:04 |
| adam_g | im testing locally with the documented local.conf | 21:04 |
| *** meghal has joined #openstack-ironic | 21:05 | |
| *** pas-ha has joined #openstack-ironic | 21:06 | |
| NobodyCam | BadCub: 171372 +a'd | 21:07 |
| BadCub | NobodyCam: Awesome! TY! :-) | 21:08 |
| BadCub | anyone free to do some reviewing on: https://review.openstack.org/#/c/171334/ - Client should fall back to the lower versions if necessary ? | 21:09 |
| adam_g | jroll, should work with 171406 applied | 21:09 |
| jroll | adam_g: ok, thanks | 21:09 |
| *** pradipta has quit IRC | 21:12 | |
| *** kkoski has quit IRC | 21:15 | |
| *** dprince has quit IRC | 21:32 | |
| openstackgerrit | Adam Gandelman proposed openstack/ironic: Add notes about upgrading juno->kilo to docs https://review.openstack.org/171827 | 21:40 |
| *** Sukhdev has quit IRC | 21:41 | |
| NobodyCam | are headers case senitive? | 21:42 |
| jroll | no | 21:44 |
| jroll | unless the software is really bad | 21:44 |
| jroll | http spec says no | 21:45 |
| NobodyCam | HTTP header names are case-insensitive, according to RFC 2616 | 21:45 |
| NobodyCam | :) | 21:45 |
| JayF | Lots of software treat them as case sensitive | 21:45 |
| JayF | for serious | 21:45 |
| JayF | I know what the RFC says, but I've seen both apache and node.js go a little bananas with strangely-cap'd headers | 21:46 |
| NobodyCam | i was looking at https://review.openstack.org/#/c/171334/7/ironicclient/common/http.py lines 145-6 & 155-6 | 21:46 |
| JayF | NobodyCam: I'm enough of a pedant to -1 or followup patch that to make it consistent | 21:46 |
| JayF | NobodyCam: imo it should be consistent within the same method in the client, lol | 21:47 |
| openstackgerrit | Clif Houck proposed openstack/ironic: Add support for image caching for the agent driver https://review.openstack.org/161832 | 21:47 |
| NobodyCam | JayF: I think devananda is afk. wanta fix and toss up a ew rev so I can +2 | 21:47 |
| NobodyCam | lol | 21:47 |
| JayF | NobodyCam: I already have a queue of one patch I owe, would rather not make it two :P | 21:48 |
| NobodyCam | :) | 21:48 |
| JayF | NobodyCam: that feel when you try to download a patchset for client into ironic, lol | 21:50 |
| NobodyCam | oh :-p | 21:52 |
| openstackgerrit | Jay Faulkner proposed openstack/python-ironicclient: Client should fall back to the lower versions if necessary https://review.openstack.org/171334 | 21:53 |
| JayF | NobodyCam: ^ | 21:53 |
| NobodyCam | lol I was just doing that | 21:53 |
| NobodyCam | lol | 21:53 |
| NobodyCam | Thank you JayF :) | 21:53 |
| JayF | figured if I didn't want my queue to grow | 21:54 |
| JayF | I shouldn't put it on the queue | 21:54 |
| JayF | JFDIFTW | 21:54 |
| NobodyCam | :) | 21:54 |
| NobodyCam | BadCub: https://review.openstack.org/#/c/171334 +2'd with JayF's fixup :) | 21:56 |
| BadCub | NobodyCam: awesome!! Just need another +2/+A on ^ | 21:56 |
| BadCub | adam_g: I added https://review.openstack.org/#/c/171827/ back to rc1 since you got the doc patch up. | 21:57 |
| *** trown is now known as trown|outttypeww | 21:59 | |
| *** chlong has joined #openstack-ironic | 22:01 | |
| * BadCub fixes lunch stuff | 22:01 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Open specs for Liberty https://review.openstack.org/171831 | 22:03 |
| NobodyCam | lol was that patch that was on your queue? | 22:04 |
| JayF | NobodyCam: yep | 22:04 |
| NobodyCam | :-p *ducks* | 22:04 |
| JayF | NobodyCam: updating template for liberty and getting rendering working for it | 22:04 |
| JayF | NobodyCam: so JoshNang and aweeks can push up the spec they're pairing on and have it actually worjk | 22:04 |
| NobodyCam | JayF: awesome!!! great forthought :) | 22:05 |
| JayF | well it opens today, right? | 22:05 |
| JayF | aren't we cutting an RC today or tomorrow? | 22:05 |
| NobodyCam | tomorrow I think, but really not until rc-1 was cut... butI'll deffer to BadCub on that one! | 22:06 |
| NobodyCam | I think rc-1 will be cut monday | 22:06 |
| NobodyCam | *think* | 22:06 |
| NobodyCam | will cehck with BadCub once he returns with food stuffs | 22:07 |
| jroll | FYI, just tested this in devstack, seems to work as expected https://review.openstack.org/#/c/171767/ | 22:08 |
| jroll | needs unit tests, nomming a sandwich and then doing that | 22:08 |
| JayF | NobodyCam: cool, I thought deadline for RC1 was tomorrow for some reason | 22:08 |
| NobodyCam | JayF: I beleive we have ffe for that :) | 22:09 |
| *** pas-ha has quit IRC | 22:09 | |
| *** jamielennox is now known as jamielennox|away | 22:11 | |
| BadCub | NobodyCam: JayF , the extension for RC1 should not really impact L specs much at all | 22:13 |
| NobodyCam | BadCub: so we open tomorrow? | 22:14 |
| BadCub | NobodyCam: let me get absolute clarification | 22:15 |
| BadCub | But from my perspective, I see no issues with opening specs for L tomorrow | 22:16 |
| NobodyCam | same here +++ | 22:16 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Open specs for Liberty https://review.openstack.org/171831 | 22:17 |
| BadCub | we only have a couple things left to land for rc1 too. | 22:17 |
| NobodyCam | w00 h00 | 22:17 |
| BadCub | Thanks JayF ^^ :) | 22:18 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Open specs for Liberty https://review.openstack.org/171831 | 22:18 |
| *** dttocs has joined #openstack-ironic | 22:19 | |
| jroll | ugh, I hate the word microversion | 22:20 |
| jroll | it's just a version. | 22:21 |
| BadCub | jroll: be one with the microversion. Be at peace and share its tranquil nature | 22:21 |
| BadCub | lol | 22:21 |
| NobodyCam | well kinda a "sub" version | 22:22 |
| NobodyCam | because thia is all the v1 api | 22:22 |
| NobodyCam | s/thia/this/ | 22:22 |
| jroll | JayF: left a thing | 22:22 |
| jroll | I mean, it's a minor version | 22:22 |
| NobodyCam | yea | 22:22 |
| jroll | nowhere else in the world have I seen the word microversion, other than openstack | 22:23 |
| * BadCub goes to be one with a smoke | 22:23 | |
| NobodyCam | lol that sounds dangerious | 22:23 |
| jroll | lol | 22:23 |
| NobodyCam | adam_g: looking at 171827 is that linked anywhere I didn't see it on http://docs-draft.openstack.org/27/171827/1/check/gate-ironic-docs/df9a434//doc/build/html/ | 22:25 |
| adam_g | NobodyCam, oh, maybe not | 22:25 |
| *** achanda has quit IRC | 22:28 | |
| *** achanda has joined #openstack-ironic | 22:31 | |
| NobodyCam | :) | 22:32 |
| *** dttocs has quit IRC | 22:32 | |
| openstackgerrit | Adam Gandelman proposed openstack/ironic: Add notes about upgrading juno->kilo to docs https://review.openstack.org/171827 | 22:34 |
| NobodyCam | adam_g: TY :) | 22:35 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Don't pass boot_option: local for whole disk images https://review.openstack.org/171767 | 22:38 |
| jroll | devananda: NobodyCam ^ that should be good to go, tested in devstack | 22:38 |
| adam_g | that reminds me of a funny tshirt i once saw | 22:39 |
| *** alexpilotti has quit IRC | 22:39 | |
| adam_g | :) | 22:39 |
| jroll | heh | 22:39 |
| jroll | I want one of those shirts | 22:39 |
| NobodyCam | http://www.customink.com/designs/ironic2/45931546-15537897/sdfu/?pc=SDFU-40124&cm_mmc=sdfu-_-Winter2014-_-Day84-_-designfront | 22:40 |
| jroll | also one of those shirts. | 22:40 |
| * BadCub wants that shirt | 22:40 | |
| NobodyCam | ok I should order them soonish | 22:41 |
| BadCub | That would be an awesome shirt for summit | 22:41 |
| openstackgerrit | Merged openstack/ironic: Fixup post-merge comments on cleaning document https://review.openstack.org/171372 | 22:41 |
| devananda | NobodyCam: /me wants it in white on grey | 22:43 |
| NobodyCam | so grey tee? | 22:43 |
| jroll | or on black like a chalkboard :D | 22:44 |
| devananda | JayF: we need to move specs approved during kilo, but not implemented, to an archival folder so that it's clear they were not completed | 22:44 |
| devananda | chalkboard++ | 22:44 |
| NobodyCam | the pic dosen't work on black so well | 22:44 |
| BadCub | chalkboard ++ | 22:44 |
| jroll | yeah, figures | 22:44 |
| devananda | easy to fix the coloring ... | 22:44 |
| NobodyCam | I'll have to install gimp | 22:44 |
| NobodyCam | unless BadCub still has photo shop loaded | 22:45 |
| BadCub | yeah, the black outlined one would be difficult, unless we reversed the colors | 22:45 |
| NobodyCam | :-p | 22:45 |
| devananda | NobodyCam: where's the original pic? | 22:45 |
| * BadCub grumbles I think I might on the Mac | 22:45 | |
| devananda | i can flip colors quickly | 22:45 |
| NobodyCam | sounds like just make the entire thing white lines and switchthe shirt to grey/black | 22:46 |
| BadCub | yep | 22:46 |
| * BadCub thinks devananda should be away at this point, but is happy to let him play with crayons and artwork | 22:46 | |
| NobodyCam | lol | 22:46 |
| JayF | devananda: is there a canonical list of those? | 22:47 |
| BadCub | JayF: https://etherpad.openstack.org/p/IronicSpecProcess | 22:47 |
| BadCub | I have all of them listed in priority order. Or at least I should have them all | 22:48 |
| JayF | BadCub: perfect, I'll move those into an "archive" area | 22:48 |
| *** Marga_ has quit IRC | 22:48 | |
| BadCub | JayF: coolness! ty :) | 22:49 |
| *** Marga_ has joined #openstack-ironic | 22:50 | |
| NobodyCam | oh look at pixie's eyes on the back :/ http://www.customink.com/lab?cid=45931546-15537897&pc=SDFU-40124&cm_mmc=sdfu-_-Winter2014-_-Day84-_-button | 22:55 |
| *** lucas-dinner has quit IRC | 22:56 | |
| NobodyCam | try this link: http://www.customink.com/designs/ironic3/beu0-00a2-gq6r/share/?pc=EMAIL-40778&cm_mmc=share-_-emailb-_-viewdesign-_-end | 22:56 |
| *** stendulker has joined #openstack-ironic | 22:56 | |
| NobodyCam | that should be grey shirt and white text | 22:57 |
| BadCub | grey looks cool. | 22:58 |
| jroll | nice | 22:59 |
| jroll | the eyes are weird, heh | 22:59 |
| NobodyCam | makes pixie look evil | 22:59 |
| pshige | oh | 22:59 |
| pshige | shipping all over the world? | 23:00 |
| * BadCub will boot up his mac tonight and see if he has Photoshop still | 23:00 | |
| jroll | anybody want to lay eyes on https://review.openstack.org/#/c/171767/ | 23:00 |
| NobodyCam | pshige: this is just something we're tossing around. | 23:00 |
| *** absubram has joined #openstack-ironic | 23:00 | |
| pshige | ok | 23:01 |
| NobodyCam | I believe everyone will be able to order their own | 23:01 |
| NobodyCam | thats the goal | 23:01 |
| NobodyCam | :) | 23:01 |
| jroll | wait, you aren't going to buy them for us? | 23:02 |
| * BadCub rolls around laughing | 23:02 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Open specs for Liberty https://review.openstack.org/171831 | 23:02 |
| NobodyCam | :p | 23:02 |
| jroll | hm | 23:03 |
| * jroll considers telling mordred to fire you guys | 23:03 | |
| NobodyCam | :( | 23:03 |
| BadCub | LOL | 23:03 |
| jroll | :D | 23:03 |
| NobodyCam | heheheh | 23:04 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Open specs for Liberty https://review.openstack.org/171831 | 23:04 |
| * BadCub ponders.... Hmmmm... Promotional expense........ | 23:04 | |
| JayF | JoshNang: The suggestion about adding state machine impact to the template makes sense | 23:04 |
| JayF | devananda: ^ agreed? | 23:04 |
| jroll | BadCub: just expense it with all the other summit stuff, nobody will notice | 23:04 |
| NobodyCam | some one could tell mark (atwood) thats how we got the hoodies provided by HP a couple of summits ago | 23:05 |
| pshige | LOL | 23:05 |
| *** chlong has quit IRC | 23:05 | |
| BadCub | jroll: They probably wouldn't lol | 23:05 |
| JayF | I'd totally buy one of those shirts if it's any color but white | 23:05 |
| JayF | I won't even try to stick HP with the bill | 23:05 |
| * jroll is a cheapskate | 23:06 | |
| BadCub | I like the grey better than white, but think black shirt would look best | 23:06 |
| NobodyCam | JayF: the last link was grey shirt with white text | 23:06 |
| JayF | NobodyCam: ah, that's somewhat better | 23:06 |
| pshige | black is cool | 23:06 |
| BadCub | will have to play with it a bit more and see what we can do to get it to work on black shirt | 23:07 |
| devananda | BadCub: NobodyCam: where's the original pic of that drawing? | 23:08 |
| pshige | again | 23:08 |
| NobodyCam | i'm looking now | 23:08 |
| pshige | devananda: I'd like to fix bug/137893 in your commit "Update installation service overview": https://review.openstack.org/#/c/166346/ as well, because my aim is nearly the same. | 23:08 |
| JayF | devananda: I know kyle made it when he was interning here, I don't know where the original-original is though | 23:08 |
| BadCub | devananda: *points at NobodyCam* he did it!!!! | 23:08 |
| pshige | May I update your commit together with other people's comments? | 23:09 |
| *** stendulker has quit IRC | 23:09 | |
| pshige | This bug is to fix documentation inconsistency of cinder-integration between "Bare Metal Installation Guide" and the "Introduction to Ironic". | 23:09 |
| BadCub | lol | 23:09 |
| devananda | JayF: hmm. yes. state machine impact. AND what part of the state machine might this new feature intersect with / be invoked from / etc | 23:09 |
| JayF | I have a nickel for the first person that comes up with good text to ^ that effect and puts it on the review for me to add :P | 23:10 |
| JayF | otherwise I'll get it shortly | 23:10 |
| devananda | pshige: sure thing, go for it | 23:10 |
| pshige | ok | 23:10 |
| devananda | JayF: have a link to the thing Kyle did? | 23:10 |
| devananda | oh. nm. you did it already | 23:11 |
| * devananda puts away his crayons and pouts | 23:11 | |
| NobodyCam | who wants the orginal | 23:11 |
| NobodyCam | I have it | 23:11 |
| *** jamielennox|away is now known as jamielennox | 23:12 | |
| NobodyCam | devananda: you too now have the orgianl | 23:13 |
| BadCub | NobodyCam: you can send it to my home email as well? | 23:14 |
| devananda | awesome, ty | 23:14 |
| * BadCub pouts devananda on head and hands him water colors instead of crayons | 23:15 | |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Update installation service overview https://review.openstack.org/166346 | 23:15 |
| BadCub | lol | 23:15 |
| NobodyCam | and all credit should go to the orginal creator of that image kylestev | 23:15 |
| pshige | lol | 23:16 |
| BadCub | pshige: you want to clean up the white space on ^^ ? | 23:18 |
| NobodyCam | BadCub: you too now have the orginal image | 23:19 |
| BadCub | awesome | 23:19 |
| pshige | oh, I will fix soon | 23:21 |
| *** absubram has quit IRC | 23:21 | |
| *** mrda-away is now known as mrda | 23:21 | |
| mrda | Morning Ironic | 23:22 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Open specs for Liberty https://review.openstack.org/171831 | 23:22 |
| SpamapS | http://logs.openstack.org/37/137737/6/check-tripleo/check-tripleo-ironic-overcloud-f20puppet-nonha/013e91f/console.html#_2015-04-07_17_01_21_764 | 23:23 |
| SpamapS | anybody know what might have caused that? | 23:23 |
| SpamapS | http://logs.openstack.org/37/137737/6/check-tripleo/check-tripleo-ironic-overcloud-f20puppet-nonha/013e91f/logs/seed_logs/openstack-ironic-api.txt.gz | 23:23 |
| SpamapS | thats the log from ironic-api | 23:23 |
| jroll | SpamapS: latest client by default requests version 1.6, if the server can't handle that it will break in this fashion | 23:24 |
| jroll | this is a horrible bug we're working on fixing | 23:24 |
| jroll | devananda: ^ fyi | 23:24 |
| SpamapS | ahhhhh ok | 23:24 |
| SpamapS | so a client release probably broke it? | 23:25 |
| jroll | yes | 23:26 |
| jroll | maybe monday? | 23:26 |
| jroll | I forget | 23:26 |
| SpamapS | Yeah, this failed yesterday | 23:26 |
| jroll | if y'all deploy latest ironic it should be fine | 23:27 |
| jroll | or older client | 23:27 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Update installation service overview https://review.openstack.org/166346 | 23:27 |
| SpamapS | jroll: http://logstash.openstack.org/#eyJzZWFyY2giOiJcIk5vdEFjY2VwdGFibGU6IE5vdCBBY2NlcHRhYmxlIChIVFRQIDQwNilcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiMTcyODAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQyODUzNTY2MzQyNn0= | 23:28 |
| SpamapS | jroll: so probably just a recheck for tripleo | 23:28 |
| SpamapS | stopped failing already | 23:29 |
| SpamapS | jroll: got a bug# so I can reference it? | 23:29 |
| jroll | SpamapS: aha, yeah | 23:29 |
| jroll | yep, one sec | 23:29 |
| SpamapS | will just go ahead and recheck all these changes actually | 23:30 |
| jroll | https://bugs.launchpad.net/tripleo/+bug/1441170 | 23:30 |
| openstack | Launchpad bug 1441170 in python-ironicclient "ironicclient "Not Acceptable" error" [High,In progress] - Assigned to Jay Faulkner (jason-oldos) | 23:30 |
| jroll | SpamapS: ^ | 23:30 |
| BadCub | pshige: ty for takign care of that :) | 23:30 |
| BadCub | taking even | 23:30 |
| SpamapS | jroll: ty very much! :) | 23:31 |
| jroll | np :) | 23:31 |
| openstackgerrit | Alex Weeks proposed openstack/ironic-specs: Add implement-rescue-mode spec https://review.openstack.org/171878 | 23:33 |
| jroll | YES | 23:33 |
| jroll | aweeks: JoshNang <3 | 23:34 |
| NobodyCam | oh nice! | 23:34 |
| * NobodyCam put on list for reading tomorrow mornings coffee | 23:34 | |
| JoshNang | :D | 23:35 |
| * BadCub adds to SpecProcess Pad under new category | 23:36 | |
| * BadCub wonders why NobodyCam plans on reading his morning coffee | 23:37 | |
| BadCub | *ducks* | 23:37 |
| jroll | lol | 23:37 |
| NobodyCam | its like reading tea leafs | 23:37 |
| JoshNang | heh | 23:37 |
| NobodyCam | :-p | 23:37 |
| BadCub | lol | 23:38 |
| openstackgerrit | Alex Weeks proposed openstack/ironic-specs: Add implement-rescue-mode spec https://review.openstack.org/171878 | 23:40 |
| jroll | clif_h: also reviewed your nova patch, lookin good | 23:42 |
| clif_h | thanks! | 23:43 |
| jroll | np dude | 23:43 |
| jroll | BadCub: still need anything done today? should I just check the review day pad? | 23:44 |
| aweeks | NobodyCam: thanks :) | 23:44 |
| *** Sukhdev has joined #openstack-ironic | 23:44 | |
| jroll | NobodyCam: devananda: does image caching interest y'all at all? | 23:45 |
| NobodyCam | adam_g: I ended up -1'ing 171827 | 23:45 |
| NobodyCam | but only because it really didn't read well for me | 23:46 |
| NobodyCam | jroll: depends | 23:46 |
| jroll | on? | 23:47 |
| BadCub | jroll: The review pad is updated, so might be good to scan it over | 23:47 |
| BadCub | some new docs (easy reads) are up for review and https://review.openstack.org/#/c/171334/ Really need another +2/+A :) | 23:48 |
| BadCub | devananda: would be a very happy camper if we got https://review.openstack.org/#/c/171334/ off the "Needs Review" list :) | 23:48 |
| NobodyCam | jroll: on which images are cached how and when they are cached type things | 23:48 |
| NobodyCam | so I guess the answer to your orginal question is YES | 23:49 |
| jroll | NobodyCam: right, ofc :P | 23:49 |
| NobodyCam | :-p | 23:49 |
| jroll | ok, at the risk of being yelled at: https://github.com/rackerlabs/arsenal | 23:49 |
| NobodyCam | lol love point #1 | 23:50 |
| jroll | you should check that out, clif_h has ironic and nova patches up in gerrit as well | 23:50 |
| NobodyCam | is this image cache or a pre-deployment system.. | 23:51 |
| jroll | uh | 23:52 |
| jroll | not sure the difference :) | 23:52 |
| jroll | putting images on disk, for undeployed nodes, is the goal | 23:52 |
| clif_h | the idea is to be decisioning around caching images on nodes | 23:53 |
| jroll | so deployment is just (optionally) lay down a config drive and go | 23:53 |
| jroll | arsenal handles the business logic as to what to cache where, like clif just said | 23:53 |
| NobodyCam | its interesting idea.. I need to thinks on it | 23:54 |
| clif_h | its to the point of working, at least I've seen it work in devstack :) | 23:54 |
| *** Haomeng has joined #openstack-ironic | 23:54 | |
| clif_h | coupled with my ironic and nova patches | 23:55 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Open specs for Liberty https://review.openstack.org/171831 | 23:55 |
| clif_h | need to update the readme | 23:55 |
| NobodyCam | lol A big fat commit | 23:56 |
| clif_h | :D | 23:56 |
| *** Haomeng|2 has quit IRC | 23:57 | |
| * BadCub is thinking Japanese food is in order for evening feeding | 23:57 | |
| pshige | sushi? | 23:58 |
| BadCub | pshige: Bento Box of yumminess | 23:59 |
| pshige | good | 23:59 |
| BadCub | :) | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!