| jroll | ugh comments are timing out | 00:01 |
|---|---|---|
| jroll | lol | 00:01 |
| *** lucas-dinner has quit IRC | 00:01 | |
| jroll | oh there they go | 00:01 |
| jroll | NobodyCam: +2'd | 00:02 |
| jroll | have a good night :) | 00:02 |
| rloo | NobodyCam: just looked at 114281, will wait for jenkins first. so later tonight if i remember. | 00:09 |
| *** rloo is now known as rloo_afk | 00:09 | |
| ramineni | jroll: Hi | 00:16 |
| kylestev | ramineni: jroll left the office for the evening a few minutes ago | 00:18 |
| kylestev | just fyi | 00:18 |
| ramineni | kylestav: oh..thanks for the info | 00:19 |
| kylestev | np | 00:20 |
| *** jcoufal has quit IRC | 00:22 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Use LLDP to get switch port mapping https://review.openstack.org/92627 | 00:33 |
| *** chuckC_ has quit IRC | 00:56 | |
| *** ChuckC has quit IRC | 00:58 | |
| *** ChuckC has joined #openstack-ironic | 00:58 | |
| *** ChuckC has quit IRC | 01:04 | |
| *** krtaylor has quit IRC | 01:21 | |
| *** ramineni has quit IRC | 01:27 | |
| *** krtaylor has joined #openstack-ironic | 01:32 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: WIP: Add send-data-to-ceilometer support for pxe_ipminative driver https://review.openstack.org/112486 | 01:32 |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Add send-data-to-ceilometer support for pxe_ipminative driver https://review.openstack.org/112486 | 01:34 |
| *** nosnos has joined #openstack-ironic | 01:42 | |
| *** shakamunyi has quit IRC | 02:01 | |
| openstackgerrit | A change was merged to openstack/ironic: Allow rebuild of node in ERROR and DEPLOYFAIL state https://review.openstack.org/114281 | 02:03 |
| mmitchell | ^ OH YES! | 02:06 |
| *** ChuckC has joined #openstack-ironic | 02:16 | |
| *** chuckC_ has joined #openstack-ironic | 02:29 | |
| *** shakamunyi has joined #openstack-ironic | 02:34 | |
| *** harlowja is now known as harlowja_away | 02:35 | |
| *** shakamunyi has quit IRC | 02:41 | |
| *** shakamunyi has joined #openstack-ironic | 02:42 | |
| *** ramineni has joined #openstack-ironic | 02:56 | |
| *** vinbs has joined #openstack-ironic | 03:07 | |
| *** rushiagr_away has quit IRC | 03:12 | |
| *** rushiagr_away has joined #openstack-ironic | 03:14 | |
| *** rloo_afk has quit IRC | 03:15 | |
| *** vinbs has quit IRC | 03:17 | |
| *** krtaylor has quit IRC | 03:28 | |
| *** tatyana has joined #openstack-ironic | 03:30 | |
| *** ramineni1 has joined #openstack-ironic | 03:40 | |
| *** krtaylor has joined #openstack-ironic | 03:42 | |
| *** ramineni has quit IRC | 03:43 | |
| *** tatyana has quit IRC | 03:44 | |
| *** krtaylor has quit IRC | 03:46 | |
| *** eghobo has joined #openstack-ironic | 03:48 | |
| *** vinbs has joined #openstack-ironic | 03:49 | |
| *** rushiagr_away is now known as rushiagr | 03:59 | |
| *** tatyana has joined #openstack-ironic | 04:02 | |
| *** krtaylor has joined #openstack-ironic | 04:04 | |
| *** tatyana has quit IRC | 04:19 | |
| *** eghobo has quit IRC | 04:31 | |
| *** eghobo has joined #openstack-ironic | 04:32 | |
| *** pcrews has quit IRC | 04:46 | |
| *** tatyana has joined #openstack-ironic | 04:46 | |
| *** rakesh_hs has joined #openstack-ironic | 04:50 | |
| openstackgerrit | Gregory Haynes proposed a change to openstack/ironic: WIP: Add conductor_hostname property to nodes https://review.openstack.org/109688 | 04:56 |
| *** yuanying has joined #openstack-ironic | 04:59 | |
| openstackgerrit | Michael Davies proposed a change to openstack/ironic: Nova ironic driver review update requests to p4 https://review.openstack.org/115493 | 05:08 |
| *** eguz has joined #openstack-ironic | 05:10 | |
| *** eghobo has quit IRC | 05:10 | |
| *** tatyana has quit IRC | 05:28 | |
| *** bmahalakshmi has joined #openstack-ironic | 05:28 | |
| *** vinbs has quit IRC | 05:30 | |
| *** bvivek has joined #openstack-ironic | 05:30 | |
| *** sirushti has joined #openstack-ironic | 05:50 | |
| *** eguz has quit IRC | 05:53 | |
| *** vinbs has joined #openstack-ironic | 06:00 | |
| *** aswadr has joined #openstack-ironic | 06:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/114135 | 06:11 |
| *** tatyana has joined #openstack-ironic | 06:26 | |
| *** yuanying has quit IRC | 06:28 | |
| *** jcoufal has joined #openstack-ironic | 06:31 | |
| *** killer_prince has joined #openstack-ironic | 06:31 | |
| *** vinbs has quit IRC | 06:38 | |
| *** jcoufal has quit IRC | 06:38 | |
| *** yuanying has joined #openstack-ironic | 06:39 | |
| *** shakamunyi has quit IRC | 06:43 | |
| *** shakamunyi has joined #openstack-ironic | 06:44 | |
| *** ifarkas has joined #openstack-ironic | 06:47 | |
| killer_prince | Hi All, does anyone know if ironic now provides tenant isolation using neutron.? | 06:47 |
| *** shakamunyi has quit IRC | 06:47 | |
| *** shakamunyi has joined #openstack-ironic | 06:48 | |
| *** vinbs has joined #openstack-ironic | 06:52 | |
| *** pcrews has joined #openstack-ironic | 06:54 | |
| *** dtantsur|afk is now known as dtantsur | 06:55 | |
| dtantsur | Morning Ironic | 06:55 |
| mrda | dtantsur: \o | 06:55 |
| dtantsur | o/ | 06:56 |
| Haomeng | dtantsur: morning:) | 06:58 |
| Haomeng | killer_prince: yes, ironic is using neutron to setup network especial for dhcp/tftp/pxe | 06:59 |
| Haomeng | killer_prince: what is your concern? | 07:00 |
| *** pradipta_away is now known as pradipta | 07:01 | |
| killer_prince | Haomeng: sorry, was afk. | 07:09 |
| killer_prince | I have a devstack setup where I have couple of BMs connected to one of the interface via a standard switch. I would like to test multi tenant environment with network isolation. | 07:10 |
| Haomeng | killer_prince: you can use vlan to isolate them by vlan tag from pyhsical switch | 07:12 |
| Haomeng | that is out of ironic scope I think | 07:12 |
| Haomeng | :) | 07:12 |
| killer_prince | right.. but i wanted it to managed by neutron is possible.. | 07:13 |
| Haomeng | killer_prince: the information for you from ironic is that our ironic follows neutron and controled by neutron for networking | 07:13 |
| Haomeng | killer_prince: yes\ | 07:13 |
| killer_prince | so you mean to say that, righ now, neutron has capabilities to manage physical switches and so it can provide tenant network isolation | 07:14 |
| killer_prince | for BMs too.. | 07:14 |
| Haomeng | killer_prince: neutron can not manage physical switch | 07:14 |
| Haomeng | but neutron can use vlan | 07:14 |
| Haomeng | co-work with physical switch | 07:14 |
| JoshNang | killer_prince: physical switch management is targeted for Juno | 07:14 |
| killer_prince | but then if i need to move machines from one tenant to others then I will have to update the vlans on switches manually.. is that a correct understanding..? | 07:15 |
| *** derekh has joined #openstack-ironic | 07:16 | |
| JoshNang | as of right now, i think so. we had some proof of concept code up and a custom neutron plugin to enable vlan management with the agent driver (we're doing vlan toggling for OnMetal) | 07:19 |
| JoshNang | but i don't expect the code to land in tree until Kilo, unfortunately | 07:19 |
| JoshNang | killer_prince: here's the gist of what we had/are running in production: https://review.openstack.org/#/c/84795/98/ironic/drivers/modules/agent_utils.py | 07:21 |
| *** jcoufal has joined #openstack-ironic | 07:27 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic: Add UEFI based deployment support in Ironic https://review.openstack.org/114357 | 07:35 |
| *** chenglch has joined #openstack-ironic | 07:40 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic: Add UEFI based deployment support in Ironic https://review.openstack.org/114357 | 07:47 |
| *** pcrews has quit IRC | 08:08 | |
| *** bvivek has quit IRC | 08:20 | |
| mrda | jroll, comstud: For your viewing pleasure: https://review.openstack.org/115540 | 08:22 |
| openstackgerrit | A change was merged to openstack/ironic: Add Parallels virtualisation type https://review.openstack.org/114719 | 08:23 |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Fix misspelled class name AmbigiousAuthSystem https://review.openstack.org/114916 | 08:23 |
| *** shakamunyi has quit IRC | 08:25 | |
| *** ndipanov_gone is now known as ndipanov | 08:27 | |
| *** pelix has joined #openstack-ironic | 08:32 | |
| killer_prince | JoshNang: Haomeng: one more thing, if i start using Vlans then do i need to specify it when creating a tenant so that it will get used or ironic does it automatically..? | 08:32 |
| *** yuanying has quit IRC | 08:34 | |
| *** killer_prince2 has joined #openstack-ironic | 08:36 | |
| *** killer_prince2 is now known as lazy_prince | 08:37 | |
| *** killer_prince has quit IRC | 08:37 | |
| *** lazy_prince is now known as killer_prince | 08:43 | |
| *** igordcard has joined #openstack-ironic | 08:44 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Take iSCSI deploy out of pxe driver https://review.openstack.org/111232 | 08:47 |
| *** athomas has joined #openstack-ironic | 08:50 | |
| *** ifarkas has quit IRC | 08:52 | |
| *** ifarkas has joined #openstack-ironic | 08:54 | |
| *** lucasagomes has joined #openstack-ironic | 08:56 | |
| Haomeng | killer_prince: make sure the vlan tag id is same with your physical node port | 09:05 |
| lucasagomes | ramineni1, rakesh_hs howdy is this bp completed? https://blueprints.launchpad.net/ironic/+spec/cleanup-all-imagecaches | 09:06 |
| lucasagomes | ramineni1, rakesh_hs I see the patch there get merged, just wondering if there's something else to do if not I will mark it as implemented | 09:07 |
| ramineni1 | lucas, you can mark it as implemented | 09:15 |
| lucasagomes | ramineni1, awesome! | 09:15 |
| lucasagomes | ramineni1, also, wanted to ask about https://blueprints.launchpad.net/ironic/+spec/ilo-virtualmedia-ipa | 09:15 |
| lucasagomes | ramineni1, I haven't seen any code for that yet, are you guys planning to put the code up soon? | 09:15 |
| ramineni1 | lucas, yes planning to post the patch soon , by this week | 09:17 |
| lucasagomes | ramineni1, alright :) thanks for that! | 09:17 |
| lucasagomes | the other one is already marked as implemented then :) cheers | 09:17 |
| ramineni1 | :) | 09:17 |
| *** MattMan has joined #openstack-ironic | 09:21 | |
| *** bvivek has joined #openstack-ironic | 09:21 | |
| *** viktors|afk is now known as viktors | 09:26 | |
| *** yuanying has joined #openstack-ironic | 09:35 | |
| mrda | Night Ironic - see you tomorrow | 09:39 |
| lucasagomes | mrda, good night! | 09:44 |
| *** bvivek has quit IRC | 09:45 | |
| *** bvivek has joined #openstack-ironic | 09:51 | |
| *** chenglch has quit IRC | 09:58 | |
| *** igordcard has quit IRC | 10:03 | |
| *** yuanying has quit IRC | 10:09 | |
| *** openstackgerrit has quit IRC | 10:10 | |
| *** bvivek2 has joined #openstack-ironic | 10:15 | |
| *** bvivek has quit IRC | 10:16 | |
| lucasagomes | jroll, NobodyCam if you guys have a time, mind having a look at https://review.openstack.org/#/c/104850/ | 10:37 |
| lucasagomes | cheers | 10:37 |
| *** ramineni1 has quit IRC | 10:44 | |
| *** ramineni has joined #openstack-ironic | 10:45 | |
| *** nosnos has quit IRC | 10:57 | |
| *** ramineni has quit IRC | 10:58 | |
| *** yuanying has joined #openstack-ironic | 11:02 | |
| *** ramineni has joined #openstack-ironic | 11:02 | |
| *** igordcard has joined #openstack-ironic | 11:08 | |
| *** bmahalakshmi has quit IRC | 11:27 | |
| lucasagomes | jroll, here's the rebased iPXE patch for devstack (tested locally) https://review.openstack.org/#/c/99677/ | 11:28 |
| *** lucasagomes is now known as lucas-hungry | 11:29 | |
| *** yuanying has quit IRC | 11:32 | |
| *** bvivek2 has quit IRC | 11:36 | |
| *** ramineni1 has joined #openstack-ironic | 11:39 | |
| *** ramineni has quit IRC | 11:40 | |
| *** vinbs has quit IRC | 11:57 | |
| *** yuanying has joined #openstack-ironic | 12:02 | |
| *** yuanying has quit IRC | 12:08 | |
| *** vinbs has joined #openstack-ironic | 12:25 | |
| *** pradipta is now known as pradipta_away | 12:33 | |
| *** lucas-hungry is now known as lucasagomes | 12:41 | |
| *** rakesh_hs has quit IRC | 12:42 | |
| *** killer_prince has quit IRC | 12:43 | |
| *** yuanying has joined #openstack-ironic | 13:04 | |
| *** yuanying has quit IRC | 13:09 | |
| *** ramineni1 has quit IRC | 13:10 | |
| *** ramineni has joined #openstack-ironic | 13:12 | |
| *** rloo has joined #openstack-ironic | 13:17 | |
| *** vinbs has quit IRC | 13:22 | |
| *** bvivek has joined #openstack-ironic | 13:26 | |
| jroll | morning lucasagomes and everyone else :) | 13:28 |
| lucasagomes | jroll, morning | 13:29 |
| rloo | morning jroll. afternoon lucasagomes. | 13:31 |
| jroll | lucasagomes: will take a look at those patches, thanks | 13:31 |
| jroll | hey, morning rloo :) | 13:31 |
| lucasagomes | rloo, morning | 13:31 |
| lucasagomes | jroll, awesome, thanks! | 13:31 |
| rloo | lucasagomes: wrt the tempest test 113276, couple of questions. | 13:31 |
| lucasagomes | rloo, yup | 13:31 |
| *** vinbs has joined #openstack-ironic | 13:32 | |
| rloo | lucasagomes: the commit title was changed to use 'scenario' instead of 'api' | 13:32 |
| rloo | lucasagomes: i thought there was a previous comment that it was api? | 13:32 |
| ramineni | jroll: Hi | 13:32 |
| lucasagomes | rloo, yeah I put scenario when I first created the patch, mostly because I didn't know what I was talking about | 13:32 |
| lucasagomes | rloo, then people pointed out to me that it was just 'api tests' | 13:32 |
| jroll | ramineni: hey :) what's up | 13:32 |
| lucasagomes | so I changed that | 13:33 |
| rloo | lucasagomes: so why did you change it back to 'scenario'? | 13:33 |
| ramineni | jroll, have few questions regarding https://bugs.launchpad.net/ironic/+bug/1358694 | 13:33 |
| rloo | lucasagomes: and I just noticed, 'manageament' is misspelled now (in the commit). | 13:33 |
| rloo | lucasagomes: sorry 'managament' | 13:34 |
| ramineni | jroll, you meant accepting genric 'password' etc .. as part of node-create for all the drivers? | 13:34 |
| lucasagomes | rloo, ohhhh gosh I changed it back to scenario because it was deva that updated my commit message before | 13:34 |
| lucasagomes | and in my local I had the old one | 13:34 |
| lucasagomes | >.< | 13:34 |
| rloo | lucasagomes: that explains it. that burned me once before too. | 13:34 |
| lucasagomes | rloo, will update that | 13:34 |
| jroll | ramineni: oh yeah, we talked about this yesterday... you will usually only need one set of credentials for all drivers, so we want to use just 'password', and allow 'ilo_password', 'ipmi_password' etc to override that | 13:35 |
| ramineni | jroll, didnt quite get that part , can you clarify it | 13:35 |
| rloo | lucasagomes: i had one more question. you changed the return value of the get_* to return the body only. | 13:35 |
| rloo | lucasagomes: if it was a general method, it should return the response and body. | 13:35 |
| jroll | ramineni: we don't want to have ilo driver looking for ipmi_password then, because we'll have to keep backwards compat for that, it will be hard... you would eventually end up looking for ilo_password, ipmi_password, password | 13:36 |
| rloo | lucasagomes: I was wondering if it should still return resp, body, but whatever calls it, doesn't use the resp part. | 13:36 |
| ramineni | jroll, ok , ya | 13:36 |
| lucasagomes | rloo, yeah it was doing that before, but first people asked me to test the return code at the client part | 13:36 |
| lucasagomes | so I was using resp only in the client | 13:36 |
| lucasagomes | rloo, but I continued returning resp and body | 13:37 |
| rloo | lucasagomes: hmm. yeah, let's see what they say now. we just want to get it merged. | 13:37 |
| lucasagomes | rloo, and then another reviwer pointed to me that resp wasn't being used on the tests, so I removed it from the return | 13:37 |
| jroll | ramineni: so just to be clear, we shouldn't fix that bug this cycle, we should wait until kilo and make all drivers better :) | 13:37 |
| lucasagomes | rloo, yup, pretty much... imo I wouldn't test it on the client side and wouldn't bother returning resp as well | 13:37 |
| rloo | lucasagomes: ok thx. I'll +1 it after you update the commit. | 13:38 |
| lucasagomes | rloo, but I'm in the whatever-mode on, so I just want it merged | 13:38 |
| ramineni | jroll , so as of now if we want to use mixed drivers , i.e for management and console ..we want to use ipmi as part of ilodriver , user needs to provide both ilo and ipmi credentails right? is there any alternative to make it better | 13:38 |
| lucasagomes | it's a fairly small patch and it's taking ages to get merged | 13:38 |
| ramineni | jroll, any suggestions :) | 13:38 |
| rloo | lucasagomes: well, maybe those tempest folks are busy. (I don't know what our track record is but I suspect people might say the same about ironic reviews.) | 13:39 |
| lucasagomes | rloo, yeah can happen :) | 13:40 |
| jroll | ramineni: my siggestion is that your script to register nodes with ironic should add both ilo_password and ipmi_password :) | 13:41 |
| jroll | lucasagomes: +1 for whatever-mode :P | 13:41 |
| *** krtaylor has quit IRC | 13:42 | |
| lucasagomes | jroll, heh yeah if they tell me to add a os.system("rm -rf /") | 13:43 |
| lucasagomes | I do | 13:43 |
| lucasagomes | jk :P | 13:43 |
| jroll | lol | 13:44 |
| ramineni | jroll, :) , node-create is manual as of now , so can't do anything in that part i guess now | 13:44 |
| jroll | ramineni: manual for who? it's easy to write a script that creates nodes | 13:44 |
| jroll | from a csv or json file or whatever | 13:44 |
| *** vinbs has quit IRC | 13:45 | |
| jroll | (and if you have more than three servers in ironic, you *should* be using a script | 13:45 |
| jroll | ) | 13:45 |
| *** pcrews has joined #openstack-ironic | 13:45 | |
| ramineni | jroll , oh ok .. got it :) | 13:45 |
| lucasagomes | rloo, updated, thanks for pointing that out | 13:51 |
| rloo | thx lucasagomes! | 13:51 |
| *** eghobo has joined #openstack-ironic | 13:55 | |
| jroll | lucasagomes: looking at https://review.openstack.org/#/c/104850/11/ironic/drivers/drac.py | 13:59 |
| jroll | for licenses do we require the "Copyright 2014 Red Hat" bit? | 13:59 |
| lucasagomes | jroll, thanks optional | 14:00 |
| lucasagomes | that's | 14:00 |
| lucasagomes | (hands out of control) | 14:00 |
| jroll | ha ok | 14:00 |
| lucasagomes | jroll, the thing is, all the code is owned by the foundation anyway, so that copyright line is just bollocks afaict | 14:01 |
| lucasagomes | lemme dig more into it | 14:01 |
| lucasagomes | but fo sure it's optional | 14:01 |
| jroll | yeah agree | 14:02 |
| jroll | don't worry about it :) | 14:02 |
| lucasagomes | https://wiki.openstack.org/wiki/LegalIssuesFAQ#Copyright_Headers | 14:02 |
| jroll | oh hm, I need to update IPA driver to use ipmitool.IPMIManagement don't I | 14:02 |
| jroll | or is it backwards compat for now? | 14:02 |
| jroll | https://review.openstack.org/#/c/104850/11/ironic/drivers/modules/drac/client.py | 14:04 |
| jroll | line 32 | 14:04 |
| jroll | hate it | 14:04 |
| *** ramineni1 has joined #openstack-ironic | 14:05 | |
| *** yuanying has joined #openstack-ironic | 14:05 | |
| *** ramineni has quit IRC | 14:07 | |
| lucasagomes | jroll, hmm yeah, sounds like a good idea to have the agent drivers able to configure things like boot device | 14:07 |
| jroll | lucasagomes: they do configure boot device already :| | 14:08 |
| lucasagomes | jroll, the pxe driver calls the mgmt interface to set the boot device, maybe the agent should do that as well? | 14:08 |
| lucasagomes | ah | 14:08 |
| jroll | but through power interface I think | 14:09 |
| lucasagomes | i c | 14:09 |
| jroll | that... still works, right? | 14:10 |
| jroll | I guess it must | 14:10 |
| lucasagomes | jroll, it was calling the utils right? | 14:11 |
| lucasagomes | well better double check | 14:11 |
| *** yuanying has quit IRC | 14:11 | |
| jroll | oh you're right | 14:11 |
| jroll | cool :) | 14:11 |
| *** vdrok has joined #openstack-ironic | 14:14 | |
| *** Poornima has joined #openstack-ironic | 14:15 | |
| jroll | lucasagomes: +A on the drac thing | 14:19 |
| *** linggao has joined #openstack-ironic | 14:19 | |
| lucasagomes | jroll, \o/ awesome thanks! | 14:20 |
| jroll | np :) | 14:22 |
| jroll | running devstack for ipxe now | 14:23 |
| lucasagomes | awesome | 14:23 |
| *** rwsu has quit IRC | 14:24 | |
| vdrok | Hello Ironic, I'm Vladyslav Drok. | 14:25 |
| jroll | lucasagomes: so we're down to 3 specs that have code in ironic to be reviewed \o/ | 14:27 |
| jroll | vdrok: hi, welcome :) | 14:27 |
| vdrok | Could you please take a look at couple of my patches: https://review.openstack.org/#/c/110634/ and https://review.openstack.org/#/c/112049/ ? | 14:27 |
| lucasagomes | jroll, yup! that's going pretty well | 14:27 |
| vdrok | jroll: hi! | 14:27 |
| *** ramineni1 has quit IRC | 14:28 | |
| jroll | whoaaaa, remove _ injection is huge | 14:29 |
| jroll | but I like the idea :) | 14:29 |
| vdrok | :) | 14:29 |
| jroll | everyone needs to stop hacking the modules global :P | 14:29 |
| *** Poornima has quit IRC | 14:30 | |
| *** vinbs has joined #openstack-ironic | 14:31 | |
| NobodyCam | good morning Ironic | 14:33 |
| *** ifarkas has quit IRC | 14:34 | |
| *** Poornima has joined #openstack-ironic | 14:34 | |
| jroll | vdrok: we'll get to those, thanks for the ping :) | 14:34 |
| jroll | NobodyCam: morning! :) | 14:34 |
| NobodyCam | morning jroll | 14:34 |
| vdrok | jroll: thanks | 14:34 |
| vdrok | NobodyCam: morning! | 14:35 |
| *** vinbs_ has joined #openstack-ironic | 14:35 | |
| NobodyCam | good morning vdrok :) | 14:35 |
| dtantsur | Morning, jroll, rloo, NobodyCam. Hi, vdrok! | 14:35 |
| *** eghobo has quit IRC | 14:36 | |
| NobodyCam | morning dtantsur | 14:36 |
| rloo | hiya dtantsur! | 14:36 |
| jroll | morning dtantsur :) | 14:36 |
| NobodyCam | morning rloo | 14:36 |
| vdrok | Hi dtantsur! | 14:36 |
| rloo | morning NobodyCam! | 14:36 |
| *** rwsu has joined #openstack-ironic | 14:36 | |
| lucasagomes | morning NobodyCam vdrok | 14:37 |
| lucasagomes | morning dtantsur | 14:37 |
| lucasagomes | well afternoon really | 14:37 |
| dtantsur | yeah, g'afternoon :) | 14:37 |
| vdrok | evening :) lucasagomes | 14:38 |
| lucasagomes | : | 14:38 |
| lucasagomes | :D | 14:38 |
| NobodyCam | morning lucasagomes | 14:38 |
| *** vinbs has quit IRC | 14:38 | |
| *** vinbs has joined #openstack-ironic | 14:39 | |
| *** vinbs_ has quit IRC | 14:41 | |
| jroll | lucasagomes: hmm, I wonder if we should support ipxe in the agent driver, or just wait until kilo when we split things out | 14:42 |
| jroll | probably just wait for kilo I think | 14:42 |
| lucasagomes | jroll, yeah... def it would be very good for the agent | 14:44 |
| lucasagomes | since the ramdisk is bigger and all | 14:44 |
| jroll | yeah... just not sure if it's worth the work | 14:44 |
| lucasagomes | so depend on tftp sucks for it | 14:44 |
| lucasagomes | but right now, hmm | 14:44 |
| lucasagomes | yeah kilo seems to be grand | 14:44 |
| lucasagomes | K-1 | 14:44 |
| jroll | we should start keeping a list of specs we need to write for kilo :P | 14:45 |
| lucasagomes | jroll, we do | 14:45 |
| jroll | oh? | 14:45 |
| lucasagomes | jroll, on that spreadsheet there's a tab | 14:45 |
| lucasagomes | for kilo | 14:45 |
| jroll | oh right | 14:45 |
| * lucasagomes lemme grab the link | 14:45 | |
| jroll | cool, thanks for reminder :P | 14:45 |
| jroll | I have it :) | 14:45 |
| lucasagomes | alright, yeah add there please :) | 14:45 |
| jroll | <3 http://bit.ly/ironic-whiteboard | 14:45 |
| lucasagomes | heh :D | 14:45 |
| *** ifarkas has joined #openstack-ironic | 14:46 | |
| lucasagomes | added there | 14:47 |
| jroll | huh, I thought I add edit privs | 14:47 |
| jroll | :/ | 14:47 |
| lucasagomes | jroll, u don't | 14:47 |
| lucasagomes | ? | 14:47 |
| jroll | also... I was going to add "separate BootInterface from DeployInterface" | 14:47 |
| lucasagomes | lemme see if I can give u | 14:47 |
| jroll | idk, I typed and it doesn't work | 14:47 |
| jroll | :P | 14:47 |
| jroll | I know I did at one point | 14:47 |
| lucasagomes | Jim Rollenhagen Can edit | 14:48 |
| lucasagomes | jroll, ^ are u logged? | 14:48 |
| jroll | huh. | 14:48 |
| jroll | yeah, tried both accounts | 14:48 |
| jroll | does it show which email? | 14:48 |
| dtantsur | should we also add unifying credentials there? | 14:49 |
| jroll | yes | 14:49 |
| lucasagomes | jroll, yup | 14:49 |
| jroll | oh huh | 14:49 |
| lucasagomes | sending on pm | 14:49 |
| jroll | re-logged in | 14:49 |
| jroll | worked ok | 14:49 |
| dtantsur | and discovery lol | 14:49 |
| jroll | thanks :) | 14:49 |
| lucasagomes | alright :) | 14:49 |
| jroll | I don't want to deal with talking about discovery :| | 14:50 |
| lucasagomes | dtantsur, well we can add it there fo sure... this are ideas I bet | 14:51 |
| dtantsur | jroll, nobody wants :) | 14:51 |
| jroll | yeah, I'm mostly kidding | 14:51 |
| *** bmahalakshmi has joined #openstack-ironic | 14:55 | |
| * jroll is getting selfish on this spreadsheet | 14:57 | |
| *** vdrok has quit IRC | 14:57 | |
| *** vdrok has joined #openstack-ironic | 14:57 | |
| *** jgrimm has joined #openstack-ironic | 14:58 | |
| *** vdrok has quit IRC | 15:00 | |
| *** vdrok_ has joined #openstack-ironic | 15:00 | |
| *** shakamunyi has joined #openstack-ironic | 15:00 | |
| *** ifarkas has quit IRC | 15:02 | |
| *** vdrok_ has quit IRC | 15:02 | |
| *** vdrok has joined #openstack-ironic | 15:02 | |
| *** vdrok has quit IRC | 15:03 | |
| *** vdrok has joined #openstack-ironic | 15:04 | |
| *** yuanying has joined #openstack-ironic | 15:07 | |
| *** bvivek has quit IRC | 15:08 | |
| lucasagomes | jroll, iPXE + devstack worked for u? | 15:11 |
| *** yuanying has quit IRC | 15:12 | |
| *** openstackgerrit has joined #openstack-ironic | 15:13 | |
| *** jcoufal has quit IRC | 15:17 | |
| *** vinbs has quit IRC | 15:26 | |
| jroll | lucasagomes: hmm, no, tempest failed :/ | 15:29 |
| jroll | hrm, I might not have it enabled though | 15:30 |
| jroll | lucasagomes: I'll poke at it a little later, gotta go afk for a bit | 15:30 |
| lucasagomes | heh yeah... well idk why it would fail because of iPXE tho | 15:30 |
| lucasagomes | jroll, ack! | 15:30 |
| jroll | unless it doesn't pxe boot :P | 15:30 |
| jroll | Details: Timed out waiting for node d7244d63-870f-4175-9f69-a533ead51fd0 to reach power_state state(s) ['power on'] | 15:30 |
| jroll | might be unrelated tho, I haven't touched my devstack box in a few days | 15:31 |
| jroll | bbl | 15:31 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: return power state ERROR instead of raising exception https://review.openstack.org/115300 | 15:31 |
| *** eghobo has joined #openstack-ironic | 15:31 | |
| *** eghobo has quit IRC | 15:33 | |
| *** vinbs has joined #openstack-ironic | 15:44 | |
| *** rushiagr is now known as rushiagr_away | 15:44 | |
| *** rushiagr_away is now known as rushiagr | 15:44 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: handle invalid seamicro_api_version https://review.openstack.org/115692 | 15:59 |
| *** vinbs has quit IRC | 16:07 | |
| *** yuanying has joined #openstack-ironic | 16:09 | |
| *** vinbs has joined #openstack-ironic | 16:09 | |
| *** tatyana has quit IRC | 16:13 | |
| *** yuanying has quit IRC | 16:14 | |
| *** vinbs has quit IRC | 16:22 | |
| *** killer_prince has joined #openstack-ironic | 16:25 | |
| *** killer_prince is now known as lazy_prince | 16:25 | |
| *** k4n0 has quit IRC | 16:25 | |
| rloo | NobodyCam: wrt 115300, yuriyz had suggested using warning. our ipmi* stuff uses warning too, but ssh/seamicro/ilo use error. Do you prefer error? | 16:26 |
| NobodyCam | we're rasing a error on the next line, I kinda feel like that should be an error | 16:29 |
| NobodyCam | I'm happy to ping yuriyz about that if your like | 16:30 |
| NobodyCam | s/your/you'd/ | 16:30 |
| rloo | NobodyCam: I'm fine either way. I'll make it an error and see if yuriy pong's back. | 16:31 |
| *** penick has joined #openstack-ironic | 16:31 | |
| NobodyCam | :) | 16:31 |
| *** rushiagr is now known as rushiagr_away | 16:32 | |
| *** krtaylor has joined #openstack-ironic | 16:35 | |
| lucasagomes | rloo, NobodyCam +1 for error | 16:36 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: return power state ERROR instead of an exception https://review.openstack.org/115300 | 16:36 |
| rloo | thx lucasagomes. Those 1's add up ;) | 16:36 |
| NobodyCam | has anyone looked at the proxy nova commands issue at all? | 16:36 |
| lucasagomes | rloo, the way I see warning is that the code fallbacked for something else and not completed failed (like it's doing on that patch, it's raising an exception right after the warning) | 16:36 |
| *** viktors is now known as viktors|afk | 16:37 | |
| *** rushiagr_away is now known as rushiagr | 16:37 | |
| rloo | lucasagomes: i think there's a bug open about cleaning up our logs. it would be good to try to write up when to use which log levels. | 16:37 |
| lucasagomes | rloo, there's another warning being raised there... (I added a comment on the previous patch-set but I think I was too late) | 16:38 |
| lucasagomes | rloo, I think we had it somewhere yeah hmm lemme check | 16:38 |
| rloo | lucasagomes: oh, you think that should be an error too? Ok, no problem making another change. | 16:39 |
| lucasagomes | rloo, no we don't :( but this is what I follow http://paste.openstack.org/show/97829/ | 16:39 |
| NobodyCam | lucasagomes: I'm not sure about the other one. | 16:39 |
| NobodyCam | its returning a error state not rasing an error | 16:39 |
| *** Poornima has quit IRC | 16:39 | |
| NobodyCam | to me that seems like a warning | 16:39 |
| lucasagomes | NobodyCam, hmm right | 16:40 |
| lucasagomes | I thought about error because user requested get_power_status | 16:41 |
| lucasagomes | but the operation failed | 16:41 |
| lucasagomes | NobodyCam, I see what you mean... hmm ok I'm happy with both | 16:41 |
| rloo | NobodyCam, lucasagomes: didn't deva have a spec about logging? and then decided it wasn't a spec. that spec might have more info about log levels. | 16:41 |
| lucasagomes | rloo, yeah, or if we had at least a WIKI page with some code guidelines | 16:41 |
| lucasagomes | that would help in this situations as well | 16:41 |
| rloo | lucasagomes: i think a wiki would be the best place. probably something we need to all agree on though. | 16:42 |
| lucasagomes | yeah | 16:42 |
| lucasagomes | wiki seems good for that | 16:42 |
| *** derekh has quit IRC | 16:43 | |
| *** dlaube has joined #openstack-ironic | 16:44 | |
| rloo | lucasagomes: i don't want to think about log levels right now. whoever tries to clean up the logging, we should get them to start a wiki ;) | 16:45 |
| lucasagomes | rloo, heh | 16:45 |
| lucasagomes | yeah leave that as warning it's alright | 16:46 |
| rloo | lucasagomes: ok thx. | 16:46 |
| *** dtantsur is now known as dtantsur|afk | 16:46 | |
| *** jcoufal has joined #openstack-ironic | 16:46 | |
| *** tatyana has joined #openstack-ironic | 16:52 | |
| *** ndipanov has quit IRC | 16:56 | |
| lucasagomes | alright I will call it a day, have a good night everybody! | 17:02 |
| *** lucasagomes is now known as lucas-dinner | 17:03 | |
| rloo | by lucasagomes | 17:03 |
| rloo | bye lucas-dinner ;) | 17:03 |
| *** bvivek has joined #openstack-ironic | 17:03 | |
| NobodyCam | night lucas-dinner | 17:04 |
| *** bvivek has quit IRC | 17:08 | |
| *** yuanying has joined #openstack-ironic | 17:10 | |
| *** yuanying has quit IRC | 17:15 | |
| *** harlowja_away is now known as harlowja | 17:19 | |
| *** penick has quit IRC | 17:21 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Take iSCSI deploy out of pxe driver https://review.openstack.org/111232 | 17:28 |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Add support for creating vfat disk images https://review.openstack.org/105413 | 17:32 |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Add support for interacting with swift https://review.openstack.org/105795 | 17:36 |
| *** rameshg87 has joined #openstack-ironic | 17:37 | |
| openstackgerrit | A change was merged to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/114135 | 17:39 |
| *** igordcard has quit IRC | 17:39 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: IloVirtualMediaIscsi deploy driver https://review.openstack.org/113865 | 17:42 |
| rameshg87 | NobodyCam, jroll, request you to provide thoughts on ironic-ilo-virtualmedia-driver patchsets starting with https://review.openstack.org/#/c/111232/ :) | 17:44 |
| *** jcoufal has quit IRC | 17:44 | |
| *** bmahalakshmi has quit IRC | 17:45 | |
| *** rameshg87 has quit IRC | 17:46 | |
| *** pelix has quit IRC | 17:47 | |
| *** tatyana has quit IRC | 17:50 | |
| *** rushiagr is now known as rushiagr_away | 17:53 | |
| *** dlaube has quit IRC | 17:54 | |
| *** dlaube has joined #openstack-ironic | 17:54 | |
| jroll | rame<tab> sigh | 17:55 |
| jroll | night lucas-dinner :) | 17:55 |
| *** tatyana has joined #openstack-ironic | 17:58 | |
| *** penick has joined #openstack-ironic | 18:02 | |
| NobodyCam | jroll: shouldn't that be "Le Sigh" | 18:03 |
| NobodyCam | https://www.youtube.com/watch?v=sVkbw2mf51c | 18:03 |
| jroll | NobodyCam: heh, thought about it | 18:04 |
| NobodyCam | :-p | 18:04 |
| jroll | sometimes gertty really sucks at syncing | 18:05 |
| jroll | NobodyCam: this is going to be a contentious one https://review.openstack.org/#/c/112049/ | 18:08 |
| jroll | vdrok: ^^ left a review | 18:08 |
| *** eghobo has joined #openstack-ironic | 18:10 | |
| rloo | jroll: that's cuz you guys use Debug in production... | 18:11 |
| jroll | as you do | 18:11 |
| *** yuanying has joined #openstack-ironic | 18:12 | |
| jroll | (by that I mean as you should) | 18:12 |
| jroll | rloo: but even if we don't, if a driver is broken on load and unit tests don't catch that, that's a problem | 18:13 |
| rloo | jroll: yes, it would be a problelm if a unit test didn't catch it. but this wouldn't be the first time a unit test didn't catch something. | 18:14 |
| jroll | then kick the driver out of the tree | 18:14 |
| jroll | like | 18:14 |
| jroll | if the driver is never initialized in unit tests | 18:14 |
| jroll | that's a major problem | 18:15 |
| *** jeblair has joined #openstack-ironic | 18:15 | |
| rloo | jroll: i was just starting to look at that patch. i haven't decided whether i like the debug or not. ie, i don't care really. | 18:15 |
| rloo | jroll: i think i'll just move on to another patch then ;) | 18:15 |
| jroll | rloo: it's more about what deva said there | 18:15 |
| jroll | "a non-configured driver failing to load for *any* reason should be squashed and ignored." | 18:16 |
| jroll | completely agree. | 18:16 |
| jroll | if a driver is disabled, it shouldn't affect how the code runs | 18:16 |
| jeblair | hi, i need an ironic core to weigh in on https://review.openstack.org/#/c/115248/ | 18:16 |
| rloo | jroll: ok, you've convinced me ;) | 18:16 |
| NobodyCam | I would be ok with a info:"cant load balh" | 18:16 |
| jroll | jeblair: looking | 18:16 |
| *** yuanying has quit IRC | 18:17 | |
| jroll | jeblair: so that causes the ironic job to only run for master, and not havana/icehouse, correct? | 18:18 |
| jeblair | jroll: correct | 18:18 |
| jroll | I tend to think we should run it on icehouse, we made a release | 18:18 |
| jroll | russellb: ^^ | 18:18 |
| * russellb perks up | 18:18 | |
| jroll | russellb: https://review.openstack.org/#/c/115248/ I think we should run this on icehouse | 18:18 |
| jroll | we made a release, we've been doing backports | 18:19 |
| russellb | ah, ok, i guess that makes sense | 18:19 |
| russellb | fine with me | 18:19 |
| * jroll leaves a not | 18:19 | |
| jroll | e | 18:19 |
| russellb | i'll update, thanks! | 18:19 |
| jroll | NobodyCam: did we have a havana release? I think not but that was before I joined | 18:20 |
| *** harlowja has quit IRC | 18:20 | |
| *** harlowja_ has joined #openstack-ironic | 18:20 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Use LLDP to get switch port mapping https://review.openstack.org/92627 | 18:20 |
| jroll | bbiab | 18:21 |
| jeblair | jroll, russellb: https://review.openstack.org/#/admin/projects/openstack/ironic,branches | 18:21 |
| jeblair | says there's an icehouse, but no havana | 18:21 |
| russellb | updated patch | 18:21 |
| russellb | jeblair: i should have checked that, my bad | 18:21 |
| jeblair | yay glad i asked! :) | 18:22 |
| jeblair | jroll: if you could leave a comment on the updated patch, that would be swell | 18:22 |
| *** rushiagr_away is now known as rushiagr | 18:23 | |
| *** MattMan has left #openstack-ironic | 18:28 | |
| *** eghobo has quit IRC | 18:35 | |
| *** eghobo has joined #openstack-ironic | 18:36 | |
| *** eghobo has quit IRC | 18:47 | |
| *** derekh has joined #openstack-ironic | 19:00 | |
| jroll | jeblair, russellb, +1'd, thanks for checking :) | 19:04 |
| *** penick has quit IRC | 19:06 | |
| *** rushiagr is now known as rushiagr_away | 19:10 | |
| *** yuanying has joined #openstack-ironic | 19:13 | |
| russellb | jroll: thanks for the quick feedback | 19:15 |
| jroll | np :) | 19:15 |
| *** yuanying has quit IRC | 19:19 | |
| *** penick has joined #openstack-ironic | 19:19 | |
| *** lucas-dinner has quit IRC | 19:21 | |
| *** tatyana has quit IRC | 19:22 | |
| NobodyCam | brb | 19:27 |
| NobodyCam | rloo: 115300 landing now :) | 19:43 |
| rloo | NobodyCam: thx! | 19:43 |
| NobodyCam | brb again :-p | 19:45 |
| *** jcoufal has joined #openstack-ironic | 19:48 | |
| *** aswadr has quit IRC | 19:52 | |
| dlaube | Hey guys, I'm trying to wrap my head around how one would go about launching a baremetal server that was already enrolled into ironic. I'm assuming that when I execute ironic node-create, I should be specifying some key/value pairs in the node properties as an identifier… perhaps something like hwtype=super-duper-server or hwtype=old-single-proc, etc | 19:53 |
| dlaube | if that assumption is correct, would I then make a call to the nova api and reference my hwtype somehow? | 19:53 |
| *** harlowja_ is now known as harlowja_away | 19:58 | |
| *** athomas has quit IRC | 20:07 | |
| *** yuanying has joined #openstack-ironic | 20:15 | |
| openstackgerrit | A change was merged to openstack/ironic: return power state ERROR instead of an exception https://review.openstack.org/115300 | 20:16 |
| *** yuanying has quit IRC | 20:21 | |
| *** penick has quit IRC | 20:23 | |
| *** penick has joined #openstack-ironic | 20:25 | |
| *** penick has quit IRC | 20:29 | |
| *** stevebaker has quit IRC | 20:33 | |
| *** stevebaker has joined #openstack-ironic | 20:33 | |
| *** penick has joined #openstack-ironic | 20:34 | |
| *** stevebaker has quit IRC | 20:43 | |
| *** stevebaker has joined #openstack-ironic | 20:43 | |
| *** jasondotstar has joined #openstack-ironic | 21:05 | |
| *** linggao has quit IRC | 21:05 | |
| jroll | so quiet | 21:12 |
| NobodyCam | its nice | 21:13 |
| NobodyCam | lol | 21:13 |
| *** penick has quit IRC | 21:16 | |
| rloo | the calm before the storm. I hope not. | 21:16 |
| NobodyCam | don't say that lol :-p | 21:16 |
| NobodyCam | hhehe | 21:16 |
| *** harlowja_away is now known as harlowja_ | 21:17 | |
| jroll | heh | 21:17 |
| jroll | dlaube: did you figure things out? | 21:17 |
| *** yuanying has joined #openstack-ironic | 21:17 | |
| mrda | Morning Ironic! | 21:19 |
| dlaube | jroll: I'm afraid not | 21:20 |
| mrda | (and I'm not the storm :) | 21:20 |
| *** yuanying has quit IRC | 21:22 | |
| NobodyCam | morning mrda | 21:23 |
| mrda | \o | 21:24 |
| *** penick has joined #openstack-ironic | 21:26 | |
| *** penick has quit IRC | 21:34 | |
| NobodyCam | mrda: any one you know of looking at the nova api proxy stuff? | 21:40 |
| mrda | nope | 21:41 |
| mrda | sorry NobodyCam | 21:41 |
| jroll | heya mrda :) | 21:43 |
| mrda | hey jroll | 21:44 |
| jroll | dlaube: so you want to deploy a node | 21:44 |
| jroll | dlaube: you're correct in that you would do a nova call | 21:44 |
| jroll | nova needs a flavor with that node's specs, and I think there's some magic "baremetal" things that go on the flavor | 21:45 |
| dlaube | ahh | 21:47 |
| dlaube | looking at this right now http://docs.openstack.org/user-guide/content/novaclient_commands.html#novaclient_subcommand_baremetal-node-create | 21:47 |
| dlaube | googling for flavor to see what I can find | 21:47 |
| dlaube | OoOOohh https://wiki.openstack.org/wiki/Baremetal/Historical#How_to_choose_the_value_for_flavor | 21:48 |
| JayF | I'd warn against using some of the older docs as they may reference nova-baremetal | 21:50 |
| JayF | It may work still, but just beware of dragons :) | 21:50 |
| *** penick has joined #openstack-ironic | 21:51 | |
| jroll | that's definitely baremetal | 21:52 |
| jroll | I... wouldn't use it | 21:52 |
| dlaube | thank you for the heads-up jroll | 21:53 |
| dlaube | roger that | 21:53 |
| jroll | np | 21:53 |
| dlaube | I'm still googling trying to find something more recent | 21:53 |
| * jroll looks for a thing | 21:53 | |
| * JayF hopes the thing you're looking for exists at all | 21:53 | |
| dlaube | didnt find anything related to flavors in the ironic docs | 21:53 |
| JayF | dlaube: that means you haven't written it yet, right :( | 21:53 |
| dlaube | hehe | 21:54 |
| jroll | dlaube: I always go to the code :) | 21:54 |
| jroll | https://github.com/openstack-dev/devstack/blob/master/lib/ironic#L421-428 | 21:54 |
| dlaube | JayF: darn, I knew I forgot something! | 21:54 |
| jroll | ignore the last line | 21:54 |
| dlaube | sweet! | 21:54 |
| jroll | honestly this should be all you need https://github.com/openstack-dev/devstack/blob/master/lib/ironic#L423 | 21:54 |
| jroll | maybe there's nothing special for baremetal... | 21:54 |
| jroll | idk. | 21:55 |
| dlaube | thank you jroll | 21:55 |
| jroll | not a problem | 21:55 |
| *** harlowja_ has quit IRC | 21:59 | |
| *** harlowja has joined #openstack-ironic | 22:02 | |
| *** derekh has quit IRC | 22:02 | |
| *** pcrews has quit IRC | 22:14 | |
| NobodyCam | lol I suppose this dose not count: http://paste.openstack.org/show/hNLkJ4Yjb33Z11H9LHJn/ | 22:18 |
| *** yuanying has joined #openstack-ironic | 22:18 | |
| jroll | lol | 22:18 |
| JayF | how many gigabytes are ironic | 22:20 |
| jroll | that's a boot-from-cinder server | 22:20 |
| JayF | hot | 22:20 |
| jroll | ain't it ironic | 22:21 |
| *** yuanying has quit IRC | 22:24 | |
| *** jasondotstar has quit IRC | 22:24 | |
| NobodyCam | lol | 22:25 |
| NobodyCam | brb | 22:25 |
| *** jasondotstar has joined #openstack-ironic | 22:29 | |
| *** jgrimm has quit IRC | 22:33 | |
| *** jasondotstar has quit IRC | 22:39 | |
| *** pcrews has joined #openstack-ironic | 22:54 | |
| *** shakamunyi has quit IRC | 23:13 | |
| *** eghobo has joined #openstack-ironic | 23:17 | |
| *** yuanying has joined #openstack-ironic | 23:20 | |
| *** yuanying has quit IRC | 23:24 | |
| *** yuanying has joined #openstack-ironic | 23:29 | |
| openstackgerrit | Gregory Haynes proposed a change to openstack/ironic: WIP: Add conductor_hostname property to nodes https://review.openstack.org/109688 | 23:33 |
| JayF | That is absolutely something I'd love to see as well | 23:34 |
| JayF | going to review it now | 23:34 |
| JayF | greghaynes: ^ thanks | 23:34 |
| greghaynes | :) | 23:34 |
| JayF | jroll: ^ wdyt | 23:34 |
| * greghaynes might need some help with some mocking that needs to happen for tests to pass | 23:34 | |
| JayF | greghaynes: well, I take it back. I'll review it once tests are passing ;) | 23:34 |
| JayF | heh | 23:34 |
| jroll | um | 23:34 |
| * JayF hides under a rock | 23:34 | |
| jroll | I would like to see HA actually working :) | 23:35 |
| JayF | I agree with that as well :) | 23:35 |
| greghaynes | jroll: Yea, that patch is a pre-req | 23:35 |
| JayF | greghaynes: as a note; you're going to want to file a bug about this and then add "closes-bug: 123456" to your commit message | 23:35 |
| greghaynes | I was going to poke at failover once this is done | 23:35 |
| jroll | greghaynes: right :) | 23:35 |
| jroll | well | 23:35 |
| jroll | that's a partial | 23:35 |
| greghaynes | JayF: Oh? That review seems like a feature missing, the failover for HA not working I think is a bug though | 23:35 |
| greghaynes | yea, partial makes sense | 23:36 |
| JayF | that is a bug and is filed | 23:36 |
| jroll | there's a bug for this | 23:36 |
| JayF | ah, so partial-bug on russell's bug | 23:36 |
| JayF | that makes more sense than what I said | 23:36 |
| *** eghobo has quit IRC | 23:36 | |
| jroll | https://bugs.launchpad.net/ironic/+bug/1355510 | 23:36 |
| jroll | is what I was thinking of | 23:36 |
| jroll | which is not russell's but | 23:36 |
| JayF | aha, alex's bug | 23:36 |
| JayF | well someone over here | 23:36 |
| JayF | lol | 23:36 |
| greghaynes | hehe | 23:36 |
| jroll | aweeks: https://review.openstack.org/109688 relevant to your interests | 23:37 |
| greghaynes | So, im getting permission denied failures when I run tests locally, any idea what might cause this? | 23:37 |
| jroll | greghaynes: what OS? | 23:37 |
| jroll | sounds wrong | 23:38 |
| greghaynes | seems like popen.ini | 23:38 |
| greghaynes | er, .init | 23:38 |
| greghaynes | ubuntu | 23:38 |
| jroll | right, it's trying to run something... | 23:38 |
| jroll | are you using tox? | 23:38 |
| jroll | would love a traceback | 23:38 |
| jroll | greghaynes: going back to your patch, we're going to have to do a large update when a conductor joins/parts the ring, yeah? is that going to be a problem? | 23:39 |
| jroll | oh, someone already asked, got it | 23:39 |
| greghaynes | jroll: thats basically the reason for adding the index | 23:40 |
| greghaynes | http://paste.ubuntu.com/8101944/ | 23:40 |
| jroll | yeah | 23:40 |
| greghaynes | via tox | 23:40 |
| jroll | still, large update sounds not ideal | 23:41 |
| greghaynes | well, its unavoidable, but if our hashring virtual nodes is reasonably picked it should be fairly well balanced | 23:41 |
| jroll | yeah | 23:41 |
| jroll | could still be thousands of rows, though | 23:42 |
| greghaynes | I could see some rate limiting / similar being a thing we need, but walk first ;) | 23:42 |
| rloo | jroll: are you going to fix this soon: https://review.openstack.org/#/c/112351/ | 23:43 |
| jroll | rloo: yeah, I should do that huh | 23:43 |
| jroll | (I will, likely tomorrow but maybe tonight) | 23:43 |
| *** christop1eraedo has joined #openstack-ironic | 23:44 | |
| rloo | jroll: yeah :-) don't want to have too many reviews to look at next week. I'm also thinking of taking some time off. Summer is almost over. sigh. | 23:44 |
| jroll | greghaynes: so your tox thing looks like maybe your user doesn't have permission to write to /tmp? or execute from /tmp I should say | 23:44 |
| greghaynes | ah! | 23:44 |
| jroll | rloo: indeed, I need to do the same :( | 23:44 |
| *** christopheraedo has quit IRC | 23:44 | |
| jroll | greghaynes: it's failing here https://github.com/openstack/ironic/blob/master/ironic/tests/test_utils.py#L99 | 23:44 |
| greghaynes | yep, im noexec /tmp | 23:45 |
| jroll | heh | 23:45 |
| *** Hefeweizen has quit IRC | 23:45 | |
| jroll | yep | 23:45 |
| rloo | jroll: maybe we should find out who's here next week then. That's the week before J3. Will at at Monday's meeting if I'm there :D | 23:45 |
| jroll | personally I would file a bug there | 23:45 |
| jroll | but idk | 23:45 |
| *** Hefeweizen has joined #openstack-ironic | 23:45 | |
| jroll | rloo: I'll be around until j3, at least | 23:45 |
| jroll | rloo: but I can fix up that review for you | 23:45 |
| jroll | wow, it's wednesday already | 23:45 |
| rloo | jroll: ah, then that's ok then. you can do reviews. whew. | 23:45 |
| jroll | lolllll | 23:45 |
| * jroll reviews 112351 | 23:46 | |
| greghaynes | https://bugs.launchpad.net/ironic/+bug/1359463 | 23:48 |
| jroll | thanks! | 23:48 |
| jroll | now fix it :P | 23:48 |
| greghaynes | hah | 23:48 |
| *** dhellmann has quit IRC | 23:49 | |
| *** eghobo has joined #openstack-ironic | 23:50 | |
| *** dhellmann has joined #openstack-ironic | 23:50 | |
| *** openstackgerrit has quit IRC | 23:54 | |
| *** aswadr has joined #openstack-ironic | 23:55 | |
| rloo | jroll: quick question. for the agent* drivers, no console interface? | 23:57 |
| JayF | Console is provided by boot/power driver | 23:58 |
| JayF | i.e. ipmi | 23:58 |
| JayF | not by deploy driver | 23:58 |
| *** openstackgerrit has joined #openstack-ironic | 23:59 | |
| rloo | JayF: I don't see it in ironic.drivers.agent.AgentAndIPMIToolDriver. | 23:59 |
| JayF | Hmm. | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!