| *** zackf has quit IRC | 00:00 | |
| *** ijw has joined #openstack-ironic | 00:01 | |
| rloo | zer0c00l: max i believe is 1000. unless someone fixed it, it won't go higher even if you specify it | 00:03 |
|---|---|---|
| *** rajinir has quit IRC | 00:05 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Remove unneeded disk_utils.mkfs() function https://review.openstack.org/375166 | 00:06 |
| jlvillal | zer0c00l: rloo: https://github.com/openstack/ironic/blob/96d78401098c3f2474c9bfe6d30869266f07852d/ironic/conf/api.py#L28-L31 | 00:08 |
| jlvillal | I think that is where the 1000 limit comes from. | 00:08 |
| rloo | jlvillal: that's the default. you can override it, but if i recall correctly, if you try to get more than 1000, it'll clamp it to 1000. unless it was fixed but i seem to recall someone had some reason for clamping it at 1000. (it was awhile ago) | 00:10 |
| rloo | jlvillal: might have had something to do with the max size of the BODY of the response. can't recall now | 00:10 |
| *** rbrud has quit IRC | 00:11 | |
| jlvillal | rloo: Looking at the code it looks you can override it with a config change. But I haven't tested it. | 00:14 |
| *** wajdi has joined #openstack-ironic | 00:14 | |
| rloo | jlvillal: i don't remember :) | 00:15 |
| *** rpioso has quit IRC | 00:16 | |
| * jlvillal just read that dtantsur|afk is now a core in puppet-ironic. Congrats dtantsur|afk :) | 00:17 | |
| *** tuanla has joined #openstack-ironic | 00:23 | |
| *** syed_ has quit IRC | 00:33 | |
| *** phuongnh has joined #openstack-ironic | 00:47 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Remove unneeded disk_utils.mkfs() function https://review.openstack.org/375166 | 00:50 |
| *** mtanin___ has joined #openstack-ironic | 00:52 | |
| *** moshele has joined #openstack-ironic | 00:52 | |
| *** mtanino has quit IRC | 00:53 | |
| *** ijw has quit IRC | 00:57 | |
| zer0c00l | rloo, jlvillal thanks | 00:59 |
| zer0c00l | To get all the nodes i did '--limit 0' | 00:59 |
| *** hoangcx has joined #openstack-ironic | 01:00 | |
| rloo | zer0c00l: sweet, so we did put in a way to do it... | 01:01 |
| tuanla | Hello ironic team | 01:09 |
| *** ijw has joined #openstack-ironic | 01:14 | |
| *** dprince has joined #openstack-ironic | 01:22 | |
| *** dprince has quit IRC | 01:22 | |
| openstackgerrit | Cao Xuan Hoang proposed openstack/ironic-ui: TrivialFix: Remove logging import unused https://review.openstack.org/365909 | 01:25 |
| openstackgerrit | Cao Xuan Hoang proposed openstack/ironic-ui: Clean imports in code https://review.openstack.org/365911 | 01:26 |
| *** ijw has quit IRC | 01:28 | |
| *** rloo has quit IRC | 01:32 | |
| *** Sukhdev has joined #openstack-ironic | 01:37 | |
| *** wajdi has quit IRC | 01:47 | |
| *** wajdi has joined #openstack-ironic | 01:47 | |
| *** wajdi_ has joined #openstack-ironic | 01:49 | |
| *** wajdi has quit IRC | 01:52 | |
| *** bfournie has joined #openstack-ironic | 01:55 | |
| openstackgerrit | gengchc2 proposed openstack/ironic: Fix a typo in helper.py https://review.openstack.org/375193 | 01:58 |
| *** joprovost has quit IRC | 01:59 | |
| *** mtanino has joined #openstack-ironic | 02:16 | |
| *** mtanin___ has quit IRC | 02:17 | |
| *** Sukhdev has quit IRC | 02:19 | |
| *** mtanino has quit IRC | 02:22 | |
| *** Sukhdev has joined #openstack-ironic | 02:28 | |
| *** thrash is now known as thrash|g0ne | 02:35 | |
| *** Sukhdev has quit IRC | 02:38 | |
| *** amotoki has quit IRC | 02:41 | |
| *** rama_y has quit IRC | 02:48 | |
| *** amotoki has joined #openstack-ironic | 02:53 | |
| *** rajinir has joined #openstack-ironic | 02:56 | |
| *** wajdi_ has quit IRC | 02:58 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** amotoki has quit IRC | 03:09 | |
| *** moshele has quit IRC | 03:11 | |
| *** amotoki has joined #openstack-ironic | 03:19 | |
| *** mtanino has joined #openstack-ironic | 03:26 | |
| *** rama_y has joined #openstack-ironic | 03:32 | |
| *** vikrant has joined #openstack-ironic | 03:38 | |
| *** amotoki has quit IRC | 03:39 | |
| *** mtanino has quit IRC | 03:41 | |
| *** Sukhdev has joined #openstack-ironic | 03:46 | |
| *** amotoki has joined #openstack-ironic | 03:49 | |
| *** baoli_ has joined #openstack-ironic | 03:54 | |
| *** ijw has joined #openstack-ironic | 03:56 | |
| *** baoli has quit IRC | 03:58 | |
| *** vmud213 has joined #openstack-ironic | 03:58 | |
| *** jrist has joined #openstack-ironic | 03:59 | |
| *** ijw has quit IRC | 04:01 | |
| *** sdake has quit IRC | 04:02 | |
| *** baoli has joined #openstack-ironic | 04:14 | |
| *** baoli_ has quit IRC | 04:17 | |
| *** ijw has joined #openstack-ironic | 04:17 | |
| *** ijw has quit IRC | 04:19 | |
| *** baoli has quit IRC | 04:29 | |
| *** bks has joined #openstack-ironic | 04:46 | |
| *** rajinir has quit IRC | 05:05 | |
| *** appu103_ has joined #openstack-ironic | 05:09 | |
| *** appu103_ is now known as aparnav | 05:09 | |
| *** syed_ has joined #openstack-ironic | 05:09 | |
| *** jaosorior has joined #openstack-ironic | 05:10 | |
| *** sabeen has quit IRC | 05:22 | |
| *** zackf has joined #openstack-ironic | 05:22 | |
| *** zackf has quit IRC | 05:27 | |
| *** vmud213 has quit IRC | 05:37 | |
| *** vmud213 has joined #openstack-ironic | 05:37 | |
| *** absubram has quit IRC | 05:47 | |
| *** mjura has joined #openstack-ironic | 05:52 | |
| openstackgerrit | Himanshu Kumar proposed openstack/python-ironicclient: Fixed to list all nodes which are in no-maintenance mode https://review.openstack.org/373096 | 05:56 |
| *** mjura has quit IRC | 05:59 | |
| *** mjura has joined #openstack-ironic | 06:00 | |
| openstackgerrit | Himanshu Kumar proposed openstack/python-ironicclient: Fixed to list all nodes which are in no-maintenance mode https://review.openstack.org/373096 | 06:01 |
| *** ianw is now known as ianw_pto | 06:02 | |
| *** rcernin has joined #openstack-ironic | 06:15 | |
| pas-ha | morning Ironic. TGIF :) | 06:17 |
| *** aswadr_ has joined #openstack-ironic | 06:25 | |
| *** phuongnh has quit IRC | 06:29 | |
| *** phuongnh has joined #openstack-ironic | 06:29 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses https://review.openstack.org/375045 | 06:46 |
| *** hoangcx_ has joined #openstack-ironic | 06:55 | |
| *** vsaienko has joined #openstack-ironic | 06:56 | |
| *** hoangcx has quit IRC | 06:57 | |
| *** phuongnh has quit IRC | 06:58 | |
| *** phuongnh has joined #openstack-ironic | 06:58 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses https://review.openstack.org/375045 | 07:00 |
| *** vsaienko has quit IRC | 07:00 | |
| *** vsaienko has joined #openstack-ironic | 07:01 | |
| *** Sukhdev has quit IRC | 07:11 | |
| *** vsaienko has quit IRC | 07:11 | |
| *** vsaienko has joined #openstack-ironic | 07:12 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 07:13 |
| *** syed_ has quit IRC | 07:23 | |
| *** jpich has joined #openstack-ironic | 07:37 | |
| *** hoangcx_ has quit IRC | 07:38 | |
| *** hoangcx has joined #openstack-ironic | 07:39 | |
| *** vsaienko has quit IRC | 07:42 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Remove six.moves module https://review.openstack.org/375297 | 07:47 |
| *** ohamada has joined #openstack-ironic | 07:47 | |
| *** athomas has quit IRC | 07:55 | |
| *** makowals_ has joined #openstack-ironic | 07:55 | |
| *** vsaienko has joined #openstack-ironic | 07:57 | |
| *** makowals_ has quit IRC | 07:58 | |
| *** makowals has quit IRC | 07:58 | |
| *** makowals has joined #openstack-ironic | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** amotoki_ has joined #openstack-ironic | 08:00 | |
| *** athomas has joined #openstack-ironic | 08:01 | |
| *** makowals has quit IRC | 08:01 | |
| *** amotoki has quit IRC | 08:03 | |
| *** vsaienko has quit IRC | 08:19 | |
| *** amotoki_ is now known as amotoki | 08:24 | |
| *** Romanenko_K has joined #openstack-ironic | 08:24 | |
| *** absubram has joined #openstack-ironic | 08:30 | |
| *** absubram_ has joined #openstack-ironic | 08:32 | |
| *** absubram has quit IRC | 08:34 | |
| *** absubram_ is now known as absubram | 08:34 | |
| *** absubram has quit IRC | 08:34 | |
| *** derekh has joined #openstack-ironic | 08:44 | |
| *** vnogin has left #openstack-ironic | 08:44 | |
| *** dtantsur|afk is now known as dtantsur | 08:45 | |
| dtantsur | good (so called) morning, Ironic | 08:45 |
| * dtantsur stares at the monitor in a desperate attempt to wake up | 08:45 | |
| dtantsur | thanks jlvillal :) | 08:45 |
| vdrok | morning dtantsur :) | 08:46 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib: Fix check for GPT partioned device https://review.openstack.org/375320 | 08:49 |
| dtantsur | jlvillal, we need to backport all the fixes to newton ^^^ also thanks for spotting it | 08:50 |
| dtantsur | hey vdrok | 08:50 |
| openstackgerrit | Merged openstack/ironic-lib: Add py35 to tox environments https://review.openstack.org/375000 | 08:56 |
| tuanla | morning ironic | 09:02 |
| tuanla | I have done deploy window image using ironic | 09:03 |
| *** openstackgerrit has quit IRC | 09:03 | |
| tuanla | but it have some problems relate to ipmi_tool Do you have free time d | 09:04 |
| *** openstackgerrit has joined #openstack-ironic | 09:04 | |
| tuanla | My problem because when i set pxe boot using ipmi bootdev, it not just set pxe is hightest priority but it also delete other option in boot priority. | 09:06 |
| *** pmannidi has quit IRC | 09:06 | |
| *** makowals has joined #openstack-ironic | 09:07 | |
| tuanla | hello dtantsur, | 09:07 |
| dtantsur | o/ | 09:07 |
| tuanla | hello sambetts|pto | 09:08 |
| vsaienk0 | Morning dtantsur, tuanla, lucasagomes, sambetts, milan, vdrok, pas-ha and all Ironic'ers! | 09:08 |
| dtantsur | hey vsaienk0! | 09:08 |
| tuanla | Do you guide remember my bugs "Not locate configuration file or MBR signature" | 09:08 |
| tuanla | Hello vsaienk0, | 09:09 |
| vdrok | morning vsaienk0 | 09:09 |
| dtantsur | vsaienk0, btw, will you have time to check the xenial job today? | 09:09 |
| vsaienk0 | dtantsur: the patch was merged yesterday, I've launched tests and xenial job passed | 09:09 |
| dtantsur | wow, awesome | 09:09 |
| dtantsur | vsaienk0, link handy to the test run? | 09:10 |
| vsaienk0 | dtantsur: https://review.openstack.org/#/c/369021/ gate-tempest-dsvm-ironic-pxe_ipa-postgres-ubuntu-xenial-nv | 09:10 |
| patchbot | patch 369021 - ironic - WIP: Ironic multitest initial commit | 09:10 |
| dtantsur | fantastic! thanks vsaienk0 | 09:10 |
| vsaienk0 | dtantsur: I'm going to upload patch to switch all IPXE -nv jobs to xenial today | 09:11 |
| dtantsur | vsaienk0, yes, this is the next step. thanks for tracking it. please add me as a reviewer. | 09:11 |
| vsaienk0 | dtantsur: sure | 09:12 |
| *** bcornec1 has quit IRC | 09:19 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 09:24 |
| dtantsur | mandre, hi! how is your deployment going? | 09:24 |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: Ignore Ansible retry files in .gitignore https://review.openstack.org/375350 | 09:25 |
| mandre | dtantsur: very well ironic-wise :) | 09:27 |
| dtantsur | cool, that's a relief | 09:28 |
| mandre | yeah! was this function new code? I don't recall it failing for me before | 09:30 |
| *** pester has quit IRC | 09:32 | |
| *** amotoki has quit IRC | 09:32 | |
| dtantsur | mandre, yes, we just received support for configdrive when deploying whole disk images with pxe_* drivers | 09:32 |
| dtantsur | previously a config drive won't be created | 09:32 |
| *** milan has joined #openstack-ironic | 09:33 | |
| milan | morning Ironic! :) TGIF! | 09:33 |
| dtantsur | oh yeah, TGIF | 09:33 |
| mandre | dtantsur: well... I'm glad I exercised this code :) | 09:34 |
| dtantsur | thanks for that :) now we have some gate coverage for this case too, but I'm still not sure we use config drive.. | 09:34 |
| dtantsur | vdrok or jroll (when you're here): are you sure / do you know if we are using config drive in the CI? | 09:35 |
| dtantsur | I can't find any traces of it, at least not in the whole disk job | 09:35 |
| milan | aarefiev, dtantsur FYI: the spec of the list introspections endpoint: https://review.openstack.org/#/c/375045/ | 09:35 |
| patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 09:35 |
| dtantsur | cool | 09:35 |
| milan | bike shedding time! :D | 09:35 |
| dtantsur | \o/ | 09:35 |
| *** AlexeyAbashkin has quit IRC | 09:40 | |
| dtantsur | I see FORCE_CONFIG_DRIVE=False in our localrc, so I guess I can assume we don't test it... | 09:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: DO NOT MERGE: testing config drive https://review.openstack.org/375362 | 09:49 |
| *** amotoki has joined #openstack-ironic | 09:54 | |
| *** amotoki has quit IRC | 09:55 | |
| openstackgerrit | Zhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info https://review.openstack.org/369245 | 10:01 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty https://review.openstack.org/370764 | 10:06 |
| *** thrash|g0ne is now known as thrash | 10:07 | |
| vdrok | morning milan | 10:13 |
| vdrok | dtantsur: let me check | 10:13 |
| milan | morning vdrok! :) | 10:13 |
| aarefiev | milan: nice, I'll take a look | 10:13 |
| milan | aarefiev, thanks! :) | 10:13 |
| aarefiev | thank you | 10:14 |
| *** tuanla has quit IRC | 10:15 | |
| vdrok | dtantsur: hm, yeah, in the wholedisk pxe_ssh job I see Checking for configdrive device (None) on node 29a15c7f-7b10-4740-a0d6-559c983668e0. work_on_disk | 10:19 |
| openstackgerrit | Xu Ao proposed openstack/ironic: Make ironic console supporting the nokeepalive mode https://review.openstack.org/372830 | 10:19 |
| dtantsur | vdrok, I've posted https://review.openstack.org/375362 to test forcing config drive | 10:19 |
| vdrok | it's in the log of the change when the support was added | 10:19 |
| patchbot | patch 375362 - ironic - DO NOT MERGE: testing config drive | 10:19 |
| dtantsur | morning aarefiev | 10:19 |
| aarefiev | morning dtantsur | 10:20 |
| vdrok | dtantsur: hm, I'm not sure it will work, there is no config_drive param handling in any of tempest functions called | 10:30 |
| vdrok | compute feature config drive enabled is true by default | 10:30 |
| dtantsur | I'm not sure compute feature has any relationship to it.. config_drive is passed directly to API | 10:32 |
| *** phuongnh has quit IRC | 10:33 | |
| dtantsur | I think the compute feature changes what their tempest tests are testing | 10:33 |
| dtantsur | but let's see, I'm not sure in anything :) | 10:33 |
| *** dtantsur is now known as dtantsur|bbl | 10:34 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty https://review.openstack.org/370764 | 10:35 |
| vdrok | oh, right, I see where it comes from, yeah, might work | 10:38 |
| *** dprince has joined #openstack-ironic | 10:58 | |
| *** mgould|afk is now known as mgould | 10:59 | |
| mgould | morning Ironic! | 10:59 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 11:00 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 11:03 |
| *** lucas-afk is now known as lucasagomes | 11:05 | |
| lucasagomes | morning all | 11:05 |
| tangchen | morning guys. :) | 11:05 |
| mgould | morning lucasagomes tangchen vdrok aarefiev milan | 11:07 |
| vsaienk0 | I'm wondered if sambetts is around? | 11:08 |
| tangchen | morning mgould | 11:08 |
| aarefiev | vsaienk0: he is on pto till monday | 11:09 |
| aarefiev | morning mgould tangchen | 11:10 |
| tangchen | morning aarefiev | 11:11 |
| *** ccamacho|afk is now known as ccamacho | 11:11 | |
| lucasagomes | hi there :D | 11:13 |
| *** dtantsur|bbl is now known as dtantsur | 11:20 | |
| dtantsur | morning lucasagomes, mgould | 11:20 |
| lucasagomes | morning | 11:20 |
| dtantsur | vdrok, jroll, https://review.openstack.org/#/c/375362/ shows expected breakage with old ironic-lib now, so yeah, we are not testing config drive in CI yet | 11:20 |
| patchbot | patch 375362 - ironic - DO NOT MERGE: testing config drive | 11:20 |
| vdrok | dtantsur: ++ just seen that | 11:21 |
| dtantsur | now I wonder if we need to just merge this patch, or do we need a new set of gates (omg!) to test config drive? | 11:21 |
| vdrok | Morning lucasagomes and mgould ! | 11:21 |
| vdrok | If rather merge this one | 11:21 |
| vdrok | *I'd | 11:22 |
| *** aswadr_ has quit IRC | 11:22 | |
| dtantsur | then we'll stop testing metadata service... I wonder if it's important | 11:22 |
| *** hoangcx has quit IRC | 11:23 | |
| dtantsur | let's wait for jroll to chat about it | 11:23 |
| dtantsur | also lucasagomes, wdyt about switching our CI to testing config drive? | 11:23 |
| * dtantsur fixes commit message for the patch | 11:24 | |
| lucasagomes | dtantsur, I'm bit out of context... if we use pxe_ssh + whole disk image it won't use the config drive? | 11:24 |
| vdrok | Metadata should be tested in other places right? And config drive is the thing that affects deployment directly... | 11:25 |
| lucasagomes | (I know it's not failing due that ironic-lib error shown by 375362) | 11:25 |
| dtantsur | lucasagomes, our CI does not cover config drive, that's the problem | 11:25 |
| dtantsur | for no combinations of drivers and images iiuc | 11:26 |
| dtantsur | vdrok, I'm also inclined to think that testing config drive is more important than testing essentially neutron | 11:26 |
| vdrok | Yup | 11:27 |
| lucasagomes | dtantsur, oh right, so ++ we should def test it | 11:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Test building config drive in ironic_tempest_plugin https://review.openstack.org/375362 | 11:27 |
| dtantsur | with updated commit message ^^^ | 11:27 |
| lucasagomes | I agree configdrive > metadata in importance for us | 11:27 |
| dtantsur | ok, so the patch is ready, please vote :) | 11:30 |
| openstackgerrit | Merged openstack/ironic-ui: update homepage with developer documentation page https://review.openstack.org/373125 | 11:30 |
| * dtantsur writes to the ML | 11:32 | |
| lucasagomes | dtantsur, ++ exactly, better for consensus there | 11:32 |
| openstackgerrit | Merged openstack/ironic-ui: Clean imports in code https://review.openstack.org/365911 | 11:34 |
| openstackgerrit | Merged openstack/ironic-ui: TrivialFix: Remove logging import unused https://review.openstack.org/365909 | 11:34 |
| openstackgerrit | Merged openstack/ironic-ui: Minor state-machine fixes/updates https://review.openstack.org/374974 | 11:34 |
| *** amotoki has joined #openstack-ironic | 11:38 | |
| tangchen | lucasagomes, dtantsur, vdrok, milan: Hi guys, please help to check this simple fix when you have time. https://review.openstack.org/#/c/357639/ | 11:39 |
| patchbot | patch 357639 - python-ironicclient - Support i18n for osc plugin help msg | 11:39 |
| tangchen | thanks very much | 11:39 |
| openstackgerrit | Merged openstack/ironic-python-agent: Add py35 to tox environments https://review.openstack.org/375046 | 11:43 |
| lucasagomes | tangchen, commented | 11:44 |
| lucasagomes | tangchen, it's missing i18n in some bits there | 11:44 |
| lucasagomes | see commit message comment as well | 11:44 |
| lucasagomes | other than that it LGTM, thanks! | 11:44 |
| tangchen | lucasagomes: Thanks, will follow that | 11:45 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Support for nested properties with unicode for ironic node https://review.openstack.org/249186 | 11:45 |
| TheJulia | Good morning | 11:46 |
| mgould | morning TheJulia | 11:50 |
| TheJulia | o/ | 11:50 |
| dtantsur | morning TheJulia | 11:51 |
| mat128 | o/ | 11:52 |
| mgould | morning mat128 | 11:52 |
| lucasagomes | TheJulia, morning | 11:52 |
| lucasagomes | someone has seem Nisha around ? | 11:53 |
| *** jrist has quit IRC | 11:56 | |
| dtantsur | I haven't for quite some time | 11:56 |
| openstackgerrit | Merged openstack/bifrost: Update reno for Newton https://review.openstack.org/374912 | 11:57 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 12:01 |
| *** ccamacho is now known as ccamacho|lunch | 12:02 | |
| lucasagomes | dtantsur, right, there's an odd check added which prevents whole disk image + uefi unless we use local boot. Which seems bogus because if it's a whole disk image we already assumes it's going to boot from the disk | 12:02 |
| lucasagomes | https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L338-L341 | 12:02 |
| lucasagomes | jroll, ^ around. You use whole disk image right? Do you set "boot_option:local" as a capability ? | 12:03 |
| lucasagomes | assume* | 12:03 |
| dtantsur | lucasagomes, how does it prevent UEFI? | 12:04 |
| lucasagomes | dtantsur, it's another check, this one: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L220-L229 | 12:05 |
| lucasagomes | which is called at validate() | 12:05 |
| lucasagomes | dtantsur, for bios mode you can use whole disk image w/o needing to set "boot_option:local" but for UEFI you can't | 12:05 |
| dtantsur | weird | 12:06 |
| lucasagomes | yeah | 12:06 |
| dtantsur | which reminds me, I should start working on switching to local boot by default | 12:06 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 12:06 |
| dtantsur | wow, this is even worse: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L243-L244 | 12:06 |
| dtantsur | we should really, really think about feature parity between our different code paths' | 12:07 |
| lucasagomes | dtantsur, heh yeah I have never tested trusted boot | 12:07 |
| dtantsur | e.g. this means that trusted boot won't work for tripleo, as we use local boot there... | 12:08 |
| lucasagomes | but for uefi it really seems bogus, I've commented out that check and just deployed a whole disk image w/o setting local boot and it works | 12:08 |
| lucasagomes | becuase whole disk already assumes local boot AFAICT | 12:08 |
| dtantsur | lucasagomes, then remove it; we just have to make sure netboot was not explicitly requested, but such check should go elsewhere (and apply for both BIOS and UEFI) | 12:08 |
| *** jrist has joined #openstack-ironic | 12:08 | |
| *** catintheroof has quit IRC | 12:08 | |
| lucasagomes | dtantsur, yeah should be a different check for it | 12:09 |
| * lucasagomes cooks a patch | 12:09 | |
| * dtantsur undigs his local boot RFE | 12:09 | |
| TheJulia | lucasagomes: Nisha has been on leave, but I think she should have been back by now | 12:09 |
| lucasagomes | TheJulia, a-ha, thanks for the info | 12:10 |
| dtantsur | lucasagomes, there is something strange about iLO: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ilo/deploy.py#L338-L346 | 12:13 |
| dtantsur | I wonder if it's really intended.. if so, we can move the check to the ilo driver | 12:13 |
| lucasagomes | dtantsur, I think that would be the best idea, that's why I wanted to ask the iLO folks | 12:14 |
| *** vikrant has quit IRC | 12:17 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty https://review.openstack.org/370764 | 12:25 |
| xavierr | good morning Ironic | 12:37 |
| xavierr | morning TheJulia mgould mat128 lucasagomes dtantsur | 12:37 |
| TheJulia | good morning xavierr | 12:37 |
| lucasagomes | xavierr, good morning | 12:38 |
| *** rcernin has quit IRC | 12:38 | |
| *** daemontool has joined #openstack-ironic | 12:40 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Support for nested properties with unicode for ironic node https://review.openstack.org/249186 | 12:41 |
| milan | morning lucasagomes mgould tangchen TheJulia xavierr! :) | 12:44 |
| *** pcaruana|afk| has joined #openstack-ironic | 12:44 | |
| lucasagomes | hi there | 12:44 |
| vdrok | morning TheJulia mat128 and xavierr | 12:46 |
| *** sdake has joined #openstack-ironic | 12:48 | |
| dtantsur | morning xavierr, milan | 12:48 |
| milan | dtantsur, morning :) | 12:49 |
| mat128 | morning xavierr, vdrok and everyone else :) | 12:49 |
| milan | morning mat128 :) | 12:49 |
| dtantsur | hey mat128 | 12:49 |
| vdrok | dtantsur: lucasagomes aha - https://github.com/openstack-dev/devstack/commit/7682ea88a6ab8693b215646f16748dbbc2476cc4 | 12:51 |
| lucasagomes | vdrok, damn hah | 12:51 |
| *** vmud213 has quit IRC | 12:51 | |
| lucasagomes | vdrok, dtantsur perhaps we could just enable it for the ironic tests then ? | 12:52 |
| xavierr | morning milan and vdrok o/ | 12:52 |
| dtantsur | lucasagomes, yes, this is another option | 12:52 |
| dtantsur | lucasagomes, vdrok, feel free to propose a patch | 12:53 |
| *** david-lyle has joined #openstack-ironic | 12:56 | |
| *** aparnav has quit IRC | 12:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Introduce default_boot_option configuration option https://review.openstack.org/375458 | 12:58 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 12:59 |
| *** pcaruana|afk| has quit IRC | 13:00 | |
| *** pcaruana has quit IRC | 13:01 | |
| *** pcaruana has joined #openstack-ironic | 13:02 | |
| *** vsaienko has joined #openstack-ironic | 13:02 | |
| *** mjura has quit IRC | 13:03 | |
| *** rcernin has joined #openstack-ironic | 13:06 | |
| mgould | morning xavierr | 13:07 |
| xavierr | vdrok: I did a refactor in inspection patch but not sure you'll like hehehe | 13:07 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 13:07 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port. https://review.openstack.org/259354 | 13:07 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing chassis-create command https://review.openstack.org/293634 | 13:07 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-get command https://review.openstack.org/342060 | 13:07 |
| vdrok | xavierr: well, let's see what others think if you don't like how things look after refactor | 13:08 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 13:08 |
| vdrok | maybe that's fine as is | 13:08 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-port-list. https://review.openstack.org/360636 | 13:08 |
| xavierr | vdrok: sure :) | 13:08 |
| *** alaski is now known as lascii | 13:09 | |
| *** e0ne has joined #openstack-ironic | 13:11 | |
| *** jaosorior has quit IRC | 13:13 | |
| *** e0ne has quit IRC | 13:14 | |
| *** jaosorior has joined #openstack-ironic | 13:14 | |
| *** cdearborn has joined #openstack-ironic | 13:14 | |
| mat128 | rloo (not online?) | 13:15 |
| mat128 | rloo: if you ever see this, I will update my reviews with your latest comments | 13:16 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Introduce default_boot_option configuration option https://review.openstack.org/375458 | 13:17 |
| openstackgerrit | Charlle Daniel Da Silva Vieira Dias proposed openstack/ironic: Implements the validation to verify if the node is free or in use by https://review.openstack.org/372762 | 13:19 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Support for nested properties with unicode for ironic node https://review.openstack.org/249186 | 13:19 |
| *** mtanino has joined #openstack-ironic | 13:19 | |
| vdrok | dtantsur: lucasagomes https://review.openstack.org/375467 | 13:20 |
| patchbot | patch 375467 - openstack-dev/devstack - Force config drive when using ironic virt driver | 13:20 |
| mat128 | vdrok: ^ agreed, is there any test that ran with configdrive=False? | 13:22 |
| mat128 | if so, im not sure how they worked | 13:22 |
| *** e0ne has joined #openstack-ironic | 13:22 | |
| vdrok | mat128: force_config_drive was true by default for all hypervisors before https://github.com/openstack-dev/devstack/commit/7682ea88a6ab8693b215646f16748dbbc2476cc4 | 13:22 |
| dtantsur | vdrok, please update the ML thread with your patch | 13:23 |
| vdrok | so we can either add config_drive=True for all tempest tests that do create_server or set it globally in nova conf | 13:23 |
| vdrok | dtantsur: yup, will do now | 13:23 |
| mat128 | vdrok, dtantsur: if there any way to have Ironic work without the configdrive? | 13:24 |
| mat128 | s/if/is | 13:24 |
| vdrok | yes, it just won't be written :) | 13:24 |
| mat128 | unless you route 169.254.169.254 somewhere, your baremetal will never be able to talk to the md server | 13:24 |
| dtantsur | mat128, yes, route is somewhere :) | 13:24 |
| mat128 | hmm | 13:25 |
| dtantsur | it's not impossible, but I guess using config drives is more reliable | 13:25 |
| * mat128 keeps thinking of baremetal network as real vlans and routers | 13:25 | |
| mat128 | but there are other options | 13:25 |
| mat128 | forgive me :) | 13:25 |
| dtantsur | mat128, note that in our CI we use VMs, so Neutron probably does the right thing wrt MD server | 13:25 |
| *** rcernin has quit IRC | 13:26 | |
| mat128 | but our CI is simulating a real scenario, so we should always have a test that does not rely on the MD server being "reachable" | 13:26 |
| mat128 | anyway, the patch is about making configdrive mandatory for Ironic, so I shouldn't be concerned | 13:26 |
| *** rbudden has joined #openstack-ironic | 13:26 | |
| mat128 | s/for Ironic/for Ironic in CI/ | 13:26 |
| openstackgerrit | Charlle Daniel Da Silva Vieira Dias proposed openstack/ironic: Implements the verification if the node is free or in use by Ironic https://review.openstack.org/372762 | 13:32 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Move check for UEFI + Whole disk images https://review.openstack.org/375481 | 13:33 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Fail-Fast feature https://review.openstack.org/370016 | 13:33 |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 13:33 |
| *** vsaienko has quit IRC | 13:33 | |
| *** Goneri has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove unneeded try..except in heartbeat https://review.openstack.org/375482 | 13:34 |
| *** ekarlso_ has joined #openstack-ironic | 13:37 | |
| *** vsaienko has joined #openstack-ironic | 13:41 | |
| *** baoli has joined #openstack-ironic | 13:45 | |
| *** baoli_ has joined #openstack-ironic | 13:46 | |
| *** kragniz has quit IRC | 13:48 | |
| *** kragniz has joined #openstack-ironic | 13:48 | |
| *** baoli has quit IRC | 13:49 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 13:49 |
| *** rbudden has quit IRC | 13:51 | |
| *** rcernin has joined #openstack-ironic | 13:52 | |
| *** hshiina has joined #openstack-ironic | 13:53 | |
| *** e0ne has quit IRC | 13:53 | |
| *** dansmith is now known as superdan | 13:55 | |
| *** syed_ has joined #openstack-ironic | 13:56 | |
| *** srobert has joined #openstack-ironic | 13:57 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Remove unnecessary option from plugin settings https://review.openstack.org/375501 | 13:57 |
| *** sabeen has joined #openstack-ironic | 13:59 | |
| *** jjohnson2 has quit IRC | 14:01 | |
| *** jheroux has joined #openstack-ironic | 14:02 | |
| *** joprovost has joined #openstack-ironic | 14:03 | |
| *** rbrud has joined #openstack-ironic | 14:08 | |
| *** rbudden has joined #openstack-ironic | 14:09 | |
| *** vsaienko has quit IRC | 14:10 | |
| *** daemontool has quit IRC | 14:12 | |
| *** daemontool_ has joined #openstack-ironic | 14:12 | |
| jlvillal | dtantsur, Thanks for back-porting the GPT patch | 14:13 |
| *** daemontool__ has joined #openstack-ironic | 14:13 | |
| dtantsur | you're welcome! (also good morning and happy Friday) | 14:13 |
| jlvillal | dtantsur, Thanks :) | 14:13 |
| jlvillal | dtantsur, When it is time, I hope you have a great weekend too :) | 14:13 |
| dtantsur | soooooon, veeeeeryyy sooooooon! | 14:14 |
| jlvillal | nice | 14:14 |
| * dtantsur has something like this in his mind: http://cdn.acidcow.com/pics/20120224/soon_06.jpg | 14:14 | |
| vdrok | good morning jlvillal | 14:15 |
| *** vsaienko has joined #openstack-ironic | 14:16 | |
| jlvillal | Good morning vdrok :) | 14:16 |
| jlvillal | dtantsur, heh | 14:16 |
| mgould | :-) | 14:17 |
| *** daemontool_ has quit IRC | 14:17 | |
| *** Goneri has quit IRC | 14:17 | |
| dtantsur | bfournie, milan, speaking of the LLDP commands, I'd prefer not to commit to any API with them | 14:19 |
| dtantsur | so I was thinking that just extending CLI is better.. I'm open to other ideas | 14:19 |
| *** ccamacho|lunch is now known as ccamacho | 14:20 | |
| milan | dtantsur, bfournie I wanted to see whether a short-status listing command might want to include any more information | 14:20 |
| milan | if that's not the case, maybe the lldp command should be different from statuses subcommand | 14:21 |
| milan | my 2C ;) | 14:21 |
| dtantsur | I'd prefer not to create commands just for the sake of having more commands | 14:21 |
| dtantsur | building a good CLI is not fun (or rather type 2 fun as mgould would put it) on its own already | 14:21 |
| milan | it's not for that sake :) | 14:21 |
| dtantsur | then what for? | 14:21 |
| dtantsur | it's essentially two commands with listing of all nodes | 14:21 |
| milan | not to have unrelated bits sharing same subcommand | 14:22 |
| dtantsur | in what way are they "unrelated"? | 14:22 |
| milan | because the point of the first one is to give a brief info | 14:22 |
| milan | whereas if you wan to use more details you'd better use equivalent of a show | 14:22 |
| milan | "show" | 14:22 |
| milan | nope? | 14:22 |
| dtantsur | I'm not totally against it, but all OSC "list" commands have --long variant which shows ALL fields | 14:23 |
| dtantsur | (usually that looks insane, try it) | 14:23 |
| milan | including data from plugins? | 14:23 |
| dtantsur | so it's not necessary brief with all possible options. it will still be the same by default | 14:23 |
| dtantsur | well, I'm afraid for ironic it will include driver_info and driver_internal_info, but I need to check it | 14:24 |
| dtantsur | (also LLDP data does not come from a plugin, it's just optional <- my bad) | 14:24 |
| dtantsur | tested: openstack baremetal node list --long looks super unreadable | 14:25 |
| dtantsur | it does include driver_info on every line | 14:25 |
| milan | OK, if it's a precedence, what can we do about it | 14:26 |
| milan | #pixiesay -meh | 14:26 |
| dtantsur | pixie? | 14:26 |
| milan | oh, that thing is gone? | 14:27 |
| dtantsur | :( | 14:27 |
| dtantsur | milan, actually the Interfaces column is an easy win: it does not require LLDP data, we can derive it from inventory | 14:27 |
| dtantsur | for those who wonder what we're discussing: https://review.openstack.org/#/c/374381/ | 14:28 |
| patchbot | patch 374381 - ironic-inspector-specs - Display LLDP information via new commands | 14:28 |
| *** sabeen1 has joined #openstack-ironic | 14:28 | |
| bfournie | dtantsur: yes the interfaces is always there independent of lldp data | 14:29 |
| milan | yeah, interfaces sound OK | 14:29 |
| *** vsaienko has quit IRC | 14:30 | |
| dtantsur | and that's the only place where we're reusing an existing command, right? | 14:30 |
| bfournie | dtantsur: yes | 14:30 |
| dtantsur | cool! | 14:31 |
| *** sabeen has quit IRC | 14:31 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 14:32 |
| dtantsur | bfournie, the spec still does not quite render, see http://docs-draft.openstack.org/81/374381/7/check/gate-ironic-inspector-specs-docs-ubuntu-xenial/eadc8ff//doc/build/html/specs/lldp-reporting.html | 14:36 |
| dtantsur | bfournie, you can test with `tox -edocs` locally, then opening doc/build/html/index.html | 14:36 |
| bfournie | dtantsur: hmm, OK, will try that and fix, thanks | 14:37 |
| *** wajdi has joined #openstack-ironic | 14:39 | |
| *** joprovost1 has joined #openstack-ironic | 14:42 | |
| *** joprovost has quit IRC | 14:44 | |
| *** joprovost1 is now known as joprovost | 14:44 | |
| *** causten has joined #openstack-ironic | 14:47 | |
| *** causten has quit IRC | 14:48 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Fail-Fast feature https://review.openstack.org/370016 | 14:54 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 14:55 |
| *** amotoki has quit IRC | 14:58 | |
| milan | Good night Ironic | 15:00 |
| * milan be gone :P | 15:00 | |
| *** milan has quit IRC | 15:01 | |
| *** xavierr is now known as xavierr|lunch | 15:03 | |
| openstackgerrit | Charlle Daniel Da Silva Vieira Dias proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 15:03 |
| mgould | milan: good night! | 15:03 |
| *** jistr is now known as jistr|call | 15:03 | |
| *** vishwanathj has joined #openstack-ironic | 15:05 | |
| *** wajdi_ has joined #openstack-ironic | 15:06 | |
| *** helloworld42 has joined #openstack-ironic | 15:07 | |
| *** mat128 has quit IRC | 15:08 | |
| *** wajdi has quit IRC | 15:08 | |
| *** mat128 has joined #openstack-ironic | 15:08 | |
| *** ijw has joined #openstack-ironic | 15:11 | |
| *** helloworld42 has quit IRC | 15:11 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Use six.StringIO instead of six.moves.StringIO https://review.openstack.org/375297 | 15:13 |
| *** jheroux has quit IRC | 15:14 | |
| *** tuan_luong has joined #openstack-ironic | 15:14 | |
| *** amotoki has joined #openstack-ironic | 15:14 | |
| tuan_luong | Hello jroll d | 15:14 |
| tuan_luong | hi sambetts|pto | 15:14 |
| *** ijw has quit IRC | 15:16 | |
| *** ijw has joined #openstack-ironic | 15:16 | |
| *** rpioso has joined #openstack-ironic | 15:17 | |
| *** amotoki has quit IRC | 15:17 | |
| *** tuan_luong has quit IRC | 15:19 | |
| *** jheroux has joined #openstack-ironic | 15:19 | |
| *** rcernin has quit IRC | 15:19 | |
| *** lucasagomes is now known as lucas-hungry | 15:20 | |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Refactor provision state so all actions can use inherited take_action https://review.openstack.org/364552 | 15:21 |
| *** tuan_luong has joined #openstack-ironic | 15:22 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Fix tempest plugin to update port connectivity data https://review.openstack.org/375570 | 15:22 |
| *** srobert has quit IRC | 15:23 | |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add --wait to OSC provisioning commands https://review.openstack.org/354806 | 15:24 |
| *** jistr|call is now known as jistr | 15:26 | |
| *** jistr is now known as jistr|biab | 15:28 | |
| *** rajinir has joined #openstack-ironic | 15:32 | |
| rajinir | mjturek: MI, do you release the node in post_hook? | 15:33 |
| rajinir | mjturek: In post_test_hook | 15:33 |
| *** jistr|biab is now known as jistr | 15:36 | |
| *** bks has quit IRC | 15:39 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix output of "ironic node-list --detail" ironic client command https://review.openstack.org/375579 | 15:42 |
| *** zackf has joined #openstack-ironic | 15:44 | |
| *** ss7pro has joined #openstack-ironic | 15:45 | |
| *** ss7pro has quit IRC | 15:47 | |
| *** ss7pro has joined #openstack-ironic | 15:47 | |
| *** absubram has joined #openstack-ironic | 15:48 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Fix output of "ironic node-list --detail" ironic client command https://review.openstack.org/375579 | 15:50 |
| *** MattMan has quit IRC | 15:50 | |
| *** MattMan has joined #openstack-ironic | 15:50 | |
| dtantsur | calling it a day, see you and have a nice weekend! | 15:50 |
| mgould | dtantsur: good night! | 15:50 |
| *** dtantsur is now known as dtantsur|afk | 15:50 | |
| *** lukl has quit IRC | 15:51 | |
| *** rama_y has quit IRC | 15:52 | |
| jlvillal | dtantsur|afk: Dos ve dos! | 15:54 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Fail-Fast feature https://review.openstack.org/370016 | 15:56 |
| * jlvillal thinks we need to backport the DB patch to stable/mitaka http://logs.openstack.org/77/365977/1/gate/gate-ironic-python27-db-ubuntu-trusty/bb07cca/console.html | 15:56 | |
| *** lukl has joined #openstack-ironic | 15:56 | |
| * jlvillal looks into it... | 15:57 | |
| *** bnemec is now known as beekneemech | 15:58 | |
| *** Goneri has joined #openstack-ironic | 15:58 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Increase timeout for migration-related tests https://review.openstack.org/375585 | 16:03 |
| *** dprince has quit IRC | 16:05 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Increase timeout for migration-related tests https://review.openstack.org/375590 | 16:10 |
| openstackgerrit | Merged openstack/ironic: Fix a typo in helper.py https://review.openstack.org/375193 | 16:11 |
| openstackgerrit | Merged openstack/ironic: Fixes wrong steps to perform migration of nodes https://review.openstack.org/374841 | 16:13 |
| jlvillal | vdrok: I assume you still want to get this one in? https://review.openstack.org/#/c/306010/ | 16:14 |
| patchbot | patch 306010 - nova - Update instance node on rebuild only when it is re... | 16:14 |
| vdrok | jlvillal: yup | 16:15 |
| *** ohamada has quit IRC | 16:15 | |
| vdrok | good night dtantsur|afk | 16:15 |
| *** aries_ is now known as ariesyous | 16:16 | |
| jlvillal | vdrok: I'll review it. Maybe having it move to the top of the date list will help :) | 16:16 |
| vdrok | jlvillal: tho I'm not sure if it is still relevant, as we have hash ring in nova now, need to check | 16:17 |
| jlvillal | Oh | 16:17 |
| *** jistr is now known as jistr|afk | 16:18 | |
| * jlvillal thinks nova's manager.py could maybe use some refactoring at almost 7000 lines... | 16:18 | |
| openstackgerrit | Merged openstack/ironic: Add i18n _() to string https://review.openstack.org/374905 | 16:19 |
| openstackgerrit | Merged openstack/ironic: Remove unneeded try..except in heartbeat https://review.openstack.org/375482 | 16:19 |
| openstackgerrit | Merged openstack/ironic: [api-ref] Correcting type of r_addresses parameter https://review.openstack.org/365551 | 16:19 |
| jlvillal | vdrok: On https://review.openstack.org/#/c/306010/6//COMMIT_MSG Line 11. Should that be "instance RECREATE"? | 16:20 |
| patchbot | patch 306010 - nova - Update instance node on rebuild only when it is re... | 16:20 |
| *** jaosorior has quit IRC | 16:21 | |
| vdrok | jlvillal: CLI command is nova rebuild | 16:21 |
| vdrok | ironic does not support recreate iirc | 16:21 |
| vdrok | rebuild is called both during nova rebuild and during instance migration to another host,node which is recreate, if i understood this correctly | 16:23 |
| vdrok | rebuild_instance is called... | 16:23 |
| jlvillal | vdrok: I guess I'm confused by the commit message because the function is called 'rebuild_instance' so I was just thinking we are always doing a rebuild in this function. | 16:24 |
| jlvillal | vdrok: I'm reading the code more to understand better :) | 16:24 |
| vdrok | yeah, that is confusing, just grep rebuild_instance, it's called not only during rebuild :) | 16:24 |
| *** Romanenko_K has quit IRC | 16:24 | |
| jlvillal | vdrok: And since there is a test on 'recreate' is why I thought maybe it should say 'recreate' in the commit message. | 16:24 |
| jlvillal | vdrok: Ah | 16:24 |
| * jlvillal knows very little Nova code | 16:25 | |
| *** hshiina has quit IRC | 16:25 | |
| vdrok | jlvillal: ah, so there is a small explanation in param migration docstring - the two ways to get here are 'rebuild' and 'evacuate' | 16:26 |
| vdrok | and recreate is in the commit message as the parameter that is used to decide whether it is rebuild or evacuate is called recreate | 16:27 |
| jlvillal | vdrok: Thanks. I did a +1, but I'll admit I don't understand all of it. Hopefully some Nova cores will look at it :) | 16:28 |
| *** lucas-hungry is now known as lucasagomes | 16:29 | |
| NobodyCam | Good Morning Ironic'ers and ..... | 16:29 |
| jlvillal | vdrok: FYI: I backported the DB migration timeout patch to stable/newton and stable/mitaka | 16:29 |
| NobodyCam | TGIF! | 16:29 |
| jlvillal | NobodyCam: Good morning :) | 16:29 |
| vdrok | thank you jlvillal, I'm looking at the code too, it seems that the bug still valid - https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1169 | 16:30 |
| vdrok | the driver will try to PATCH instance.node, so it has to be a correct one | 16:31 |
| vdrok | morning NobodyCam, TGIF indeed :) | 16:31 |
| NobodyCam | morning vdrok | 16:32 |
| *** rama_y has joined #openstack-ironic | 16:32 | |
| NobodyCam | :) | 16:32 |
| NobodyCam | morning jlvillal :) | 16:32 |
| openstackgerrit | Merged openstack/ironic: DevStack: Use Jinja2 for templating when creating new VMs https://review.openstack.org/374345 | 16:33 |
| *** makowals has quit IRC | 16:36 | |
| *** causten has joined #openstack-ironic | 16:44 | |
| *** causten has quit IRC | 16:49 | |
| *** ccamacho is now known as ccamacho|out | 16:49 | |
| mgould | morning NobodyCam | 16:50 |
| *** zackf has quit IRC | 16:51 | |
| NobodyCam | morning mgould... happy Friday | 16:52 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Configure nodes/environment to boot in UEFI mode https://review.openstack.org/374988 | 16:54 |
| lucasagomes | NobodyCam, morning TGIF | 16:55 |
| NobodyCam | good morning lucasagomes ... Happy friday to you :) | 16:55 |
| lucasagomes | :D | 16:55 |
| *** derekh has quit IRC | 16:56 | |
| openstackgerrit | Merged openstack/bifrost: Allow user to get iPXE files https://review.openstack.org/367635 | 16:58 |
| openstackgerrit | Merged openstack/bifrost: Refactor env-setup.sh https://review.openstack.org/371883 | 16:58 |
| openstackgerrit | Merged openstack/bifrost: Workaround openSUSE ansible_os_family issue https://review.openstack.org/371082 | 16:58 |
| openstackgerrit | Merged openstack/bifrost: Install process SuSE support https://review.openstack.org/367637 | 16:58 |
| openstackgerrit | Merged openstack/bifrost: SuSE support for VM node creation https://review.openstack.org/370385 | 16:58 |
| *** jistr|afk is now known as jistr | 16:59 | |
| NobodyCam | :) | 16:59 |
| *** makowals has joined #openstack-ironic | 17:00 | |
| *** vsaienko has joined #openstack-ironic | 17:03 | |
| *** ccamacho|out has quit IRC | 17:05 | |
| *** trown is now known as trown|lunch | 17:06 | |
| *** makowals has quit IRC | 17:07 | |
| *** tuan_luong has quit IRC | 17:08 | |
| *** catintheroof has joined #openstack-ironic | 17:09 | |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager. https://review.openstack.org/366150 | 17:10 |
| *** adu has joined #openstack-ironic | 17:12 | |
| mjturek | hey rajinir yes! | 17:20 |
| mjturek | rajinir: let me get you a paste of that too | 17:20 |
| *** daemontool__ has quit IRC | 17:21 | |
| *** athomas has quit IRC | 17:22 | |
| mjturek | rajinir: http://paste.openstack.org/show/582798/ | 17:24 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Support for creating UEFI VMs https://review.openstack.org/374346 | 17:26 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Configure nodes/environment to boot in UEFI mode https://review.openstack.org/374988 | 17:26 |
| *** zackf has joined #openstack-ironic | 17:28 | |
| *** zackf has joined #openstack-ironic | 17:28 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 17:28 |
| *** jpich has quit IRC | 17:29 | |
| *** xavierr|lunch is now known as xavierr | 17:31 | |
| mariojv | yuriyz: i took out driver_info and driver_internal_info from the notification payload there ^ and i asked searchlight what fields they'd find useful to keep in | 17:38 |
| *** ariesyous has left #openstack-ironic | 17:38 | |
| mariojv | i'm not totally sure, but i think that they just want us to expose fields that an operator would find useful to search on | 17:38 |
| mariojv | so for example, if i wanted to find all nodes that have a particular driver and see what notifications have been emitted in the past hour for them, having the "driver" field would be necessary | 17:39 |
| mariojv | driver_info and driver_internal_info are probably less important; i doubt anyone would ever search on driver_internal_info | 17:40 |
| mariojv | but i've never spoken to any people who use searchlight with another project, so i'm not 100% on what all the use cases for an operator who uses searchlight with ironic would be | 17:41 |
| mariojv | my initial feeling is we should probably lean towards including more fields than we think are necessary rather than less | 17:42 |
| mariojv | we can always change the fields later if someone requests it, too | 17:42 |
| openstackgerrit | Charlle Daniel Da Silva Vieira Dias proposed openstack/ironic: Clarify when oneview node can be managed by ironic https://review.openstack.org/372735 | 17:47 |
| vdrok | lucasagomes: hm, so in https://review.openstack.org/375481 that check is only needed for ilo servers that boot via pxe? | 17:51 |
| patchbot | patch 375481 - ironic - Move check for UEFI + Whole disk images | 17:51 |
| lucasagomes | vdrok, yeah I left it for iLO because I don't really understand why they perform that check | 17:52 |
| vdrok | looking at the comment there, it says nothing specific about ilo hardware, seems like it was added as a kind of additional caution | 17:52 |
| lucasagomes | vdrok, I can deploy whole disk image + uefi with no problem | 17:52 |
| vdrok | yep, might be better for some ilo folks to have a look | 17:53 |
| lucasagomes | vdrok, rigth yeah, do you think we should remove it all ? | 17:53 |
| vdrok | I cannot say that, I have not tried :) | 17:53 |
| lucasagomes | vdrok, yeah. perhaps I should add 'em in the review. Lemme do it | 17:53 |
| vdrok | yup, thanks | 17:53 |
| lucasagomes | added Nisha there | 17:53 |
| vdrok | lucasagomes: I've also added stendulker and myself, other than that piece it lgtm :) | 17:54 |
| lucasagomes | vdrok, thank you! | 17:55 |
| openstackgerrit | Clark Boylan proposed openstack/ironic: Support https swift in devstack plugin https://review.openstack.org/375649 | 17:59 |
| *** krtaylor has quit IRC | 18:01 | |
| *** rpioso has quit IRC | 18:05 | |
| *** trown|lunch is now known as trown | 18:06 | |
| vdrok | good night everyone, see you on monday :) | 18:09 |
| jroll | lucasagomes: we don't set boot_option:local, but also don't believe we use UEFI | 18:10 |
| jroll | dtantsur|afk: sorry, was traveling this morning, will reply to your thread | 18:10 |
| lucasagomes | jroll, thanks. Which is fine because that's the inconsistency: w/o uefi you don't need to set it to local but with uefi you have to | 18:11 |
| mariojv | is every node attribute returnable via api shown via the node detail call? i can find out but curious if someone has a quick answer | 18:12 |
| jroll | lucasagomes: I woke up at 4am to get on a plane this morning, I don't have the brainpower to think about that, but it does seem wrong :) | 18:12 |
| mariojv | searchlight has said they only find attributes returned via API useful in notifications, so i want to try to narrow down any unneeded ones that my notifications patch might be sending out | 18:12 |
| lucasagomes | jroll, heh yeah, no worries | 18:13 |
| jroll | easy review to unblock a devstack thing, btw https://review.openstack.org/#/c/375649/1 | 18:13 |
| patchbot | patch 375649 - ironic - Support https swift in devstack plugin | 18:13 |
| jroll | our job here will test it https://review.openstack.org/#/c/372374/ | 18:13 |
| patchbot | patch 372374 - openstack-dev/devstack - Turn on tls proxy by default for testing | 18:13 |
| lucasagomes | jroll, +2 revoked the +a until it's tested | 18:16 |
| jroll | lucasagomes: cool, ty | 18:16 |
| lucasagomes | I ill keep an eye on it | 18:16 |
| mariojv | var name is correct https://github.com/openstack-dev/devstack/blob/master/lib/swift#L35 | 18:17 |
| jroll | lucasagomes: also just sent an email (and pavlo mentioned the history as well) about the configdrive thing, I'm +2 on that change | 18:22 |
| lucasagomes | jroll, great! | 18:23 |
| *** adu has quit IRC | 18:25 | |
| * jroll goes back to not working, have a good weekend y'all | 18:26 | |
| *** adu has joined #openstack-ironic | 18:26 | |
| jroll | oh, now that I look, this does belong in devstack | 18:27 |
| *** e0ne has joined #openstack-ironic | 18:29 | |
| *** rpioso has joined #openstack-ironic | 18:29 | |
| *** aNuposic has joined #openstack-ironic | 18:34 | |
| *** praneshp has joined #openstack-ironic | 18:35 | |
| aNuposic | Hi Folks, I am trying to create a node using ironic node-create but want to know what shall i give as the address to this command if I am running devstack? | 18:36 |
| aNuposic | should it be 127.0.0.1? | 18:36 |
| *** vsaienko has quit IRC | 18:41 | |
| lucasagomes | aNuposic, yup | 18:41 |
| lucasagomes | aNuposic, are you using *_ipmitool driver? If so just pay attention to the port | 18:42 |
| lucasagomes | which should match with the virtualbmc port | 18:42 |
| lucasagomes | (see vbmc list) | 18:42 |
| aNuposic | lucasagomes: I am using fake driver | 18:42 |
| lucasagomes | aNuposic, ah right... so I guess you don't even need an addres | 18:43 |
| lucasagomes | address* | 18:43 |
| *** e0ne has quit IRC | 18:43 | |
| aNuposic | lucasgomes:oh is it | 18:44 |
| aNuposic | just username and password? | 18:44 |
| aNuposic | lucasgomes: or not required that too? When are these options used then? | 18:45 |
| *** zackf has quit IRC | 18:45 | |
| aNuposic | lucasgomes: I guess i need an ipmi address pointing to 127.0.0.1 | 18:46 |
| lucasagomes | aNuposic, I don't think you need anything for the fake driver | 18:46 |
| * lucasagomes checks | 18:46 | |
| *** e0ne has joined #openstack-ironic | 18:47 | |
| lucasagomes | aNuposic, btw, try to run "ironic driver-properties <driver name>" | 18:49 |
| lucasagomes | and it tells you what is required and what's not | 18:49 |
| aNuposic | lucasgomes:ah i see | 18:51 |
| aNuposic | lucasgomes: what is A1 A2, B1 and B2? | 18:51 |
| aNuposic | not getting | 18:51 |
| lucasagomes | aNuposic, well that's the fake driver, it's used for unittests only | 18:52 |
| lucasagomes | so all information / properties are mocked | 18:52 |
| lucasagomes | it's not a real driver (as the name points out) | 18:52 |
| aNuposic | lucasgomes: ah hmm. | 18:53 |
| openstackgerrit | Clark Boylan proposed openstack/ironic: Support https in devstack plugin https://review.openstack.org/375649 | 18:53 |
| aNuposic | lucasgomes: so no address required? | 18:54 |
| aNuposic | lucasgomes: i was not clear from the driver-properties | 18:54 |
| devananda|lunch | interesting. I'm looking at the context object on the conductor, and it does not seem to be inheriting the "user" attribute from the API service, even for commands like node-update | 18:56 |
| *** devananda|lunch is now known as devananda | 18:56 | |
| lucasagomes | aNuposic, yeah you don't need anything | 18:56 |
| lucasagomes | aNuposic, out of curiosity, why fake? | 18:56 |
| *** mkam has joined #openstack-ironic | 18:56 | |
| aNuposic | lucasgomes: i am just trying to test one feature | 18:57 |
| aNuposic | lucasgomes: but let me double check | 18:57 |
| aNuposic | lucasgomes: if just fake is required | 18:57 |
| aNuposic | lucasgomes: oops i was mistaken other nodes are using agent_ipmitool and so will use the same driver for the node i am creating | 18:58 |
| aNuposic | lucasgomes: my bad | 18:58 |
| lucasagomes | no worries | 18:59 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Support for creating UEFI VMs https://review.openstack.org/374346 | 19:00 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Configure nodes/environment to boot in UEFI mode https://review.openstack.org/374988 | 19:00 |
| aNuposic | lucasgomes: so now ipmi_address seems required from properties and so that would be pointing to 127.0.0.1 | 19:01 |
| aNuposic | lucasgomes: i will make sure about the port | 19:02 |
| lucasagomes | aNuposic, you can't use the already created nodes ? Cause adding more by hand requires a bit more things than invoking "ironic node-create" | 19:03 |
| lucasagomes | you need a VM, a virtual BMC to control that VM and so on | 19:04 |
| lucasagomes | create the bridge as well | 19:04 |
| lucasagomes | (see devstack/tools/script/create-node.sh script) | 19:04 |
| lucasagomes | I mean ironic/devstack/... | 19:04 |
| aNuposic | lucasgomes: I can use but then I need to run node-update command but with that I am not getting what type of image (qcow2,vmlinuz) are compatible with already created node | 19:06 |
| aNuposic | lucasgomes: hmm I see. Seems creating a node is a lengthy process | 19:07 |
| lucasagomes | aNuposic, right... not sure if I follow the "type of image" bits there tho | 19:08 |
| aNuposic | lucasgomes: i can see these details of node already created http://pastebin.com/9QZmDsi5 | 19:08 |
| devananda | this definitely looks odd to me: http://paste.openstack.org/raw/582813/ | 19:08 |
| mrtenio | release | 19:09 |
| devananda | lucasagomes: any idea why our context is losing the "user" and "tenant" values when it goes over RPC ? | 19:09 |
| lucasagomes | devananda, hmmmm def odd, no idea off the top of my head | 19:09 |
| mrtenio | Sorry, I was trying to search for release :P | 19:09 |
| devananda | k. /me keeps digging | 19:09 |
| aNuposic | lucasgomes: I mean i need to run disk-image-create command to build the image like this "disk-image-create ubuntu baremetal localboot dhcp-all-interfaces local-config -o my-image" | 19:10 |
| *** david-lyle_ has joined #openstack-ironic | 19:10 | |
| aNuposic | lucasgomes: this gives me qcow2,vmlinux and initrd images | 19:11 |
| aNuposic | lucasgomes: I am just not sure are they compatible with the node already created? Do I have to care about that? | 19:11 |
| lucasagomes | aNuposic, tl;dr yes. But see, the .qcow is the image of the root filesystem, the .vmlinuz is the kernel and the .initrd is the initramfs | 19:12 |
| lucasagomes | aNuposic, take a look here: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements | 19:12 |
| devananda | it preserves other attributes, like "roles" , not just admin. http://paste.openstack.org/raw/582814/ | 19:13 |
| lucasagomes | it will make more sense | 19:13 |
| *** david-lyle has quit IRC | 19:13 | |
| lucasagomes | devananda, very odd seems everything is preserved but user & tenant, lemme take a look at the code see if I can spot something | 19:16 |
| *** e0ne_ has joined #openstack-ironic | 19:16 | |
| *** e0ne has quit IRC | 19:17 | |
| devananda | lucasagomes: found it | 19:17 |
| devananda | lucasagomes: common/context.py def from_dict | 19:17 |
| devananda | we actually strip them out -- but there's no explanation why | 19:17 |
| *** e0ne_ has quit IRC | 19:17 | |
| lucasagomes | oh two pop's there | 19:17 |
| lucasagomes | hmm | 19:17 |
| *** catintheroof has quit IRC | 19:17 | |
| lucasagomes | devananda, seems I've added that when porting to oslo.messaging | 19:18 |
| devananda | :) | 19:18 |
| lucasagomes | it's been a while heh 2014 | 19:18 |
| * lucasagomes thinks | 19:18 | |
| *** e0ne has joined #openstack-ironic | 19:18 | |
| *** e0ne has quit IRC | 19:19 | |
| lucasagomes | https://review.openstack.org/#/c/88307/ | 19:19 |
| *** e0ne_ has joined #openstack-ironic | 19:19 | |
| patchbot | patch 88307 - ironic - Port to oslo.messaging (MERGED) | 19:19 |
| *** e0ne_ has quit IRC | 19:19 | |
| aNuposic | lucasgomes: ah hmm. Cool so I see that the command I am running would be good with the image. | 19:19 |
| devananda | wow, yea, looong time ago | 19:20 |
| *** e0ne has joined #openstack-ironic | 19:20 | |
| *** e0ne has quit IRC | 19:21 | |
| aNuposic | lucasgomes: And add that to the image service | 19:21 |
| devananda | removing those two pops causes these tests to fail: ironic.tests.unit.common.test_context.RequestContextTestCase.test_from_dict and ironic.tests.unit.common.test_rpc.TestRequestContextSerializer.test_deserialize_context | 19:22 |
| lucasagomes | devananda, i'm trying to look where I got it from, cause I remember I based the work on Nova I think | 19:25 |
| lucasagomes | devananda, worth testing removing it and fixing the tests | 19:25 |
| devananda | yah. posting that shortly | 19:25 |
| lucasagomes | I don't see why anything else would break | 19:25 |
| devananda | I would like to have access to that information in the conductor :) | 19:25 |
| devananda | as we are adding multi-user things, knowing what user requestedthe action that the conductor is taking is ... well, important | 19:26 |
| devananda | we have the token, but I can't think of any reason to _fetch_ this from keystone when we already have it | 19:26 |
| devananda | mariojv: I believe we should actually hide driver_internal_info from most (if not all) users | 19:31 |
| lucasagomes | devananda, very important and cheap to have that information there (since we already have it at the api level) | 19:34 |
| lucasagomes | I don't see why not to remove those pop's at the top of my head | 19:34 |
| mariojv | thanks devananda, i agree, or else it's not really "internal" info and should go somewhere else imo | 19:37 |
| mariojv | i'm still debating myself for whether we should include sanitized driver_info in node notifications | 19:38 |
| mariojv | (sanitized meaning strip out sensitive credentials) | 19:38 |
| lucasagomes | mariojv, IMO sounds like we should remove sensitive data like passwords from the notifications | 19:40 |
| mariojv | searchlight generally wants fields that are exposed via API in notifications, but i can't think of a really good use case for searching based on driver_info | 19:40 |
| mariojv | lucasagomes: agreed, we definitely want to sanitize things like ipmi_password before the notification goes on the wire | 19:41 |
| mariojv | we may even want to remove instance_info from the notification, or at least pieces of it | 19:41 |
| *** absubram has quit IRC | 19:41 | |
| mariojv | including a large base64 configdrive in there seems like a very large waste | 19:42 |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 19:42 |
| lucasagomes | mariojv, yes! We have a bug about it, I hope the configdrive will disappear from the instance_info this cycle! | 19:44 |
| mariojv | oh, nice | 19:44 |
| lucasagomes | but yeah we should def remove it (it might contain the root password for the image, so it's a security concern as well) | 19:44 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Include user and tenant in serialized context https://review.openstack.org/375689 | 19:44 |
| devananda | gotta run to a dr's appt -- back in a few hours | 19:44 |
| *** devananda is now known as devananda|bbl | 19:44 | |
| mariojv | ok, i'll remove instance_info, too | 19:45 |
| devananda|bbl | mariojv: "search for nodes where driver_info.ipmi_address is on the 10.200.*.* network" | 19:45 |
| mariojv | yeah that could be useful ^ | 19:45 |
| devananda|bbl | mariojv: "search for nodes where instance_info does not contain a config drive" | 19:45 |
| devananda|bbl | if you give people data, they will find a use for it ;) | 19:46 |
| * devananda|bbl really goes now | 19:46 | |
| mariojv | there's an oslo lib we use for sanitizing really sensitive fields before sending anything over the api right? | 19:46 |
| * mariojv looks | 19:46 | |
| devananda|bbl | mariojv: yep | 19:47 |
| lucasagomes | mariojv, mask_dict_password | 19:47 |
| mariojv | perfect, thanks | 19:47 |
| *** mkam has quit IRC | 19:51 | |
| *** [1]rpioso has joined #openstack-ironic | 19:55 | |
| *** absubram has joined #openstack-ironic | 19:56 | |
| *** rpioso has quit IRC | 19:56 | |
| *** absubram_ has joined #openstack-ironic | 19:57 | |
| *** joprovost has quit IRC | 19:59 | |
| *** sdake has quit IRC | 20:00 | |
| *** krtaylor has joined #openstack-ironic | 20:00 | |
| *** absubram has quit IRC | 20:01 | |
| *** absubram_ is now known as absubram | 20:01 | |
| vdrok | lucasagomes: devananda|bbl ++ those two pops seem unnecessary. I also have this one, will revive it https://review.openstack.org/295371 | 20:02 |
| patchbot | patch 295371 - ironic - Add user and project domains to ironic context | 20:02 |
| *** ijw has quit IRC | 20:02 | |
| vdrok | Was working OK the last time I tested | 20:03 |
| devananda|bbl | vdrok: I see. first glance that looks good. I think I changed the way we handle show_password though, so you may want to check that that still works as you expect | 20:04 |
| vdrok | devananda|bbl: yup, will check tomorrow :) | 20:05 |
| lucasagomes | vdrok, ++ | 20:05 |
| *** david-lyle_ is now known as david-lyle | 20:06 | |
| lucasagomes | ok, I'm calling it a day | 20:08 |
| openstackgerrit | Clark Boylan proposed openstack/ironic: Support https in devstack plugin https://review.openstack.org/375649 | 20:09 |
| lucasagomes | have a good evening all, enjoy the weekend | 20:09 |
| lucasagomes | talk soon :-) | 20:09 |
| *** lucasagomes is now known as lucas-afk | 20:09 | |
| *** absubram has quit IRC | 20:10 | |
| *** ijw has joined #openstack-ironic | 20:10 | |
| vdrok | Good night lucas-afk | 20:10 |
| *** ijw has quit IRC | 20:14 | |
| *** rajinir has quit IRC | 20:15 | |
| *** praneshp has quit IRC | 20:22 | |
| *** clenimar has quit IRC | 20:26 | |
| *** adu has quit IRC | 20:36 | |
| *** lascii is now known as alaski | 20:41 | |
| *** jheroux has quit IRC | 20:45 | |
| *** krtaylor has quit IRC | 20:47 | |
| *** praneshp has joined #openstack-ironic | 20:56 | |
| *** wajdi_ has quit IRC | 21:05 | |
| jlvillal | Regarding: https://review.openstack.org/#/c/375467/ | 21:07 |
| patchbot | patch 375467 - openstack-dev/devstack - Force config drive when using ironic virt driver | 21:07 |
| jlvillal | Are we worried at all if that lands right now it would break the gate? | 21:07 |
| *** trown is now known as trown|outtypewww | 21:09 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: See if force config drive breaks the gate https://review.openstack.org/375732 | 21:09 |
| aNuposic | lucasgomes: I updated the node with ramdisk and kernel | 21:09 |
| aNuposic | lucasgomes: it was much easier | 21:10 |
| jlvillal | vdrok: ^^^ | 21:10 |
| openstackgerrit | Clark Boylan proposed openstack/ironic: Support https in devstack plugin https://review.openstack.org/375649 | 21:10 |
| jlvillal | aNuposic: Isn't that what I suggested earlier? ;) | 21:10 |
| jlvillal | aNuposic: I think lucas-afk is gone gone gone for the weekend. | 21:10 |
| jlvillal | aNuposic: Glad you got it working! | 21:11 |
| aNuposic | jlvillal: Yeah actually. But we thought to first go with creating a new node cause I was not sure if the image I create would be compatible with the node already created | 21:11 |
| aNuposic | i mean with the deploy kernel and deploy ramdisk | 21:12 |
| jlvillal | aNuposic: Okay. Well hopefully you have it all working now | 21:12 |
| aNuposic | jlvillal: But yeah as you both suggested it was very length process | 21:12 |
| jlvillal | aNuposic: Yeah :( | 21:12 |
| aNuposic | and so did node-uodate | 21:12 |
| aNuposic | jlvillal: What is the way to check node is up and running? I mean my node-validate is showing http://pastebin.com/pAi6RPtL | 21:13 |
| aNuposic | also I don't see the hostname or IP to ssh into and check | 21:13 |
| jlvillal | aNuposic: ironic node-show should show it | 21:13 |
| jlvillal | aNuposic: Looks like you need to fix the deploy info | 21:14 |
| aNuposic | jlvillal: hmmm it shows checksum should also be provided | 21:14 |
| jlvillal | aNuposic: That's what I mean | 21:15 |
| aNuposic | jlvillal: but not sure where to get it from | 21:15 |
| aNuposic | ;) | 21:15 |
| vdrok | jlvillal: it does basically the same as dtantsur|afk patch, and it passed. But never hurts to checj | 21:15 |
| jlvillal | vdrok: Okay. Because dtantsur|afk's patch didn't pass. | 21:15 |
| jlvillal | vdrok: I thought that was because we are waiting for a fix to land. | 21:15 |
| jlvillal | vdrok: ironic-lib release??? I'm not sure | 21:15 |
| jlvillal | vdrok: Ah I see. Only the non-voting job fails. | 21:16 |
| jlvillal | wholedisk | 21:16 |
| vdrok | ? I remember it did pass.. one job failed but it's non voting and the fail was expected as ironic-lib was not released yet | 21:16 |
| jlvillal | aNuposic: I'm not sure off the top of my head. I forget how the checksum gets generated. | 21:16 |
| jlvillal | vdrok: right | 21:17 |
| jlvillal | aNuposic: Let me take a look | 21:17 |
| vdrok | Yeah, I'm from phone so a bit slow :) | 21:17 |
| jlvillal | vdrok: You should go enjoy your weekend :) | 21:19 |
| *** praneshp has quit IRC | 21:21 | |
| *** thrash is now known as thrash|g0ne | 21:21 | |
| vdrok | jlvillal: btw thanks for bringing up the Nova fix, it's on the way to merge already | 21:21 |
| jlvillal | vdrok: Yeah. Woo hoo! :) | 21:21 |
| jlvillal | vdrok: Thanks for fixing it! | 21:21 |
| vdrok | :) | 21:21 |
| jlvillal | aNuposic: Look here: http://docs.openstack.org/developer/ironic/deploy/install-guide.html And search for 'checksum' | 21:23 |
| aNuposic | jlvillal: Ok let me have a look at it. | 21:24 |
| *** praneshp has joined #openstack-ironic | 21:28 | |
| *** rbudden has quit IRC | 21:28 | |
| aNuposic | jlvillal: cool i got alpha numeric hash | 21:28 |
| aNuposic | jlvillal: will add that to the instance_info | 21:29 |
| aNuposic | jlvillal: now deploy seems to be true. :) console is false, that's just fine i guess | 21:31 |
| jlvillal | aNuposic: I think so. You aren't using the console. | 21:31 |
| aNuposic | jlvillal: I am not. | 21:31 |
| *** joprovost has joined #openstack-ironic | 21:31 | |
| aNuposic | jlvillal: I should ssh login to the node after the deployment is that right? | 21:32 |
| jlvillal | aNuposic: I don't know. Depends on what you are trying to do. | 21:32 |
| aNuposic | jlvillal: ok. actually no need of ssh login. That I was going to do earlier to see if node is up and running | 21:38 |
| aNuposic | jlvillal: Now ready to run node-set-provision with config drive | 21:40 |
| *** joprovost has quit IRC | 21:42 | |
| *** cdearborn has quit IRC | 21:45 | |
| xavierr | have a good night all | 22:05 |
| xavierr | see you all on monday :) | 22:05 |
| *** xavierr is now known as xavierr|afk | 22:05 | |
| *** rpioso has joined #openstack-ironic | 22:08 | |
| *** jrist has quit IRC | 22:08 | |
| *** [1]rpioso has quit IRC | 22:14 | |
| *** zackf has joined #openstack-ironic | 22:19 | |
| *** zackf has quit IRC | 22:24 | |
| jlvillal | xavierr|afk: Good night | 22:48 |
| *** ppiela has quit IRC | 23:00 | |
| *** jrist has joined #openstack-ironic | 23:00 | |
| *** jrist has quit IRC | 23:03 | |
| *** jrist has joined #openstack-ironic | 23:04 | |
| *** vishwanathj has quit IRC | 23:05 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: See if force config drive breaks the gate https://review.openstack.org/375732 | 23:06 |
| openstackgerrit | linbing proposed openstack/ironic-python-agent: * Force LANGUAGE=en_US in test runs * Force the value of LC_ALL to be en_US.UTF-8 https://review.openstack.org/375765 | 23:07 |
| *** sdake has joined #openstack-ironic | 23:17 | |
| *** rama_y has quit IRC | 23:38 | |
| *** ijw has joined #openstack-ironic | 23:39 | |
| *** aNuposic has quit IRC | 23:39 | |
| *** rpioso has quit IRC | 23:43 | |
| *** ijw has quit IRC | 23:49 | |
| *** rbudden has joined #openstack-ironic | 23:58 | |
| *** rbudden has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
