*** ijw has joined #openstack-ironic | 00:02 | |
devananda | NobodyCam: ah, jogo's pinning on stable/juno | 00:03 |
---|---|---|
devananda | nope. /me lands it | 00:03 |
devananda | NobodyCam: actually it was failing previous revisions | 00:04 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 00:04 |
*** hj-hp has quit IRC | 00:06 | |
*** penick has joined #openstack-ironic | 00:06 | |
NobodyCam | devananda: I was just rebuilding to test it | 00:07 |
NobodyCam | ofc had to also update all my caches too | 00:08 |
NobodyCam | :-p | 00:08 |
*** wanyen has joined #openstack-ironic | 00:11 | |
openstackgerrit | Joshua Harlow proposed openstack/ironic: Update the states generator and regenerate the image https://review.openstack.org/158102 | 00:11 |
wanyen | Hi BadCub, | 00:11 |
BadCub | heya wanyen | 00:12 |
wanyen | BadCub, there are a few iLO driver specs that have been approved but not yet on ironic kilo3 LP. Can you add them? I will give you the links. | 00:12 |
BadCub | absolutely! | 00:13 |
wanyen | Implement Cleaning Operations for iLO drivers https://blueprints.launchpad.net/ironic/+spec/ilo-cleaning-support | 00:13 |
openstackgerrit | Merged openstack/ironic: Remove unused modules from ironic/openstack/common https://review.openstack.org/158264 | 00:13 |
jogo | devananda: ? did it break something | 00:13 |
wanyen | BadCub, another one - In-band RAID configuration using IP https://blueprints.launchpad.net/ironic/+spec/inband-raid-configuration | 00:14 |
wanyen | BadCub,, one more: New driver interface for RAID configuration https://blueprints.launchpad.net/ironic/+spec/ironic-generic-raid-interface | 00:15 |
wanyen | Devananda and BadCub, also, can you raise priority of https://blueprints.launchpad.net/ironic/+spec/uefi-secure-boot from low to medium? It’s iLO driver’s top priority item. Thanks! | 00:17 |
BadCub | ations for iLO drivers https://blueprints.launchpad.net/ironic/+spec/ilo-cleaning-support appears to already be approved for K3 | 00:17 |
BadCub | ack, my copy functionality has gone insane | 00:17 |
wanyen | BadCub, all the blueprints and specs that I justed mentioned have been approved but not yet on Kilo3 launchpad. | 00:18 |
BadCub | DOH. My bad. My mouse is gone insane, gimmie a sec and I will nail them down | 00:19 |
wanyen | BadCub, ty | 00:19 |
BadCub | wanyen UEFI and Cleaning were already tagged for K3. I just updated RAID for K3 | 00:23 |
BadCub | wanyen: devananda still sets priority on things :-) | 00:24 |
wanyen | BadCub, ty | 00:28 |
wanyen | Devananda, can you consider raising priority for secure boot https://blueprints.launchpad.net/ironic/+spec/uefi-secure-boot from low to medium? It’s iLO driver’s top priority item. Thanks! | 00:29 |
BadCub | wanyen, yw :-) | 00:29 |
*** Marga_ has quit IRC | 00:40 | |
*** Marga_ has joined #openstack-ironic | 00:41 | |
*** Marga_ has quit IRC | 00:41 | |
devananda | jroll: you started on the pluggable network provider work -- how's that going? chance of being done for kilo? | 00:45 |
devananda | wanyen: done | 00:45 |
jroll | devananda: distractions | 00:45 |
devananda | wanyen: I updated several proirities just now | 00:45 |
jroll | idk, I could probably get it close this week | 00:45 |
jroll | haven't gotten any feedback on current patches | 00:45 |
jroll | (yes I know tests are failing, that isn't the point) | 00:46 |
devananda | jroll: it's at least a High priority for me | 00:46 |
devananda | once I have some time to start reviewing deeper things | 00:46 |
jroll | ok, cool | 00:46 |
jroll | devananda: can you take a rough look tonight and I'll hack on it a bit tomorrow? | 00:46 |
devananda | after spending most of today going over the BPs and specs with BadCub I think I have a better picture of htings | 00:46 |
devananda | still feels like there are several items we're not tracking well | 00:46 |
devananda | but that's probably entirely my fault. like the microver stuff | 00:47 |
jroll | heh | 00:47 |
*** ChuckC has quit IRC | 00:56 | |
openstackgerrit | Devananda van der Veen proposed openstack/python-ironicclient: Add support for logical names https://review.openstack.org/158520 | 00:56 |
devananda | and like that one | 00:57 |
devananda | oh, which needs to be based on the microver support in the client, otherwise the server will rejecet it | 00:57 |
NobodyCam | lol which you have a -1 on atm lol :-p | 00:59 |
devananda | right | 01:00 |
devananda | because it does it wrong | 01:00 |
devananda | i may just fix that up too | 01:00 |
NobodyCam | oh sure devananda wakes up at 5 pm :-p | 01:00 |
NobodyCam | hehehe | 01:01 |
* BadCub rubs eyes | 01:01 | |
mrda | devananda: oh, thanks for that. I was waiting for 155624 first. | 01:01 |
NobodyCam | morning mrda :) | 01:02 |
mrda | :) | 01:02 |
mrda | hey NobodyCam | 01:02 |
devananda | mrda: ohhai! welcome | 01:03 |
devananda | mrda: if you feel like doing some related stuff - I think we should have a "latest" version in the server, and the client should default to it | 01:03 |
jroll | o.o | 01:04 |
devananda | eg, instead of only accepting numbers, the server should appect X-OpenStack-ironic-API-Version="latest" | 01:04 |
devananda | and then just do that | 01:04 |
mrda | Since the microversion support one was still very much a work in progress, I was delaying :) | 01:04 |
mrda | oh, ok. Sure. | 01:04 |
devananda | because otherwise the client has to magicallyknow what the latest version is | 01:04 |
mrda | I'll put something up | 01:04 |
devananda | which, well, means a bunch of other magic | 01:04 |
devananda | nova does ^ and I like it | 01:04 |
devananda | thanks! | 01:04 |
openstackgerrit | Merged openstack/ironic: Return required properties for agent deploy driver https://review.openstack.org/158313 | 01:04 |
* devananda runs off to a 5pm meeting | 01:04 | |
mrda | so, just on 155624, it should *always* send the microversion header now, right? | 01:05 |
*** penick has quit IRC | 01:07 | |
*** adam_g is now known as adam_g_out | 01:09 | |
*** devlaps has quit IRC | 01:11 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning States https://review.openstack.org/153444 | 01:15 |
*** david-lyle is now known as david-lyle_afk | 01:19 | |
*** achanda has quit IRC | 01:21 | |
devananda | mrda: yes | 01:27 |
* BadCub thinks it is time to fix dinner and hide for the evening | 01:27 | |
*** ParsectiX has quit IRC | 01:35 | |
*** ParsectiX has joined #openstack-ironic | 01:36 | |
NobodyCam | mmmmmm dinner | 01:37 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning in DriverInterfaces https://review.openstack.org/157208 | 01:42 |
*** krtaylor has quit IRC | 01:43 | |
*** BadCub has quit IRC | 01:45 | |
*** ijw has quit IRC | 01:48 | |
*** krtaylor has joined #openstack-ironic | 01:55 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement execute clean steps https://review.openstack.org/155561 | 01:59 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement execute clean steps https://review.openstack.org/155561 | 02:01 |
*** yog_ has quit IRC | 02:32 | |
*** rloo_afk has quit IRC | 02:45 | |
*** spandhe has quit IRC | 02:48 | |
*** ramineni has joined #openstack-ironic | 02:49 | |
*** yuanying has quit IRC | 02:54 | |
*** wanyen has quit IRC | 03:14 | |
*** yog_ has joined #openstack-ironic | 03:24 | |
*** dmellado has quit IRC | 03:27 | |
*** dmellado has joined #openstack-ironic | 03:29 | |
*** Marga_ has joined #openstack-ironic | 03:30 | |
*** yuanying has joined #openstack-ironic | 03:36 | |
*** jlvillal_remote has joined #openstack-ironic | 03:38 | |
*** killer_prince has quit IRC | 03:40 | |
*** oomichi has quit IRC | 03:41 | |
*** Marga_ has quit IRC | 03:43 | |
*** spandhe has joined #openstack-ironic | 03:44 | |
*** Marga_ has joined #openstack-ironic | 03:44 | |
*** spandhe_ has joined #openstack-ironic | 03:47 | |
*** spandhe has quit IRC | 03:48 | |
*** spandhe_ is now known as spandhe | 03:48 | |
*** harlowja is now known as harlowja_away | 04:14 | |
*** harlowja_away is now known as harlowja | 04:15 | |
*** devlaps has joined #openstack-ironic | 04:17 | |
*** pensu has joined #openstack-ironic | 04:51 | |
*** ParsectiX has quit IRC | 04:55 | |
*** ParsectiX has joined #openstack-ironic | 04:55 | |
*** krtaylor has quit IRC | 04:55 | |
*** ParsectiX has quit IRC | 04:57 | |
*** ParsectiX has joined #openstack-ironic | 04:58 | |
*** spandhe has quit IRC | 04:59 | |
*** killer_prince has joined #openstack-ironic | 05:05 | |
*** killer_prince is now known as lazy_prince | 05:06 | |
*** krtaylor has joined #openstack-ironic | 05:07 | |
*** krtaylor has quit IRC | 05:14 | |
*** devlaps has quit IRC | 05:17 | |
*** krtaylor has joined #openstack-ironic | 05:26 | |
*** eghobo has joined #openstack-ironic | 05:29 | |
*** lazy_prince has quit IRC | 05:29 | |
*** killer_prince has joined #openstack-ironic | 05:31 | |
*** killer_prince is now known as lazy_prince | 05:31 | |
*** achanda has joined #openstack-ironic | 05:34 | |
*** jrist is now known as jrist-afk | 05:35 | |
*** takadayuiko has joined #openstack-ironic | 05:38 | |
*** killer_prince has joined #openstack-ironic | 05:40 | |
*** lazy_prince has quit IRC | 05:40 | |
*** killer_prince is now known as lazy_prince | 05:40 | |
*** ParsectiX has quit IRC | 05:47 | |
*** ParsectiX has joined #openstack-ironic | 05:47 | |
*** coolsvap_ is now known as coolsvap | 05:57 | |
openstackgerrit | Merged stackforge/proliantutils: HPSSA: Fix the disk sorting issue on create https://review.openstack.org/157832 | 05:58 |
openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add Cleaning Operations for iLO drivers https://review.openstack.org/157715 | 06:00 |
openstackgerrit | Merged stackforge/proliantutils: HPSSA: Move test outputs to a single file https://review.openstack.org/157862 | 06:03 |
*** harlowja is now known as harlowja_away | 06:11 | |
*** ParsectiX has quit IRC | 06:13 | |
*** andreykurilin_ has joined #openstack-ironic | 06:14 | |
*** eghobo has quit IRC | 06:19 | |
openstackgerrit | sandhya proposed openstack/ironic: Implement Chassis Level Discovery Tool https://review.openstack.org/158577 | 06:32 |
*** eghobo has joined #openstack-ironic | 06:38 | |
*** eghobo has quit IRC | 06:38 | |
*** pcrews has quit IRC | 06:39 | |
*** eghobo has joined #openstack-ironic | 06:41 | |
*** pradipta has joined #openstack-ironic | 06:42 | |
*** andreykurilin_ has quit IRC | 06:44 | |
*** viktors1 has joined #openstack-ironic | 06:45 | |
*** eghobo has quit IRC | 06:46 | |
*** pcrews has joined #openstack-ironic | 06:46 | |
*** eghobo has joined #openstack-ironic | 06:48 | |
*** lazy_prince has quit IRC | 06:54 | |
*** Nisha has joined #openstack-ironic | 06:55 | |
*** killer_prince has joined #openstack-ironic | 06:58 | |
*** killer_prince is now known as lazy_prince | 06:58 | |
*** jcoufal has joined #openstack-ironic | 07:03 | |
*** dlpartain has joined #openstack-ironic | 07:06 | |
*** dlpartain has left #openstack-ironic | 07:06 | |
openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add tests for ilo_deploy driver https://review.openstack.org/142985 | 07:16 |
*** eghobo has quit IRC | 07:23 | |
*** viktors1 has quit IRC | 07:28 | |
*** jcoufal_ has joined #openstack-ironic | 07:32 | |
*** chlong has quit IRC | 07:34 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Root partition should be marked as bootable https://review.openstack.org/156587 | 07:35 |
*** jcoufal has quit IRC | 07:35 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add disk allocator https://review.openstack.org/158052 | 07:35 |
*** coolsvap is now known as coolsvap_ | 07:36 | |
*** viktors1 has joined #openstack-ironic | 07:36 | |
*** Nisha has quit IRC | 07:37 | |
*** coolsvap_ is now known as coolsvap | 07:39 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: RAID creation should return current config https://review.openstack.org/158033 | 07:42 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Common changes for secure boot support https://review.openstack.org/153974 | 07:43 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: RAID creation should return current config https://review.openstack.org/158033 | 07:44 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Common changes for secure boot support https://review.openstack.org/153974 | 07:45 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 07:45 |
*** Marga_ has quit IRC | 07:45 | |
*** Marga_ has joined #openstack-ironic | 07:46 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 07:46 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add disk allocator https://review.openstack.org/158052 | 07:46 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver https://review.openstack.org/154816 | 07:47 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Common changes for secure boot support https://review.openstack.org/153974 | 07:48 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 07:48 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 07:49 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver https://review.openstack.org/154816 | 07:49 |
*** viktors1 has quit IRC | 07:53 | |
*** viktors1 has joined #openstack-ironic | 07:53 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 07:56 |
*** Marga_ has quit IRC | 07:59 | |
*** jcoufal has joined #openstack-ironic | 08:01 | |
*** jcoufal_ has quit IRC | 08:05 | |
*** ndipanov has joined #openstack-ironic | 08:08 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add disk allocator https://review.openstack.org/158052 | 08:08 |
openstackgerrit | Michael Davies proposed openstack/ironic: Add support for 'latest' in microversion header https://review.openstack.org/158601 | 08:10 |
mrda | devananda: ^^^ Still need to test, but here's the 1st revision (just rebuilding a devstack) | 08:11 |
openstackgerrit | Merged stackforge/proliantutils: ILO: Update get_persistent_boot for UEFI https://review.openstack.org/158205 | 08:14 |
*** athomas has joined #openstack-ironic | 08:26 | |
*** priteau has joined #openstack-ironic | 08:27 | |
*** ifarkas has joined #openstack-ironic | 08:27 | |
*** takadayuiko has quit IRC | 08:34 | |
*** teju has joined #openstack-ironic | 08:37 | |
*** dtantsur|afk is now known as dtantsur | 08:37 | |
dtantsur | Morning Ironic | 08:37 |
*** Nisha has joined #openstack-ironic | 08:38 | |
*** saripurigopi has joined #openstack-ironic | 08:39 | |
*** oomichi_ has joined #openstack-ironic | 08:40 | |
*** achanda has quit IRC | 08:42 | |
*** rameshg87 has joined #openstack-ironic | 08:46 | |
*** achanda has joined #openstack-ironic | 08:49 | |
*** bauzas has joined #openstack-ironic | 08:50 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: setup.py to work without requirements.txt https://review.openstack.org/158609 | 08:54 |
*** jistr has joined #openstack-ironic | 08:54 | |
*** erwan_taf has joined #openstack-ironic | 08:55 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: setup.py to work without requirements.txt https://review.openstack.org/158609 | 08:55 |
*** pradipta has quit IRC | 08:59 | |
*** yarker has quit IRC | 09:04 | |
*** teju has quit IRC | 09:05 | |
*** rfchapman has quit IRC | 09:05 | |
*** achanda has quit IRC | 09:06 | |
*** lucasagomes has joined #openstack-ironic | 09:11 | |
openstackgerrit | Merged stackforge/ironic-discoverd: setup.py to work without requirements.txt https://review.openstack.org/158609 | 09:13 |
*** vdrok_afk is now known as vdrok | 09:16 | |
vdrok | morning ironic | 09:16 |
*** viktors|afk is now known as viktors | 09:20 | |
*** derekh has joined #openstack-ironic | 09:20 | |
*** romcheg has joined #openstack-ironic | 09:21 | |
rameshg87 | lucasagomes, hi | 09:22 |
gilliard | Good morning Ironic ;) | 09:22 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Support for non-Glance image references https://review.openstack.org/136741 | 09:22 |
rameshg87 | morning vdrok, gilliard, dtantsur, lucasagomes :) | 09:22 |
dtantsur | o/ | 09:22 |
vdrok | morning rameshg87, gilliard, dtantsur, lucasagomes | 09:23 |
lucasagomes | morning all :) | 09:24 |
*** ndipanov has quit IRC | 09:30 | |
*** ndipanov has joined #openstack-ironic | 09:30 | |
*** MattMan has quit IRC | 09:32 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add the image extension (for local boot) https://review.openstack.org/157469 | 09:36 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add iscsi extension https://review.openstack.org/155727 | 09:36 |
*** openstackgerrit has quit IRC | 09:36 | |
*** MattMan has joined #openstack-ironic | 09:36 | |
saripurigopi | are there any guide lines for writing the mock tests for new driver? | 09:36 |
*** openstackgerrit has joined #openstack-ironic | 09:36 | |
lucasagomes | saripurigopi, not really :( I would use other drivers tests as an example tho | 09:43 |
saripurigopi | lucasagomes, :-) I'm doing the same, just wanted to know. | 09:44 |
lucasagomes | ack :) | 09:44 |
rameshg87 | lucasagomes, hey | 09:53 |
lucasagomes | rameshg87, hi there | 09:55 |
rameshg87 | lucasagomes, i wanted to check regarding root device hints | 09:56 |
rameshg87 | lucasagomes, i see you are planning to do this in dib - https://review.openstack.org/#/c/142178/2/elements/deploy-ironic/init.d/70-ironic-root-device | 09:56 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Fix reboot logic of iRMC Power Driver https://review.openstack.org/158614 | 09:56 |
rameshg87 | lucasagomes, just a min, i think i jumped in too soon | 09:58 |
*** jcoufal has quit IRC | 09:59 | |
*** jcoufal has joined #openstack-ironic | 09:59 | |
lucasagomes | rameshg87, right, I gotta a patch on DIB | 09:59 |
lucasagomes | I need to polish it up but it works | 09:59 |
lucasagomes | rameshg87, I want to do it in IPA too | 10:00 |
rameshg87 | lucasagomes, i had a question on how you get wwn. | 10:00 |
lucasagomes | rameshg87, lsblk? | 10:00 |
rameshg87 | lucasagomes, yeah | 10:00 |
rameshg87 | lucasagomes, but i think i figured out. i am writing some functional tests in proliantutils module | 10:01 |
rameshg87 | lucasagomes, so i will assert that what wwn i get from hp utility is same as what is reported by lsblk | 10:01 |
lucasagomes | rameshg87, http://paste.openstack.org/show/181114/ the options from lsblk | 10:01 |
lucasagomes | ah nice, ok :) | 10:01 |
lucasagomes | yeah that should work | 10:02 |
rameshg87 | lucasagomes, https://review.openstack.org/#/c/158122/1/proliantutils/functests/test_hpssa.py L98 | 10:02 |
*** athomas has quit IRC | 10:02 | |
rameshg87 | lucasagomes, i will assert size and wwn reported by our tool and lsblk are same. | 10:03 |
*** pelix1 has joined #openstack-ironic | 10:03 | |
rameshg87 | lucasagomes, so it matches when ironic tries to use them later on | 10:03 |
lucasagomes | +1! | 10:04 |
openstackgerrit | Imre Farkas proposed stackforge/ironic-discoverd: eDeploy plugin to expose cmdb data https://review.openstack.org/158618 | 10:04 |
lucasagomes | rameshg87, I will polish that DIB patch as soon as I get some time | 10:04 |
lucasagomes | the comments are most about bash style etc | 10:04 |
*** harlowja_away has quit IRC | 10:04 | |
*** pelix1 is now known as pelix | 10:04 | |
rameshg87 | lucasagomes, great .. thanks :) | 10:13 |
rameshg87 | lucasagomes, wwn is always an 16-byte number ? any idea ? | 10:14 |
rameshg87 | lucasagomes, our tool returned unique identifier as "600508B1001C456EF1BDF9C0605EF7DD" | 10:14 |
rameshg87 | lucasagomes, where as lsblk returned WWN="0x600508b1001c456e" | 10:14 |
lucasagomes | rameshg87, hmm gotta check it out | 10:14 |
rameshg87 | lucasagomes, first 16-bytes seem to be same :) o | 10:15 |
rameshg87 | okay:) | 10:15 |
lucasagomes | maybe we should normalize it too, like before passing to the ramdisk? | 10:15 |
lucasagomes | like first 16 hex | 10:15 |
rameshg87 | lucasagomes, what does normalize mean here ? | 10:15 |
lucasagomes | lower case | 10:15 |
rameshg87 | lucasagomes, okay | 10:15 |
lucasagomes | and I we do the same on the ramdisk side | 10:16 |
rameshg87 | lucasagomes, so ramdisk would expect 0x prefix as well, right ? | 10:16 |
lucasagomes | rameshg87, right now yes, but we can sync it | 10:16 |
rameshg87 | lucasagomes, okay | 10:16 |
lucasagomes | cause right now I just test against the lsblk output | 10:16 |
rameshg87 | lucasagomes, yeah, okay | 10:16 |
rameshg87 | lucasagomes, please have a look at when you get some time: https://review.openstack.org/#/c/156587/ | 10:21 |
rameshg87 | lucasagomes, the root partition should be made bootable fix :) | 10:21 |
rameshg87 | lucasagomes, i changed it to do only for localboot | 10:21 |
lucasagomes | oh sure will do | 10:22 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 10:25 |
*** erwan_taf has quit IRC | 10:26 | |
*** erwan_taf has joined #openstack-ironic | 10:27 | |
*** yog_ has quit IRC | 10:28 | |
openstackgerrit | Merged stackforge/proliantutils: HPSSA: RAID creation should return current config https://review.openstack.org/158033 | 10:30 |
*** ndipanov has quit IRC | 10:32 | |
openstackgerrit | Merged openstack/ironic: Update the states generator and regenerate the image https://review.openstack.org/158102 | 10:44 |
*** ndipanov has joined #openstack-ironic | 10:46 | |
*** takadayuiko has joined #openstack-ironic | 10:51 | |
*** yuanying has quit IRC | 10:56 | |
*** ramineni has quit IRC | 11:04 | |
*** subscope has quit IRC | 11:05 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add disk allocator https://review.openstack.org/158052 | 11:06 |
openstackgerrit | Merged openstack/ironic: Minor changes to InspectInterface https://review.openstack.org/158436 | 11:16 |
*** subscope has joined #openstack-ironic | 11:17 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE drivers support for IPA https://review.openstack.org/155728 | 11:22 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Local boot support for IPA https://review.openstack.org/157723 | 11:22 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE driver: Deprecate pxe_deploy_{ramdisk, kernel} https://review.openstack.org/158644 | 11:22 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Fixes for issue found during functional testing https://review.openstack.org/158645 | 11:23 |
*** pensu has quit IRC | 11:23 | |
mrda | . | 11:29 |
openstackgerrit | Michael Davies proposed openstack/ironic: Add support for 'latest' in microversion header https://review.openstack.org/158601 | 11:29 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Fixes for issue found during functional testing https://review.openstack.org/158645 | 11:34 |
*** Nisha has quit IRC | 11:34 | |
*** naohirot has joined #openstack-ironic | 11:38 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Support for non-Glance image references https://review.openstack.org/136741 | 11:40 |
*** GheRivero has quit IRC | 11:41 | |
*** GheRivero has joined #openstack-ironic | 11:41 | |
openstackgerrit | Michael Davies proposed openstack/ironic: Add support for 'latest' in microversion header https://review.openstack.org/158601 | 11:45 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add functional tests for create https://review.openstack.org/158122 | 11:46 |
*** rameshg87 has quit IRC | 11:47 | |
*** Nisha has joined #openstack-ironic | 11:49 | |
*** saripurigopi has quit IRC | 11:49 | |
openstackgerrit | Michael Davies proposed openstack/ironic: Add support for 'latest' in microversion header https://review.openstack.org/158601 | 11:54 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE drivers support for IPA https://review.openstack.org/155728 | 11:54 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE driver: Deprecate pxe_deploy_{ramdisk, kernel} https://review.openstack.org/158644 | 11:54 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Local boot support for IPA https://review.openstack.org/157723 | 11:54 |
*** chlong has joined #openstack-ironic | 11:56 | |
*** kbs has joined #openstack-ironic | 12:02 | |
*** oomichi_ has quit IRC | 12:21 | |
*** takadayuiko has quit IRC | 12:29 | |
lucasagomes | jroll, JayF bug on devstack + agent https://bugs.launchpad.net/devstack/+bug/1425059 | 12:32 |
openstack | Launchpad bug 1425059 in devstack "Ironic: IPA fails to build the ramdisk" [Undecided,New] | 12:32 |
lucasagomes | right now our guidelines inlcude "IRONIC_BUILD_DEPLOY_RAMDISK=False" so we download the ramdisk, but building it with devstack is not working :( I will take a look when I get some time too | 12:33 |
lucasagomes | just pinging you to see if u have any idea, the log is in the bug ticket | 12:34 |
*** lucasagomes is now known as lucas-hungry | 12:36 | |
*** chlong has quit IRC | 12:44 | |
*** dlpartain has joined #openstack-ironic | 12:46 | |
*** dlpartain has quit IRC | 12:55 | |
*** athomas has joined #openstack-ironic | 12:56 | |
*** Nisha has quit IRC | 12:57 | |
*** jjohnson2 has joined #openstack-ironic | 12:58 | |
*** lazy_prince is now known as killer_prince | 12:59 | |
openstackgerrit | Merged openstack/ironic: Ensure state values are 15 characters or less https://review.openstack.org/158393 | 13:02 |
*** yog_ has joined #openstack-ironic | 13:04 | |
jroll | morning all :) | 13:06 |
TheJulia | good morning | 13:06 |
jroll | lucas-hungry: looks like your dns is bad or archive.ubuntu.com is down... dunno what you want me to do :P | 13:07 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: WIP: add module for in-band inspection using ironic-discoverd https://review.openstack.org/156562 | 13:12 |
dtantsur | jroll, TheJulia, morning | 13:16 |
jroll | morning dtantsur | 13:16 |
*** dprince has joined #openstack-ironic | 13:18 | |
lucas-hungry | jroll, heh nothing, just pointing you to the bug | 13:27 |
lucas-hungry | jroll, it's odd because I can build the ramdisk when not running with devstack | 13:28 |
lucas-hungry | on the same machine | 13:28 |
*** lucas-hungry is now known as lucasagomes | 13:28 | |
lucasagomes | so that's why I think it's a devstack misconfiguration or something | 13:28 |
jroll | lucasagomes: hrm, I did a recheck on an ipa patch to see if this is working in the gate | 13:28 |
jroll | lucasagomes: I just assume fedora null-routes archive.ubuntu.com :P | 13:29 |
lucasagomes | lol | 13:29 |
lucasagomes | maybe :P | 13:29 |
jroll | lucasagomes: this is the patch I rechecked https://review.openstack.org/#/c/157768/ | 13:30 |
lucasagomes | jroll, but you build the ramdisk in gate? or it downloads it? | 13:31 |
jroll | lucasagomes: in ipa, we build it | 13:31 |
jroll | or for ipa patches | 13:31 |
jroll | we build from source | 13:31 |
jroll | then after merge we build and upload a new ramdisk | 13:32 |
jroll | and ironic patches download that | 13:32 |
lucasagomes | I see, but it doesn't run with devstack does it? | 13:33 |
jroll | mmm. | 13:33 |
jroll | well, yes it does | 13:33 |
jroll | the agent_ssh-src job runs devstack | 13:33 |
lucasagomes | I see, it may be on fedora only then :( | 13:34 |
lucasagomes | I will take a look later see if I can find something | 13:34 |
*** ChuckC has joined #openstack-ironic | 13:45 | |
*** kbs has quit IRC | 13:49 | |
*** rloo has joined #openstack-ironic | 13:54 | |
*** kbs has joined #openstack-ironic | 13:54 | |
*** dlpartain has joined #openstack-ironic | 14:02 | |
*** dlpartain has left #openstack-ironic | 14:02 | |
*** rameshg87 has joined #openstack-ironic | 14:20 | |
*** rameshg87_ has joined #openstack-ironic | 14:28 | |
lucasagomes | check-tempest-dsvm-ironic-pxe_ssh: SUCCESS, gate fixed for the IPA patches :D | 14:29 |
*** rameshg87 has quit IRC | 14:31 | |
trown | dtantsur: is the func test on ironic-discoverd failing for you on master? | 14:34 |
* dtantsur is checking | 14:35 | |
dtantsur | trown, no. are you sure you got the latest master? | 14:35 |
trown | I just did a `git remote update; git pull` might be something odd here | 14:36 |
dtantsur | trown, what's your current commit and what's an error? | 14:37 |
trown | dtantsur, the problem was between the keyboard and chair | 14:38 |
*** BadCub has joined #openstack-ironic | 14:38 | |
trown | I must have missed git pull | 14:38 |
BadCub | Morning Ironic | 14:38 |
dtantsur | heh these Tuesday qualifies as Monday :D | 14:38 |
dtantsur | BadCub, morning | 14:39 |
dtantsur | * these = this | 14:39 |
trown | hehe | 14:39 |
victor_lowther | Morning Ironic | 14:39 |
dtantsur | victor_lowther, morning | 14:39 |
openstackgerrit | Victor Lowther proposed openstack/ironic: Add DRAC BIOS config vendor passthru API https://review.openstack.org/150488 | 14:40 |
*** jlvillal_sfo_air has joined #openstack-ironic | 14:40 | |
rloo | morning trown, BadCub, victor_lowther, lucasagomes, dtantsur | 14:41 |
*** jlvillal_sfo_air is now known as jlvllal_rem | 14:41 | |
lucasagomes | rloo, morning | 14:41 |
rloo | dtantsur: if this is monday, you going to work on friday too? :-) | 14:41 |
jlvllal_rem | Is the tip broken? Or is it just my tests which are failing? | 14:41 |
victor_lowther | if I could get some reviews on https://review.openstack.org/150488, that would be awesome. | 14:41 |
rloo | jlvllal_rem: i noticed one of your patches failing. are there others? | 14:41 |
trown | rloo, good morning | 14:42 |
jlvllal_rem | rloo: I think that might be on the python-ironicclient. The py34 test seems to die. | 14:42 |
jlvllal_rem | But I am more concerned with 'ironic'. | 14:42 |
jlvllal_rem | I will try a fresh clone and see what happens. I am getting an error at the moment | 14:43 |
rloo | jlvllal_rem: so your unit tests are failing? | 14:43 |
jlvllal_rem | rloo: Yes. And I did a 'git reset --hard' to get to the tip and still failing. | 14:44 |
jlvllal_rem | So I am doing a fresh 'git clone' and new tox run. | 14:44 |
rloo | jlvllal_rem: maybe rebuild your tox environment? | 14:44 |
jlvllal_rem | I will see what happens | 14:44 |
rloo | jlvllal_rem: ok. I just tried it on my dev, and it works. I think I rebuilt tox yesterday. | 14:45 |
jlvllal_rem | rloo: Okay great :) | 14:45 |
rloo | jlvllal_rem: things could break if an updated 3rd party package was released | 14:45 |
jlvllal_rem | Thanks. And you did a 'git pull' | 14:45 |
rloo | jlvllal_rem: yup. | 14:45 |
jlvllal_rem | Great. Thanks! | 14:45 |
jlvllal_rem | And it worked for me to. I guess my environment got funky | 14:46 |
rloo | good, that was an easy one to fix ;) | 14:46 |
vdrok | morning everyone | 14:46 |
jlvllal_rem | rloo: But the python-ironicclient gate seems broken. On the py34 test. | 14:46 |
jlvllal_rem | Maybe that has been fixed in the last 10 hours. | 14:47 |
rloo | jlvllal_rem: yeah. I -1'd that patch so why don't you fix it and try again. i don't have a py34 environment to try. it seems odd that it is consistently failing there. makes me think it has something to do with the change although i don't see how. | 14:47 |
vdrok | There is a question - it seems that at the moment we cannot download images without using cache for pxe driver | 14:47 |
vdrok | https://bugs.launchpad.net/ironic/+bug/1425105 | 14:48 |
openstack | Launchpad bug 1425105 in Ironic "Node can not be deployed if image cache is not used" [Undecided,New] - Assigned to Vladyslav Drok (vdrok) | 14:48 |
rloo | vdrok: thx for fixing it ;) | 14:48 |
vdrok | Should we restore this possibility? | 14:48 |
vdrok | rloo, I have not fixed it yet :) | 14:48 |
vdrok | rloo, I don't know should it be fixed or maybe that part of code can be removed | 14:49 |
jlvllal_rem | rloo: Thanks for the catch on E711. The perils of cut-n-paste | 14:49 |
rloo | vdrok: oh, you just reported that bug. I thought it had already been triaged etc. | 14:49 |
openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Change utils.get_ipmi_address(node) to always return IPv4 address https://review.openstack.org/157012 | 14:49 |
rloo | vdrok: why does it matter. why would you NOT want to use the cache? | 14:50 |
jroll | yeah, I'd like to hear a use case on that | 14:50 |
openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient: For flake8 check, make the 'E12' ignore be more granular https://review.openstack.org/157929 | 14:50 |
jroll | morning jlvllal_rem vdrok rloo :) | 14:50 |
rloo | morning jroll | 14:51 |
vdrok | I don't know, maybe if I do deployment only once? | 14:51 |
vdrok | morning jroll | 14:51 |
vdrok | this code for some reason allows it https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/image_cache.py#L89-L97 | 14:51 |
jlvllal_rem | jroll: morning | 14:52 |
vdrok | oh, and morning rloo :) | 14:52 |
rloo | afternoon vdrok! | 14:52 |
vdrok | morning jlvillal_rem | 14:52 |
mdbooth | jroll devananda: Do you have a moment to review https://review.openstack.org/#/c/158269/1, at least to express an opinion on whether it would interfere with Ironic? | 14:53 |
dtantsur | rloo, o/ | 14:53 |
rloo | vdrok: the code does change over time. I don't know the history/reason for that, so I can't answer. I suppose one could just delete that 'if' part if it never gets there | 14:55 |
rloo | vdrok: on the other hand, if in the future, someone has a usecase for not using the cache, that code might be useful. dunno. | 14:55 |
rloo | vdrok: so I don't see that the bug you opened is a bug. It just describes a behaviour of what the code is doing. It doesn't say why it shouldn't do that. | 14:56 |
NobodyCam | good morning Ironicers, says the man make'n coffee | 14:57 |
*** jjohnson2 has quit IRC | 14:57 | |
jroll | mdbooth: I haven't had enough coffee yet. ComputeNode.host == CONF.host, yes? | 14:57 |
*** kbs has quit IRC | 14:57 | |
mdbooth | jroll: ack | 14:57 |
jroll | cool | 14:57 |
vdrok | rloo, yeah, right, at first I though it's broken because we don't check that we can write to dest if it exists, but then I tried to deploy without cache and got this problem instead | 14:57 |
*** killer_prince is now known as lazy_prince | 14:58 | |
rloo | vdrok: so the code isn't easily understandable :-( | 14:58 |
jroll | mdbooth: +1'd, it won't break *my* deployment, I'm not sure about others but I think it's fine | 14:58 |
mdbooth | jroll: Cool, honesty in review :) | 15:00 |
jroll | yep :) | 15:01 |
*** jrist-afk is now known as jrist | 15:03 | |
naohirot | good morning all | 15:03 |
NobodyCam | morning naohirot :) | 15:03 |
naohirot | NobodyCam: Hi | 15:03 |
NobodyCam | :) | 15:03 |
naohirot | NobodyCam: I'd like to consult with you about iRMC priority | 15:04 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 15:05 |
naohirot | NobodyCam: I noticed that it seems iRMC has the lowest priority | 15:05 |
jroll | naohirot: devananda sets priorities | 15:05 |
naohirot | jroll: Okay, maybe I need to talk to devananda, but | 15:06 |
naohirot | jroll: NobodyCam: You know, my mission is to put fujitsu driver into the ironic | 15:06 |
*** alexpilotti has quit IRC | 15:07 | |
naohirot | jogo: NobodyCam: I need to hear from core team how this priority set. | 15:07 |
naohirot | naohirot: sorry s/jogo/jroll/ | 15:08 |
NobodyCam | naohirot: Jroll is core :) | 15:08 |
naohirot | NobodyCam: Yes, I know | 15:08 |
NobodyCam | and he is correct devananda set the priority | 15:08 |
jroll | I don't understand the question | 15:08 |
NobodyCam | :) | 15:08 |
naohirot | jroll: I need to explain why and how iRMC got the current priority. | 15:09 |
naohirot | jroll: to my company. | 15:09 |
*** kbs has joined #openstack-ironic | 15:09 | |
naohirot | jroll: I don't have any information. | 15:09 |
NobodyCam | naohirot: are you getting this from the Ironic Priorities & Planning spreed sheet? | 15:10 |
naohirot | jroll: From my point of view, everything has been decided in the place I cannot take part in. | 15:10 |
jroll | naohirot: then ask devananda, he sets the priorities. to the best of my knowledge, we don't have secret meetings or anything to decide them. | 15:11 |
jroll | if we do have them, then 1) I wasn't invited; and 2) that's bad and we should feel bad | 15:11 |
naohirot | NobodyCam: the priority sheet is also the source , but main source is https://launchpad.net/ironic/+milestone/kilo-3 | 15:11 |
jroll | holy cow, we have so much work to do | 15:12 |
rloo | jroll, naohirot: I think that is a fair question. How are the priorities set? | 15:12 |
* jroll is scared | 15:12 | |
jroll | rloo: devananda clicks things. | 15:12 |
dtantsur | I don't really disagree with drivers having lower priorities | 15:13 |
jroll | I assume they are based on things like, how many deployers will use this feature, etc | 15:13 |
dtantsur | we have too many principal things to work on/introduce etc | 15:13 |
rloo | jroll: I understand. but clearly people want to know. | 15:13 |
jroll | right. should ask deva. | 15:13 |
naohirot | dtantsur: my manager ask that why irmc has lower priority than iLO, AMT, or Drac. | 15:14 |
rloo | if left to me, the high priority things are infrastructure things. then the drivers. and I'd like to put some sort of tariff on the drivers; they need to participate in X reviews, for one review from me. | 15:14 |
rloo | but that's what I think internally, that isn't what I do ;) | 15:14 |
NobodyCam | naohirot: I believe the priorities are set on several factors, Importance to project, closeness to completition, amount of time needed to review, amount others (i'm sure) | 15:14 |
BadCub | I think NobodyCam's assumption to be pretty much on target | 15:15 |
NobodyCam | naohirot: just because I item does not have a high priority does not mean it won't land | 15:15 |
naohirot | NobodyCam: If so, I need those information to convince myself and my company. | 15:15 |
NobodyCam | that information is for the reviewers to plan there reviews | 15:15 |
rloo | NobodyCam: it doesn't mean it won't land, but the chances are lower because eg I don't have time to review all this stuff, so I'll review the higher priority things. | 15:15 |
dtantsur | naohirot, to be honest I never care about priorities in Launchpad when reviewing, so please don't overestimate them | 15:16 |
rloo | naohirot: you and everyone else can help by reviewing too. I'd like to see more reviewers that are non-core-reviewers. | 15:16 |
naohirot | NobodyCam: You know I submitted spec and code pretty much earlier, and revised as quickly as possible if I got comments. | 15:16 |
NobodyCam | rloo: yes, higher protity items are most likly "more difficult" to review | 15:16 |
jroll | yeah, I try to ignore priority as well | 15:17 |
NobodyCam | naohirot: yes I do know. | 15:17 |
naohirot | dtantsur: But my company is really cares about :) | 15:17 |
devananda | naohirot: because irmc has already landed a driver this cycle, so I feel the remaining work (vmedia support), while good, is lower priority than getting some attention for the ilo and drac drivers | 15:18 |
jroll | naohirot: as I understand it, priorities are just like... how critical to the project, etc. iRMC is not as critical to making ironic useful as say, cleaning or RAID | 15:18 |
dtantsur | naohirot, every company wants its stuff to land asap :) and I don't know why ilo is higher than irmc, but I do understand why generic changes and generic drivers are higher | 15:18 |
NobodyCam | good morning devananda :) | 15:18 |
devananda | naohirot: for comparison, in k1 and k2 cycles, we completed 5 and 4 blueprints, respectively | 15:18 |
jroll | morning deva :) | 15:18 |
NobodyCam | morning dtantsur :) | 15:18 |
dtantsur | morning, NobodyCam, devananda | 15:18 |
devananda | naohirot: we have 23 targeted to k3 .... | 15:18 |
BadCub | morning devananda | 15:18 |
naohirot | devananda: good morning :) | 15:18 |
devananda | I am still revising priorities as well | 15:19 |
naohirot | devananda: My company planned everything based on Kilo has iRMC 3 drivers. | 15:20 |
devananda | naohirot: as others have said, the priority is an indication from me to reviewers that "hey, this is more important to making Ironic as a whole project better, please spend more time over here" | 15:21 |
devananda | there are certain features which I personally really want, because my team needs them, which I've set as LOW proirity. for example, non-glance image refs | 15:21 |
devananda | naohirot: your company should not plan based on expectations about a community project | 15:22 |
naohirot | devananda: Okay, I understand your intention about the priority | 15:22 |
devananda | naohirot: you, like every other company, are participating in a community project. the best way to get your work landed is to help others land their work by reviewing it | 15:23 |
naohirot | devananda: As you know that company needs some reason when company change their plan. | 15:23 |
*** devlaps has joined #openstack-ironic | 15:24 | |
devananda | naohirot: sure. there's nothing preventing it from landing in kilo | 15:24 |
*** devlaps has quit IRC | 15:24 | |
naohirot | devananda: are you saying that nothing prevent from landing iRMC in kilo? | 15:25 |
devananda | naohirot: that is what I said | 15:25 |
naohirot | devananda: for instance, when iRMC management driver bumped from kilo-2 to kilo-3, I could not answer to my manager why it happened. | 15:26 |
naohirot | devananda: I'm realy worry about happing same thing from kilo-3 to L series. | 15:27 |
dtantsur | naohirot, because we have too many things to review and land... to top it all some people have their own priorities, which is unavoidable, as we all work for some company :) | 15:27 |
rloo | honestly. let's be realistic here. I will be. I only have so much time, and I can only review so much. So I tend to review based on priority. which to me means that I most likely will NOT get to naohirot's changes. | 15:28 |
devananda | naohirot: it bumped from k2 to k3 for a very simple reason -- it was not landed when the k2 deadline arrived | 15:28 |
naohirot | devananda: I know that core team is busy, therefore we need to discuss the priority and need consensus about the priority | 15:29 |
devananda | naohirot: code reviews take a lot of time. the core team is small and has a HUGE amount of reviews to do. if you want your code to land faster, you should review other developers' code too | 15:29 |
naohirot | devananda: Yes, I'll do that, no problem :) | 15:30 |
naohirot | devananda: I submitted my code as earlier as possible, so that core team can review enough before the dead line. | 15:31 |
naohirot | devananda: I think iRMC management code reached to certain level of quality, I tested it with Power driver using several fujitsu servers. | 15:33 |
*** anderbubble has joined #openstack-ironic | 15:36 | |
*** dprince has quit IRC | 15:36 | |
*** jcoufal has quit IRC | 15:37 | |
naohirot | devananda: I'd like you to review the management code before really you become busy for kilo-3 deadline | 15:37 |
*** jcoufal has joined #openstack-ironic | 15:37 | |
*** absubram has joined #openstack-ironic | 15:37 | |
*** jlvllal_rem has quit IRC | 15:40 | |
*** mgoddard1 has joined #openstack-ironic | 15:41 | |
* naohirot good night all | 15:42 | |
*** naohirot has quit IRC | 15:42 | |
*** mgoddard has quit IRC | 15:43 | |
*** david-lyle_afk is now known as david-lyle | 15:47 | |
openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Change utils.get_ipmi_address(node) to always return IPv4 address https://review.openstack.org/157012 | 15:48 |
rloo | vdrok: are you OK if I change your bug to invalid? https://bugs.launchpad.net/ironic/+bug/1425105 | 15:54 |
openstack | Launchpad bug 1425105 in Ironic "Node can not be deployed if image cache is not used" [Undecided,New] - Assigned to Vladyslav Drok (vdrok) | 15:54 |
*** hj-hp has joined #openstack-ironic | 15:55 | |
vdrok | rloo, idk, if you think that this functional is not needed then yes, i'll just put up a patch removing the code | 15:56 |
rloo | vdrok: i don't even know that we should delete the code. | 15:56 |
vdrok | rloo, i'll do it tomorrow, so we can wait to see what others think | 15:57 |
vdrok | rloo, but if you want, go ahead :) | 15:58 |
rloo | vdrok: for the reason that if you write a method 'foo' that does x or y, and method 'bar' calls 'foo' but only ever causes x, does that mean y should be deleted? | 15:58 |
vdrok | rloo, also because it seems that that piece does not work even if gets called | 15:59 |
vdrok | rloo, on rebuild | 15:59 |
rloo | vdrok: ok, if that piece doesn't work, then yes, it should be deleted ;) | 16:00 |
vdrok | rloo, I'll update the decription to be a bit clearer | 16:00 |
rloo | vdrok: in that case, you could reword the bug to say that it doesn't work even if it was possible to invoke it, so it should be deleted ;) | 16:01 |
rloo | vdrok: thx | 16:01 |
vdrok | rloo, np | 16:01 |
rameshg87_ | devananda, dtantsur, shouldn't we just lay down some better rules for vendor drivers | 16:02 |
rameshg87_ | devananda, dtantsur, just a suggestion | 16:02 |
rameshg87_ | devananda, dtantsur, for example: https://review.openstack.org/#/c/152857/5/ironic/drivers/modules/amt/management.py | 16:02 |
*** coolsvap is now known as coolsvap_ | 16:02 | |
rameshg87_ | nobody much gets what's _set_boot_device_order _enable_boot_config | 16:02 |
rameshg87_ | unless they know about amt hardware | 16:03 |
rameshg87_ | shouldn't the stuffs like set_boot_device, get_boot_device() be just a call to some library of the vendor module | 16:03 |
rameshg87_ | all these stuffs can stay there making the ironic driver module much simpler and easy to review | 16:04 |
rameshg87_ | wdyt ? | 16:04 |
NobodyCam | lucasagomes: is the Expose configdrive stuff all wrapped up? all reviews taged to the BP are merged! | 16:10 |
jlvillal | rloo: Any chance that the python34 test is broken on the python-ironicclient project? | 16:10 |
rloo | jlvillal: no idea. did it fail again? | 16:10 |
jlvillal | rloo: I'm just not sure how the config change to flake8 would break python34. | 16:10 |
jlvillal | Yes :( | 16:10 |
lucasagomes | NobodyCam, hey yes | 16:10 |
lucasagomes | NobodyCam, well need docs | 16:10 |
rloo | jlvillal: take a look at some other patches to the client to see if they failed too. | 16:10 |
lucasagomes | I will put some docs up, but code-wise it's good :) | 16:10 |
jlvillal | rloo: And if so, who can fix it? | 16:10 |
NobodyCam | ahh okay I'll update the spreed sheet | 16:10 |
NobodyCam | :) | 16:11 |
rloo | jlvillal: infra folks | 16:11 |
jlvillal | rloo: Thanks | 16:11 |
devananda | lucasagomes: and testing? | 16:11 |
lucasagomes | devananda, tempest tests? yeah hmm I could add that too | 16:11 |
devananda | lucasagomes: ta :) | 16:11 |
lucasagomes | devananda, lemme right down | 16:11 |
lucasagomes | devananda, btw, re IPA, gate patch is now merged and patches are passing | 16:12 |
rloo | jlvillal: looks like there is something wrong there. I see another patch failed there too. | 16:12 |
rloo | https://review.openstack.org/#/c/158520/ | 16:12 |
lucasagomes | you prefer to create a new job to test the IPA ramdisk, or switch from the existing one? | 16:12 |
jlvillal | rloo: Agreed | 16:12 |
* lucasagomes is working on the devstack integration | 16:12 | |
lucasagomes | NobodyCam, also, local boot need docs too :) | 16:13 |
lucasagomes | and root device hints | 16:13 |
*** dprince has joined #openstack-ironic | 16:13 | |
*** MattMan has left #openstack-ironic | 16:13 | |
lucasagomes | it's on my todo list, but I've been busy with some other bits and didn't put a patch up for that yet | 16:13 |
jlvillal | rloo: I'm asking over on #openstack-infra to see if anyone can help. Not sure if there is a particular person I should ping. | 16:13 |
*** MattMan has joined #openstack-ironic | 16:13 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/158750 | 16:14 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 16:16 |
jroll | jlvillal: 2015-02-24 15:37:27.114 | Build timed out (after 40 minutes). Marking the build as failed. | 16:17 |
NobodyCam | devananda: I'm going to add "Doc's" t the priorities and planning spread sheet? | 16:17 |
jroll | makes me hrm | 16:17 |
jlvillal | jroll: I'm over in #openstack-infra asking guy | 16:17 |
jroll | yeah, I see now | 16:18 |
jroll | I don't think it's an infra problem | 16:18 |
jlvillal | He is suggesting that something is wrong with the package. I'm thinking huh? | 16:18 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 16:18 |
jroll | clarkb is pretty good at this stuff fwiw :P | 16:18 |
jroll | it's likely a dependency that was updated | 16:18 |
jroll | did you run tox -r -epy34 locally? | 16:18 |
jlvillal | jroll: Not yet. My system didn't have python 34 | 16:19 |
openstackgerrit | Lucas Alvares Gomes proposed stackforge/ironic-discoverd: eDeploy plugin to not overwrite node's capabilities https://review.openstack.org/158757 | 16:19 |
jlvillal | I will try on my Trusty system | 16:19 |
jroll | yeah, I'd do that first | 16:19 |
*** rameshg87_ has quit IRC | 16:19 | |
jroll | I actually kind of wonder if testr is broken on 3.4, but kind of doubt it | 16:19 |
* lucasagomes thinks he have seen a problem with mock and 3.4 | 16:22 | |
lucasagomes | seems fixed now: https://bugs.launchpad.net/ubuntu/+source/python-mock/+bug/1283742 | 16:23 |
openstack | Launchpad bug 1283742 in python-mock (Ubuntu Trusty) "python-mock tests fail with python3.4" [High,Fix released] - Assigned to Barry Warsaw (barry) | 16:23 |
NobodyCam | dtantsur: just checking on 156562... woundering when it will be ready to un-WIP? | 16:23 |
dtantsur | NobodyCam, today-tomorrow when I write some tests and also test it in real world. Actually it should be rebased on top of Nisha's last patch, so that we can test it... | 16:24 |
dtantsur | NobodyCam, but you review would help ;) | 16:24 |
NobodyCam | :) ack will do :) | 16:25 |
NobodyCam | and awesome thank you :) | 16:25 |
*** vdrok is now known as vdrok_afk | 16:25 | |
openstackgerrit | Merged stackforge/ironic-discoverd: Change utils.get_ipmi_address(node) to always return IPv4 address https://review.openstack.org/157012 | 16:29 |
*** jcoufal has quit IRC | 16:30 | |
jlvillal | jroll: Am I doing something wrong? http://paste.openstack.org/show/181310/ My command: tox -v -v -epy34 | 16:31 |
jlvillal | jroll: I snipped out the stuff before the error. | 16:32 |
jroll | jlvillal: interesting, not sure | 16:32 |
jroll | I wonder if it's using 2.7 pbr or pip or something | 16:32 |
jlvillal | jroll: This is Ubuntu 14.04. Has been updated | 16:32 |
jroll | dunno, I don't tend to run 3.4 locally :/ | 16:33 |
jroll | or httpretty is broken or something | 16:33 |
jlvillal | jroll: Me either :) It is Python 3.4.0 | 16:33 |
* jroll distracted atm | 16:33 | |
jlvillal | jroll: Okay. Thanks | 16:33 |
NobodyCam | dtantsur: quick question: on line 132 of https://review.openstack.org/#/c/156562/3/ironic/drivers/modules/discoverd.py | 16:33 |
NobodyCam | why id and uuid? | 16:34 |
*** lazy_prince is now known as killer_prince | 16:35 | |
*** jcoufal has joined #openstack-ironic | 16:37 | |
dtantsur | NobodyCam, copy-paste to the win! :) | 16:44 |
NobodyCam | :) | 16:45 |
NobodyCam | heheh I commented on it | 16:45 |
NobodyCam | overall LGTM... couple of nits :-p | 16:45 |
dtantsur | yeah thanks! | 16:45 |
*** jcoufal has quit IRC | 16:47 | |
*** jcoufal has joined #openstack-ironic | 16:57 | |
*** mgoddard1 has quit IRC | 16:58 | |
*** mgoddard has joined #openstack-ironic | 16:59 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: WIP: add module for in-band inspection using ironic-discoverd https://review.openstack.org/156562 | 17:06 |
dtantsur | will be finishing ^^^ tomorrow | 17:06 |
NobodyCam | :) | 17:07 |
NobodyCam | you out for the rest of the day dtantsur ? | 17:07 |
dtantsur | NobodyCam, yeah, it's 6pm and it's that kind of Tuesday that feels like Monday | 17:07 |
dtantsur | so I'm finishing here | 17:07 |
NobodyCam | have a great night dtantsur :) | 17:07 |
dtantsur | g'night | 17:08 |
*** dtantsur is now known as dtantsur|afk | 17:08 | |
*** romcheg has quit IRC | 17:15 | |
openstackgerrit | Lucas Alvares Gomes proposed stackforge/ironic-discoverd: eDeploy plugin to not overwrite node's capabilities https://review.openstack.org/158757 | 17:17 |
*** jistr has quit IRC | 17:18 | |
*** rwsu-afk is now known as rwsu | 17:18 | |
*** erwan_taf has quit IRC | 17:19 | |
*** coolsvap_ is now known as coolsvap | 17:20 | |
lucasagomes | when people get some time and wants to review something in IPA please take a look at https://review.openstack.org/#/c/155727/ | 17:22 |
lucasagomes | thanks :) | 17:22 |
NobodyCam | lucasagomes: I'll take a look in a couple of minutes | 17:22 |
lucasagomes | NobodyCam, cheers, no rush | 17:22 |
lucasagomes | I'm working on the devstack now to enabled IPA + pxe_ssh to run | 17:23 |
openstackgerrit | Merged openstack/ironic: Add tests for ilo_deploy driver https://review.openstack.org/142985 | 17:23 |
lucasagomes | so you guys can test it as well | 17:23 |
devananda | lucasagomes: what's the status of the switch to IPA in our gate? | 17:23 |
devananda | lucasagomes: is there a patch up for devstack or ...? | 17:23 |
lucasagomes | devananda, should be all good | 17:23 |
lucasagomes | devananda, working on that now | 17:23 |
lucasagomes | so gate is unblocked now, we get our 1G | 17:24 |
lucasagomes | devananda, question is, are we creating a new job? or replacing the existing one with IPA? | 17:24 |
devananda | cool. ya, saw that land | 17:24 |
devananda | lucasagomes: oh. so dib should still run the old job in their gate | 17:24 |
lucasagomes | I will create a switch on devstack IRONIC_USE_IPA_RAMDISK=True/False | 17:24 |
lucasagomes | devananda, yeah, let's create another job for the IPA ramdisk, so we keep both running | 17:25 |
lucasagomes | running = being tested | 17:25 |
lucasagomes | devananda, btw if u wanna try locally, should be fairly easy | 17:25 |
devananda | jroll: planning to attend the cross project meeting today? | 17:26 |
devananda | jroll: there's going to be a discussion about release cycle management / cadence. based on your comments yesterday, you might be interested | 17:26 |
*** penick has joined #openstack-ironic | 17:26 | |
jroll | devananda: I have a conflict :/ | 17:26 |
jroll | but I'll be lurking | 17:26 |
devananda | jroll: see https://wiki.openstack.org/wiki/Release_Cycle_Management/Liberty_Tracking | 17:26 |
lucasagomes | apply this to ironic https://review.openstack.org/#/c/155728/ | 17:27 |
lucasagomes | build the IPA image with https://review.openstack.org/#/c/155727/ | 17:27 |
lucasagomes | and that's it, you will be able to use pxe_* with either ramdisk | 17:27 |
lucasagomes | no extra config | 17:27 |
lucasagomes | for those who wants to test ^ | 17:27 |
devananda | awesome | 17:27 |
*** viktors is now known as viktors|afk | 17:27 | |
JoshNang | sweet! | 17:27 |
jlvillal | jroll: Argh, that took awhile. So my issue was that I have LANG=C in my environment. Once I let it default to: LANG=en_US.UTF-8 then httpretty installed. | 17:27 |
devananda | TheJulia: ^ | 17:27 |
*** penick has quit IRC | 17:27 | |
jroll | jlvillal: fun | 17:28 |
jlvillal | jroll: Now to go to the next step... :) | 17:28 |
devananda | jroll: IIRC, ipa can pull an image from any source, not just glance/swift, right? | 17:32 |
*** athomas has quit IRC | 17:33 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning in DriverInterfaces https://review.openstack.org/157208 | 17:35 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning States https://review.openstack.org/153444 | 17:35 |
jroll | devananda: any http url | 17:35 |
devananda | jroll: thought so. ty | 17:36 |
jroll | np | 17:36 |
openstackgerrit | Merged openstack/ironic: Fix reboot logic of iRMC Power Driver https://review.openstack.org/158614 | 17:38 |
JoshNang | jroll: it does require a checksum of the image to be included though. https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/standby.py#L165 | 17:40 |
jroll | JoshNang: good point | 17:40 |
jroll | devananda: ^^ | 17:40 |
JoshNang | we could make it only check the checksum if the key is there | 17:41 |
devananda | ahh ty | 17:41 |
*** derekh has quit IRC | 17:44 | |
lucasagomes | [off-topic] awesome talk: http://www.ted.com/talks/miguel_nicolelis_brain_to_brain_communication_has_arrived_how_we_did_it | 17:45 |
NobodyCam | JoshNang: hahah I added a comment to ps 5 of 157208 just as you pushed up ps 6 | 17:45 |
NobodyCam | :0p | 17:45 |
JoshNang | NobodyCam: ha, no worries, that's an easy fix | 17:46 |
JoshNang | thanks :) | 17:46 |
NobodyCam | :-p | 17:46 |
*** Marga_ has joined #openstack-ironic | 17:48 | |
*** hj-hp has quit IRC | 17:48 | |
*** hj-hp has joined #openstack-ironic | 17:49 | |
jroll | states.VERBS are for API verbs yes? | 17:54 |
*** ParsectiX has joined #openstack-ironic | 17:54 | |
lucasagomes | jroll, confirmed, yeah the building IPA with devstack only affects fedora... I will investigate when I get some time | 17:54 |
jroll | lucasagomes: that's really strange | 17:54 |
lucasagomes | jroll, should be some fw rule, not sure | 17:54 |
lucasagomes | jroll, I can build IPA on fedora, if not running with devstack :) | 17:55 |
jroll | lucasagomes: you'd think if building docker ubuntu images was broken on fedora, someone would have said something | 17:55 |
jroll | oh | 17:55 |
jroll | gotcha | 17:55 |
jroll | maybe devstack configures docker weird | 17:55 |
lucasagomes | no, it's just some odd misconfiguration that devstack might be doing on fedora | 17:55 |
lucasagomes | jroll, yeah maybe that | 17:55 |
lucasagomes | gotta look, but I will leave it at side for now | 17:56 |
lucasagomes | the bug is open there too if someone wants to take a look | 17:56 |
jroll | I can't imagine they're firewalling archive.ubuntu.com or port 443 | 17:57 |
lucasagomes | jroll, yeah no idea, I worked around it by building the ramdisk myself and adding to files/* | 17:58 |
lucasagomes | :) | 17:58 |
lucasagomes | tested on ubuntu now just to see if happens there too, but apparently not | 17:58 |
jroll | yep :) | 17:58 |
lucasagomes | and with that said, I will call it a day | 17:59 |
lucasagomes | need a break :) | 17:59 |
lucasagomes | have a good night everyone! | 17:59 |
lucasagomes | NobodyCam, btw, lemme know if my reply satisfy ur comment in https://review.openstack.org/#/c/157723/ | 18:00 |
lucasagomes | cheers! | 18:00 |
*** lucasagomes is now known as lucas-dinner | 18:00 | |
jroll | night lucas-dinner :) | 18:00 |
*** coolsvap is now known as coolsvap_ | 18:00 | |
* victor_lowther ifarkas: Some replies to your comments. | 18:01 | |
*** spandhe has joined #openstack-ironic | 18:01 | |
NobodyCam | have a good night lucas-dinner | 18:03 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Missing mock causing long tests https://review.openstack.org/158799 | 18:03 |
*** Nisha has joined #openstack-ironic | 18:03 | |
JoshNang | easy review ^ | 18:03 |
*** jcoufal has quit IRC | 18:04 | |
*** jcoufal has joined #openstack-ironic | 18:04 | |
jroll | JoshNang: how was that passing before :x | 18:04 |
JoshNang | no idea | 18:04 |
JayF | shouldn't we find out? lol | 18:04 |
JoshNang | but when i rebased one of my patches, all the sudden they were taking...quite awhile | 18:05 |
JoshNang | ohh, set_failed_state catching the exception if it can't power off the node | 18:05 |
jroll | right, and this test expects a deploy fail | 18:06 |
jroll | ya | 18:06 |
JayF | cool | 18:06 |
* jroll +2 | 18:06 | |
*** ndipanov has quit IRC | 18:07 | |
*** jcoufal has quit IRC | 18:09 | |
jlvillal | jroll: I'm going to leave it to a better developer than me to figure it out. I have almost zero knowledge of testr, so unsure how to proceed :( Since it is tip of tree, I would assume something in the testing environment has changed??? | 18:09 |
jroll | jlvillal: yeah, probably a bumped dependency tbh | 18:10 |
*** devlaps has joined #openstack-ironic | 18:11 | |
*** mgoddard1 has joined #openstack-ironic | 18:13 | |
*** mgoddard has quit IRC | 18:15 | |
*** mgoddard1 has quit IRC | 18:19 | |
*** hj-hp has quit IRC | 18:19 | |
*** hj-hp has joined #openstack-ironic | 18:19 | |
*** harlowja has joined #openstack-ironic | 18:20 | |
*** ChuckC has quit IRC | 18:20 | |
*** ChuckC has joined #openstack-ironic | 18:21 | |
*** MattMan has left #openstack-ironic | 18:22 | |
*** harlowja has quit IRC | 18:24 | |
*** harlowja has joined #openstack-ironic | 18:24 | |
NobodyCam | gah: +1219 lines | 18:29 |
*** Marga_ has quit IRC | 18:32 | |
rloo | devananda: I'm a procrastinator. I had offered to look at resource-locking in kilo. Should I bother, or is it better for me to spend time on the existing priorities? | 18:32 |
*** Marga_ has joined #openstack-ironic | 18:32 | |
*** mgoddard has joined #openstack-ironic | 18:32 | |
*** Nisha_away has joined #openstack-ironic | 18:34 | |
*** Nisha has quit IRC | 18:34 | |
jlvillal | Nisha: I added some extra testing to your patch series. Hopefully you like it :) | 18:35 |
*** penick has joined #openstack-ironic | 18:35 | |
*** yginger5 is now known as yjiang5 | 18:35 | |
*** priteau has quit IRC | 18:36 | |
*** ParsectiX has quit IRC | 18:37 | |
*** ParsectiX has joined #openstack-ironic | 18:38 | |
*** ParsectiX has quit IRC | 18:41 | |
devananda | rloo: by resource locking, you mean the distributed locking with non-mysql backend? | 18:41 |
*** ParsectiX has joined #openstack-ironic | 18:41 | |
rloo | devananda: I mean actually looking at what was discussed at the kilo summit and seeing what might be doable | 18:42 |
devananda | jroll: since you have strong feelings on it, could you go comment on 157757 in the next hour? | 18:42 |
devananda | jroll: iow, before the TC meeting where we discuss it | 18:42 |
rloo | devananda: https://etherpad.openstack.org/p/kilo-ironic-resource-locking | 18:43 |
devananda | jroll: because folks are going to ask, very directly, why we're not proposing it to oslo | 18:43 |
devananda | jroll: and I don't have strong feelings on it, so I won't defend it | 18:43 |
devananda | rloo: oh, right | 18:43 |
devananda | rloo: yea, punt on it. too late in the cycle now. no one felt strongly enough to write a spec or really work on it | 18:44 |
rloo | devananda: yeah, I had forgotten most of that. (Good thing we took notes.) | 18:44 |
devananda | rloo: except for the rackspace folks who wrote the zookeeper backend | 18:44 |
rloo | devananda: ok. i'll focus on the existing priorities then. | 18:44 |
rloo | devananda: thx | 18:44 |
jroll | devananda: great... | 18:45 |
* rloo thinks it sometimes helps to procrastinate | 18:45 | |
devananda | bbiab | 18:45 |
*** krtaylor has quit IRC | 18:48 | |
*** achanda has joined #openstack-ironic | 18:52 | |
*** penick has quit IRC | 18:53 | |
*** penick_ has joined #openstack-ironic | 18:53 | |
Nisha_away | jlvillal, there | 18:56 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection https://review.openstack.org/149823 | 18:58 |
*** krtaylor has joined #openstack-ironic | 19:00 | |
jroll | devananda: commented on 157757 | 19:01 |
*** jlvillal_remote has quit IRC | 19:04 | |
openstackgerrit | Victor Lowther proposed openstack/ironic: Add DRAC BIOS config vendor passthru API https://review.openstack.org/150488 | 19:14 |
*** Nisha_away has quit IRC | 19:18 | |
*** Nisha_away has joined #openstack-ironic | 19:19 | |
*** hj-hp has quit IRC | 19:19 | |
*** ChuckC_ has joined #openstack-ironic | 19:23 | |
*** hj-hp has joined #openstack-ironic | 19:24 | |
*** penick_ has quit IRC | 19:24 | |
*** ChuckC has quit IRC | 19:25 | |
*** ChuckC has joined #openstack-ironic | 19:26 | |
openstackgerrit | Merged openstack/ironic-python-agent: Add additional logging and return results for IPA extensions https://review.openstack.org/152755 | 19:27 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning States https://review.openstack.org/153444 | 19:27 |
*** ChuckC_ has quit IRC | 19:28 | |
*** hj-hp has quit IRC | 19:30 | |
*** hj-hp has joined #openstack-ironic | 19:31 | |
*** achanda has quit IRC | 19:33 | |
*** devlaps has quit IRC | 19:33 | |
*** dprince has quit IRC | 19:35 | |
*** achanda has joined #openstack-ironic | 19:41 | |
*** hj-hp has quit IRC | 19:44 | |
*** trown is now known as trown|brb | 19:50 | |
NobodyCam | brb | 19:51 |
ifarkas | victor_lowther, ok, thanks! I will check it tomorrow morning | 20:04 |
*** hj-hp has joined #openstack-ironic | 20:13 | |
*** hj-hp has quit IRC | 20:14 | |
*** trown|brb is now known as trown | 20:16 | |
*** Marga_ has quit IRC | 20:18 | |
*** hj-hp has joined #openstack-ironic | 20:19 | |
*** Marga_ has joined #openstack-ironic | 20:19 | |
*** datajerk1 has joined #openstack-ironic | 20:21 | |
*** ifarkas has quit IRC | 20:23 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Add documentation for VirtualBox drivers https://review.openstack.org/146307 | 20:24 |
*** datajerk has quit IRC | 20:25 | |
*** rfchapman has joined #openstack-ironic | 20:26 | |
rfchapman | Hello all | 20:26 |
NobodyCam | hey hey rfchapman :) | 20:27 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning in DriverInterfaces https://review.openstack.org/157208 | 20:28 |
rfchapman | NobodyCam: is it possible for an image to become corrupted in some way? I created a fedora 20 image several days ago and it seemed to be working with the hardware i'm using (hp dl365g1). It was failing late into the image install but was getting futher along than it is now. Now it doesn't even complete the first stage of baremetal install. It hangs at "uhci_hcd 0000:00:04.4: HCRESET not completed yet!" System Halted | 20:28 |
NobodyCam | rfchapman: I suppose it could :/ | 20:30 |
*** hj-hp has quit IRC | 20:30 | |
*** Marga_ has quit IRC | 20:30 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement Cleaning in DriverInterfaces https://review.openstack.org/157208 | 20:31 |
NobodyCam | rfchapman: your not even booting into the deploy image? if I understand correct? | 20:31 |
rfchapman | NobodyCam: Yesterday it was failing at the partitioning, I found that the disks were 73Gig drives and not 80Gig. I created another flavor with 50gig drive and then it started haning at he forementioned spot. | 20:32 |
rfchapman | NobodyCam: correct. I'ts the initrd that fails. | 20:33 |
NobodyCam | hummm... How did you create the deployment image (kernel and ramdisk) | 20:33 |
rfchapman | nothing else has changed. | 20:34 |
NobodyCam | may be worth recrating | 20:34 |
NobodyCam | recreating even | 20:34 |
rfchapman | I created it with the diskimage-builder as outlined in the ironic baremetal installation page. | 20:36 |
*** faizan has joined #openstack-ironic | 20:39 | |
*** Nisha_away is now known as Nisha | 20:40 | |
rfchapman | NobodyCam: So you think I should re-create the image and try again? | 20:41 |
jlvillal | jroll: Made a little progress. Appears to be: ironicclient.tests.test_shell.ShellTestNoMox.test_node_list which is hanging | 20:42 |
jlvillal | Why I don't know... | 20:42 |
NobodyCam | rfchapman: couldn't hurt | 20:42 |
jroll | jlvillal: perhaps the client isn't mocked properly or something? | 20:42 |
jroll | jlvillal: e.g. hanging trying to reach the server | 20:42 |
jlvillal | jroll: I will look. Thanks. | 20:42 |
NobodyCam | rfchapman: may just reload the images in glance? | 20:42 |
jroll | jlvillal: unclear why that would have ever passed but yeah | 20:42 |
rfchapman | NobodyCam: OK, I'll try that just after grab a bite to eat. brb | 20:43 |
NobodyCam | :) | 20:43 |
jlvillal | jroll: The two NoMox tests seem to hang. | 20:44 |
jlvillal | investigating... | 20:44 |
jroll | maybe mox is ded on 3.4 | 20:44 |
jroll | or wait, that's NoMox | 20:44 |
faizan | devananda: jroll: NobodyCam: hello all, Please take look at https://review.openstack.org/#/c/157636/ for adding "ironic-lib" to openstack | 20:45 |
faizan | jroll: Thanks for your comments on https://review.openstack.org/#/c/157757/ | 20:46 |
*** Nisha has quit IRC | 20:46 | |
NobodyCam | faizan: :) will do | 20:47 |
devananda | faizan: that is going to be discussed by the TC soon, if we have time today | 20:47 |
devananda | faizan: jroll: please join #openstack-meeting -- though I'm not sure we will get to it today, so it may have to wait for next week | 20:48 |
*** marzif_ has joined #openstack-ironic | 20:49 | |
jroll | devananda: I have another meeting, but I'll be lurking and may speak up | 20:50 |
jroll | sometimes I wish I didn't have opinions | 20:50 |
devananda | heh | 20:52 |
mrda | Morning Ironic | 20:56 |
NobodyCam | morning mrda :) | 20:56 |
devananda | looks like no time | 20:57 |
*** hj-hp has joined #openstack-ironic | 20:58 | |
jroll | I really don't understand the point of oslo | 21:01 |
jroll | is it so that common code can have a PTL? | 21:02 |
devananda | jroll: so that we don't all create different db abstraction layers | 21:02 |
devananda | jroll: and different messaging layers. and different service libraries. and different config parsing code. | 21:02 |
devananda | .... and ... | 21:02 |
jroll | well sure, but why does a common library need to be called oslo | 21:02 |
devananda | it doesn't. unless all of openstack is going to use it | 21:02 |
mrda | jroll: someone has to be responsible for, and nuture the common code | 21:03 |
devananda | if it really is just part of ironic, great - it should be in our domain | 21:03 |
devananda | if it's code that other projects will want/need, it should be in the designated place to share that sort of thing | 21:03 |
mrda | ...it's better than copying and pasting the same code all around the place | 21:03 |
devananda | I think that's al lthe TC folks are asking - a little diligence in answering that | 21:03 |
jroll | why can we not have an oslo program that houses a bunch of libraries | 21:03 |
jroll | and if a library starts being used by multiple projects, move it to the right umbrella | 21:04 |
devananda | huh? | 21:04 |
jroll | like | 21:04 |
jroll | in this particular case: | 21:04 |
jroll | put ironic-lib or whatever in ironic | 21:04 |
jroll | if another project starts using it, then move it to the oslo program | 21:05 |
jroll | the name doesn't need to change | 21:05 |
*** mgoddard has quit IRC | 21:11 | |
*** andreykurilin_ has joined #openstack-ironic | 21:12 | |
*** faizan has quit IRC | 21:12 | |
devananda | BadCub: you're missing the project meeting | 21:13 |
devananda | BadCub: and I just mentioned you | 21:13 |
jroll | BadCub: he said you're bad | 21:13 |
mrda | lol | 21:13 |
*** andreykurilin_ has quit IRC | 21:17 | |
*** andreykurilin__ has joined #openstack-ironic | 21:17 | |
BadCub | devananda I know man... I should really get up there | 21:17 |
BadCub | like immediately! | 21:17 |
NobodyCam | hehehe | 21:17 |
devananda | BadCub: it is in #openstack-meeting ... | 21:18 |
BadCub | on my way | 21:18 |
*** ParsectiX has quit IRC | 21:19 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Root partition should be marked as bootable https://review.openstack.org/156587 | 21:19 |
*** ParsectiX has joined #openstack-ironic | 21:19 | |
*** andreykurilin_ has joined #openstack-ironic | 21:22 | |
*** andreykurilin__ has quit IRC | 21:22 | |
*** jcoufal has joined #openstack-ironic | 21:25 | |
jlvillal | GheRivero: ping | 21:28 |
NobodyCam | jlvillal: I hope he's sleeping now.. | 21:29 |
NobodyCam | I believe he's back home | 21:29 |
jlvillal | NobodyCam: Okay. I was trying to ask him or David Hu a question on a unit test in python-ironicclient. As I think they are the authors. | 21:29 |
jlvillal | And it is the test which seems to be hanging the Python 3.4 test. I don't know why it is hanging. | 21:30 |
*** Marga_ has joined #openstack-ironic | 21:32 | |
*** marzif_ has quit IRC | 21:32 | |
*** ijw has joined #openstack-ironic | 21:36 | |
*** sdake has quit IRC | 21:37 | |
*** Marga_ has quit IRC | 21:40 | |
*** Marga_ has joined #openstack-ironic | 21:41 | |
*** ParsectiX has quit IRC | 21:41 | |
*** Marga_ has quit IRC | 21:42 | |
*** ParsectiX has joined #openstack-ironic | 21:42 | |
*** Marga_ has joined #openstack-ironic | 21:42 | |
*** Marga_ has quit IRC | 21:42 | |
*** Marga_ has joined #openstack-ironic | 21:43 | |
*** Marga_ has quit IRC | 21:44 | |
*** Marga_ has joined #openstack-ironic | 21:44 | |
*** Marga_ has quit IRC | 21:45 | |
*** Marga_ has joined #openstack-ironic | 21:46 | |
*** Marga_ has quit IRC | 21:47 | |
*** Marga_ has joined #openstack-ironic | 21:47 | |
*** sdake has joined #openstack-ironic | 21:47 | |
*** sdake has quit IRC | 21:48 | |
*** Marga_ has quit IRC | 21:51 | |
*** Marga_ has joined #openstack-ironic | 21:52 | |
*** Marga_ has quit IRC | 21:53 | |
*** sdake has joined #openstack-ironic | 21:53 | |
*** Marga_ has joined #openstack-ironic | 21:53 | |
*** Marga_ has quit IRC | 21:54 | |
*** Marga_ has joined #openstack-ironic | 21:54 | |
mordred | jroll: there are libs in oslo that are not called oslo | 22:08 |
mordred | jroll: pbr springs to mind | 22:08 |
openstackgerrit | Michael Davies proposed openstack/ironic: Add support for 'latest' in microversion header https://review.openstack.org/158601 | 22:08 |
*** Hefeweizen has quit IRC | 22:10 | |
*** pelix has quit IRC | 22:10 | |
NobodyCam | mrda: oh cool ^^^^ | 22:10 |
NobodyCam | and hey hey mordred :) | 22:12 |
jroll | mordred: that's what I'm sayin | 22:12 |
*** ijw has quit IRC | 22:13 | |
openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient: Do not use httpretty 0.8.7 as it breaks testing https://review.openstack.org/158893 | 22:19 |
jlvillal | jroll: I think I fixed it :) ^^^^ | 22:20 |
jroll | woot | 22:20 |
jlvillal | Okay, "fixed" may be too strong a word ;) | 22:20 |
* jlvillal has no idea what the the root cause of httpretty breaking is. | 22:21 | |
NobodyCam | devananda: BadCub: I think the blueprint : https://blueprints.launchpad.net/ironic/+spec/irmc-virtualmedia-deploy-driver is mismarked as approved. | 22:21 |
jroll | httpretty>=0.8.0,!=0.8.1,!=0.8.2,!=0.8.3,!=0.8.7 | 22:21 |
jroll | must be great software | 22:21 |
NobodyCam | lol | 22:22 |
NobodyCam | ya | 22:22 |
BadCub | .me goes to look now | 22:22 |
jroll | NobodyCam: agree, https://review.openstack.org/#/c/134865/ is the spec | 22:22 |
mrda | jroll: Just means they made a few oopses on release | 22:22 |
mrda | not an excuse though | 22:23 |
jroll | mrda: that's a lot of oppses | 22:23 |
jroll | idk | 22:23 |
BadCub | NobodyCam that one is not on my spreadsheet yet. hmmmm | 22:23 |
BadCub | https://blueprints.launchpad.net/ironic/+spec/inband-raid-configuration i supdated to pending approval | 22:25 |
jlvillal | jroll: :) Probably could clean that up to say >=0.8.4,!=0.8.7 | 22:26 |
jroll | jlvillal: meh | 22:26 |
jroll | jlvillal: you may need to patch global-requirements, btw | 22:26 |
* BadCub hatesw IceChat copy/past features. | 22:27 | |
jlvillal | jroll: Where is that? What is that? | 22:27 |
jroll | jlvillal: uh oh | 22:27 |
jroll | you're gonna hate this | 22:27 |
* jlvillal prepares himself... | 22:27 | |
jroll | jlvillal: https://github.com/openstack/requirements#global-requirements-for-openstack-projects | 22:27 |
jroll | so you'd have to patch https://github.com/openstack/requirements/blob/master/global-requirements.txt#L206 | 22:28 |
jlvillal | There be a lot of words in that there document ;) | 22:28 |
jlvillal | jroll: Okay. I will work on a patch for that. Thanks! | 22:29 |
BadCub | NobodyCam take care of. | 22:30 |
NobodyCam | :) Thank you BadCub :) | 22:30 |
jroll | jlvillal: awesome, thanks :) | 22:30 |
BadCub | NP. devananda and I talked about this one this morning, hence it not being on my "approved" spreadsheet. :-p | 22:31 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Root partition should be marked as bootable https://review.openstack.org/156587 | 22:33 |
jlvillal | jroll: https://review.openstack.org/#/c/158897/ | 22:34 |
*** jcoufal has quit IRC | 22:35 | |
*** Marga_ has quit IRC | 22:35 | |
*** Marga_ has joined #openstack-ironic | 22:36 | |
*** openstackgerrit has quit IRC | 22:37 | |
*** openstackgerrit has joined #openstack-ironic | 22:37 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 22:40 |
jroll | jlvillal: awesome. fwiw the infra team mostly reviews that, you might poke clarkb again with that patch since he'll be interested what broke | 22:41 |
jlvillal | jroll: I subtly (I hope) just did :) | 22:41 |
jlvillal | jroll: Amazing how long it took me to come up with one line code change :( | 22:42 |
*** kbs has quit IRC | 22:42 | |
jlvillal | The joy of software :D | 22:43 |
jroll | jlvillal: such is software | 22:43 |
jroll | heh | 22:43 |
jroll | :D | 22:43 |
openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient: Do not use httpretty 0.8.7 as it breaks testing https://review.openstack.org/158893 | 22:45 |
*** Marga_ has quit IRC | 22:49 | |
*** Marga_ has joined #openstack-ironic | 22:49 | |
*** achanda has quit IRC | 22:54 | |
*** hj-hp has quit IRC | 22:55 | |
*** hj-hp has joined #openstack-ironic | 22:57 | |
*** lucas-dinner has quit IRC | 22:59 | |
*** harlowja is now known as harlowja_away | 23:02 | |
NobodyCam | doc patch: https://review.openstack.org/#/c/146307 has one +2 | 23:04 |
*** harlowja_away is now known as harlowja | 23:05 | |
*** achanda has joined #openstack-ironic | 23:13 | |
jlvillal | If anyone wants a review, just let me know. Otherwise I will see what is on my watched list. | 23:18 |
*** purp has joined #openstack-ironic | 23:19 | |
*** hj-hp has quit IRC | 23:19 | |
*** chlong has joined #openstack-ironic | 23:19 | |
jroll | jlvillal: I got an easy one coming :D | 23:22 |
jlvillal | jroll: Okay! I can handle an easy one, I hope ;) | 23:23 |
*** spandhe has quit IRC | 23:23 | |
jroll | there you are sir | 23:23 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Remove unused code from agent vendor lookup() https://review.openstack.org/158912 | 23:23 |
jroll | enjoy | 23:23 |
jroll | it should pass tests, I didn't run locally | 23:24 |
* NobodyCam -1's just for fun | 23:24 | |
* NobodyCam also ducks | 23:24 | |
NobodyCam | lol | 23:24 |
jroll | x.x | 23:24 |
NobodyCam | hehehe | 23:24 |
*** andreykurilin_ has quit IRC | 23:28 | |
NobodyCam | humm jroll my only comment is I would ask lucas what he inteneded. as he just added that 12 days ago. | 23:29 |
jroll | NobodyCam: he refactored that code to another file | 23:30 |
jroll | that was from us and probably remnants of crappy downstream code | 23:30 |
*** anderbubble has quit IRC | 23:31 | |
NobodyCam | ya looking at the commit now | 23:32 |
NobodyCam | lol great way to folks to read your email. add "Free beer and pizza!" to the title | 23:35 |
*** absubram has quit IRC | 23:36 | |
jroll | heh | 23:37 |
*** ijw has joined #openstack-ironic | 23:38 | |
NobodyCam | jroll: is it okay that i'm sad you didn't have to modify any tests to remove that :( | 23:41 |
*** ijw has quit IRC | 23:44 | |
jlvillal | Is there a way to have your patch depend on another patch that is in a different project? Maybe a line added to the commit message? | 23:46 |
NobodyCam | jlvillal: I hear tell there is | 23:46 |
jlvillal | We had a "Depends-Change-Id: XXXXXXXXXXXX" thing in my last build system | 23:47 |
jroll | NobodyCam: it's bad, but now there's less code not tested :) | 23:47 |
jroll | jlvillal: Depends-On: $CHANGE_ID | 23:47 |
jlvillal | jroll: Thanks | 23:47 |
NobodyCam | :) | 23:47 |
NobodyCam | jroll: :) | 23:47 |
jlvillal | jroll: I will try that with my python-ironicclient patch :) | 23:47 |
jroll | jlvillal: awesome | 23:48 |
jroll | (it works, I've used it, it's fantastic) | 23:48 |
*** achanda has quit IRC | 23:48 | |
openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient: Do not use httpretty 0.8.7 as it breaks testing https://review.openstack.org/158893 | 23:49 |
*** chlong_ has joined #openstack-ironic | 23:50 | |
jlvillal | jroll: Cool. I see it in the Zuul page with the two patches :) | 23:50 |
*** achanda has joined #openstack-ironic | 23:53 | |
NobodyCam | :) kewl | 23:54 |
NobodyCam | j* just fyi here a link to the free beer and pizza: http://lists.openstack.org/pipermail/openstack/2015-February/011660.html | 23:58 |
NobodyCam | it's up by you | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!