| adam_g | JayF, just got done, actually. woulda been 5 if those caches were used :) | 00:00 |
|---|---|---|
| JayF | it was about enough time for pong | 00:00 |
| *** zdiN0bot has quit IRC | 00:01 | |
| adam_g | JayF, if you're planning on doing a lot of devstack work you may want to consider keeping a base VM image warm with its dependencies cached. i use the scripts here to do it similar to the gate: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/nodepool/scripts | 00:01 |
| devananda | g'night all, gotta run out again!~ | 00:02 |
| adam_g | devananda, cheers | 00:02 |
| adam_g | JayF, after a fresh devstack run, you should have at least 2 neutron ports created, a router port and a dhcp port. there shouold be a dnsmasq process running, serving config from /opt/stack/data/neutron/dhcp/$net_id | 00:04 |
| JayF | that dnsmasq process is still running with bind-interfaces | 00:04 |
| JayF | so is it not possible to run multiple dhcp-agents on the same physical box? | 00:04 |
| JayF | because we'll pretty much need to send different dhcp opts for the agent vs pxe driver | 00:05 |
| JayF | and/or hardcode some of it into the pxe config pieces | 00:05 |
| JayF | adam_g: I see the same thing now, fwiw, although --conf-file= is a really strange option to see | 00:05 |
| *** epim has quit IRC | 00:06 | |
| adam_g | JayF, there is a reason for that, i dont remember what it is, tho :) | 00:08 |
| JayF | I think for now my concerns are mildly abated | 00:08 |
| JayF | although the more I find out about how this works under the hood, the more I dislike it | 00:08 |
| *** epim has joined #openstack-ironic | 00:10 | |
| *** epim has quit IRC | 00:12 | |
| JayF | it must implement multiple dhcp agents via tags in dnsmasq | 00:13 |
| JayF | although that means I don't think it'd be possible to expose a dhcp range out one interface/vlan and not another one | 00:14 |
| adam_g | JayF, what are the dhcp requirements for the agent? | 00:14 |
| JayF | I got started down this rabbithole trying to figure out how to get agent and pxe driver running alongside | 00:14 |
| JayF | the agent ramdisk will have different kernel options required, among other things | 00:15 |
| JayF | JoshNang is working on that piece now, and I think we figured out the basics of the how | 00:15 |
| JayF | I've just kept digging to see if running this neutron-based dhcp stuff is viable at all for a larger deployment | 00:15 |
| JayF | and I'm thinking no, for a bunch of reasons, the clearest of which is having bad history with dnsmasq at scale | 00:16 |
| JoshNang | so for kernel params, I think we'll just need to make our own pxe_config.template with the params | 00:17 |
| JayF | JoshNang: something to think of, is that some people might want to use the agent with a ramdisk built with DIB or something other than our coreos image -- should we look at making that template user-specifyable in some cases? | 00:18 |
| *** epim has joined #openstack-ironic | 00:18 | |
| JayF | i.e. the kernel params/pxe config is almost as much tied to how the ramdisk was created as it is what's inside it | 00:19 |
| *** eguz has joined #openstack-ironic | 00:19 | |
| JoshNang | it's already an ironic config option | 00:21 |
| JayF | good, so we can specify an ipxe bootstrap if we wanted to, for instance | 00:21 |
| adam_g | JayF, i may be wrong, but i believe dnsmasq is a deployment choice on the neutron side and used in the reference ovs/linux configuration | 00:22 |
| JayF | it is, but afaict it is the only choice | 00:22 |
| JayF | and we'll have an option in the agent driver to say 'let us handle dhcp/pxe' | 00:22 |
| JayF | and for any production deployment I manage, we'll be flipping that switch | 00:23 |
| *** eghobo has quit IRC | 00:23 | |
| Shrews | JayF: curious... while waiting for nova to boot an instance, have you noticed if qemu-system-* is taking 100% cpu? I'm seeing that in a fresh devstack install, then things timeout and nova rolls over to a new node and does the same thing. | 00:25 |
| Shrews | but i'm trying on different h/w, so wondering if it's my setup | 00:26 |
| JayF | well I would give you an answer, except I apparently can't get anything booted anymore | 00:27 |
| JayF | ERROR (BadRequest): Invalid key_name provided. (HTTP 400) (Request-ID: req-77040e0c-cceb-4a96-a6ec-a3dce8184cce) | 00:27 |
| Shrews | :( | 00:28 |
| *** epim has quit IRC | 00:29 | |
| *** zdiN0bot has joined #openstack-ironic | 00:33 | |
| *** zdiN0bot has quit IRC | 00:38 | |
| *** zdiN0bot has joined #openstack-ironic | 00:44 | |
| *** zdiN0bot1 has joined #openstack-ironic | 00:49 | |
| *** rloo has quit IRC | 00:49 | |
| *** zdiN0bot has quit IRC | 00:49 | |
| *** zdiN0bot1 has quit IRC | 00:53 | |
| *** Shrews has quit IRC | 00:58 | |
| *** Shrews has joined #openstack-ironic | 01:00 | |
| *** harlowja has joined #openstack-ironic | 01:00 | |
| adam_g | oh hum | 01:06 |
| adam_g | i wonder if this broke us https://review.openstack.org/#/c/88384/ | 01:10 |
| *** zdiN0bot has joined #openstack-ironic | 01:20 | |
| *** zdiN0bot1 has joined #openstack-ironic | 01:22 | |
| anteaya | devananda: ask away | 01:23 |
| anteaya | devananda: sorry I missed you | 01:24 |
| *** zdiN0bot has quit IRC | 01:25 | |
| *** zdiN0bot1 has quit IRC | 01:27 | |
| *** eguz has quit IRC | 01:33 | |
| *** nosnos has joined #openstack-ironic | 01:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/89234 | 01:41 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/89244 | 01:48 |
| *** jrist has quit IRC | 02:02 | |
| *** coolsvap has quit IRC | 02:05 | |
| *** jrist has joined #openstack-ironic | 02:10 | |
| *** coolsvap has joined #openstack-ironic | 02:10 | |
| *** zdin0bot has joined #openstack-ironic | 02:11 | |
| *** max_lobur has quit IRC | 02:33 | |
| *** newell has quit IRC | 02:45 | |
| *** NobodyCam has quit IRC | 02:46 | |
| *** zdin0bot has quit IRC | 02:52 | |
| *** zdin0bot has joined #openstack-ironic | 02:53 | |
| *** harlowja is now known as harlowja_away | 03:09 | |
| *** zdin0bot1 has joined #openstack-ironic | 03:28 | |
| *** zdin0bot has quit IRC | 03:32 | |
| *** NobodyCam has joined #openstack-ironic | 03:51 | |
| NobodyCam | gah my cloud irc crashed | 03:52 |
| NobodyCam | will finish rebuild tomorrow | 03:52 |
| *** eghobo has joined #openstack-ironic | 04:01 | |
| *** coolsvap has quit IRC | 04:05 | |
| *** coolsvap has joined #openstack-ironic | 04:07 | |
| *** harlowja_away is now known as harlowja | 04:18 | |
| *** lazy_prince has joined #openstack-ironic | 04:25 | |
| *** urulama has joined #openstack-ironic | 04:49 | |
| *** vkozhukalov has joined #openstack-ironic | 05:03 | |
| *** rwsu has quit IRC | 05:07 | |
| *** nosnos has quit IRC | 05:08 | |
| *** urulama has quit IRC | 05:21 | |
| *** rwsu has joined #openstack-ironic | 05:23 | |
| *** max_lobur has joined #openstack-ironic | 05:25 | |
| *** max_lobur1 has joined #openstack-ironic | 05:33 | |
| *** max_lobur has quit IRC | 05:35 | |
| *** nosnos has joined #openstack-ironic | 05:35 | |
| *** eghobo has quit IRC | 05:39 | |
| *** sabah has joined #openstack-ironic | 05:39 | |
| *** eghobo has joined #openstack-ironic | 05:39 | |
| *** zdin0bot1 has quit IRC | 05:40 | |
| davidlenwell | devananda: you around ? | 05:41 |
| davidlenwell | was following along with this .. http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html | 05:42 |
| davidlenwell | has anyone tested this stuff with supermicro boards? | 05:42 |
| *** eguz has joined #openstack-ironic | 05:46 | |
| davidlenwell | getting WARNING wsme.api [-] Client-side error: BMC inaccessible for node 8431e2c9-cda5-4e1a-a321-7985900aa4fa: [Errno 2] No such file or directory | 05:48 |
| *** eghobo has quit IRC | 05:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/88508 | 06:07 |
| *** vkozhukalov has quit IRC | 06:08 | |
| *** zdin0bot has joined #openstack-ironic | 06:10 | |
| *** harlowja is now known as harlowja_away | 06:30 | |
| *** zdiN0bot1 has joined #openstack-ironic | 06:46 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Update Ironic User Guide post landing for I08faac6 https://review.openstack.org/89818 | 06:46 |
| *** ndipanov has joined #openstack-ironic | 06:48 | |
| *** zdiN0bot1 has quit IRC | 06:51 | |
| *** foexle has joined #openstack-ironic | 06:52 | |
| *** ifarkas has joined #openstack-ironic | 06:59 | |
| *** lsmola has joined #openstack-ironic | 07:09 | |
| *** eguz has quit IRC | 07:13 | |
| *** lucas-dinner has quit IRC | 07:17 | |
| *** ifarkas has quit IRC | 07:25 | |
| *** sabah has quit IRC | 07:26 | |
| *** ifarkas has joined #openstack-ironic | 07:26 | |
| *** yuriyz has joined #openstack-ironic | 07:27 | |
| *** mrda is now known as mrda_away | 07:29 | |
| *** yuriyz has quit IRC | 07:31 | |
| *** yuriyz has joined #openstack-ironic | 07:33 | |
| *** vkozhukalov has joined #openstack-ironic | 07:50 | |
| *** romcheg1 has joined #openstack-ironic | 07:52 | |
| *** foexle has quit IRC | 07:55 | |
| Mikhail_D_wk | Morning all!!! :) | 08:02 |
| *** nosnos has quit IRC | 08:18 | |
| *** derekh has joined #openstack-ironic | 08:18 | |
| *** jistr has joined #openstack-ironic | 08:21 | |
| *** lucasagomes has joined #openstack-ironic | 08:23 | |
| dtantsur | Morning Ironic | 08:26 |
| *** foexle has joined #openstack-ironic | 08:32 | |
| *** zdin0bot has quit IRC | 08:35 | |
| *** athomas has joined #openstack-ironic | 08:39 | |
| davidlenwell | morning | 08:39 |
| *** viktors|afk is now known as viktors | 08:45 | |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code https://review.openstack.org/72418 | 08:46 |
| *** martyntaylor has joined #openstack-ironic | 08:48 | |
| *** zdin0bot has joined #openstack-ironic | 09:05 | |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Sync latest code and reuse exceptions from oslo https://review.openstack.org/71500 | 09:05 |
| *** sabah has joined #openstack-ironic | 09:06 | |
| *** zdin0bot has quit IRC | 09:10 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Update Ironic User Guide post landing for 41af7d6b https://review.openstack.org/89818 | 09:35 |
| *** stephenpearson has joined #openstack-ironic | 09:49 | |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/ironic: Renaming ironicclient exceptions in nova driver https://review.openstack.org/90040 | 09:50 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code https://review.openstack.org/72418 | 09:56 |
| *** zdin0bot has joined #openstack-ironic | 10:06 | |
| *** zdin0bot has quit IRC | 10:10 | |
| *** nosnos has joined #openstack-ironic | 10:15 | |
| agordeev | g'morning Ironic :) | 10:27 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add ManagementInterface https://review.openstack.org/86063 | 10:52 |
| lucasagomes | morning agordeev | 10:54 |
| *** zdin0bot has joined #openstack-ironic | 11:07 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Overwrite instance_exists in Nova Ironic Driver https://review.openstack.org/88611 | 11:10 |
| *** zdin0bot has quit IRC | 11:11 | |
| agordeev | lucasagomes: morning. how are you? | 11:15 |
| lucasagomes | agordeev, all is well, urself? | 11:15 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMITool to use the new ManagementInterface https://review.openstack.org/86092 | 11:15 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: SeaMicro to use the new ManagementInterface https://review.openstack.org/86328 | 11:15 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMINative to use the new ManagementInterface https://review.openstack.org/86588 | 11:16 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: SSH virsh to use the new ManagementInterface https://review.openstack.org/89884 | 11:17 |
| agordeev | lucasagomes: pretty good :) | 11:18 |
| lucasagomes | :) | 11:19 |
| dtantsur | morning, agordeev, lucasagomes :) | 11:29 |
| lucasagomes | hey hey morning dtantsur :) | 11:30 |
| *** romcheg1 has quit IRC | 11:42 | |
| agordeev | dtantsur: morning | 11:42 |
| *** sabah has quit IRC | 11:53 | |
| *** zdin0bot has joined #openstack-ironic | 12:08 | |
| *** zdin0bot has quit IRC | 12:12 | |
| *** jdob has joined #openstack-ironic | 12:16 | |
| NobodyCam | good morning Ironic | 12:36 |
| agordeev | NobodyCam: morning :) | 12:37 |
| dtantsur | morning, NobodyCam | 12:37 |
| NobodyCam | morning agordeev dtantsur ... working on my irssi and ofc mak'n coffee | 12:40 |
| *** nosnos has quit IRC | 12:42 | |
| *** rloo has joined #openstack-ironic | 12:46 | |
| *** linggao has joined #openstack-ironic | 12:49 | |
| lucasagomes | morning NobodyCam | 12:50 |
| NobodyCam | morning lucasagomes | 12:52 |
| lucasagomes | NobodyCam, when you get some time: https://review.openstack.org/#/c/88307/ | 12:55 |
| lucasagomes | (I know it's big and painful to review that :( ) | 12:56 |
| NobodyCam | brb | 12:58 |
| *** NobodyCam has quit IRC | 12:58 | |
| *** athomas has quit IRC | 12:58 | |
| *** NobodyCam has joined #openstack-ironic | 12:58 | |
| NobodyCam | ok irssi is back | 12:58 |
| NobodyCam | lucasagomes: can you send me that ping again | 12:59 |
| NobodyCam | s/ping/url/ | 12:59 |
| lucasagomes | NobodyCam, yeah | 12:59 |
| lucasagomes | <lucasagomes> NobodyCam, when you get some time: https://review.openstack.org/#/c/88307/ | 12:59 |
| lucasagomes | <lucasagomes> (I know it's big and painful to review that :( ) | 12:59 |
| NobodyCam | :) | 12:59 |
| NobodyCam | TY | 12:59 |
| lucasagomes | :) | 12:59 |
| rloo | hi lucasagomes, when you have a few minutes, would like to discuss this with you: https://review.openstack.org/#/c/73005/6/ironic/drivers/modules/ssh.py | 13:04 |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Remove py3kcompat module https://review.openstack.org/87656 | 13:04 |
| lucasagomes | rloo, morning :) sure | 13:06 |
| NobodyCam | lucasagomes: how did you pick 100 workers? (default=100) | 13:06 |
| NobodyCam | morning rloo | 13:06 |
| lucasagomes | NobodyCam, the default was 64 before from the old rpc_thread_pool_size option | 13:06 |
| rloo | Morning NobodyCam ;) | 13:07 |
| lucasagomes | NobodyCam, I didn't pick it from anywhere, I just came up with that umber | 13:07 |
| lucasagomes | number* | 13:07 |
| *** athomas has joined #openstack-ironic | 13:07 | |
| lucasagomes | NobodyCam, just looks like a reasonable number | 13:07 |
| lucasagomes | NobodyCam, the 64 was pick in the same way (take a look at https://review.openstack.org/#/c/6792/) | 13:08 |
| *** zdin0bot has joined #openstack-ironic | 13:08 | |
| lucasagomes | NobodyCam, do u think it's too much to have 100 workers? should we take less? I really have no good idea of what a good number would be | 13:09 |
| NobodyCam | no I had just read the "Can we change rpc_thread_pool_size default value" thread on ml. | 13:09 |
| lucasagomes | yeah | 13:09 |
| rloo | lucasagomes: wrt ManagementInterface: https://review.openstack.org/#/c/86063/12/ironic/drivers/modules/fake.py | 13:10 |
| NobodyCam | and "Given DB calls all block the process right now, there's really not much advantage to a larger pool size. 64" was just stuck in my head | 13:10 |
| lucasagomes | rloo, so... yeah good point about the one-and-only-one or two-and-only-two way | 13:10 |
| lucasagomes | yeah | 13:11 |
| rloo | lucasagomes: what do you think about having a _boot_device=boot_devices.PXE, and set_boot_device can set this. | 13:11 |
| rloo | lucasagomes: umm, just thinking. I guess what you have is OK since boot_device can only be one value. Forget this conversation. | 13:11 |
| lucasagomes | rloo, heh, ok | 13:12 |
| *** zdin0bot has quit IRC | 13:13 | |
| *** coolsvap is now known as coolsvap|afk | 13:13 | |
| lucasagomes | rloo, so back to the optional/mandatory options... maybe we should not have this one-and-only-one option (or two, three etc...) we can have some priority order. E.g for the ssh driver, you can use password or key_file, if both r specified we are going to use the key_file | 13:13 |
| lucasagomes | rloo, so we would just need to document in the driver it | 13:13 |
| *** coolsvap|afk is now known as coolsvap | 13:13 | |
| rloo | lucasagomes: are you OK if I move them all to required instead of options? I think if we are going to 'err', we might as well have the user error on providing than not providing the info. | 13:14 |
| rloo | lucasagomes: also, there is a 3rd option key_content (in the latest code, my patch is old). Makes it more interesting. | 13:14 |
| lucasagomes | rloo, yeah, that would be a bit missleading tho, because the user will think that he needs to specify both options | 13:15 |
| lucasagomes | key_file and password | 13:15 |
| lucasagomes | yeah key_content now as well heh | 13:15 |
| lucasagomes | damn | 13:15 |
| * lucasagomes thinks a better way to express it | 13:15 | |
| lucasagomes | about* | 13:15 |
| rloo | key_file, key_content, and/or password. ha ha. I thought if we put it in required and the description says one of these are required? | 13:15 |
| rloo | and yes, I think it may make more sense to have an ordering/priority instead of *only* one. I believe we do that elsewhere (but it isn't documented, can't remember where I saw it.) | 13:16 |
| lucasagomes | rloo, right, yeah I think that required + the help string saying that only one of those are required works | 13:17 |
| lucasagomes | I really want to have a better way to handle credentials in ironic | 13:18 |
| NobodyCam | doh we still have module=py3kcompat I thought there was a patch to remove that | 13:18 |
| rloo | lucasagomes: ok thx! I'll open another bug to do the priority (seems like too much to change it in this review) | 13:18 |
| rloo | NobodyCam: I just ok'd the removal for ironicclient. is it in ironic still? | 13:18 |
| NobodyCam | its in openstack-common.conf | 13:19 |
| NobodyCam | :-p | 13:19 |
| lucasagomes | rloo, ack thank u | 13:20 |
| rloo | NobodyCam: I feel like there are all these invisible workers. How does anyone know to remove py3kcompat anyway? | 13:21 |
| lucasagomes | NobodyCam, mind submitting a patch to remove it? | 13:21 |
| lucasagomes | rloo, hmmm you grep the code and see if there's anyone actually using it | 13:21 |
| lucasagomes | if not, you can remove | 13:21 |
| lucasagomes | heh | 13:21 |
| lucasagomes | that's how I do :P | 13:22 |
| NobodyCam | ya need to check that first | 13:22 |
| NobodyCam | lucasagomes: LGTM, I need to test this change. will do that shortly and then +2 | 13:22 |
| rloo | lucasagomes: oh yeah, I meant, how does anyone know that we should even be removing py3kcompat? | 13:22 |
| lucasagomes | NobodyCam, yay thanks! | 13:22 |
| NobodyCam | I have a +1 on it now ... | 13:22 |
| lucasagomes | NobodyCam, ack :) | 13:22 |
| rloo | lucasagomes: unless you knew about the original bug about it | 13:22 |
| *** matty_dubs|gone is now known as matty_dubs | 13:24 | |
| lucasagomes | heh idk accident maybe? they might open openstack-common.conf and see it there and start wondering "hmm why it's still there" | 13:24 |
| lucasagomes | dunno really | 13:24 |
| lucasagomes | NobodyCam, do you think that we should expose in the API things like set/get boot device? | 13:28 |
| NobodyCam | testing the patch now | 13:30 |
| NobodyCam | humm | 13:30 |
| NobodyCam | vs via vendor passthru? | 13:31 |
| *** rloo has quit IRC | 13:34 | |
| *** rloo has joined #openstack-ironic | 13:34 | |
| lucasagomes | NobodyCam, yeah, well the blueprint I've is to remove it from the vendor_passthru | 13:35 |
| lucasagomes | NobodyCam, but what I dunno is to know if we ever wanted it to be exposed via the rest api or not | 13:35 |
| *** lazy_prince has quit IRC | 13:35 | |
| NobodyCam | lucasagomes: I see it being much more usefull for things like windows and whole disk deploys | 13:35 |
| NobodyCam | but ya I kinda do feel it is good to expose | 13:36 |
| lucasagomes | NobodyCam, right, but for those the driver could set the device it wants internally | 13:36 |
| lucasagomes | like we have for pxe | 13:36 |
| lucasagomes | NobodyCam, ack | 13:36 |
| *** andreykurilin has quit IRC | 13:37 | |
| NobodyCam | better to me consistent for all drivers, and I can see someone wantng to pxe deploy a whole disk image | 13:38 |
| NobodyCam | slow thou it may be | 13:38 |
| NobodyCam | :-p | 13:38 |
| lucasagomes | heh | 13:38 |
| lucasagomes | right I will expose those via api, I put some info in the bp link | 13:38 |
| lucasagomes | NobodyCam, https://blueprints.launchpad.net/ironic/+spec/promote-set-boot-device | 13:39 |
| NobodyCam | :) | 13:40 |
| *** jbjohnso has joined #openstack-ironic | 13:43 | |
| *** coolsvap is now known as coolsvap|afk | 13:57 | |
| *** romcheg1 has joined #openstack-ironic | 13:58 | |
| dtantsur | NobodyCam, devananda, btw, I hope I fixed https://blueprints.launchpad.net/ironic/+spec/pxe-master-images-caching Could you have a look again? | 14:00 |
| NobodyCam | dtantsur: sure... I will in a few min | 14:01 |
| dtantsur | thanks! | 14:01 |
| *** zdin0bot has joined #openstack-ironic | 14:09 | |
| NobodyCam | gah day oh updates | 14:10 |
| *** zdin0bot1 has joined #openstack-ironic | 14:11 | |
| *** zdin0bot has quit IRC | 14:13 | |
| *** zdin0bot1 has quit IRC | 14:15 | |
| NobodyCam | lucasagomes: woo hoo tests running again :-p | 14:30 |
| lucasagomes | NobodyCam, yay! | 14:30 |
| lucasagomes | NobodyCam, btw, there's anyone working on the tripleo undercloud test for ironic | 14:31 |
| NobodyCam | :-p brb... quick walkies and mo jo :) | 14:31 |
| lucasagomes | right now it's non voting, but it stills consume a lot of time and resource, if there's anybody working on it right now idk maybe we should disable it? | 14:31 |
| NobodyCam | lucasagomes: yes I am.. I just got what I think is a good test out of the gate lastnight | 14:34 |
| NobodyCam | what I am not seeing in the logs is the ping back from the node | 14:35 |
| NobodyCam | lucasagomes: http://logs.openstack.org/29/85529/20/check-tripleo/check-tripleo-ironic-undercloud-precise/963d091/ | 14:35 |
| lucasagomes | NobodyCam, ah I see, good stuff! | 14:36 |
| lucasagomes | yea i was unsure, hope we get that working soon | 14:36 |
| NobodyCam | its driving me nuts... I would love to see the console of the node :-p | 14:38 |
| lucasagomes | hah I can figure out :( | 14:39 |
| NobodyCam | if I saw the same locally it wouldn't be that bad. but it working locally for me :-p | 14:39 |
| lucasagomes | the console.txt there fails in the stack create | 14:39 |
| lucasagomes | oh damn, wait when those happens | 14:40 |
| lucasagomes | the hardest | 14:40 |
| lucasagomes | s/wait/I hate/g | 14:40 |
| lucasagomes | I need coffee lol | 14:40 |
| lucasagomes | I thought about I hate and wrote wait | 14:40 |
| NobodyCam | lol | 14:40 |
| NobodyCam | lucasagomes: here the api log (http://paste.openstack.org/show/FyzKqUh5ENo8NgbYenVN/) note no post to vender_passthru from the node. | 14:41 |
| NobodyCam | lol and conductor log is all of two lines. | 14:42 |
| NobodyCam | http://paste.openstack.org/show/3Bug63GIrp0Th0cegfZr/ | 14:42 |
| NobodyCam | no ping from the bm node we are deploying to. | 14:43 |
| lucasagomes | NobodyCam, you got debug enabled? | 14:43 |
| NobodyCam | with nothing in the conductor log I am assuming if there was a issue there would be a error in there | 14:43 |
| lucasagomes | that log in the conductor is very short | 14:43 |
| NobodyCam | lol I tried | 14:44 |
| lucasagomes | you should have a bunch of | 14:44 |
| lucasagomes | 2014-04-24 15:43:12.308 16130 DEBUG ironic.drivers.modules.ssh [-] Checking Node: IronicFedora's Mac address. _get_hosts_name_for_node /opt/stack/ironic/ironic/drivers/modules/ssh.py:331 | 14:44 |
| lucasagomes | 2014-04-24 15:43:12.401 16130 DEBUG ironic.drivers.modules.ssh [-] Checking Node: baremetal_0's Mac address. _get_hosts_name_for_node /opt/stack/ironic/ironic/drivers/modules/ssh.py:331 | 14:44 |
| lucasagomes | 2014-04-24 15:43:12.495 16130 DEBUG ironic.drivers.modules.ssh [-] Found Mac address: 52:54:00:e0:c1:be _get_hosts_name_for_node /opt/stack/ironic/ironic/drivers/modules/ssh.py:344 | 14:44 |
| lucasagomes | 2014-04-24 15:44:12.547 16130 DEBUG ironic.drivers.modules.ssh [-] Retrieved Node List: ['', 'IronicFedora', 'baremetal_0', 'IronicUbuntu', 'ubuntuprecise', '', ''] _get_hosts_name_for_node /opt/stack/ironic/ironic/drivers/modules/ssh.py:326 | 14:44 |
| lucasagomes | in the conductor log | 14:44 |
| lucasagomes | NobodyCam, right... hmm that makes things more difficult without the debug enabled | 14:44 |
| NobodyCam | lucasagomes: I can get debug log this way: https://review.openstack.org/#/c/85101/ | 14:45 |
| NobodyCam | but I cant set cross project dependices :-( | 14:45 |
| jroll | good morning Ironic | 14:45 |
| NobodyCam | good morning jroll | 14:45 |
| lucasagomes | NobodyCam, heh fair enough hardcoding it | 14:46 |
| lucasagomes | NobodyCam, there's no template for debug={{debug}} or something like that? | 14:46 |
| NobodyCam | lucasagomes: I tried many many ways to force ironic to do debug loging | 14:46 |
| lucasagomes | NobodyCam, and the debug= option was present in the config? | 14:47 |
| NobodyCam | lucasagomes: I see see that ssh trafic on the review where I was able to set debug | 14:47 |
| lucasagomes | jroll, morning | 14:47 |
| NobodyCam | lucasagomes: I can not get debug into the conf file with changing the tripleo element | 14:48 |
| lucasagomes | right | 14:48 |
| lucasagomes | NobodyCam, other elments does have a debug in it? | 14:48 |
| NobodyCam | other? | 14:48 |
| lucasagomes | like not hardcoded but as a template | 14:48 |
| lucasagomes | others | 14:48 |
| lucasagomes | idk nova, ceilometer etc... | 14:48 |
| lucasagomes | elements | 14:48 |
| lucasagomes | NobodyCam, | 14:49 |
| lucasagomes | ../tripleo-image-elements/elements/cinder/os-apply-config/etc/cinder/cinder.conf:debug={{cinder.debug}} | 14:50 |
| lucasagomes | ../tripleo-image-elements/elements/heat/os-config-applier/etc/heat/heat.conf:debug={{heat.debug}} | 14:50 |
| NobodyCam | yes we can add that be I'm not sure OoO would land anything that enabled that. | 14:50 |
| lucasagomes | I don't see any problem in having something like ^ | 14:51 |
| lucasagomes | http://paste.openstack.org/show/76954/ | 14:51 |
| NobodyCam | the tooling is good and SHOULD be added | 14:51 |
| openstackgerrit | A change was merged to openstack/ironic: Sync oslo/common/excutils https://review.openstack.org/87345 | 14:52 |
| *** zdiN0bot has joined #openstack-ironic | 14:53 | |
| NobodyCam | humm fatal: Couldn't find remote ref refs/changes/07/88307/6 | 14:54 |
| NobodyCam | fatal: The remote end hung up unexpectedly | 14:54 |
| lucasagomes | :( | 14:55 |
| devananda | mornin! | 14:56 |
| dtantsur | devananda, hi! | 14:56 |
| agordeev | devananda: morning :) | 14:57 |
| rloo | hi devananda! just the man I wanted to see ;) | 14:57 |
| NobodyCam | good morning devananda | 14:57 |
| devananda | NobodyCam: add "debug=True" to ironic.conf ? | 14:57 |
| agordeev | jroll: morning | 14:57 |
| lucasagomes | devananda, morning | 14:57 |
| jroll | morning devananda agordeev lucasagomes NobodyCam | 14:57 |
| rloo | devananda: what was the story wrt seamicro and unit testing. we aren't unit testing? | 14:57 |
| *** zdiN0bot has quit IRC | 14:58 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Accept new parameters for `prepare_image` https://review.openstack.org/86723 | 14:58 |
| NobodyCam | devananda: yes the issue is getting OoO to that in master so we can get debug log out of the gate. I can try again | 14:58 |
| lucasagomes | rloo, we are? we skip the tests if the python-seamicroclient is not installed tho | 14:58 |
| rloo | lucasagomes: but how does python-seamicroclient get installed if it isn't in requirements? | 14:58 |
| lucasagomes | rloo, by hand | 14:58 |
| lucasagomes | pip install python-seamicroclient | 14:59 |
| lucasagomes | (maybe we should add it to the test-requirements idk) | 14:59 |
| devananda | rloo: ah. thanks for reminding me. iirc, python-seamicroclient shouldn't have been added to requirements (and was removed) so we're not running the unit tests in the gate | 14:59 |
| agordeev | jroll: could you take a look at https://etherpad.openstack.org/p/ipa-discovery-ext and provide a feed back? | 14:59 |
| rloo | devananda: lucasagomes: ok, so we can add by hand, but those unit tests won't run in the gate. and that's fine? | 15:00 |
| devananda | NobodyCam: ah, in the gate, right. IMO, yea, tripleo gate checks should have debug logging on | 15:00 |
| lucasagomes | rloo, I see... hmm not really IMO we should run those in gate | 15:00 |
| devananda | rloo: lucasagomes: when ironic becomes integrated, our gate == everyone's gate | 15:00 |
| jroll | agordeev: yes, I can't right this moment but will later today | 15:01 |
| agordeev | jroll: also the reply about swift-temp-url. I think the ability of getting image from unauthentificated http URL will be great | 15:01 |
| agordeev | jroll: no problem, we're not in a hurry :) | 15:01 |
| rloo | lucasagomes: devananda: so is that something we/I should note in the developer something. if we make a change to 3rd party drivers, we need to manually test ourselves? | 15:02 |
| jroll | agordeev: +1 - that exists today in the agent as it just gets a URL and downloads it. as far as ironic, how do you see that working with nova? | 15:02 |
| devananda | rloo: lucasagomes: should a third-party driver be part of the global openstack gate? we can include the unit tests there, but we definitely can't include functional tests there. | 15:02 |
| lucasagomes | devananda, hmm right. I kinda feel that we should run the seamicro unittests in the gate | 15:02 |
| rloo | how do other projects (nova, neutron) do it? | 15:03 |
| devananda | and i would actually argue that, even though we /could/ run the unit tests, we shouldn't, because we dont have any control on the python-seamicroclient library | 15:03 |
| lucasagomes | devananda, otherwise we going to change things in the drivers interface and won't detect for those third-party drivers | 15:03 |
| lucasagomes | and won't detect any failure | 15:04 |
| devananda | hm | 15:04 |
| rloo | devananda: eg, didn't you find a bug with some function that was added to seamicro driver, I think the 'task' or something was missing from the arg list? is it OK for those bugs to be there until someone (eg from seamicro) finds/fixes it? | 15:04 |
| devananda | can we run the unit tests w/o depending on the external lib? | 15:04 |
| lucasagomes | devananda, we probably could create a FakeSeaMicro client thing | 15:04 |
| devananda | lemme have some coffee before this gets much deeper :) | 15:05 |
| devananda | i'm not reay awake yet | 15:05 |
| lucasagomes | ack | 15:05 |
| rloo | no worries. later is OK. | 15:05 |
| lucasagomes | sth like https://github.com/openstack/ironic/blob/master/ironic/nova/tests/virt/ironic/test_driver.py#L50-L92 | 15:07 |
| rloo | lucasagomes: I suppose people providing a 3rd party driver could also provide Fake* stubs for their library calls? [I certainly don't want to write that.] | 15:11 |
| NobodyCam | bbt..brb | 15:12 |
| lucasagomes | rloo, yeah, I suppose they could architect the unittests in a way that won't depend on having the lib installed | 15:14 |
| lucasagomes | idk if it's a good or bad thing, because if the lib changes the tests will still pass | 15:14 |
| *** coolsvap|afk is now known as coolsvap | 15:15 | |
| agordeev | jroll: to tell the truth, we're not going to use nova for our case. But in general, idk the pleasant way how to pass the credentials to the agent for letting him work with other openstack components seamlessly | 15:16 |
| jroll | lucasagomes, rloo, why not just have the seamicro client in test-requirements.txt | 15:17 |
| agordeev | jroll: also, i'm done for today. have a nice day, see you later! | 15:18 |
| jroll | agordeev: hmm. even, how do you see that working with ironic? | 15:18 |
| jroll | agordeev: ah, ok, we can talk later. have a good night :) | 15:18 |
| lucasagomes | jroll, yeah, mentioned it but... <devananda> and i would actually argue that, even though we /could/ run the unit tests, we shouldn't, because we dont have any control on the python-seamicroclient library | 15:19 |
| * lucasagomes dunno much about what should[n't] be in gate | 15:20 | |
| rloo | lucasagomes: but where do you/we draw the line wrt 'control' over other libraries? | 15:20 |
| lucasagomes | rloo, dunno | 15:22 |
| lucasagomes | I think it's more about it's related to the third-party stuff or not | 15:22 |
| lucasagomes | also in order to have the seamicro client in the test-requirements we first need to have it in the global requirements of openstack | 15:23 |
| lucasagomes | so idk whether it will/should be accepted there | 15:23 |
| rloo | lucasagomes: oh, I didn't realize that. So all the projects' requirements are subsets of the global requirements? | 15:24 |
| lucasagomes | rloo, yeah | 15:24 |
| rloo | might as well just use global requirements then ;) | 15:24 |
| rloo | ^^ just kidding. | 15:25 |
| lucasagomes | heh then if u want to install only ironic u would have to install the dependencies of _all_ other projects :P | 15:25 |
| rloo | yeah, that's the problem ;) | 15:25 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Fix expected status code in Ironic lookup https://review.openstack.org/90118 | 15:26 |
| jroll | lucasagomes: oh, right. global requirements. | 15:27 |
| *** flwang__ has joined #openstack-ironic | 15:27 | |
| * jroll :( | 15:27 | |
| *** yuriyz has quit IRC | 15:30 | |
| flwang__ | devananda: around? | 15:30 |
| devananda | flwang__: sorta. still on myu first cup of coffee :) | 15:31 |
| devananda | re teh requirements discussion ^^^, yes - ya'll are on the right track. global requirements. | 15:31 |
| *** dshulyak has quit IRC | 15:31 | |
| stephenpearson | Going to have to ask you lot again .. Trying to get a node booted using fake_pxe, but no matter what I do n-sch keeps saying "Filter RetryFilter returned 0 hosts". The Request Spec in that log file doesn't make too much sense to me, but I'm pretty sure that the ironic node props match up with the nova flavor. | 15:32 |
| flwang__ | devananda: would you like discuss a bit the signed images? | 15:32 |
| flwang__ | devananda: I'm from Glance team :) | 15:32 |
| lucasagomes | stephenpearson, fake_pxe? | 15:33 |
| *** dshulyak has joined #openstack-ironic | 15:33 | |
| stephenpearson | lucasgomes: The ironic driver, yes. Assume that's correct. | 15:34 |
| lucasagomes | stephenpearson, right, when you do a ironic node-list, the node does have a valid power state? | 15:35 |
| lucasagomes | "power off", "power on" ? | 15:35 |
| devananda | flwang__: signed images ... yes. eventually, we'd like to be able to verify the images that ironic writes to the target node immediately prior to booting it | 15:35 |
| lucasagomes | stephenpearson, the fake_pxe doesn't do much, it won't start any vm or anything... we use ssh_pxe to do that | 15:35 |
| lucasagomes | so the ssh driver is our driver to mock other power drivers | 15:36 |
| devananda | stephenpearson: "fake" in this case means there's no power driver. if you are testing with IPMI, you want the pxe_ipmitool driver. if you're testing with VMs, you want pxe_ssh driver | 15:36 |
| devananda | stephenpearson: but that is tangential to the scheduler problem | 15:36 |
| flwang__ | devananda: so any comments about how to store the key? | 15:36 |
| stephenpearson | lucasgomes: I was thinking I could do the power bit by hand. As expected it's showing 'power state = None'. | 15:36 |
| devananda | flwang__: not yet. i haven' tlooked into the glance aspects of this at all | 15:36 |
| flwang__ | devananda: do you think barbican is a good place? | 15:36 |
| lucasagomes | yeah pxe_ssh* not ssh_pxe | 15:37 |
| lucasagomes | stephenpearson, right so it power_state == None the driver won't detect it as an available node for nova | 15:37 |
| flwang__ | devananda: so from the Ironinc POV, it just want to verify if the image is signed by a secure source/org before booting, is it? | 15:37 |
| stephenpearson | I actually don't have IPMI (using a vmware node) and I don't care about power control just yet. | 15:37 |
| devananda | flwang__: perhaps? i think i see the logic -- the resource is in glance, but the verification happens in ironic | 15:37 |
| devananda | flwang__: that's half of it | 15:38 |
| lucasagomes | https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/driver.py#L290 | 15:38 |
| devananda | flwang__: ironic writes the image to a node's /dev/sd0 | 15:38 |
| *** foexle has quit IRC | 15:38 | |
| lucasagomes | stephenpearson, right, you can use the ssh driver to power on/off vmware machines (I never tested tho, I always use kvm) | 15:38 |
| devananda | flwang__: depending on the driver used, it may have been transferred over the wire in different ways | 15:38 |
| lucasagomes | stephenpearson, but one way, try to set a valid boot device for that fake driver and see if the sch will now detect it | 15:39 |
| devananda | flwang__: i think a final verification step will eventually be needed -- ensure the signature of the bytes *written* to the target node | 15:39 |
| lucasagomes | stephenpearson, ironic node-set-power-state $NODE on | 15:39 |
| lucasagomes | stephenpearson, ironic node-set-power-state $NODE off | 15:39 |
| devananda | flwang__: match the signature of the image in glance (ie, ensure it wasn't tampered with in transit) | 15:39 |
| flwang__ | devananda: so you mean checksum is not enough to make sure if the image is tampered or not, right? | 15:41 |
| stephenpearson | lucasgomes: no, setting power state manually didn't help. Still get a scheduling error immediately. | 15:41 |
| devananda | flwang__: are you familiar with UEFI and secure boot mechanisms? | 15:41 |
| lucasagomes | stephenpearson, right it might take a time to propagate it to nova | 15:41 |
| lucasagomes | stephenpearson, check n-cpu for | 15:41 |
| lucasagomes | 2014-04-24 16:40:02.439 AUDIT nova.compute.resource_tracker [-] Free ram (MB): 512 | 15:41 |
| lucasagomes | 2014-04-24 16:40:02.439 AUDIT nova.compute.resource_tracker [-] Free disk (GB): 10 | 15:41 |
| lucasagomes | 2014-04-24 16:40:02.439 AUDIT nova.compute.resource_tracker [-] Free VCPUS: 1 | 15:41 |
| lucasagomes | something similar to that ^ | 15:41 |
| stephenpearson | ok will do | 15:41 |
| stephenpearson | thx | 15:41 |
| lucasagomes | (or just start/stop n-cpu) | 15:42 |
| lucasagomes | to speed up things | 15:42 |
| flwang__ | devananda: not really, any link we can learn? | 15:42 |
| flwang__ | s/we/I | 15:42 |
| devananda | flwang__: looking | 15:42 |
| *** hemna_ has joined #openstack-ironic | 15:42 | |
| NobodyCam | stephenpearson: can you paste a nova flavor list / ironic node-list | 15:42 |
| stephenpearson | Oh damn, now n-cpu won't start. :) | 15:43 |
| NobodyCam | stephenpearson: error? | 15:43 |
| stephenpearson | 2014-04-24 16:42:39.733 ERROR nova.openstack.common.threadgroup [-] 795b24c3-177c-4a4b-b60b-8e4fddc2789e is not a valid node managed by this compute host. | 15:43 |
| NobodyCam | node stuck in a deploy state? | 15:44 |
| lucasagomes | stephenpearson, is this node in maintenance mode or something? | 15:45 |
| stephenpearson | Not sure. Gimme minute. | 15:45 |
| lucasagomes | ack | 15:46 |
| devananda | flwang__: https://www.linuxfoundation.org/sites/main/files/lf_uefi_secure_boot_open_platforms.pdf | 15:46 |
| devananda | flwang__: https://wiki.ubuntu.com/SecurityTeam/SecureBoot | 15:46 |
| devananda | flwang__: these are not very easy to digest... i dont have a simple reference handy, sorry | 15:46 |
| flwang__ | devananda: cool, thanks a lot | 15:46 |
| flwang__ | devananda: so, can we start this from a small footprint? :) | 15:47 |
| stephenpearson | lucasgomes, NobodyCam: Curious - as soon as I set the power state to off in ironic then n-cpu starts up again. | 15:47 |
| flwang__ | devananda: actually, in my thought, signed images has two typical scenarios, | 15:48 |
| NobodyCam | stephenpearson: :) | 15:48 |
| lucasagomes | stephenpearson, right I know the reason, there's a patch up fixing it | 15:48 |
| flwang__ | devananda: 1. glance verify if the images from a trusted source | 15:48 |
| lucasagomes | stephenpearson, we have two functions in the driver get_available_nodes and node_is_valid (or something like that) they should do the same checks, but right now it's not | 15:48 |
| lucasagomes | stephenpearson, there's a patch upstream fixing it | 15:49 |
| devananda | flwang__: here is my attempt at a naive summary: a PK needs to be loaded into the firmware of the machines ahead of time, then when a signed image is booted on the hardware, the TPM verifies both the bootloader and the kernel prior to starting the OS | 15:49 |
| lucasagomes | stephenpearson, https://review.openstack.org/#/c/88476/3/ironic/nova/virt/ironic/driver.py | 15:49 |
| flwang__ | devananda: 2. glance clients/consumers consume the images to verify if it's secure/not tampered | 15:49 |
| lucasagomes | stephenpearson, so, right now in trunk, the node_is_available is checking for power_off, that's why it's failing if it's power_on | 15:49 |
| lucasagomes | it's a known problem and is already being fixed | 15:50 |
| flwang__ | devananda: so ironic user cases is in my #2 scenarios and it's a typical one | 15:50 |
| lucasagomes | stephenpearson, so, can u see that AUDIT log? | 15:50 |
| stephenpearson | lucasgomes: ah, I see. Interesting that something different seems to be happening after setting power=off rather than None. | 15:50 |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic: Fix workaround for the "device is busy" problem https://review.openstack.org/90126 | 15:50 |
| flwang__ | devananda: yep, from the glance POV, we need to figure out how to store the PK(public key) | 15:51 |
| lucasagomes | stephenpearson, yeah, so the driver have to check if the node has a valid state or is not in maintenance before exposing it to nova (sanity checks) | 15:51 |
| lucasagomes | stephenpearson, also as ur using a fake driver, it doesn't power on or off ur vm, you will need to do it by hand (and set the power state to on in ironic as well) | 15:52 |
| flwang__ | devananda: and glance is most like a middle man at here if the PK is stored in the 3rd party, such as Barbican | 15:52 |
| devananda | stephenpearson: which is going to be complicated -- there are multiple power on/off steps during the boot process | 15:52 |
| lucasagomes | heh yeah | 15:52 |
| devananda | stephenpearson: and i'm not sure that the fake_pxe driver *can* complete a deployment | 15:52 |
| flwang__ | devananda: or maybe we can store PK with in glance db/respository | 15:52 |
| devananda | stephenpearson: it's not intended to be used that way | 15:52 |
| lucasagomes | you would be better using our ssh driver to mock the power driver instead of fake | 15:52 |
| lucasagomes | fake is more for unittests | 15:52 |
| devananda | right | 15:52 |
| stephenpearson | devananda, lucasgomes: Ok thanks. So I should be using pxe_ssh or pxe_ipmiutil or something like that? | 15:53 |
| *** zdiN0bot has joined #openstack-ironic | 15:53 | |
| stephenpearson | If I want to build an 'external' node rather than run a unit test I mean. | 15:54 |
| devananda | stephenpearson: pxe_ssh | 15:54 |
| devananda | stephenpearson: and set the node.driver_info['ssh_virt_type'] = 'vmware' | 15:54 |
| stephenpearson | devananda: Ok, got it. But isn't pxe_ssh for kvm/virsh only? | 15:54 |
| stephenpearson | Oh right | 15:54 |
| devananda | no | 15:54 |
| stephenpearson | Yeah, read your second comment :) | 15:54 |
| devananda | it uses SSH to issue commands to a remote host -- that's it. there are hooks in there for vbox/virsh/vmware | 15:55 |
| lucasagomes | stephenpearson, theorically it does support vbox and vmware as well | 15:55 |
| lucasagomes | but I never tested | 15:55 |
| lucasagomes | vbox gives me a kernel panic (in my host machine) and vmware I've no license :) | 15:55 |
| stephenpearson | vmware fusion, just to make things more interesting :) | 15:55 |
| devananda | NobodyCam added the vbox code and arata notsu added the vmware | 15:55 |
| lucasagomes | devananda, btw, ^ I added set_boot_device to the ssh driver | 15:56 |
| lucasagomes | devananda, but only for virsh, since I can't test vbox or vmware | 15:56 |
| devananda | lucasagomes: \o/ | 15:56 |
| devananda | awesome | 15:56 |
| lucasagomes | devananda, it works, heh doing the api now | 15:56 |
| lucasagomes | https://review.openstack.org/#/c/89884/ | 15:56 |
| *** zdiN0bot has quit IRC | 15:58 | |
| *** lsmola has quit IRC | 16:00 | |
| *** krtaylor has quit IRC | 16:08 | |
| *** eghobo has joined #openstack-ironic | 16:11 | |
| *** matty_dubs is now known as matty_dubs|lunch | 16:13 | |
| *** ndipanov has quit IRC | 16:14 | |
| *** viktors is now known as viktors|afk | 16:14 | |
| *** athomas has quit IRC | 16:19 | |
| *** eguz has joined #openstack-ironic | 16:23 | |
| *** newell has joined #openstack-ironic | 16:24 | |
| *** eghobo has quit IRC | 16:27 | |
| *** athomas has joined #openstack-ironic | 16:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/89234 | 16:27 |
| devananda | lucasagomes: fantastic. now we need programatic control of whether or not to create a boot loader | 16:29 |
| lucasagomes | devananda, :) so this thing about the boot loader is tricky | 16:30 |
| lucasagomes | devananda, because it's like the conductor installing the bootloader onto the iscsi disk right? | 16:30 |
| lucasagomes | devananda, it can lead to a lot of problems, if we use grub for e.g, grub-install will look at the host and install the bootloader according t it | 16:31 |
| devananda | a) pxe driver installs it over iscsi b) pxe driver copies a whole-disk-image that already has one c) agent writes one locally | 16:31 |
| devananda | lucasagomes: right | 16:31 |
| lucasagomes | devananda, b) and c) seems fine | 16:31 |
| lucasagomes | a) seems problematic | 16:31 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/89244 | 16:34 |
| lucasagomes | devananda, as an e.g https://review.openstack.org/#/c/36861/ | 16:35 |
| NobodyCam | humm ok I seem to not be able to get dib to build with patches.... sigh ..... | 16:37 |
| lucasagomes | devananda, but anyway, can be done... if we avoid grub and use syslinux for e.g it would be less problematic | 16:38 |
| *** jistr has quit IRC | 16:40 | |
| devananda | rloo: lucasagomes: chatted with clark just now. tldr... | 16:51 |
| devananda | third-party drivers that can't be functionally tested in the gate should not have their external dependencies added to requirements, and unit tests just need to mock them | 16:51 |
| *** harlowja_away is now known as harlowja | 16:51 | |
| lucasagomes | devananda, ah thanks... We need a bug then to mock the seamicro client in our unittests | 16:52 |
| lucasagomes | and not skip it | 16:52 |
| devananda | yep | 16:52 |
| * devananda files it | 16:52 | |
| lucasagomes | thaks | 16:52 |
| lucasagomes | thanks* | 16:52 |
| rloo | devananda, so there shouldn't be any unit tests for those drivers that use other libraries? like eg tests/drivers/test_seamicro.py? | 16:52 |
| devananda | rloo: there should be unit tests | 16:53 |
| devananda | rloo: but they need to mock the external lib so they can run without it | 16:53 |
| devananda | rloo: the unit test should be testing the *driver* not the external lib | 16:53 |
| devananda | test_ipmitool doesn't actually run ipmitool :) | 16:53 |
| rloo | devananda: ok, so the owners of the drivers need to also do the mocking of any external libraries. | 16:53 |
| devananda | yep | 16:54 |
| devananda | this wasn't clear to me when we landed that driver | 16:54 |
| rloo | devananda. I just don't want to be reviewing stuff that I don't really care about ;) | 16:54 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Fix expected status code in Ironic lookup https://review.openstack.org/90118 | 16:54 |
| *** zdiN0bot has joined #openstack-ironic | 16:54 | |
| *** LarsN has joined #openstack-ironic | 16:54 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 16:55 | |
| *** zdiN0bot has quit IRC | 16:59 | |
| NobodyCam | brb | 16:59 |
| *** matty_dubs|lunch is now known as matty_dubs | 16:59 | |
| devananda | rloo: filed https://bugs.launchpad.net/ironic/+bug/1312321 to track this | 17:06 |
| *** martyntaylor has quit IRC | 17:07 | |
| rloo | thx devananda. is it a 'must' that this is provided? We should make it clear if it is. | 17:07 |
| *** coolsvap is now known as coolsvap|afk | 17:08 | |
| devananda | rloo: not sure what you mean by "must" | 17:08 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Expose {set,get}_boot_device in the API https://review.openstack.org/90151 | 17:09 |
| rloo | devananda: I actually had a unit test for FakeSeaMicroDriver and PXEAndSeaMicroDriver, will think about if/where to put it if anywhere. | 17:09 |
| rloo | devananda: there are some requirements for 3rd party drivers to be in ironic, right? | 17:09 |
| rloo | devananda: requirements, in order for them to be in ironic. One of them is/should be to provide unit tests with mocks for libraries, right? | 17:10 |
| devananda | rloo: https://wiki.openstack.org/wiki/Ironic/Testing | 17:11 |
| devananda | feel free to edit :) | 17:11 |
| lucasagomes | right, I'm done for today, gotta get the train to go back home | 17:12 |
| lucasagomes | have a good night everyone, rloo devananda NobodyCam | 17:12 |
| rloo | bye lucasagomes! | 17:12 |
| NobodyCam | night lucasagomes | 17:12 |
| lucasagomes | if you guys have some time please take a look at the reviews of the management interface blueprint, the linear dependency of the patches is getting long and being hard to maintain (need another +2 https://review.openstack.org/#/c/86063)/ | 17:13 |
| rloo | thx devananda. I'll update the Testing later today. | 17:13 |
| *** Mikhail_D_ltp has quit IRC | 17:15 | |
| *** derekh has quit IRC | 17:15 | |
| *** lucasagomes has quit IRC | 17:15 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 17:19 | |
| *** zdin0bot has joined #openstack-ironic | 17:21 | |
| *** vkozhukalov has quit IRC | 17:27 | |
| *** romcheg1 has quit IRC | 17:29 | |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic: Adds support for deploying Disk Images https://review.openstack.org/73054 | 17:32 |
| devananda | hemna_: hi! cinder question for you | 17:34 |
| *** zdin0bot has quit IRC | 17:34 | |
| hemna_ | yes | 17:35 |
| devananda | hemna_: http://git.openstack.org/cgit/openstack/requirements/commit/?id=669056d88937a8c629020ee1b89602843cb6f0c4 | 17:35 |
| devananda | hemna_: compared to https://bugs.launchpad.net/ironic/+bug/1312321 | 17:35 |
| *** eguz has quit IRC | 17:35 | |
| devananda | hemna_: context is the long conversation i jsut had with clarkb in -infra re: third party libs in global requirements | 17:35 |
| hemna_ | heh | 17:36 |
| hemna_ | so... | 17:36 |
| devananda | hemna_: is the hp3par lib actually tested in the gate? -- my guess is "no" | 17:36 |
| *** eghobo has joined #openstack-ironic | 17:36 | |
| hemna_ | the lib isn't tested in the gate | 17:36 |
| hemna_ | but the unit tests for the 3par drivers in cinder need the lib due to checking for raised exceptions coming from the 3parlib. | 17:36 |
| hemna_ | it's been a sore point for us for a while | 17:37 |
| hemna_ | I have a TODO for Juno to remove the client lib from the unit tests. Still haven't figured out how to do it correctly yet though. | 17:37 |
| hemna_ | and maintain the tests for the exceptions being raised. | 17:37 |
| devananda | ah | 17:37 |
| devananda | so we're in a similar spot with the pyuthon-seamicro lib | 17:38 |
| devananda | at the end of icehouse, i added a skip to those unit tests | 17:38 |
| devananda | since it's not a requirement for ironic | 17:38 |
| hemna_ | I'm not sure what the right solution is to be honest. | 17:38 |
| devananda | and clark rightly pointed out that a unit test for driver code shouldn't depend on specific functionality in an _external_ library | 17:38 |
| devananda | when it's supposed to test the driver -- not the library | 17:39 |
| hemna_ | unless we create a middle man object that takes the library exceptions and wraps them in another that can be mocked without the need to import the client lib. | 17:39 |
| hemna_ | it shouldn't | 17:39 |
| devananda | create a mock that raises those exceptions within the unit test | 17:39 |
| hemna_ | but when you want to test the driver for correctly detecting exceptions and handling those raised by the library..... | 17:39 |
| devananda | basically, emlate the expected behavior without actually importing the library | 17:40 |
| hemna_ | can you mock a library's exceptions w/o importing the library ? | 17:40 |
| *** romcheg1 has joined #openstack-ironic | 17:40 | |
| *** epim has joined #openstack-ironic | 17:40 | |
| hemna_ | catch 22 no ? | 17:40 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Drivers determine acceptable power states https://review.openstack.org/86744 | 17:42 |
| *** kmartin has joined #openstack-ironic | 17:44 | |
| *** krtaylor has joined #openstack-ironic | 17:45 | |
| hemna_ | devananda, paste.openstack.org/show/76983/ | 17:48 |
| hemna_ | for example, one of the tests we have in our driver unit tests looking for an exception that is raised by the client lib. | 17:48 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Drivers may expose a top-level passthru API https://review.openstack.org/81919 | 17:52 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 17:54 |
| *** lucas-afk has joined #openstack-ironic | 17:55 | |
| *** zdin0bot has joined #openstack-ironic | 17:55 | |
| *** mdenny has joined #openstack-ironic | 18:03 | |
| *** tatyana has joined #openstack-ironic | 18:04 | |
| *** ifarkas has quit IRC | 18:05 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Revert "Fix expected status code in Ironic lookup" https://review.openstack.org/90170 | 18:05 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Fix expected status code in Ironic heartbeat https://review.openstack.org/90171 | 18:05 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Drivers determine acceptable power states https://review.openstack.org/86744 | 18:09 |
| devananda | hemna_: ok. i meant for rohan to fix this, but since it wasn't clear how, and you had the same problem, here :) | 18:09 |
| hemna_ | heh | 18:10 |
| *** stephenpearson has quit IRC | 18:10 | |
| hemna_ | only thing I can think of is to mirror the exceptions module from the lib and pull it into the unit test itself. | 18:11 |
| hemna_ | icky, but I'm not sure what else to do | 18:11 |
| *** athomas has quit IRC | 18:11 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Mock seamicro module in unit tests if not present https://review.openstack.org/90174 | 18:11 |
| devananda | hemna_: ^ | 18:11 |
| hemna_ | ok thanks for the magic. I'll see if I can do that as well. | 18:13 |
| hemna_ | :) | 18:13 |
| devananda | welcome! | 18:13 |
| davidlenwell | hi devananda.. hi NobodyCam | 18:19 |
| davidlenwell | did you guys see my comment from the wee hours of the morning | 18:20 |
| davidlenwell | ? | 18:20 |
| JoshNang | devananda: so i'm working on factoring the pxe/tftp stuff that would be common to all drivers out of the pxe driver. however, that's going to move a lot of config options out of the pxe group into a different group, which is going to be a backwards incompatible change. not sure where to go with this. | 18:20 |
| devananda | davidlenwell: nope | 18:21 |
| davidlenwell | devananda: I was walking througbh the dev quickstart on my cluster at home | 18:21 |
| devananda | JoshNang: there is a way to signal options as having a deprecated location | 18:21 |
| davidlenwell | tried to add nodes .. got some funny failures | 18:21 |
| JoshNang | devananda: sweet. that makes sense. thanks! | 18:22 |
| davidlenwell | are there known issues with supermicro? | 18:22 |
| devananda | JoshNang: see eg. nova/virt/libvirt/driver.py | 18:22 |
| devananda | JoshNang: or look in the oslo.config docs... it should be there :) | 18:22 |
| devananda | davidlenwell: is your cluster a supermicro cluster? | 18:23 |
| davidlenwell | yes | 18:23 |
| devananda | ahh | 18:23 |
| devananda | well | 18:23 |
| JoshNang | devananda: you rock. thanks! | 18:23 |
| devananda | it's not tested in teh gate. so i dont know if it works or not | 18:23 |
| devananda | davidlenwell: please file bugs! | 18:23 |
| davidlenwell | devananda: okay .. well yes .. that | 18:23 |
| devananda | and fix them ;) | 18:23 |
| davidlenwell | I'll be testing it today | 18:23 |
| davidlenwell | again | 18:23 |
| davidlenwell | durring normal hours | 18:23 |
| davidlenwell | so I can rule out user error before filing a bug | 18:24 |
| devananda | cool | 18:24 |
| *** epim has quit IRC | 18:24 | |
| devananda | davidlenwell: fwiw, some of the unit tests weren't running in the gate -- i just filed https://review.openstack.org/#/c/90174/1 to get them running again | 18:25 |
| davidlenwell | my cluster is supermicro but it does have ipmi 2 supposedly .. piston cloud installs on it without issue | 18:25 |
| davidlenwell | and we don't support anything but impi | 18:25 |
| davidlenwell | ipmi | 18:25 |
| devananda | davidlenwell: rohan kanade is the lead from seamicro // author of that driver. he's not on irc right now (nic: k4n0) but would probably be delighed to hear that you're using seamicro | 18:26 |
| devananda | davidlenwell: wait. supermicro or seamicro? | 18:26 |
| devananda | oh sorry | 18:26 |
| davidlenwell | supermicro | 18:26 |
| *** zdin0bot has quit IRC | 18:26 | |
| devananda | right | 18:26 |
| devananda | i misread - was just fixing seamicro so that's what i thought you said | 18:26 |
| davidlenwell | supermicro ipmi 2 | 18:26 |
| devananda | davidlenwell: you're just using ipmi then | 18:26 |
| devananda | yea. no knkown issues. that should work | 18:26 |
| davidlenwell | yep | 18:26 |
| *** epim has joined #openstack-ironic | 18:27 | |
| davidlenwell | I'll spend the time today and insure I'm not just having a networking problem before I file any bugs | 18:27 |
| NobodyCam | morning davidlenwell :) | 18:27 |
| davidlenwell | hi NobodyCam | 18:28 |
| davidlenwell | are you on your way to atlanta? | 18:28 |
| * devananda needs to find food before the next conference | 18:28 | |
| NobodyCam | yep in texas atm | 18:28 |
| davidlenwell | I was just in palm springs | 18:28 |
| NobodyCam | lol :) | 18:28 |
| davidlenwell | but figured you already left so i' didn't try to call ya | 18:28 |
| davidlenwell | went to coachella | 18:28 |
| NobodyCam | to hot there now | 18:28 |
| davidlenwell | stayed in palm desert | 18:28 |
| NobodyCam | nice | 18:28 |
| davidlenwell | yeah it was 99 degrees in the day time | 18:28 |
| *** blamar_ has joined #openstack-ironic | 18:29 | |
| *** blamar_ is now known as blamar | 18:29 | |
| *** derekh has joined #openstack-ironic | 18:31 | |
| *** igor__ has joined #openstack-ironic | 18:35 | |
| *** martyntaylor has joined #openstack-ironic | 18:47 | |
| *** martyntaylor has left #openstack-ironic | 18:51 | |
| *** zdiN0bot has joined #openstack-ironic | 18:56 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Implement API to get driver properties https://review.openstack.org/73005 | 18:57 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Decouple state inspection and availability check https://review.openstack.org/88476 | 18:58 |
| *** zdiN0bot has quit IRC | 19:00 | |
| *** BadCub has joined #openstack-ironic | 19:02 | |
| openstackgerrit | linggao proposed a change to openstack/ironic: Support serial console access https://review.openstack.org/64100 | 19:02 |
| *** epim has quit IRC | 19:03 | |
| *** vkozhukalov has joined #openstack-ironic | 19:04 | |
| linggao | Hi matty_dubs | 19:04 |
| *** blamar has quit IRC | 19:12 | |
| *** notq has joined #openstack-ironic | 19:13 | |
| *** rloo has quit IRC | 19:23 | |
| *** rloo has joined #openstack-ironic | 19:23 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Uniquely identify builds to prevent race https://review.openstack.org/88383 | 19:29 |
| *** mdenny has quit IRC | 19:29 | |
| *** mdenny has joined #openstack-ironic | 19:30 | |
| *** KurtMartin has joined #openstack-ironic | 19:30 | |
| *** kmartin has quit IRC | 19:33 | |
| *** Mikhail_D_ltp has left #openstack-ironic | 19:35 | |
| *** rloo has quit IRC | 19:36 | |
| *** rloo has joined #openstack-ironic | 19:36 | |
| *** rloo has quit IRC | 19:37 | |
| *** rloo has joined #openstack-ironic | 19:38 | |
| *** rloo has quit IRC | 19:42 | |
| *** rloo has joined #openstack-ironic | 19:42 | |
| openstackgerrit | linggao proposed a change to openstack/ironic: Handle API error for XML client https://review.openstack.org/88448 | 19:42 |
| *** rloo has quit IRC | 19:44 | |
| *** coolsvap|afk has quit IRC | 19:44 | |
| *** rloo has joined #openstack-ironic | 19:44 | |
| *** rloo has quit IRC | 19:52 | |
| *** rloo has joined #openstack-ironic | 19:53 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Accept new parameters for `prepare_image` https://review.openstack.org/86723 | 20:00 |
| *** rloo has quit IRC | 20:02 | |
| *** rloo has joined #openstack-ironic | 20:02 | |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic: Adds support for deploying Disk Images https://review.openstack.org/73054 | 20:05 |
| *** harlowja is now known as harlowja_away | 20:06 | |
| *** epim has joined #openstack-ironic | 20:09 | |
| *** epim has quit IRC | 20:16 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Accept new parameters for `prepare_image` https://review.openstack.org/86723 | 20:24 |
| *** epim has joined #openstack-ironic | 20:31 | |
| NobodyCam | *CORES* review jam monday 8:00am | 20:36 |
| *** lucas-afk has quit IRC | 20:37 | |
| *** derekh has quit IRC | 20:39 | |
| NobodyCam | brb... quick walkies | 20:49 |
| *** jdob has quit IRC | 20:52 | |
| *** lucas-afk has joined #openstack-ironic | 20:53 | |
| *** zdiN0bot has joined #openstack-ironic | 20:57 | |
| *** notq has quit IRC | 20:59 | |
| *** harlowja_away is now known as harlowja | 21:01 | |
| *** zdiN0bot has quit IRC | 21:02 | |
| * devananda goes semi-afk for the rest of the day to review conference proposals, blueprints, and run some errands | 21:10 | |
| *** linggao has quit IRC | 21:10 | |
| NobodyCam | :) | 21:11 |
| devananda | romcheg: ping | 21:11 |
| romcheg1 | devananda: Pong | 21:11 |
| devananda | romcheg1: hi! can you comment on http://summit.openstack.org/cfp/details/275 ? | 21:12 |
| devananda | romcheg1: and what's the status of rally integration for ironic? | 21:12 |
| devananda | romcheg1: will there be real numbers available to look at during that session? | 21:13 |
| romcheg1 | devananda: I managed to deal with some incompatibilities. | 21:13 |
| romcheg1 | Now implementing some benchmarks | 21:13 |
| romcheg1 | There are a few folks here working on benchmarking different parts | 21:14 |
| romcheg1 | We plan to show some stats during the session | 21:14 |
| devananda | romcheg1: matty_dubs is also working on some benchmarks, i believe. you should collaborate :) | 21:15 |
| romcheg1 | My results will be ready in a day or two | 21:15 |
| romcheg1 | devananda: yes, we discussed that. He works on a different part :) | 21:15 |
| devananda | romcheg1: great. please be ready to outline your status / test method / results in an etherpad by end of next week | 21:16 |
| romcheg1 | devananda: will be ready | 21:16 |
| *** jbjohnso has quit IRC | 21:16 | |
| devananda | there are 3 sessions proposed about performance -- i'm goign to merge them | 21:16 |
| romcheg1 | devananda: cool | 21:18 |
| romcheg1 | I will leave a comment on the session in the morning. Now don't have access to that | 21:18 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Starting refactor https://review.openstack.org/90233 | 21:20 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Factoring out PXE and TFTP functions https://review.openstack.org/90233 | 21:22 |
| *** epim has quit IRC | 21:26 | |
| *** epim has joined #openstack-ironic | 21:30 | |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added disk utils https://review.openstack.org/86163 | 21:34 |
| NobodyCam | hey hey romcheg1 :) | 21:34 |
| NobodyCam | you see my note? | 21:34 |
| romcheg1 | Morning NobodyCam | 21:34 |
| NobodyCam | *CORES* review jam monday 8:00am <- | 21:34 |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added list and report disk utils https://review.openstack.org/88602 | 21:34 |
| romcheg1 | Yup | 21:35 |
| NobodyCam | w00t :) | 21:36 |
| romcheg1 | Hope I won't have to escape by Monday :) | 21:38 |
| NobodyCam | ummm .. .yes ! | 21:38 |
| NobodyCam | that would be not good | 21:38 |
| romcheg1 | NobodyCam: http://goo.gl/DjEhUl | 21:41 |
| romcheg1 | NobodyCam: That's just about where I am :) | 21:41 |
| NobodyCam | oh no | 21:42 |
| NobodyCam | thats not good that the same stuff they did un the north | 21:42 |
| NobodyCam | s/un/in/ | 21:43 |
| NobodyCam | and we get news like http://www.ibtimes.com/russias-use-unmarked-troops-simferopol-crimea-shady-not-illegal-1559425 | 21:45 |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added list and report disk utils https://review.openstack.org/88602 | 21:48 |
| romcheg1 | NobodyCam: Doesn't sound like the biggest problem there :) | 21:49 |
| *** derekh has joined #openstack-ironic | 21:52 | |
| *** mdenny has quit IRC | 21:54 | |
| *** mdenny has joined #openstack-ironic | 21:55 | |
| *** KurtMartin has quit IRC | 22:00 | |
| *** KurtMartin has joined #openstack-ironic | 22:00 | |
| *** jgrimm has quit IRC | 22:02 | |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added partition extension https://review.openstack.org/90241 | 22:03 |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Sync latest code and reuse exceptions from oslo https://review.openstack.org/71500 | 22:06 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/ironic: Renaming ironicclient exceptions in nova driver https://review.openstack.org/90040 | 22:07 |
| NobodyCam | romcheg1: its not! but thats our news | 22:10 |
| devananda | jroll: russell_h: how do you guys feel if I schedule the IPA session first, and take the opening ~10 minutes for project introduction / status rep | 22:11 |
| jroll | devananda: as long as I can trust everyone to let you keep to 10 minutes :) | 22:12 |
| jroll | should be fine | 22:13 |
| jroll | hey y'all, this just needs one more +2 :) https://review.openstack.org/#/c/81919/ | 22:14 |
| devananda | jroll: great. scheduling done (for now) | 22:15 |
| devananda | http://junodesignsummit.sched.org/ | 22:16 |
| devananda | FYI folks, Ironic is now tuesday morning slots | 22:16 |
| jroll | cool | 22:18 |
| *** max_lobur1 has quit IRC | 22:19 | |
| *** lucas-afk has quit IRC | 22:25 | |
| *** epim has quit IRC | 22:30 | |
| *** epim has joined #openstack-ironic | 22:38 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Revert "Fix expected status code in Ironic lookup" https://review.openstack.org/90170 | 22:40 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Fix expected status code in Ironic heartbeat https://review.openstack.org/90171 | 22:41 |
| *** zdin0bot has joined #openstack-ironic | 22:42 | |
| NobodyCam | jroll: I really would like russell to give it the +1 befor landing | 22:43 |
| JayF | NobodyCam: fwiw; I know they've been coordinating on it in our internal IRC channel all day | 22:44 |
| JayF | russell_h: https://review.openstack.org/#/c/81919/ NobodyCam wants you to +1 that then he'll land it ;) | 22:44 |
| JayF | (not exactly what you said but that's what I'm hearing ;D) | 22:44 |
| NobodyCam | JayF: russell_h started the patch.. we all take over each others patches but I've always liked the owner to bless it | 22:45 |
| *** derekh has quit IRC | 22:45 | |
| JayF | I know, I'm sure he'll be thrilled to toss a +1 on that once so we can get it in :) | 22:45 |
| JayF | he's just operating on CST this week so not sure if he's still around today | 22:45 |
| NobodyCam | lol as am I... | 22:45 |
| NobodyCam | so with that I'm going to step afk, but will be back to check on things | 22:46 |
| jroll | NobodyCam: that's fair, thanks :) | 22:50 |
| russell_h | NobodyCam: yep, jroll's changes LGTM | 22:51 |
| russell_h | I mean, so do mine :) | 22:52 |
| jroll | \o/ | 22:52 |
| *** zdin0bot has quit IRC | 22:52 | |
| *** vkozhukalov has quit IRC | 22:55 | |
| *** mdenny has quit IRC | 22:56 | |
| *** epim has quit IRC | 22:58 | |
| *** zdiN0bot has joined #openstack-ironic | 22:59 | |
| *** dkehn_ has joined #openstack-ironic | 23:00 | |
| *** zdiN0bot has quit IRC | 23:03 | |
| *** dkehnx has quit IRC | 23:04 | |
| *** lifeless has quit IRC | 23:04 | |
| *** epim has joined #openstack-ironic | 23:05 | |
| *** lifeless has joined #openstack-ironic | 23:05 | |
| russell_h | alright, I'm off, its margarita time in TX | 23:06 |
| JayF | don't waste away again | 23:06 |
| JayF | in San Antonio (margaritaville) | 23:06 |
| JayF | :) | 23:07 |
| russell_h | lol | 23:07 |
| *** epim has quit IRC | 23:11 | |
| *** romcheg1 has quit IRC | 23:14 | |
| *** eguz has joined #openstack-ironic | 23:19 | |
| *** eghobo has quit IRC | 23:23 | |
| *** eghobo has joined #openstack-ironic | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!