| *** r-daneel has joined #openstack-ironic | 00:09 | |
| *** achanda has joined #openstack-ironic | 00:11 | |
| *** mtanino_ has quit IRC | 00:13 | |
| *** tsekiyama has quit IRC | 00:14 | |
| *** [1]cdearborn has joined #openstack-ironic | 00:17 | |
| *** bnemec has quit IRC | 00:18 | |
| *** Haomeng has joined #openstack-ironic | 00:19 | |
| *** achanda has quit IRC | 00:19 | |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #openstack-ironic | 00:23 | |
| *** achanda has joined #openstack-ironic | 00:25 | |
| *** smoriya has joined #openstack-ironic | 00:27 | |
| *** puranamr has quit IRC | 00:36 | |
| *** asd112z has joined #openstack-ironic | 00:41 | |
| *** [1]cdearborn has quit IRC | 00:43 | |
| *** natorious is now known as zz_natorious | 00:46 | |
| *** bnemec has joined #openstack-ironic | 00:47 | |
| *** Marga_ has joined #openstack-ironic | 00:51 | |
| *** Haomeng|2 has joined #openstack-ironic | 00:52 | |
| *** praneshp has quit IRC | 00:53 | |
| *** shoutm has joined #openstack-ironic | 00:54 | |
| *** Marga_ has quit IRC | 00:54 | |
| *** Marga_ has joined #openstack-ironic | 00:55 | |
| *** Haomeng has quit IRC | 00:55 | |
| *** shoutm_ has quit IRC | 00:57 | |
| *** Marga_ has quit IRC | 01:00 | |
| *** Marga_ has joined #openstack-ironic | 01:00 | |
| *** meghal has joined #openstack-ironic | 01:03 | |
| *** zhenguo has joined #openstack-ironic | 01:05 | |
| *** meghal has quit IRC | 01:05 | |
| *** shoutm has quit IRC | 01:10 | |
| *** shoutm has joined #openstack-ironic | 01:14 | |
| *** meghal has joined #openstack-ironic | 01:15 | |
| *** meghal has quit IRC | 01:17 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Update iRMC vmedia spec to reflect changes made during code review https://review.openstack.org/217452 | 01:21 |
|---|---|---|
| *** jcoufal has joined #openstack-ironic | 01:25 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add node-create example to enroll section of install-guide https://review.openstack.org/209251 | 01:29 |
| *** jcoufal has quit IRC | 01:29 | |
| *** r-daneel has quit IRC | 01:35 | |
| *** bnemec has quit IRC | 01:41 | |
| *** chenglch has joined #openstack-ironic | 01:42 | |
| *** dims has joined #openstack-ironic | 01:43 | |
| *** chenglch has quit IRC | 01:47 | |
| *** chenglch has joined #openstack-ironic | 01:49 | |
| *** ndipanov has quit IRC | 02:00 | |
| *** rloo has quit IRC | 02:06 | |
| openstackgerrit | lei zhang proposed openstack/ironic: Add port type on port option https://review.openstack.org/217019 | 02:15 |
| *** Marga_ has quit IRC | 02:28 | |
| *** asd112z has quit IRC | 02:50 | |
| *** hakimo_ has joined #openstack-ironic | 02:52 | |
| *** hakimo has quit IRC | 02:55 | |
| *** dims has quit IRC | 02:56 | |
| *** zz_natorious is now known as natorious | 03:02 | |
| openstackgerrit | gaoxiaoyong proposed openstack/ironic: Revise "needs value" to "need a value" in "/ironic/api/controllers/v1/types.py" https://review.openstack.org/217007 | 03:05 |
| *** yuanying_ has joined #openstack-ironic | 03:06 | |
| *** ramineni has joined #openstack-ironic | 03:06 | |
| *** yuanying_ has quit IRC | 03:07 | |
| *** yuanying has quit IRC | 03:10 | |
| *** ramineni has quit IRC | 03:11 | |
| *** ramineni has joined #openstack-ironic | 03:12 | |
| *** ramineni1 has joined #openstack-ironic | 03:13 | |
| *** ramineni2 has joined #openstack-ironic | 03:14 | |
| *** natorious is now known as zz_natorious | 03:16 | |
| *** ramineni has quit IRC | 03:16 | |
| *** ramineni1 has quit IRC | 03:18 | |
| *** ramineni2 has quit IRC | 03:19 | |
| *** Nisha has joined #openstack-ironic | 03:24 | |
| *** ramineni has joined #openstack-ironic | 03:33 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix minor spelling errors https://review.openstack.org/217007 | 03:34 |
| *** ramineni1 has joined #openstack-ironic | 03:35 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix minor spelling/grammar errors https://review.openstack.org/217007 | 03:36 |
| *** ramineni2 has joined #openstack-ironic | 03:37 | |
| *** ramineni has quit IRC | 03:38 | |
| *** ramineni has joined #openstack-ironic | 03:39 | |
| *** ramineni1 has quit IRC | 03:40 | |
| *** ramineni1 has joined #openstack-ironic | 03:41 | |
| *** ramineni2 has quit IRC | 03:42 | |
| *** ramineni_ has joined #openstack-ironic | 03:43 | |
| *** ramineni has quit IRC | 03:44 | |
| *** alex_xu has quit IRC | 03:45 | |
| *** ramineni1 has quit IRC | 03:45 | |
| *** alex_xu has joined #openstack-ironic | 03:46 | |
| *** irenab has joined #openstack-ironic | 04:00 | |
| *** yuanying has joined #openstack-ironic | 04:01 | |
| *** jamielennox is now known as jamielennox|away | 04:04 | |
| *** jamielennox|away is now known as jamielennox | 04:05 | |
| *** achanda has quit IRC | 04:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/216562 | 04:14 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/217150 | 04:14 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/216966 | 04:14 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/217481 | 04:14 |
| openstackgerrit | lei zhang proposed openstack/ironic: Add mininum and maxinum on port option https://review.openstack.org/217019 | 04:19 |
| *** puranamr has joined #openstack-ironic | 04:27 | |
| *** yuanying has quit IRC | 04:32 | |
| *** Nisha has quit IRC | 04:37 | |
| *** yuanying has joined #openstack-ironic | 04:37 | |
| *** links has joined #openstack-ironic | 04:38 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Make end-points discoverable via Ironic API https://review.openstack.org/205895 | 04:39 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Added CORS support middleware to Ironic https://review.openstack.org/199769 | 04:43 |
| *** achanda has joined #openstack-ironic | 04:44 | |
| *** puranamr has quit IRC | 04:50 | |
| *** irenab has quit IRC | 05:05 | |
| *** blakec has joined #openstack-ironic | 05:07 | |
| *** openstackgerrit has quit IRC | 05:16 | |
| *** openstackgerrit has joined #openstack-ironic | 05:17 | |
| *** lazy_prince has joined #openstack-ironic | 05:36 | |
| *** smoriya has quit IRC | 05:38 | |
| *** dims has joined #openstack-ironic | 05:46 | |
| *** enikanorov has joined #openstack-ironic | 05:49 | |
| *** Marga_ has joined #openstack-ironic | 05:51 | |
| *** Marga_ has quit IRC | 05:51 | |
| *** Marga_ has joined #openstack-ironic | 05:51 | |
| *** dims has quit IRC | 05:52 | |
| *** blakec has quit IRC | 05:56 | |
| *** yuanying has quit IRC | 05:58 | |
| lintan | Hi, rameshg87 | 06:01 |
| *** yuikotakada has joined #openstack-ironic | 06:55 | |
| *** saripurigopi has joined #openstack-ironic | 06:59 | |
| *** praneshp has joined #openstack-ironic | 07:00 | |
| *** blakec has joined #openstack-ironic | 07:05 | |
| *** achanda has quit IRC | 07:06 | |
| *** blakec1 has joined #openstack-ironic | 07:10 | |
| *** blakec has quit IRC | 07:13 | |
| *** ifarkas has joined #openstack-ironic | 07:13 | |
| *** e0ne has joined #openstack-ironic | 07:18 | |
| *** praneshp has quit IRC | 07:35 | |
| *** dims has joined #openstack-ironic | 07:35 | |
| *** mgoddard has joined #openstack-ironic | 07:36 | |
| *** dims has quit IRC | 07:40 | |
| *** achanda has joined #openstack-ironic | 07:51 | |
| *** blakec1 has quit IRC | 07:55 | |
| *** shoutm_ has joined #openstack-ironic | 07:57 | |
| *** shoutm has quit IRC | 08:00 | |
| *** e0ne has quit IRC | 08:01 | |
| *** yuanying has joined #openstack-ironic | 08:03 | |
| *** karimb has joined #openstack-ironic | 08:09 | |
| *** boris-42 has quit IRC | 08:10 | |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/bifrost: Correct bug with default value where required https://review.openstack.org/217109 | 08:15 |
| *** jistr has joined #openstack-ironic | 08:15 | |
| *** lucasagomes has joined #openstack-ironic | 08:22 | |
| *** ukalifon1 has joined #openstack-ironic | 08:28 | |
| *** karimb has quit IRC | 08:29 | |
| *** ndipanov has joined #openstack-ironic | 08:30 | |
| *** shoutm_ has quit IRC | 08:32 | |
| *** pelix has joined #openstack-ironic | 08:33 | |
| *** derekh has joined #openstack-ironic | 08:40 | |
| openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/217204 | 08:40 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix logging and exceptions messages in ipminative driver https://review.openstack.org/217071 | 08:42 |
| *** dlpartain has joined #openstack-ironic | 08:48 | |
| *** dlpartain has quit IRC | 08:49 | |
| *** achanda has quit IRC | 08:50 | |
| *** jistr has quit IRC | 08:54 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Base IronicObject on VersionedObject https://review.openstack.org/213602 | 08:55 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Use oslo.versionedobjects remotable decorators https://review.openstack.org/217000 | 08:55 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Using ObjectRegistry to register class https://review.openstack.org/217584 | 08:55 |
| *** jistr has joined #openstack-ironic | 08:56 | |
| *** e0ne has joined #openstack-ironic | 08:59 | |
| *** wuhg has joined #openstack-ironic | 09:00 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Fix functional tests job https://review.openstack.org/217135 | 09:01 |
| *** jistr has quit IRC | 09:01 | |
| *** jistr has joined #openstack-ironic | 09:05 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add vendor interface to ipminative driver https://review.openstack.org/215185 | 09:11 |
| *** dtantsur|afk is now known as dtantsur | 09:12 | |
| dtantsur | Morning Ironic | 09:12 |
| yuikotakada | dtantsur, morning :) | 09:12 |
| dtantsur | o/ | 09:12 |
| *** UForgotten has quit IRC | 09:13 | |
| dtantsur | yuikotakada, looks like none of inspector talks were approved.. | 09:16 |
| yuikotakada | dtantsur, yeah :( Otherwise I expected... | 09:16 |
| dtantsur | yeah... | 09:17 |
| dtantsur | not so many talks at all, I would say | 09:17 |
| dtantsur | well, we can have a presentation on Friday, like in Vancouver :) | 09:18 |
| dtantsur | unfortunately it means that trown is not coming | 09:18 |
| dtantsur | (I suppose) | 09:18 |
| yuikotakada | dtantsur, this Friday? oh :D In fact, I will do in LinuxCon EU | 09:18 |
| dtantsur | yuikotakada, I meant summit Friday, during the contributor meetup | 09:19 |
| yuikotakada | dtantsur, ah :D yup, we can | 09:19 |
| openstackgerrit | Merged openstack/ironic: Add links for UEFI secure boot support to iLO driver documentation https://review.openstack.org/216528 | 09:22 |
| openstackgerrit | Sam Betts proposed openstack/ironic-specs: CIMC PXE Drivers for Ironic https://review.openstack.org/217598 | 09:29 |
| *** MattMan has quit IRC | 09:29 | |
| *** chenglch has quit IRC | 09:38 | |
| marios | hello ironic - i have a possibly silly question but i am stuck trying to find the answer. Is it the case that the ironic driver doesn't support specification of a /swap space when deploying a nova instance (i.e. --swap to nova boot cli). | 09:44 |
| marios | dtantsur: lucasagomes ^^^ thanks guys | 09:44 |
| lucasagomes | marios, we do support creating a swap partiton, tho I haven't tried the --swap option | 09:45 |
| lucasagomes | we use the swap that is specified in the flavo | 09:45 |
| lucasagomes | flavor* | 09:45 |
| lucasagomes | maybe the --swap parameter is broken for us and we will have to fix it | 09:45 |
| lucasagomes | marios, https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L63-L64 | 09:46 |
| marios | lucasagomes: ok, good to know. i have tried with a flavor that specified swap but couldn't get it to work | 09:46 |
| lucasagomes | marios, right I will take a look | 09:46 |
| dtantsur | oh | 09:46 |
| lucasagomes | marios, thanks btw, I will build an env and give it a try with swap | 09:47 |
| marios | lucasagomes: i though maybe this was related (and a recent thing) | 09:47 |
| marios | http://specs.openstack.org/openstack/ironic-specs/specs/backlog/partition-image-support-for-agent-driver.html | 09:47 |
| marios | (some things still pending there) | 09:47 |
| lucasagomes | marios, right. So Ironic can deploy 2 types of images: Full disk or partition images | 09:48 |
| lucasagomes | currently the agent_ drivers only supports Full disk images | 09:48 |
| lucasagomes | so if you're using it you will have to create the swap partition at image building time | 09:48 |
| marios | lucasagomes: right. thanks this confirms what i have come to understand, well it increases my understanding slightly anyway ;) | 09:48 |
| lucasagomes | :-) | 09:48 |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/217150 | 09:49 |
| *** dims has joined #openstack-ironic | 09:50 | |
| *** naohirot has quit IRC | 09:56 | |
| *** VikasC has joined #openstack-ironic | 09:58 | |
| *** MattMan has joined #openstack-ironic | 10:00 | |
| *** ramineni_ has quit IRC | 10:00 | |
| *** yuikotakada has quit IRC | 10:05 | |
| *** jistr has quit IRC | 10:06 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/217203 | 10:08 |
| *** zhenguo has quit IRC | 10:10 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Make our README friendly to OpenStack release-tools https://review.openstack.org/216799 | 10:12 |
| *** maurosr has quit IRC | 10:14 | |
| *** maurosr has joined #openstack-ironic | 10:17 | |
| *** jistr has joined #openstack-ironic | 10:18 | |
| openstackgerrit | Merged openstack/ironic-inspector: Clean up README https://review.openstack.org/216797 | 10:19 |
| *** Marga_ has quit IRC | 10:24 | |
| openstackgerrit | Merged openstack/ironic: format links in the readme to work with the release notes tools https://review.openstack.org/216758 | 10:28 |
| *** Haomeng|2 has quit IRC | 10:39 | |
| *** PradeepV has joined #openstack-ironic | 10:56 | |
| PradeepV | Hi Ironic, I created a image using "disk-image-create ubuntu vm" , what is the default username of this image, how can we login into the baremetal instance created through this image | 10:58 |
| lucasagomes | PradeepV, I don't think it has one? You will need cloud-init to setup a key for you | 11:03 |
| lucasagomes | PradeepV, you can use other elements like "stackuser" or "devuser" (I think that' show they are called) to create a user for you | 11:03 |
| lucasagomes | ask at #tripleo they may help you better than I can | 11:03 |
| mrda | PradeepV: you want to include 'local-config' so it'll include ~/.ssh/id_rsa.pub as an authorized_keys entry | 11:03 |
| mrda | then it should allow you to 'ssh root@yournewhost' using your ssh key for auth | 11:04 |
| mrda | of course lucasagomes' suggestion is also great :-) | 11:05 |
| lucasagomes | mrda, good morning! | 11:06 |
| mrda | lucasagomes: for my it's almost bed :) | 11:06 |
| mrda | s/my/me/ | 11:06 |
| lucasagomes | mrda, oh I mean UGT | 11:06 |
| lucasagomes | mrda, good evening :-) UGT | 11:07 |
| * mrda doesn't know what timezone UGT is :) | 11:07 | |
| lucasagomes | mrda, http://www.total-knowledge.com/~ilya/mips/ugt.html | 11:07 |
| mrda | Nice to overlap with you lucasagomes | 11:07 |
| PradeepV | lucasagomes and mrda, thank you for your help | 11:07 |
| PradeepV | i will try | 11:07 |
| openstackgerrit | Merged openstack/ironic: Remove broken workaround code for old mock. https://review.openstack.org/216444 | 11:07 |
| lucasagomes | mrda, yeah man! So you're back to australia? How was the texan holidays? | 11:08 |
| mrda | oh :) | 11:08 |
| mrda | lucasagomes: Seattle and San Antonio were nice | 11:08 |
| mrda | Good to see a lot of the Ironic gang | 11:08 |
| lucasagomes | yeah! looking fwd to see the folks in Tokyo | 11:08 |
| mrda | Looking forward to hanging out with you again of course | 11:09 |
| lucasagomes | ++ | 11:09 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Fixes config drive creation failure in UEFI boot mode https://review.openstack.org/215097 | 11:17 |
| PradeepV | Hi mrda: how can i include the local-config, can you give more inputs | 11:17 |
| *** alex_xu has quit IRC | 11:20 | |
| *** dlpartain has joined #openstack-ironic | 11:21 | |
| mrda | disk-image-create ubuntu baremetal dhcp-all-interfaces local-config -o my-image | 11:21 |
| mrda | PradeepV: ^^ | 11:21 |
| PradeepV | thank you mrda: :) | 11:22 |
| *** alex_xu has joined #openstack-ironic | 11:23 | |
| PradeepV | will it include cloud init package also | 11:23 |
| *** dlpartain has left #openstack-ironic | 11:24 | |
| mrda | I think local-config does | 11:24 |
| mrda | But I'm not an expert in this, I just try things until they work :) | 11:25 |
| *** UForgotten has joined #openstack-ironic | 11:26 | |
| *** dprince has joined #openstack-ironic | 11:34 | |
| sonuk | kindly review https://review.openstack.org/#/c/213654/. Comments are most welcome. | 11:41 |
| *** e0ne has quit IRC | 11:42 | |
| *** sonuk has quit IRC | 11:43 | |
| *** VikasC has quit IRC | 11:44 | |
| *** saripurigopi has quit IRC | 11:44 | |
| *** thrash|g0ne is now known as thrash | 11:45 | |
| *** lucasagomes is now known as lucas-hungry | 11:48 | |
| *** trown|outttypeww is now known as trown | 11:53 | |
| *** getvasanth has joined #openstack-ironic | 11:55 | |
| openstackgerrit | Lucio Seki proposed stackforge/pyghmi: Add reset bmc command https://review.openstack.org/217670 | 11:56 |
| openstackgerrit | Lucio Seki proposed stackforge/pyghmi: Add NMI diag command https://review.openstack.org/217671 | 11:56 |
| getvasanth | Has anyone resolved this error "No VIFs found for node" | 11:56 |
| trown | good UGT morning Ironic | 12:01 |
| getvasanth | trown: Good Morning | 12:01 |
| openstackgerrit | Lucio Seki proposed stackforge/pyghmi: Add NMI diag command https://review.openstack.org/217676 | 12:05 |
| dtantsur | morning getvasanth, trown | 12:08 |
| dtantsur | trown, looks like we're out of luck with our talks :( | 12:08 |
| getvasanth | dtantsur: Has anyone resolved this error "No VIFs found for node" | 12:09 |
| dtantsur | getvasanth, could you give some context? | 12:09 |
| openstackgerrit | John Trowbridge proposed openstack/ironic-inspector: Store and expose introspection data https://review.openstack.org/213159 | 12:09 |
| trown | dtantsur: ya I saw :( | 12:10 |
| getvasanth | dtansur: I have openstack setup with ironic, and now i am trying to boot the baremetal | 12:10 |
| trown | getvasanth: are you trying to boot directly from Ironic? Nova sets up the VIFs | 12:10 |
| getvasanth | dtantsur: at that time Ironic-conductor throws an error saying "No VIFs found for node" | 12:10 |
| *** viktors has joined #openstack-ironic | 12:11 | |
| dtantsur | getvasanth, are you using nova? | 12:11 |
| getvasanth | dtantsur: you mean, i have to boot with "nova boot"? | 12:11 |
| getvasanth | dtantsur: Now i am trying to boot driectly | 12:12 |
| trown | getvasanth: ya that will not work without manually creating the VIFs | 12:12 |
| getvasanth | trown: how do i create it manually? | 12:13 |
| trown | getvasanth: you could look at https://github.com/openstack/bifrost to get an idea how to do that | 12:13 |
| trown | getvasanth: or you could use nova | 12:13 |
| getvasanth | trown: thanks for the link, I will check that and get back to you, | 12:13 |
| getvasanth | dtantsur: thanks | 12:13 |
| *** dims has quit IRC | 12:14 | |
| getvasanth | trown: which is better to use nova to boot the baremetal? | 12:15 |
| trown | getvasanth: that is simpler if you have nova in your environment | 12:15 |
| trown | bifrost is a project that uses Ironic without nova | 12:15 |
| getvasanth | trown: i have nova installed, but didnt try to boot with it, is there any docs to follow, other than the install guide? | 12:16 |
| trown | getvasanth: http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html | 12:17 |
| trown | getvasanth: the devstack section has some example `nova boot` commands at the end | 12:17 |
| *** jcoufal has joined #openstack-ironic | 12:19 | |
| getvasanth | trown: thanks for the link, i will go through it | 12:19 |
| TheJulia | good morning | 12:20 |
| sinval | good morning TheJulia and Ironicers | 12:24 |
| getvasanth | Good morning | 12:24 |
| *** mtanino has joined #openstack-ironic | 12:24 | |
| dtantsur | TheJulia, sinval, morning | 12:28 |
| *** dims has joined #openstack-ironic | 12:29 | |
| *** openstackgerrit has quit IRC | 12:31 | |
| *** openstackgerrit has joined #openstack-ironic | 12:32 | |
| *** Marga_ has joined #openstack-ironic | 12:35 | |
| *** jcoufal has quit IRC | 12:35 | |
| *** lucas-hungry is now known as lucasagomes | 12:37 | |
| NobodyCam | good morning TheJulia lucas-hungry dtantsur trown sambetts sinval | 12:37 |
| *** amotoki_ has quit IRC | 12:37 | |
| NobodyCam | its early here.. :-p | 12:37 |
| dtantsur | NobodyCam, morning! | 12:37 |
| NobodyCam | :) | 12:38 |
| dtantsur | just wanted to say it must be early for you :) | 12:38 |
| NobodyCam | yea.. like 5:30 am :-p | 12:38 |
| NobodyCam | so sssshhhhh I'm not here yet :-p | 12:38 |
| BadCub | morning dtantsur NobodyCam lucasagomes TheJulia trown sambetts sinval | 12:38 |
| dtantsur | morning BadCub | 12:39 |
| lucasagomes | BadCub, NobodyCam TheJulia sinval getvasanth trown good ugt morning! | 12:39 |
| * lucasagomes hopes he didn't forget anyone | 12:39 | |
| BadCub | hehehehe | 12:39 |
| NobodyCam | :) | 12:39 |
| *** wuhg has quit IRC | 12:41 | |
| *** puranamr has joined #openstack-ironic | 12:42 | |
| *** PradeepV has quit IRC | 12:43 | |
| *** e0ne has joined #openstack-ironic | 12:44 | |
| *** afaranha has joined #openstack-ironic | 12:44 | |
| *** jcoufal has joined #openstack-ironic | 12:49 | |
| *** puranamr has quit IRC | 12:50 | |
| *** puranamr has joined #openstack-ironic | 12:52 | |
| *** Marga_ has quit IRC | 12:56 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Resync with openstack/requirements https://review.openstack.org/216681 | 12:56 |
| *** baoli has joined #openstack-ironic | 12:59 | |
| *** linggao has joined #openstack-ironic | 12:59 | |
| *** rloo has joined #openstack-ironic | 13:05 | |
| *** Marga_ has joined #openstack-ironic | 13:07 | |
| *** jjohnson2 has joined #openstack-ironic | 13:09 | |
| *** mtanino has quit IRC | 13:11 | |
| sambetts | Morning NobodyCam, BadCub o/ | 13:13 |
| *** nicodemos has joined #openstack-ironic | 13:15 | |
| *** puranamr has quit IRC | 13:15 | |
| NobodyCam | mornign sambetts :) | 13:16 |
| dtantsur | sambetts, morning | 13:18 |
| sambetts | o/ dtantsur | 13:19 |
| *** ukalifon1 has quit IRC | 13:20 | |
| *** viktors is now known as viktors|away | 13:24 | |
| *** zhenguo has joined #openstack-ironic | 13:25 | |
| *** ukalifon1 has joined #openstack-ironic | 13:27 | |
| *** Nisha_away has joined #openstack-ironic | 13:32 | |
| zigo | Hi there! | 13:32 |
| zigo | I'm trying to build a package for ironic-discoverd. | 13:32 |
| dtantsur | zigo, hi! for Kilo? | 13:32 |
| zigo | When running unit tests, I have this issue: http://paste.debian.net/307319/ | 13:32 |
| zigo | Though when I install python-ironic-discoverd (ie: the python code for it), then unit tests are passing. | 13:33 |
| zigo | dtantsur: Yes, for Kilo. | 13:33 |
| zigo | So version 1.1.0. | 13:33 |
| zigo | I've added PYTHONPATH=. but this doesn't help. | 13:33 |
| zigo | How can I run unit tests when building package without these errors then? | 13:33 |
| getvasanth | trown: hi, i am trying too boot through nova now, but I get a strange error "No valid host was found. there are not enough hosts available" | 13:34 |
| trown | getvasanth: that would mean that nova scheduler can not match any nodes to your boot request | 13:35 |
| getvasanth | trown: instance fails to spawn | 13:35 |
| dtantsur | zigo, sorry, I don't quite get you. Unit tests fail when building a package, but don't fail if you install a package and run them from it? | 13:36 |
| zigo | dtantsur: Unit tests fail when building a package, but don't fail if you install a package and THEN build the package again. | 13:36 |
| zigo | (ie: not running the unit tests from the installed package, but really when building it from the source directory) | 13:37 |
| getvasanth | trown: how does nova knows which one to boot | 13:37 |
| *** thiagop has joined #openstack-ironic | 13:37 | |
| zigo | So something is trying to get stuff form /usr/lib/python2.7/dist-packages/ironic_discoverd | 13:37 |
| thiagop | good morning Ironic | 13:38 |
| dtantsur | zigo, mmm, I think it's because tests rely on setuptools entry points, so you have to install tests somewhere (be it destination directory or temporary build directory) | 13:39 |
| dtantsur | (install everything, not just tests, sorry) | 13:39 |
| dtantsur | thiagop, morning | 13:39 |
| zigo | dtantsur: So you mean, it's trying to call /usr/bin/ironic-discoverd ? | 13:39 |
| getvasanth | trown: in nova.conf driver is set to ironic, I am not understanding this. | 13:39 |
| thiagop | dtantsur: o/ | 13:40 |
| dtantsur | zigo, no. we (like many other projects) use stevedore and setuptools entry points to attach plugins | 13:40 |
| dtantsur | zigo, tests that fail for you use plugins. but do find them, meta information is requires, that is generated on installing | 13:41 |
| trown | getvasanth: could you put up a paste of `ironic node-show <uuid>` for one of the nodes, as well as the nova boot command you ran and `nova flavor-show <uuid>` for the flavor you are using? | 13:43 |
| trown | getvasanth: there is some mismatch between the flavor and your nodes | 13:43 |
| getvasanth | trown: here is the info http://pastebin.ca/3131551 | 13:45 |
| trown | getvasanth: oh...the node is in "deploy failed" state...presumably from your earlier attempts | 13:49 |
| getvasanth | trown: oh! what do i do now | 13:49 |
| trown | getvasanth: you need to move it to manageable and the available: | 13:50 |
| trown | ironic node-set-provision-state 9833d3ad-a282-4c04-9615-a9aa19e28153 manage | 13:50 |
| trown | ironic node-set-provision-state 9833d3ad-a282-4c04-9615-a9aa19e28153 provide | 13:50 |
| trown | getvasanth: then you can retry nova boot | 13:51 |
| getvasanth | trwon: i dont have the option manage | 13:51 |
| getvasanth | [root@ironic ~(keystone_admin)]# ironic node-set-provision-state 9833d3ad-a282-4c04-9615-a9aa19e28153 manage | 13:51 |
| getvasanth | usage: ironic node-set-provision-state <node id> <provision state> | 13:51 |
| getvasanth | ironic node-set-provision-state: error: argument <provision state>: invalid choice: 'manage' (choose from 'active', 'deleted', 'rebuild') | 13:51 |
| *** mtanino has joined #openstack-ironic | 13:52 | |
| *** mjturek1 has quit IRC | 13:53 | |
| trown | getvasanth: I think you can go from deleted to manageable | 13:53 |
| *** mjturek1 has joined #openstack-ironic | 13:54 | |
| getvasanth | trown: ok will delete and do | 13:54 |
| trown | getvasanth: it also looks like local_gb on the node is very small...did you manually populate the properties? | 13:55 |
| getvasanth | trown: yes | 13:55 |
| trown | getvasanth: so local_gb is 10, but m1.small has disk of 20 | 13:55 |
| trown | so it will not match | 13:55 |
| getvasanth | trown: updated to 20 now, | 13:57 |
| getvasanth | trown: i didnt understand the manage & provide parts | 13:57 |
| trown | getvasanth: nova will only schedule to nodes in available state | 13:57 |
| trown | getvasanth: provide is the verb that puts a node in available state | 13:58 |
| getvasanth | trown: i think i need to put the node in maintenance mode and then bring it back | 14:00 |
| trown | getvasanth: what provision state is it in now? | 14:02 |
| trown | getvasanth: can you run `ironic node-set-provision-state 9833d3ad-a282-4c04-9615-a9aa19e28153 manage` | 14:02 |
| getvasanth | trown: provision state is none | 14:02 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add vendor interface to ipminative driver https://review.openstack.org/215185 | 14:02 |
| trown | getvasanth: you may even be able to go from none directly to available... | 14:04 |
| trown | ironic node-set-provision-state 9833d3ad-a282-4c04-9615-a9aa19e28153 provide | 14:04 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Correct bug with default value where required https://review.openstack.org/217109 | 14:04 |
| getvasanth | trown: http://pastebin.ca/3131589 its not moving to provide | 14:05 |
| trown | getvasanth: its in 'available' thats what we want | 14:06 |
| trown | getvasanth: it is a bit confusing because the verb is 'provide' but the target state for that verb is 'available' | 14:06 |
| getvasanth | trown: yes little confusing | 14:06 |
| *** krtaylor has quit IRC | 14:07 | |
| *** viktors|away is now known as viktors | 14:14 | |
| *** shoutm has joined #openstack-ironic | 14:18 | |
| openstackgerrit | John Trowbridge proposed openstack/ironic-inspector: Store and expose introspection data https://review.openstack.org/213159 | 14:18 |
| openstackgerrit | Merged stackforge/pyghmi: Add NMI diag command https://review.openstack.org/217676 | 14:20 |
| *** romainh has joined #openstack-ironic | 14:21 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add pluggable credentials storage https://review.openstack.org/186056 | 14:28 |
| getvasanth | trown: this is the nova boot command"nova boot --flavor m1.small --nic net-id=1e58ad20-f3eb-44fd-ae32-2e7a0c02ff9e --image=cc805134-7260-4060-9300-1e5600f841e5 first | 14:31 |
| getvasanth | " | 14:31 |
| *** achanda has joined #openstack-ironic | 14:31 | |
| rloo | trown, getvasanth: skimmed ^^, my guess is that getvasanth is using an older (micro)version for server and/or client. | 14:33 |
| getvasanth | rloo: my controller node is a vm | 14:34 |
| rloo | trown, getvasanth: the 'manage' and 'provide' verbs were added to the client in Jan. Can't recall if we did a release with that; otherwise it is master. | 14:34 |
| *** krtaylor has joined #openstack-ironic | 14:35 | |
| *** r3mu5 has joined #openstack-ironic | 14:35 | |
| getvasanth | rloo: my ironic client version is 0.3.1 | 14:35 |
| *** Nisha_away has quit IRC | 14:35 | |
| *** romainh has left #openstack-ironic | 14:36 | |
| rloo | getvasanth: 0.7.0 seems to be the latest: https://pypi.python.org/pypi/python-ironicclient | 14:36 |
| *** links has quit IRC | 14:37 | |
| rloo | getvasanth: which ironic version are you using? | 14:37 |
| rloo | getvasanth: 0.3.1 is from october. are you using ironic-kilo? | 14:38 |
| getvasanth | rloo: yes it is in kilo and rhel7 | 14:38 |
| getvasanth | rloo: i am using kilo version | 14:39 |
| rloo | trown: ^^. | 14:39 |
| rloo | getvasanth: that explains why trown's instructions don't work. | 14:39 |
| getvasanth | rloo: understood that, but how do i proceed further with this version | 14:40 |
| *** achanda has quit IRC | 14:40 | |
| getvasanth | rloo: its keep throwing the "No VIFs found error" | 14:41 |
| getvasanth | rloo: its driving me crazy.. | 14:41 |
| rloo | getvasanth: so nova managed to get an ironic node? | 14:41 |
| rloo | getvasanth: I know almost nothing about the networking part. | 14:41 |
| getvasanth | rloo: me too :( | 14:42 |
| getvasanth | rloo: how do i check that nova has deducted the ironic node? | 14:42 |
| *** ukalifon1 has quit IRC | 14:43 | |
| *** Nisha has joined #openstack-ironic | 14:44 | |
| getvasanth | rloo: nova hypervisor-list shows the ironic node id | 14:45 |
| getvasanth | rloo: and status is also enabled | 14:45 |
| NobodyCam | good morning rloo :) | 14:46 |
| *** zz_natorious is now known as natorious | 14:47 | |
| rloo | morning NobodyCam | 14:47 |
| rloo | getvasanth: not sure what you mean by 'deducted'. | 14:47 |
| NobodyCam | :) | 14:48 |
| openstackgerrit | Merged openstack/ironic: Use JsonEncoded{Dict,List} from oslo_db https://review.openstack.org/215739 | 14:48 |
| getvasanth | rloo: "so nova managed to get an ironic node?" nova is able to see the ironic node | 14:49 |
| getvasanth | thats what i meant | 14:49 |
| *** dims has quit IRC | 14:51 | |
| rloo | getvasanth: sorry, it isn't clear to me what your problem is. | 14:51 |
| *** dims has joined #openstack-ironic | 14:51 | |
| rloo | getvasanth: sorry, gotta take off for a bit. maybe someone else can help | 14:52 |
| getvasanth | rloo: sorry, i am trying to boot the with nova, but i am getting into error loops one after another, | 14:52 |
| getvasanth | rloo: thanks for the help | 14:52 |
| *** davideagnello has quit IRC | 14:56 | |
| *** krtaylor has quit IRC | 15:02 | |
| thiagop | Folks, if you have some time to take a look on the OneView driver, we'll appreciate: https://review.openstack.org/#/c/191822/ | 15:07 |
| jlvillal | dtantsur, lucasagomes: I have updated patch based on your comments: https://review.openstack.org/#/c/213890/ If you have time to look :) | 15:07 |
| dtantsur | ack | 15:08 |
| lucasagomes | jlvillal, thanks will do | 15:08 |
| jlvillal | Thanks | 15:08 |
| openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add reset bmc command https://review.openstack.org/217759 | 15:12 |
| NobodyCam | thiagop: congratz! | 15:13 |
| Nisha | yuriyz, hi | 15:13 |
| yuriyz | hi Nisha | 15:14 |
| *** davideagnello has joined #openstack-ironic | 15:14 | |
| thiagop | Hi NobodyCam, thanks. But why? | 15:14 |
| NobodyCam | the internal email I just got | 15:14 |
| Nisha | yuriyz, I saw your comments on the patch https://review.openstack.org/#/c/217068/5/ironic/drivers/modules/ilo/deploy.py | 15:14 |
| thiagop | NobodyCam: I didn't receive the e-mails directly, just by proxy (meaning Andrey or Erasmo) | 15:15 |
| thiagop | *I don't* | 15:16 |
| thiagop | [bad day for english speaking] | 15:16 |
| Nisha | yuriyz, ILO doesnt allow the setting boot device and inserting virtual media unless the system is in power off state | 15:18 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 15:19 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 15:19 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 15:19 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 15:19 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: [POC] pxe: support boot from Cinder iSCSI volume using iPXE https://review.openstack.org/215849 | 15:19 |
| lucasagomes | nice ^ | 15:20 |
| yuriyz | Nisha maybe power off can be postponed, because reset of active node when conductor died is bad IMO | 15:20 |
| Nisha | yuriyz, i didnt get "maybe power off can be postponed" | 15:21 |
| getvasanth | Even after the Nova is able to see the Ironic node and enabled, Not sure why it throw "No valid host was found" | 15:21 |
| getvasanth | and pointer will help | 15:22 |
| *** links has joined #openstack-ironic | 15:22 | |
| Nisha | yuriyz, i understand the concern but the ilo virtual media drivers need to do power off the system to attach the vmedia and set the bot device | 15:22 |
| *** blakec has joined #openstack-ironic | 15:24 | |
| openstackgerrit | Merged openstack/ironic-inspector: Fail introspection early if no lookup attributes can be detected https://review.openstack.org/216777 | 15:25 |
| getvasanth | trown: are you there, Even after the Nova is able to see the Ironic node and enabled, Not sure why it throw "No valid host was found" | 15:25 |
| trown | getvasanth: look at the nova scheduler logs | 15:26 |
| yuriyz | Nisha, if node in active state it already booted with some media, and instance is running, why we should do a reset/off at the same moment in take_over(), maybe we can do it later | 15:26 |
| trown | getvasanth: the reason will be there | 15:26 |
| *** r3mu5 has left #openstack-ironic | 15:28 | |
| getvasanth | trown: let me check | 15:28 |
| Nisha | yuriyz, ok let me test with the suggestion. I agree with the concern so let me see if that works | 15:31 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update documentation to install grub2 when creating the user image https://review.openstack.org/217137 | 15:32 |
| getvasanth | trown: there is no error messages in the logs, but "nova show 6d63cce7-8b96-4df8-b8cc-c17465f29085" says "No valid host found error" | 15:34 |
| getvasanth | trown: is there a way to check the nova is able to communicate with the ironic node? | 15:35 |
| *** vdrok has quit IRC | 15:35 | |
| *** Nisha has quit IRC | 15:35 | |
| *** tsekiyama has joined #openstack-ironic | 15:36 | |
| trown | getvasanth: nova does not communicate with the node | 15:37 |
| *** vdrok has joined #openstack-ironic | 15:37 | |
| trown | getvasanth: that is Ironic's job...nova just does the scheduling, and neutron VIF plugging | 15:38 |
| trown | getvasanth: you may need to enable debug logging...but there should be an indication of which host filter returned 0 hosts in the scheduler logs | 15:38 |
| *** Nisha has joined #openstack-ironic | 15:38 | |
| getvasanth | trown: ok, will enable the debug mode on nova and see | 15:39 |
| *** getvasanth__ has joined #openstack-ironic | 15:42 | |
| *** bnemec has joined #openstack-ironic | 15:42 | |
| *** shoutm has quit IRC | 15:43 | |
| *** Marga_ has quit IRC | 15:43 | |
| *** getvasanth has quit IRC | 15:43 | |
| *** romcheg1 has joined #openstack-ironic | 15:48 | |
| *** praneshp has joined #openstack-ironic | 15:49 | |
| *** Marga_ has joined #openstack-ironic | 15:49 | |
| *** praneshp_ has joined #openstack-ironic | 15:52 | |
| *** viktors is now known as viktors|afk | 15:53 | |
| *** ifarkas has quit IRC | 15:53 | |
| *** praneshp has quit IRC | 15:53 | |
| *** praneshp_ is now known as praneshp | 15:53 | |
| *** tideline has joined #openstack-ironic | 15:56 | |
| getvasanth__ | trown: got the issue, http://pastebin.ca/3131819 nova hypevisor-show says that the free_ram_mb is -512mb, where as the flavor is asking for 2gb to start the instance. | 15:57 |
| getvasanth__ | trown: ok now how do i make the nova aware that ironic node has 64gb of memory | 15:57 |
| lucasagomes | getvasanth__, just update the properties/memory_mb field in the Ironic node | 15:59 |
| lucasagomes | (it may take a while to be propagated to nova) | 15:59 |
| mariojv | hey ironic. i was working on https://bugs.launchpad.net/ironic/+bug/1435237 (ipa configdrive size) and wanted to get some feedback on a fix before i send it up for review | 15:59 |
| openstack | Launchpad bug 1435237 in Ironic "IPA: Configdrive can't be bigger than 64MB" [Medium,Triaged] - Assigned to Mario Villaplana (mario-villaplana-j) | 15:59 |
| mariojv | i basically just changed the script to check the config drive size and round up to the nearest MB, but i'm not getting optimal partition alignments when that happens: https://github.com/supermari0/ironic-python-agent/commit/bf3349f4c9a5696a280d2665a0c5733f1aa12f5f | 16:00 |
| mariojv | and i'm not really sure of a portable way to ensure that the alignment is optimal | 16:01 |
| rloo | lucasagomes, getvasanth__: free ram is -512mb. Is that 'normal'? Might be better off deleting that node from ironic and re-adding/creating to ironic. | 16:01 |
| mariojv | i've also heard that we shouldn't partition the last MB of the disk from the gentoo wiki, don't have any other sources on that though | 16:01 |
| lucasagomes | perhaps we are not using the exact ram filter ? | 16:02 |
| lucasagomes | s/we/he | 16:02 |
| JayF | mariojv: a thought; you could migrate that to python | 16:03 |
| JayF | mariojv: rather than doing more bashing | 16:03 |
| JayF | mariojv: but if you did that might wanna wait for the partition stuff in ironic-lib to land, but idk when that is | 16:03 |
| mariojv | JayF: you mean this? https://review.openstack.org/#/c/162162/ | 16:04 |
| getvasanth__ | lucasgomes: in ironic node it is added 64Gb as memory, but the nova says -512mb | 16:04 |
| JayF | mariojv: yeah, but that'll have to be merged, integrated with ipa, etc | 16:04 |
| getvasanth__ | lucasagomes:in ironic node it is added 64Gb as memory, but the nova says -512mb | 16:04 |
| JayF | like I don't think IPA uses ironic-lib yet | 16:04 |
| *** puranamr has joined #openstack-ironic | 16:04 | |
| JayF | the partitioning code in there will have to be pulled out and replaced with the lib | 16:04 |
| lucasagomes | getvasanth__, can you paste the node-show + nova hypervisor-stats commands output? | 16:06 |
| mariojv | JayF: i agree rewriting it in python would be best. that code needs to be made more portable so we can partition somewhere besides the start of the disk first, i'll start with that | 16:06 |
| JayF | mariojv: might wanna make sure you coordinate with whoever is doing the ironic-lib stuff | 16:07 |
| *** mgoddard has quit IRC | 16:07 | |
| JayF | lucasagomes: ^ do you know who's doing the work for ironic-lib if mariojv wanted to help integrate it with IPA? | 16:07 |
| mariojv | that doesn't really solve the optimal partition alignment problem, but having that abstracted away would be better than trying to hack something into a bash script | 16:07 |
| JayF | mariojv: well my hope would be that the code ironic uses for partitioning (now in ironic-lib as well) may already handle that case | 16:07 |
| *** mgoddard has joined #openstack-ironic | 16:08 | |
| lucasagomes | JayF, I think ramesh have be doing a lot of the ports from ironic to ironic-lib | 16:08 |
| lucasagomes | mariojv, ^ | 16:08 |
| *** trown is now known as trown|lunch | 16:09 | |
| *** getvasanth has joined #openstack-ironic | 16:09 | |
| *** getvasanth__ has quit IRC | 16:09 | |
| mariojv | JayF: actually the way it seems to work right now is that it just adds partitions in sequence contiguously, starting at 1MB offset for the partition table, then continuing | 16:12 |
| mariojv | but in IPA, we write to the end of disk then leave it to something else to expand the actual image partition | 16:13 |
| mariojv | s/write to the end of disk/write configdrive to the end of disk | 16:13 |
| JayF | the iscsi driver supports full disk images | 16:13 |
| JayF | does it not support full disk images + configdrive? | 16:13 |
| mariojv | i'm not sure, if it uses this then it seems to assume that the image partitions are already expanded as much as they need to be: https://github.com/openstack/ironic/blob/297f14469d23dd793de625251366380cc648fdb2/ironic/common/disk_partitioner.py#L142 | 16:14 |
| mariojv | cinerama: any insight into how iscsi + config drive currently works? we're thinking about using that in ironic-lib in IPA, but i think we'd need to make some modifications so that the configdrive gets written to the end of disk in IPA, instead of writing it immediately after the image like that would do | 16:16 |
| lucasagomes | JayF, I think that's a bug in the iscsi drivers | 16:16 |
| mariojv | using that - meaning using DiskPartitioner | 16:16 |
| lucasagomes | it doesn't seem to support full disk images with config drive partition | 16:16 |
| lucasagomes | :-( | 16:16 |
| JayF | Aha, so this is a problemset that hasn't been tackled at all | 16:16 |
| lucasagomes | at least the last time I looked at it | 16:16 |
| JayF | mariojv: my $.02, I think "magic number" partition sizes are better | 16:17 |
| JayF | mariojv: but as long as parted is run in -o optimal mode, it shouldn't allow you to make misaligned partitions on most hw | 16:17 |
| *** e0ne has quit IRC | 16:18 | |
| mariojv | it does on ours. maybe just try our best to do optimal partitions, and log a warning if it isn't | 16:18 |
| JayF | Hmm. I guess maybe doing powers of 2 is best then? I honestly don't know enough about partition alignment but I can't imagine that'd be wrong (even if space-wasteful) | 16:19 |
| mariojv | yeah, i'll have to test it. not guaranteed to make it work but it's better than just randomly guessing | 16:20 |
| mariojv | alternatively there's this with using sysfs, but again, haven't tested yet, probably not guaranteed to work everywhere, etc: http://rainbow.chard.org/2013/01/30/how-to-align-partitions-for-best-performance-using-parted/ | 16:20 |
| JayF | nice thing for ipa is | 16:20 |
| JayF | we know the place it's going to run *is* a controlled environment | 16:21 |
| mariojv | in any case switching what we currently have to python is good | 16:21 |
| mariojv | true | 16:21 |
| lucasagomes | +1 to switch from bash to python | 16:22 |
| lucasagomes | btw food for thought re partitioning: http://blivet.readthedocs.org/en/latest/intro.html | 16:24 |
| *** jistr has quit IRC | 16:24 | |
| lucasagomes | it's a python library, it's what is used for anaconda so is pretty much well tested | 16:24 |
| *** tideline_ has joined #openstack-ironic | 16:24 | |
| *** tideline has quit IRC | 16:25 | |
| *** romcheg1 has quit IRC | 16:27 | |
| *** marzif has joined #openstack-ironic | 16:27 | |
| *** Nisha_away has joined #openstack-ironic | 16:29 | |
| *** Nisha has quit IRC | 16:29 | |
| mariojv | i like it in general. it looks like it only allows you to select where on disk you put the partition if you specify the exact sectors, though | 16:29 |
| mariojv | so calling out to parted would probably be better | 16:29 |
| mariojv | but that can definitely be generalized in ironic-lib so the rest of our projects can use it | 16:30 |
| mariojv | i wonder if it would be useful to still have a config option or something to specify exact sizes for IPA users who don't want to use auto-detection of the configdrive size. brb | 16:32 |
| *** natorious is now known as zz_natorious | 16:35 | |
| *** tideline_ is now known as tideline | 16:36 | |
| lucasagomes | well configdrive size maxes out at 64MiB AFAICT | 16:37 |
| lucasagomes | so we can always create a 64MiB partition for it | 16:37 |
| * lucasagomes memory may be failing here | 16:38 | |
| *** boris-42 has joined #openstack-ironic | 16:39 | |
| *** zhenguo has quit IRC | 16:41 | |
| lucasagomes | folks I'm callign it a day | 16:43 |
| dtantsur | anybody here tried using neutron+ironic without nova? | 16:43 |
| *** derekh has quit IRC | 16:44 | |
| dtantsur | lucasagomes, g'night | 16:44 |
| mariojv | see ya lucasagomes \o | 16:44 |
| lucasagomes | came to the office, so have to get the train and all | 16:44 |
| lucasagomes | see y'all! | 16:44 |
| *** lucasagomes has quit IRC | 16:44 | |
| getvasanth | licasagomes: nova still says the ironic node has 0 mb of memory | 16:44 |
| *** links has quit IRC | 16:45 | |
| getvasanth | lucasagomes: any pointers please | 16:45 |
| mariojv | lucasagomes: you actually filed that bug btw, it's only guaranteed to be 64MB if it's in vfat format: http://docs.openstack.org/user-guide/cli_config_drive.html | 16:45 |
| mariojv | i highly doubt that it would ever be >64MB, though | 16:46 |
| *** praneshp has quit IRC | 16:48 | |
| *** tideline has quit IRC | 16:48 | |
| *** ndipanov has quit IRC | 16:50 | |
| *** krtaylor has joined #openstack-ironic | 16:52 | |
| *** albertoffb has joined #openstack-ironic | 16:54 | |
| *** marzif has quit IRC | 16:55 | |
| *** harlowja has joined #openstack-ironic | 16:55 | |
| openstackgerrit | greghaynes proposed openstack/bifrost: Support setting vlan in config drive https://review.openstack.org/216431 | 16:56 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 16:56 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 16:59 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 16:59 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 16:59 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: [POC] pxe: support boot from Cinder iSCSI volume using iPXE https://review.openstack.org/215849 | 16:59 |
| *** romcheg1 has joined #openstack-ironic | 17:00 | |
| *** lazy_prince has quit IRC | 17:03 | |
| *** Marga_ has quit IRC | 17:08 | |
| *** dtantsur is now known as dtantsur|afk | 17:09 | |
| dtantsur|afk | g'night! | 17:10 |
| *** garthb has joined #openstack-ironic | 17:10 | |
| rloo | night dtantsur|afk | 17:10 |
| *** Nisha_away has quit IRC | 17:12 | |
| *** getvasanth has quit IRC | 17:13 | |
| *** Nisha has joined #openstack-ironic | 17:15 | |
| *** trown|lunch is now known as trown | 17:20 | |
| *** bnemec has quit IRC | 17:21 | |
| *** achanda has joined #openstack-ironic | 17:22 | |
| openstackgerrit | greghaynes proposed openstack/bifrost: Add static DHCP for dynamic inventory https://review.openstack.org/217374 | 17:26 |
| *** praneshp has joined #openstack-ironic | 17:26 | |
| openstackgerrit | Merged openstack/ironic-specs: Add a new method validate_raid_config to RAIDInterface https://review.openstack.org/216809 | 17:27 |
| *** blakec has quit IRC | 17:30 | |
| *** blakec has joined #openstack-ironic | 17:32 | |
| *** Marga_ has joined #openstack-ironic | 17:34 | |
| rloo | JoshNang: wondering what the status was wrt zapping. it is a high priority but i don't see any patches for it? | 17:38 |
| rloo | JoshNang: and wrt cleaning, it is all done except the API to get the list of cleaning steps? | 17:39 |
| JoshNang | rloo: yes on cleaning, i had been waiting on the api bit because we slightly changed how they get populated in the zapping spec | 17:41 |
| JoshNang | ramesh said he was going to handle zapping, but getting the spec merged took quite some time, so i'm not sure where we stand with that | 17:41 |
| rloo | JoshNang: 'waiting on' . Are you still waiting for that? Was that the issue about which http status to return? | 17:42 |
| JoshNang | rloo: nope, i can go ahead with it now. just gotta find time :/ | 17:42 |
| rloo | JoshNang: ok, i'll follow up with ramesh. i think he has a lot on his plate then. | 17:42 |
| JoshNang | but yeah, i'm going to need the status code to get the patch merged | 17:42 |
| rloo | JoshNang: so we still have to decide on the code? i can't remember what happened if anything. | 17:43 |
| JoshNang | yeahhh i noticed he's got a lot going | 17:43 |
| JoshNang | rloo: yup, the patch is up here to decide: https://review.openstack.org/#/c/209207/ | 17:44 |
| *** e0ne has joined #openstack-ironic | 17:45 | |
| rloo | JoshNang: maybe i should send out email. if i understand correctly, our ironic version that corresponds with liberty release, will be done Sept 21-Oct9 timeframe. | 17:45 |
| *** meghal has joined #openstack-ironic | 17:45 | |
| rloo | JoshNang: thx, i'll take a look. | 17:46 |
| JoshNang | yeah, i'll try to get the cleaning api bit up today. this morning was supposed to be my "fix up all the upstream patches" morning, but distractions.. | 17:46 |
| *** tideline has joined #openstack-ironic | 17:49 | |
| rloo | JoshNang: thx. If you can get it by Monday morning, that'd be good. Or soon anyway! | 17:49 |
| JoshNang | rloo: wfm :) | 17:51 |
| *** tideline has quit IRC | 17:53 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add node-create example to enroll section of install-guide https://review.openstack.org/209251 | 17:57 |
| *** aarefiev22 has joined #openstack-ironic | 17:58 | |
| *** meghal has quit IRC | 18:01 | |
| *** garthb has quit IRC | 18:01 | |
| *** garthb has joined #openstack-ironic | 18:02 | |
| *** blakec has quit IRC | 18:03 | |
| *** getvasanth has joined #openstack-ironic | 18:05 | |
| *** blakec has joined #openstack-ironic | 18:05 | |
| *** getvasanth has quit IRC | 18:06 | |
| *** getvasanth has joined #openstack-ironic | 18:07 | |
| *** garthb has quit IRC | 18:09 | |
| *** pelix has quit IRC | 18:12 | |
| *** meghal has joined #openstack-ironic | 18:15 | |
| *** blakec has quit IRC | 18:18 | |
| *** meghal1 has joined #openstack-ironic | 18:34 | |
| *** meghal has quit IRC | 18:34 | |
| *** verdurin has quit IRC | 18:35 | |
| *** verdurin has joined #openstack-ironic | 18:39 | |
| *** tideline has joined #openstack-ironic | 18:40 | |
| *** zz_natorious is now known as natorious | 18:42 | |
| jlvillal | NobodyCam: Is this https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo/edit#gid=1338037272 the place to look for the priority things to review? | 18:44 |
| NobodyCam | yep | 18:46 |
| jlvillal | NobodyCam: Thanks | 18:46 |
| *** Marga_ has quit IRC | 18:50 | |
| *** mgoddard has quit IRC | 18:50 | |
| *** e0ne has quit IRC | 18:53 | |
| NobodyCam | :) | 18:54 |
| *** garthb has joined #openstack-ironic | 18:56 | |
| TheJulia | betherly: ping | 18:57 |
| betherly | Hey TheJulia | 18:58 |
| *** jjohnson2 has quit IRC | 19:02 | |
| TheJulia | betherly: do you have like 10 minutes to chat? | 19:03 |
| betherly | Yep :) | 19:03 |
| betherly | TheJulia: ^ | 19:03 |
| TheJulia | going to /msg | 19:03 |
| betherly | awesome ok :) | 19:03 |
| *** mgoddard has joined #openstack-ironic | 19:07 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add a versions.before() helper function https://review.openstack.org/217853 | 19:12 |
| *** puranamr has quit IRC | 19:12 | |
| garthb | Hello, I have a general question on Ram Filter failures when deploying new Nova instances on bare metal nodes... | 19:13 |
| *** puranamr has joined #openstack-ironic | 19:13 | |
| garthb | I have bare metal node successfully registered...see http://paste.openstack.org/show/429915/ | 19:13 |
| garthb | The bare metal flavor is...see http://paste.openstack.org/show/429966/ | 19:15 |
| garthb | The Ram Filter is failing as it thinks there is no ram available in the bare metal node that was registered see http://paste.openstack.org/show/429916/ | 19:16 |
| *** puranamr has quit IRC | 19:16 | |
| *** puranamr has joined #openstack-ironic | 19:17 | |
| *** puranamr has quit IRC | 19:17 | |
| garthb | Any suggestions on how to correct...thx | 19:17 |
| *** puranamr has joined #openstack-ironic | 19:17 | |
| *** jjohnson2 has joined #openstack-ironic | 19:19 | |
| *** dprince has quit IRC | 19:20 | |
| linggao | Hi Ironic folks, are the design sessions for the summit final yet or is there still time to submit a session request? | 19:22 |
| thiagop | garthb: have you checked the hypervisor created on Nova? Sometimes it takes a bit of time to update the nova with Ironic resources | 19:23 |
| rloo | garthb: don't know if this is the problem or not, but want to make sure, did you configure nova like this: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-to-use-the-bare-metal-service | 19:24 |
| rloo | linggao: as far as I know, we haven't yet started taking suggestions for design sessions. | 19:24 |
| garthb | rloo, thiagop using a devstack setup with ironic configured in...I did check nova.conf and it looks correct... | 19:26 |
| linggao | rloo, can you give a link for submitting a sessionn request? | 19:26 |
| rloo | linggao: well, that's what i mean. usually there's an etherpad. no one has started one yet. | 19:26 |
| linggao | I see. | 19:27 |
| thiagop | garthb: Nova should create an "hypervisor" for each Ironic node enrolled. Check if it's created (I think I'll be there) and check the free_memory_mb | 19:28 |
| garthb | thiagop, registered these bare metal nodes yesterday afternoon, but it's still failing this today...restarted ir-cond (just to be sure) | 19:28 |
| rloo | linggao: my suggestion is to keep an eye out for meeting minutes or email list, for mention of soliciting topics for design sessions. i suspect it won't happen for at least 2 weeks if not more. | 19:29 |
| openstackgerrit | Merged stackforge/pyghmi: Add reset bmc command https://review.openstack.org/217759 | 19:29 |
| garthb | thiagop, yes there are hypervisor nodes for each bare metal instance I created... | 19:30 |
| garthb | thiagop, checking memory_mb for each one | 19:30 |
| garthb | thiagop, they are all 0 | 19:31 |
| garthb | thiagop, that seems like it's the problem... | 19:31 |
| thiagop | garthb: if pending instances to this "fake hypervisors" are still on nova, they remain holding the resources | 19:32 |
| thiagop | garthb: you need to delete them | 19:32 |
| thiagop | garthb: if there is no instance, maybe you'll find some answers on n-cpu or n-cond logs | 19:33 |
| garthb | thiagop, running_vms is 0 for each...also I have 0 nova instances deployed, so that there should be no instances running | 19:33 |
| linggao | rloo, thanks! | 19:33 |
| thiagop | garthb: do the logs have something useful? | 19:37 |
| *** e0ne has joined #openstack-ironic | 19:37 | |
| garthb | thiagop, let me fire up another instance again and check...one moment | 19:41 |
| thiagop | ok | 19:44 |
| *** puranamr has quit IRC | 19:44 | |
| *** puranamr has joined #openstack-ironic | 19:45 | |
| *** puranamr has quit IRC | 19:48 | |
| *** nicodemos has quit IRC | 19:48 | |
| *** nicodemos has joined #openstack-ironic | 19:49 | |
| garthb | thiagop, no errors in n-cpu or n-cond...only n-sch had some details about Ramfilter returning 0 hosts... | 19:49 |
| thiagop | garthb: That's really strange... | 19:50 |
| thiagop | garthb: can you paste the output of "hypervisor show" for a node that should match this flavor? | 19:51 |
| garthb | thiagop, sure what moment | 19:51 |
| garthb | thiagop, http://paste.openstack.org/show/430017/ | 19:52 |
| thiagop | garthb: ow, it's totally empty | 19:52 |
| garthb | thiagop, shouldn't that info be populated from the ironic node-create and node-update commands? | 19:53 |
| thiagop | garthb: it shouldn't be empty this way, memory_mb should be showing the ammount of memory the server has | 19:53 |
| thiagop | vcpus too | 19:54 |
| thiagop | garthb: yes, it should | 19:54 |
| garthb | thiagop, yes agree...I suspect it's not picking up the data from the ironic node-create and node-updates I did | 19:54 |
| thiagop | garthb: in fact, nova should poll Ironic periodically to get this info. If the hypervisor is there, Nova did it's work | 19:55 |
| thiagop | garthb: let me compare my node to yours... | 19:55 |
| garthb | thiagop, yeah that makes sense...ok | 19:55 |
| *** nicodemos has quit IRC | 19:58 | |
| thiagop | garthb: ironic node-validate $NODE_UUID | 19:58 |
| thiagop | garthb: what is the output? | 19:58 |
| *** achanda has quit IRC | 19:58 | |
| garthb | thiagop, http://paste.openstack.org/show/430040/ | 19:59 |
| thiagop | garthb: the port is created for the node? | 20:00 |
| garthb | thiagop, no...do I need to do an ironic port-create? | 20:01 |
| garthb | thiagop, I didn't do that step... | 20:01 |
| *** thiagop_ has joined #openstack-ironic | 20:01 | |
| sinval | garthb: hi, thiagop is AFK now, i can help you with this | 20:03 |
| garthb | sinval, ok...thx | 20:03 |
| sinval | garthb: you have to create the port to the node, ironic portcreate | 20:04 |
| *** blakec has joined #openstack-ironic | 20:04 | |
| sinval | garthb: ironic port-create * command would work for you | 20:04 |
| garthb | sinval, ok, what do I use as the MAC address? I don't see one in ironic node-show... | 20:05 |
| thiagop_ | garthb: the MAC of the first nic of the node | 20:05 |
| thiagop_ | garthb: probably you'll see that through OneView | 20:06 |
| garthb | thiagop, ah ok...one moment... | 20:06 |
| *** puranamr has joined #openstack-ironic | 20:08 | |
| garthb | thiagop, sinval the port is now created | 20:08 |
| thiagop_ | thiagop_: Nova needs a moment to update the hypervisor, grab a coffee and then see if the hyperviser has info on memory_mb, vcpus... | 20:09 |
| thiagop_ | garthb: ^ | 20:09 |
| *** sinval_ has joined #openstack-ironic | 20:10 | |
| *** sambetts has quit IRC | 20:10 | |
| garthb | thiagop, sinval Cool...I'm starving...so I'll do better and grab lunch :) | 20:10 |
| thiagop_ | garthb: I'm almost looking for dinner here | 20:11 |
| thiagop_ | haha | 20:11 |
| sinval_ | garthb, great idea | 20:11 |
| garthb | thiagop, sinval thanks I'll check back in with you shortly and let you know the results...thanks again for your help! | 20:11 |
| thiagop_ | garthb: not at all | 20:11 |
| sinval_ | ^^ | 20:12 |
| *** e0ne has quit IRC | 20:13 | |
| *** albertoffb has quit IRC | 20:13 | |
| *** puranamr has quit IRC | 20:14 | |
| *** baoli has quit IRC | 20:14 | |
| *** aarefiev22 has quit IRC | 20:14 | |
| *** thiagop_ has quit IRC | 20:15 | |
| *** adam_g has quit IRC | 20:17 | |
| *** sambetts has joined #openstack-ironic | 20:17 | |
| *** mgoddard has quit IRC | 20:17 | |
| rloo | JoshNang: wrt clean/zap steps. to get the list of possible steps, if an agent driver, you need to ask the agent for the steps. but the agent is not always 'alive'? | 20:19 |
| *** adam_g has joined #openstack-ironic | 20:21 | |
| *** adam_g has quit IRC | 20:21 | |
| *** adam_g has joined #openstack-ironic | 20:21 | |
| *** thiagop_ has joined #openstack-ironic | 20:21 | |
| *** blakec has quit IRC | 20:22 | |
| *** achanda has joined #openstack-ironic | 20:22 | |
| *** achanda has quit IRC | 20:22 | |
| *** achanda has joined #openstack-ironic | 20:23 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 20:25 |
| JoshNang | rloo: right, that's the crux of the caching change we made in the zapping spec. basically, cache those agent (and all other) steps and return the cached steps when listing steps. it'll only fail to pull steps from the cache the first time you add a node, and that's when we have to tell the user to retry in X seconds, and we'll boot the agent to get them | 20:26 |
| rloo | JoshNang: but it kind of depends on *when* the request is made, doesn't it? eg, a node is created and in enroll state. and the user issues the request to get list of clean steps. we're going to try to boot the agent then? | 20:28 |
| JoshNang | rloo: hmm yes. if its in enroll i think we should just raise an error | 20:29 |
| rloo | JoshNang: this seems complicated. why don't we just say that request/API doesn't work with agents? :) | 20:30 |
| rloo | JoshNang: I suppose we don't want the user to ask the agent itself, for the list? (was thinking of redirect'ing) | 20:30 |
| JoshNang | rloo: heh if only | 20:31 |
| JoshNang | that won't work if your agent is on a provisioning network :/ | 20:31 |
| rloo | JoshNang: yeah, I know. although whoever issues the request might have access to that network... | 20:31 |
| rloo | JoshNang: but not the right solution anyway. | 20:32 |
| JoshNang | rloo: yeah, personally, that's what i'll likely do | 20:32 |
| rloo | JoshNang: oh really? | 20:32 |
| rloo | JoshNang: so why not go with the simple solution now, if it is an agent and it is alive, or not, do we care?, provide a redirect url? | 20:33 |
| JoshNang | rloo: i mean, that's what i already do when testing/debugging cleaning. all our steps are in the agent currently, and it has an api. | 20:33 |
| JoshNang | mmm i think this way is best for the user, but not for the developers | 20:34 |
| rloo | JoshNang: would it be easier to code this, than trying to get the api 'right' wrt retry-header/caching/etc? | 20:34 |
| rloo | JoshNang: and/or not implement this API as part of the spec... | 20:35 |
| JoshNang | rloo: definitely easier to code..but i don't think "go boot the agent yourself and query it's api yourself, and then mix in the clean steps from other drivers yourself" is a good experience :/ | 20:36 |
| rloo | JoshNang: ugh, i forgot about the other drivers. | 20:36 |
| JoshNang | yeah :( | 20:39 |
| *** krtaylor has quit IRC | 20:46 | |
| *** sinval_ has quit IRC | 20:52 | |
| *** Marga_ has joined #openstack-ironic | 20:57 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 21:01 |
| thiagop | Folks, I'm calling it a day | 21:03 |
| thiagop | have a good night everyone | 21:04 |
| *** thiagop has quit IRC | 21:04 | |
| rloo | NobodyCam: I thought a bit more about your doc updates to 209251. let me know if it makes sense. and/or if you want me to mock something up to make it clearer. | 21:05 |
| *** trown is now known as trown|outttypeww | 21:09 | |
| *** jcoufal has quit IRC | 21:10 | |
| NobodyCam | rloo: let me take a quick look :) | 21:11 |
| *** linggao has quit IRC | 21:12 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Correct bug with default value where required https://review.openstack.org/217109 | 21:13 |
| NobodyCam | rloo: Let me think about it I need to think of how to reword things | 21:15 |
| NobodyCam | But I think i understand what you thinking | 21:16 |
| rloo | NobodyCam: writing code is easier I think :) | 21:16 |
| NobodyCam | :) words are hard! :-p | 21:16 |
| *** blakec has joined #openstack-ironic | 21:16 | |
| *** romcheg1 has left #openstack-ironic | 21:27 | |
| mariojv | rloo: regarding https://review.openstack.org/#/c/217137, why is grub2 necessary for local boot as opposed to another bootloader? i know lucasagomes wrote the patch but you seemed to know about it too :) | 21:37 |
| rloo | mariojv: oh, i don't know/recall. i didn't look into it, just proofread it. i think we coded it that way. seems like we recently changed to grub2 | 21:39 |
| mariojv | alright, i'll ask lucas tomorrow, thanks | 21:39 |
| *** blakec has quit IRC | 21:39 | |
| *** jjohnson2 has quit IRC | 21:40 | |
| rloo | mariojv: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#local-boot-with-partition-images | 21:43 |
| mariojv | rloo: i see. so the guide is assuming partition images instead of whole disk images? | 21:44 |
| mariojv | i'm not that familiar with disk-image-create | 21:44 |
| *** krtaylor has joined #openstack-ironic | 21:45 | |
| rloo | mariojv: ironic doesn't handle all variations and i can't keep track of which it does and what we have or haven't documented. i suppose we should do a better job of that. | 21:45 |
| openstackgerrit | Merged openstack/bifrost: Systemd is not just for Red Hat derivatives https://review.openstack.org/217391 | 21:49 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update documentation to install grub2 when creating the user image https://review.openstack.org/217137 | 21:51 |
| openstackgerrit | Lilia Sampaio proposed openstack/python-oneviewclient: Sending OneView Client implementation https://review.openstack.org/212796 | 21:57 |
| *** penick has joined #openstack-ironic | 22:06 | |
| *** blakec has joined #openstack-ironic | 22:15 | |
| *** Nisha has quit IRC | 22:28 | |
| *** Nisha has joined #openstack-ironic | 22:28 | |
| *** blakec has quit IRC | 22:45 | |
| tonyb | Any chance I can get some eyes on this https://review.openstack.org/#/c/173121 it's a stbale/juno requirements update. | 23:07 |
| JayF | I don't think anyone here cna approve it, but I'll take a look to see if anyone would obviously impact Ironic | 23:07 |
| JayF | oh; this is for ironic | 23:08 |
| JayF | hah | 23:08 |
| JayF | still I'm not a core :) | 23:08 |
| *** shoutm has joined #openstack-ironic | 23:12 | |
| *** dims_ has joined #openstack-ironic | 23:18 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update ironic.conf.sample with tox -egenconfig https://review.openstack.org/217962 | 23:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/216562 | 23:22 |
| *** dims has quit IRC | 23:22 | |
| *** Marga_ has quit IRC | 23:27 | |
| openstackgerrit | Clif Houck proposed openstack/ironic: Use __str__ as the standard way of getting an Exception's description https://review.openstack.org/217434 | 23:33 |
| *** bnemec has joined #openstack-ironic | 23:35 | |
| JayF | clif_h: should we be using six.text_type() instead of str()? | 23:36 |
| clif_h | JayF: I'm not sure, should we? | 23:36 |
| JayF | clif_h: I guess if it passed py34 tests it's probably fine, but I thought you always had to use the six.text_type() | 23:36 |
| JayF | https://pythonhosted.org/six/#six.text_type | 23:37 |
| JayF | clif_h: ^ I think that means yes | 23:37 |
| clif_h | It passed py27,py34,pep8 for me | 23:37 |
| JayF | Hmmm. | 23:37 |
| *** dims_ has quit IRC | 23:38 | |
| JayF | clif_h: I'm having lvh look | 23:38 |
| JayF | clif_h: lvh says repr() or six.text_type() in this case | 23:39 |
| JayF | with six.text_type being preferable, so that makes sense to me | 23:39 |
| JayF | so I'll put that on the review | 23:39 |
| clif_h | repr doesn't sound like what we want | 23:46 |
| clif_h | text_type does | 23:46 |
| JoshNang | JayF: :D returning the output of upgrade bios? not a good idea. i think it just sent a megabyte or two of text back via the API | 23:49 |
| clif_h | JayF: just here, or everywhere? should I go back and look for instances of str(exc) in other places? | 23:55 |
| *** achanda_ has joined #openstack-ironic | 23:56 | |
| clif_h | I'll double check the rest of the codebase I suppose | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!