*** jgrimm is now known as zz_jgrimm | 00:09 | |
tteggel_ | anyone fancy a nice easy stable/juno +W ;) https://review.openstack.org/#/c/149959 | 00:11 |
---|---|---|
openstackgerrit | Chris Krelle proposed openstack/ironic: Fix typos in commit I68c9f9f86f5f113bb111c0f4fd83216ae0659d36 https://review.openstack.org/151412 | 00:12 |
*** tteggel_ is now known as tteggel | 00:14 | |
*** tteggel has quit IRC | 00:14 | |
*** tteggel has joined #openstack-ironic | 00:14 | |
NobodyCam | +W? | 00:15 |
NobodyCam | wow more changes in conf.sample then I expected? | 00:16 |
NobodyCam | oh I know | 00:18 |
*** zer0c00l has quit IRC | 00:21 | |
*** zer0c00l has joined #openstack-ironic | 00:22 | |
*** penick has quit IRC | 00:24 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Fix typos in commit I68c9f9f86f5f113bb111c0f4fd83216ae0659d36 https://review.openstack.org/151412 | 00:25 |
*** jjohnson2 has joined #openstack-ironic | 00:29 | |
*** smoriya has joined #openstack-ironic | 00:30 | |
*** david-lyle is now known as david-lyle_afk | 00:31 | |
*** ijw has quit IRC | 00:35 | |
*** derekh has joined #openstack-ironic | 00:39 | |
*** zhenzanz has joined #openstack-ironic | 00:39 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic https://review.openstack.org/134865 | 00:41 |
*** penick has joined #openstack-ironic | 00:41 | |
*** penick has quit IRC | 00:42 | |
*** Haomeng|2 has joined #openstack-ironic | 00:44 | |
*** Haomeng has quit IRC | 00:45 | |
*** spandhe has quit IRC | 00:46 | |
*** Nisha has quit IRC | 00:48 | |
*** bradjones has quit IRC | 00:56 | |
*** penick has joined #openstack-ironic | 00:57 | |
*** penick has quit IRC | 00:58 | |
*** chenleji has quit IRC | 00:58 | |
jroll | devananda: didn't get around to those patches today, sorry | 00:58 |
*** chenleji has joined #openstack-ironic | 00:58 | |
jiangfei | devananda: hi | 00:59 |
*** bradjones has joined #openstack-ironic | 01:00 | |
*** Marga_ has joined #openstack-ironic | 01:05 | |
*** derekh has quit IRC | 01:10 | |
*** igordcard has joined #openstack-ironic | 01:19 | |
*** achanda_ has joined #openstack-ironic | 01:19 | |
*** achanda has quit IRC | 01:23 | |
*** jjohnson2 has quit IRC | 01:23 | |
*** achanda_ has quit IRC | 01:24 | |
*** igordcard has quit IRC | 01:27 | |
*** Guest63943 has quit IRC | 01:38 | |
*** Marga_ has quit IRC | 01:44 | |
*** Marga_ has joined #openstack-ironic | 01:45 | |
*** ChuckC_ has quit IRC | 01:48 | |
*** ChuckC_ has joined #openstack-ironic | 01:49 | |
*** chenglch has joined #openstack-ironic | 01:50 | |
*** bradjones has quit IRC | 02:00 | |
*** bradjones has joined #openstack-ironic | 02:01 | |
*** nosleep77 has quit IRC | 02:07 | |
*** nosleep77 has joined #openstack-ironic | 02:11 | |
*** Marga_ has quit IRC | 02:12 | |
*** jerryz has joined #openstack-ironic | 02:13 | |
*** Marga_ has joined #openstack-ironic | 02:13 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add missing exceptions into function docstring https://review.openstack.org/151507 | 02:19 |
*** Marga__ has joined #openstack-ironic | 02:26 | |
*** Marga_ has quit IRC | 02:26 | |
*** Marga_ has joined #openstack-ironic | 02:27 | |
*** penick has joined #openstack-ironic | 02:27 | |
*** Marga__ has quit IRC | 02:31 | |
*** yog_ has joined #openstack-ironic | 02:40 | |
*** penick has quit IRC | 02:40 | |
*** ramineni has joined #openstack-ironic | 02:43 | |
*** rwsu is now known as rwsu-afk | 02:48 | |
*** zhenzanz_ has joined #openstack-ironic | 02:50 | |
*** zhenzanz has quit IRC | 02:52 | |
*** zhenzanz_ is now known as zhenzanz | 02:52 | |
*** anderbubble has joined #openstack-ironic | 02:53 | |
*** rloo has quit IRC | 02:53 | |
*** harlowja_ is now known as harlowja_away | 02:58 | |
*** bradjones has quit IRC | 03:00 | |
*** bradjones has joined #openstack-ironic | 03:03 | |
*** yog_ has quit IRC | 03:12 | |
*** adam_g_out is now known as adam_g | 03:29 | |
*** bradjones has quit IRC | 03:30 | |
*** bradjones has joined #openstack-ironic | 03:36 | |
*** Marga_ has quit IRC | 03:41 | |
*** Marga_ has joined #openstack-ironic | 03:49 | |
*** penick has joined #openstack-ironic | 03:54 | |
*** ChuckC_ has quit IRC | 03:58 | |
*** ChuckC_ has joined #openstack-ironic | 03:59 | |
*** coolsvap|afk is now known as coolsvap | 04:03 | |
*** lazy_prince has quit IRC | 04:06 | |
*** killer_prince has joined #openstack-ironic | 04:07 | |
*** killer_prince is now known as lazy_prince | 04:07 | |
*** pensu has joined #openstack-ironic | 04:08 | |
*** yog_ has joined #openstack-ironic | 04:21 | |
*** rameshg87 has joined #openstack-ironic | 04:24 | |
openstackgerrit | Sirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images https://review.openstack.org/150142 | 04:28 |
*** anderbubble has quit IRC | 04:33 | |
*** achanda has joined #openstack-ironic | 04:36 | |
*** achanda has quit IRC | 04:40 | |
*** penick_ has joined #openstack-ironic | 04:42 | |
*** penick has quit IRC | 04:45 | |
*** penick_ is now known as penick | 04:45 | |
*** achanda has joined #openstack-ironic | 04:49 | |
*** pcrews has quit IRC | 04:56 | |
*** rakesh_hs has joined #openstack-ironic | 05:04 | |
*** anderbubble has joined #openstack-ironic | 05:04 | |
*** anderbubble has quit IRC | 05:06 | |
*** penick has quit IRC | 05:17 | |
*** pensu has quit IRC | 05:17 | |
*** achanda has quit IRC | 05:20 | |
*** chenleji has quit IRC | 05:29 | |
*** chenleji has joined #openstack-ironic | 05:30 | |
*** anderbubble has joined #openstack-ironic | 05:30 | |
*** achanda has joined #openstack-ironic | 05:31 | |
*** chenleji has quit IRC | 05:34 | |
*** chenleji has joined #openstack-ironic | 05:35 | |
*** pradipta_away is now known as pradipta | 05:38 | |
*** achanda has quit IRC | 05:41 | |
*** pensu has joined #openstack-ironic | 05:42 | |
openstackgerrit | Zhenzan Zhou proposed openstack/ironic: Add config option to control password mask for node-show https://review.openstack.org/150688 | 05:51 |
*** Marga_ has quit IRC | 05:53 | |
*** Marga_ has joined #openstack-ironic | 05:54 | |
openstackgerrit | Rakesh H S proposed openstack/python-ironicclient: ironicclient node-set-maintenance to accept true/false https://review.openstack.org/150692 | 05:54 |
*** chenleji has quit IRC | 05:55 | |
*** chenleji has joined #openstack-ironic | 05:56 | |
*** achanda has joined #openstack-ironic | 05:58 | |
openstackgerrit | Zhenzan Zhou proposed openstack/ironic: Add policy mask_password to mask passwords in driver_info https://review.openstack.org/150688 | 06:21 |
*** achanda has quit IRC | 06:24 | |
*** achanda has joined #openstack-ironic | 06:25 | |
*** EmilienM is now known as EmilienM|afk | 06:29 | |
*** eghobo has joined #openstack-ironic | 06:35 | |
openstackgerrit | Merged openstack/ironic: Add 'irmc_' prefix to optional properties https://review.openstack.org/150849 | 06:37 |
*** achanda has quit IRC | 06:54 | |
*** andreykurilin_ has joined #openstack-ironic | 07:03 | |
*** eghobo has quit IRC | 07:10 | |
*** andreykurilin|ho has joined #openstack-ironic | 07:11 | |
*** andreykurilin_ has quit IRC | 07:12 | |
*** anderbubble has quit IRC | 07:34 | |
*** ndipanov has joined #openstack-ironic | 07:34 | |
*** penick_ has joined #openstack-ironic | 07:34 | |
*** ifarkas has joined #openstack-ironic | 07:36 | |
*** romcheg has joined #openstack-ironic | 07:58 | |
*** penick_ has quit IRC | 08:01 | |
*** romcheg has quit IRC | 08:03 | |
*** chlong has quit IRC | 08:08 | |
*** dtantsur|afk is now known as dtantsur | 08:24 | |
*** jmank has quit IRC | 08:24 | |
dtantsur | Morning Ironic | 08:24 |
dtantsur | finally TGIF | 08:24 |
*** jmank has joined #openstack-ironic | 08:24 | |
*** erwan_taf has joined #openstack-ironic | 08:26 | |
*** romcheg has joined #openstack-ironic | 08:29 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver https://review.openstack.org/146803 | 08:41 |
*** yjiang5 has quit IRC | 08:47 | |
*** yjiang5 has joined #openstack-ironic | 08:48 | |
*** dtantsur is now known as dtantsur|brb | 08:52 | |
*** pensu has quit IRC | 08:52 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add states required for node-inspection https://review.openstack.org/147857 | 08:52 |
*** Nisha has joined #openstack-ironic | 08:52 | |
*** athomas has joined #openstack-ironic | 08:52 | |
*** pradipta is now known as pradipta_away | 08:54 | |
*** jcoufal has joined #openstack-ironic | 08:57 | |
*** zhenzanz has quit IRC | 09:01 | |
*** ndipanov has quit IRC | 09:08 | |
*** ndipanov has joined #openstack-ironic | 09:12 | |
*** MattMan has quit IRC | 09:15 | |
*** derekh has joined #openstack-ironic | 09:19 | |
*** MattMan has joined #openstack-ironic | 09:24 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add states required for node-inspection https://review.openstack.org/147857 | 09:26 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add InspectInterface for node-introspection https://review.openstack.org/149817 | 09:26 |
*** dlpartain has joined #openstack-ironic | 09:26 | |
*** jiangfei has quit IRC | 09:30 | |
*** jistr has joined #openstack-ironic | 09:31 | |
*** lucasagomes has joined #openstack-ironic | 09:33 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Hardware Inspection https://review.openstack.org/149823 | 09:41 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add states required for node-inspection https://review.openstack.org/147857 | 09:41 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add InspectInterface for node-introspection https://review.openstack.org/149817 | 09:41 |
openstackgerrit | Imre Farkas proposed openstack/ironic-specs: DRAC vendor passthru for RAID management https://review.openstack.org/107981 | 09:46 |
openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: Add eDeploy plugin https://review.openstack.org/146599 | 09:46 |
*** coolsvap is now known as coolsvap|afk | 09:58 | |
openstackgerrit | sandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool https://review.openstack.org/134866 | 10:01 |
*** Haomeng has joined #openstack-ironic | 10:04 | |
*** Haomeng|2 has quit IRC | 10:04 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Address final comments of edf532db91 https://review.openstack.org/151589 | 10:05 |
*** dlpartain has quit IRC | 10:15 | |
*** pelix has joined #openstack-ironic | 10:20 | |
openstackgerrit | Merged openstack/ironic: Fix typos in commit I68c9f9f86f5f113bb111c0f4fd83216ae0659d36 https://review.openstack.org/151412 | 10:28 |
sambetts | morning dtantsur | 10:31 |
sambetts | and I agree TGIF | 10:32 |
*** igordcard has joined #openstack-ironic | 10:40 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 10:42 |
*** chlong has joined #openstack-ironic | 10:43 | |
*** chenglch has left #openstack-ironic | 10:44 | |
*** Nisha_away has joined #openstack-ironic | 10:44 | |
openstackgerrit | sandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool https://review.openstack.org/134866 | 10:47 |
*** Nisha has quit IRC | 10:48 | |
lucasagomes | sambetts, yo morning | 10:52 |
*** Nisha_away has quit IRC | 10:53 | |
*** Haomeng has quit IRC | 10:55 | |
*** Haomeng has joined #openstack-ironic | 10:56 | |
lucasagomes | it's only here or review.openstack.org is down? | 10:56 |
*** ramineni has quit IRC | 10:59 | |
sambetts | lucasagomes: its seems a bit unstable this morning | 11:05 |
lucasagomes | yeah | 11:05 |
*** athomas has quit IRC | 11:11 | |
*** athomas has joined #openstack-ironic | 11:18 | |
*** romcheg has quit IRC | 11:21 | |
*** dmellado has joined #openstack-ironic | 11:21 | |
dmellado | Hi all here, I'm having issues downloading python-ironicclient from pypi | 11:22 |
dmellado | do you know if there's any kind of issue? | 11:22 |
dmellado | http://paste.openstack.org/show/164109/ | 11:23 |
*** vdrok_afk is now known as vdrok | 11:23 | |
*** dtantsur|brb is now known as dtantsur | 11:25 | |
dtantsur | dmellado, some DNS problems on your side? | 11:26 |
dtantsur | (or maybe not your...) | 11:26 |
*** rameshg87 has quit IRC | 11:33 | |
lucasagomes | dmellado, haven't seem that no :/ can you try other packages to see if it's not a problem either on ur network or with pip in general? | 11:34 |
dmellado | it might be some issue with my side and pip, that was I was checking... | 11:34 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Update the expose-configdrive spec https://review.openstack.org/151605 | 11:36 |
dmellado | thanks anyway dtantsur lucasagomes | 11:37 |
lucasagomes | dmellado, np | 11:37 |
*** igordcard has quit IRC | 11:39 | |
*** ifarkas has quit IRC | 11:48 | |
*** dlpartain has joined #openstack-ironic | 11:56 | |
*** andreykurilin|ho has quit IRC | 12:00 | |
*** zhenzanz has joined #openstack-ironic | 12:12 | |
*** yog_ has quit IRC | 12:22 | |
*** DenisEgorenko1 has joined #openstack-ironic | 12:24 | |
DenisEgorenko1 | Hi guys! I have one question. I use ironic with baremetal, neutron. The base ubuntu 14.04.1 image works fine (Linux 3.13.0-44-generic #73-Ubuntu SMP Tue Dec 16 00:22:43 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux). Next, i try to test image which builded by DIB (ubuntu 14.04.1 with the same kernel). And i see next errors: | 12:30 |
DenisEgorenko1 | http://paste.openstack.org/show/164136/ | 12:30 |
DenisEgorenko1 | Execution of dd command is comleted. Any ideas? | 12:30 |
DenisEgorenko1 | If i try to run base ubuntu again - it's ok. | 12:30 |
dtantsur | hmm... lucasagomes ideas on ^^^ | 12:31 |
* lucasagomes looks | 12:32 | |
lucasagomes | DenisEgorenko1, is it a partition image or a full disk image? (did you create it with the VM element in DIB? | 12:33 |
lucasagomes | )* | 12:33 |
DenisEgorenko1 | i use next elements: base vm ubuntu hadoop oozie mysql hive | 12:34 |
lucasagomes | that blkid is trying find the fs UUID that will later passed to the pxe config file as root=UUID | 12:34 |
lucasagomes | DenisEgorenko1, right, that's why then | 12:34 |
lucasagomes | the VM element creates a full disk image | 12:35 |
DenisEgorenko1 | lucasagomes: should i use without VM? | 12:35 |
lucasagomes | and the PXE drivers for ironic right now only supports partition images for now (there's a spec merged and some code up to support full disk) | 12:35 |
lucasagomes | DenisEgorenko1, yes | 12:35 |
DenisEgorenko1 | lucasagomes: thank you, i will try. | 12:36 |
lucasagomes | just remove the vm element and try it again, perhaps we should document it better | 12:36 |
lucasagomes | but yeah, I hope the full disk image support merges soon | 12:36 |
lucasagomes | DenisEgorenko1, np | 12:36 |
lucasagomes | lemme know if that works for u | 12:36 |
DenisEgorenko1 | lucasagomes: ok, i will tell you, works it or not | 12:37 |
lucasagomes | cool :) thakns | 12:37 |
lucasagomes | thanks* | 12:37 |
*** smoriya has quit IRC | 12:45 | |
*** Marga_ has quit IRC | 12:52 | |
*** Marga_ has joined #openstack-ironic | 12:53 | |
*** lucasagomes is now known as lucas-hungry | 12:59 | |
* lucas-hungry lunch brb | 12:59 | |
*** Marga__ has joined #openstack-ironic | 13:04 | |
*** Marga_ has quit IRC | 13:04 | |
*** dlpartain has left #openstack-ironic | 13:06 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Improve testing of the Node's REST API https://review.openstack.org/150603 | 13:10 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Rename NOSTATE to AVAILABLE https://review.openstack.org/149754 | 13:10 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add MANAGEABLE state and associated transitions https://review.openstack.org/150073 | 13:10 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add support for API microversions https://review.openstack.org/150821 | 13:10 |
*** rameshg87 has joined #openstack-ironic | 13:14 | |
devananda | jroll: I believe ^ are done now | 13:14 |
*** zhenzanz has quit IRC | 13:16 | |
*** romcheg has joined #openstack-ironic | 13:16 | |
rameshg87 | devananda, can you please have a look at raid configuration spec: https://review.openstack.org/#/c/135899/ | 13:25 |
jroll | devananda: you rock, are you at FOSDEM? | 13:25 |
jroll | morning all :) | 13:25 |
rameshg87 | morning jroll | 13:26 |
jroll | heya rameshg87 | 13:26 |
dtantsur | devananda, rameshg87, jroll, morning | 13:27 |
jroll | \o dtantsur | 13:28 |
rameshg87 | dtantsur, o/ | 13:28 |
*** EmilienM|afk is now known as EmilienM | 13:28 | |
*** zhenzanz has joined #openstack-ironic | 13:29 | |
*** foexle has joined #openstack-ironic | 13:30 | |
*** dprince has joined #openstack-ironic | 13:32 | |
DenisEgorenko1 | lucas-hungry: i rebuild image, without vm element, it works fine now! Thank you so much :-) | 13:37 |
jroll | devananda: what do you think about allowing deployers to specify/override min version? This patch made me think about it, could hide creds in a newer version, but perhaps deployers want to block the older version at that point: https://review.openstack.org/#/c/150688/ | 13:39 |
jroll | dtantsur: did you see the merge conflict here? https://review.openstack.org/#/c/147523/ | 13:39 |
dtantsur | jroll, hell, I missed it... thanks for noting | 13:40 |
jroll | yeah no | 13:40 |
jroll | np* | 13:40 |
* dtantsur things that git needs a special case for Python imports | 13:42 | |
dtantsur | anyone here remembers oldschool way of setting unix password without /etc/shadow and PAM? | 13:42 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add support for driver-specific periodic tasks https://review.openstack.org/147523 | 13:43 |
jroll | dtantsur: do you see why I refuse to use DIB yet? | 13:43 |
dtantsur | jroll, 100% >_< | 13:44 |
jroll | read the code at the first midcycle I went to (last march) and kicked and screamed | 13:44 |
jroll | same for people trying to convince us to use os-*-config and rabbit as a transport layer for IPA | 13:45 |
*** rakesh_hs has quit IRC | 13:45 | |
*** deva__ has joined #openstack-ironic | 13:45 | |
deva__ | jroll: sure. seems reasonable. however | 13:46 |
deva__ | if downstream folks start adding features and raising their version independently | 13:47 |
deva__ | that'd just break all sanity | 13:47 |
deva__ | so | 13:47 |
jroll | that's something they'll need to deal with themselves, and not having a config option won't stop that | 13:47 |
jroll | deva__: for example, we've bumped the rpcapi version downstream, it's a pain point we deal with when rebasing | 13:48 |
deva__ | I'd suggest we don't do something which makes it look like we endorse that kind our change | 13:48 |
jroll | so just make min_version configurable | 13:48 |
deva__ | that makes cross cloud compatibility totally broken | 13:48 |
jroll | does it? | 13:49 |
deva__ | if two services both user the same version number | 13:49 |
deva__ | and provide something divergent | 13:49 |
deva__ | ..... | 13:49 |
jroll | oh, downstream api bumping, right | 13:50 |
deva__ | right | 13:50 |
deva__ | that idea makes me so sad | 13:50 |
jroll | so here's my thought process: we'd do a version bump for the ability to mask credentials. we probably don't want to deprecate older versions just for that. but then credentials are still exposed because someone can just request the older version. | 13:50 |
jroll | maybe we just don't do a bump for that, and break anyone depending on creds being in the response | 13:51 |
jroll | indeed | 13:51 |
* jroll wonders when nova is going to start running into that problem | 13:51 | |
deva__ | they haven't finished landing micro versions, I think | 13:51 |
jroll | right | 13:52 |
jroll | so presumably it will happen within a couple months of landing it | 13:52 |
jroll | which is really lame | 13:52 |
jroll | unless they still support extensions, I guess they probably will for a while | 13:52 |
deva__ | that's so far beyond lame | 13:52 |
deva__ | well. both probably | 13:53 |
jroll | I'd rather not support extensions in ironic simply because being a downstream consumer is really hard | 13:53 |
jroll | on the other hand, I question how much ironic needs cross-cloud compatibility, being an admin api | 13:54 |
deva__ | I would be more ragey at these ideas if I wasn't able to take a nap | 13:54 |
jroll | who is an admin on multiple clouds with different codebases? | 13:54 |
deva__ | sure. not as much an issue for us | 13:54 |
jroll | s/with different codebases/by different downstream consumers/ | 13:54 |
deva__ | this just kills users of nova | 13:54 |
jroll | sure | 13:55 |
deva__ | which really is the main entry point for users of ironic | 13:55 |
jroll | but to that end, should *we* worry about "looking like we endorse that kind of change" | 13:55 |
jroll | I guess the big question is: should we allow downstream consumers to decide when to deprecate an api version, rather than only developers? | 13:56 |
jroll | or put differently: do we want to allow that via a config option or a code change | 13:57 |
*** pensu has joined #openstack-ironic | 13:57 | |
*** Haomeng has quit IRC | 14:05 | |
*** Haomeng has joined #openstack-ironic | 14:05 | |
lucas-hungry | DenisEgorenko1, np :) | 14:05 |
*** lucas-hungry is now known as lucasagomes | 14:06 | |
jroll | heya lucasagomes \o | 14:06 |
lucasagomes | jroll, hey! morning! | 14:06 |
lucasagomes | deva__, afternoon :) | 14:07 |
jroll | lucasagomes: any thoughts on this stuff? ^ | 14:08 |
openstackgerrit | Zhenzan Zhou proposed openstack/ironic: Add policy mask_password to mask passwords in driver_info https://review.openstack.org/150688 | 14:08 |
* lucasagomes will read, 1 sec | 14:08 | |
jroll | yeah no rush | 14:09 |
BadCub_ | morning Ironic | 14:09 |
jroll | morning BadCub_ :) | 14:09 |
BadCub_ | heya jroll | 14:10 |
deva__ | deployers deciding to EOL certain capabilities before upstream seems less likely to break the world, than if they go adding their own new version numbers | 14:11 |
jroll | deva__: yeah, that's my goal with that, though we may end up with deployers allowing deprecated versions (might be ok?) | 14:13 |
jroll | deva__: or we could allow them to only set min_version higher than the code does | 14:13 |
deva__ | option two | 14:14 |
jroll | yeah | 14:14 |
jroll | we can do this on top of the rest of this stuff | 14:14 |
* jroll keeps reviewing | 14:14 | |
deva__ | ++ | 14:14 |
jroll | (for example, as soon as we update our tooling to 1.1, I'll likely deprecate it so we don't accidentally rely on it) | 14:15 |
deva__ | oh | 14:15 |
deva__ | there a bug then | 14:15 |
deva__ | if you set min version >0, but don't specify a header, it's going to be broken right now | 14:16 |
deva__ | I'm assuming you'll do the config thing in a follow on patch. should fix that then | 14:16 |
jroll | deva__: right, I just commented on that in the review | 14:17 |
deva__ | heh | 14:17 |
jroll | we should build the default from '1.%d' % MIN_VERSION | 14:17 |
deva__ | yes | 14:17 |
jroll | even without a config thing, it's going to trip us up later | 14:17 |
deva__ | I would also love to see our tempest API testing pulled in to our tree | 14:19 |
jroll | +1 | 14:20 |
jroll | have other projects done that yet? how does it work? | 14:20 |
jroll | I assume we just have tempest import from ironic or vice versa? | 14:20 |
*** openstackstatus has joined #openstack-ironic | 14:20 | |
*** ChanServ sets mode: +v openstackstatus | 14:20 | |
dtantsur | jroll, hey, does DIB build of ironic-agent has problems with RAM consumption? I remember something about it... | 14:20 |
deva__ | mtreinish tells me it is possible now | 14:21 |
jroll | dtantsur: when I tested it, it required 3-4gb of RAM just to boot | 14:21 |
deva__ | or. really close | 14:21 |
jroll | dtantsur: it also writes the image to /tmp so need to account for that too | 14:21 |
dtantsur | jroll, hell... so don't like I have any working option for my ssh ramdisk idea... | 14:21 |
jroll | deva__: cool, might be a good hacking project for one of these midcycles | 14:21 |
deva__ | yah | 14:22 |
jroll | dtantsur: use the coreos builder :D | 14:22 |
jroll | I don't know why everything has to depend on DIB, that's silly | 14:22 |
deva__ | I'm also thinking about moving all the non production drivers into a separate namespace | 14:22 |
jroll | mmm, that should be a good battle as to what's production and what isn't :P | 14:22 |
deva__ | but no idea what that looks like yet | 14:23 |
deva__ | heh | 14:23 |
deva__ | ssh and fake => bye bye | 14:23 |
jroll | naturally | 14:23 |
dtantsur | jroll, I just think of giving up on improving discoverd ramdisks... existing one works for the default case and everything else's not worth it... | 14:24 |
jroll | are people running ilo/drac in production yet? | 14:24 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add VirtualBox drivers and its modules https://review.openstack.org/146305 | 14:24 |
deva__ | that's the one that made me take want to split it | 14:24 |
Shrews | deva__: ooh, ++ for pulling tempest tests into our tree during sprint | 14:24 |
-openstackstatus- NOTICE: zuul isn't running jobs since ~10:30 utc, investigation underway | 14:24 | |
*** ChanServ changes topic to "zuul isn't running jobs since ~10:30 utc, investigation underway" | 14:24 | |
deva__ | vbox | 14:24 |
jroll | dtantsur: I kind of hate to say this, let other people build on it for their use case | 14:24 |
jroll | ah yeah | 14:25 |
deva__ | oh and reminder for folks | 14:25 |
deva__ | next week is kilo 2 | 14:26 |
deva__ | review all the things, please. prioritize by the wisdom that is launch pad | 14:26 |
jroll | dang | 14:26 |
*** lazy_prince has quit IRC | 14:27 | |
deva__ | out use your own wisdom. because launchpad sucks. even though I try to use it | 14:27 |
deva__ | *or | 14:27 |
jroll | this needs to be bumped, I assume https://blueprints.launchpad.net/ironic/+spec/implement-cleaning-states | 14:28 |
deva__ | yah | 14:28 |
jroll | mrda-away: still working on https://blueprints.launchpad.net/ironic/+spec/logical-names ? | 14:28 |
openstackgerrit | Victor Lowther proposed openstack/ironic: Add DRAC BIOS config vendor passthru API -- DO NOT MERGE https://review.openstack.org/150488 | 14:28 |
jroll | oh, that was updated last week, hmm | 14:28 |
deva__ | I review it yesterday | 14:28 |
jroll | yeah | 14:29 |
deva__ | very close. but wasn't working for me | 14:29 |
jroll | just haven't noticed action on it in a while | 14:29 |
deva__ | also, now that we have a hammer, by which I mean micro versioning, I see nails | 14:29 |
*** killer_prince has joined #openstack-ironic | 14:29 | |
*** killer_prince is now known as lazy_prince | 14:30 | |
deva__ | but adding new API fields -- definitely a version bump | 14:30 |
Shrews | deva__: micro versioning? | 14:31 |
deva__ | ok. this headache isn't going away by typing. time to close my eyes for a while, then get dinner. | 14:32 |
deva__ | Shrews: yup. I've been busy | 14:32 |
lucasagomes | yeah I like the versioning... now whether it should be configurable by the deployer or not, I'm not sure. I have to read the nova proposal too to see if I get some ideas | 14:32 |
lucasagomes | the idea of not setting a version < than the code is good | 14:32 |
jroll | lucasagomes: this is what made me think of it: https://review.openstack.org/150688 | 14:33 |
jroll | as a deployer, I want to deprecate the version that allowed viewing credentials before upstream does | 14:34 |
*** erwan_taf has quit IRC | 14:35 | |
* jroll makes a note about api versioning there as well | 14:35 | |
*** EmilienM is now known as EmilienM|afl | 14:35 | |
*** EmilienM|afl is now known as EmilienM|afk | 14:35 | |
lucasagomes | jroll, right, so 1) would be a new attribute at nodes ? | 14:40 |
jroll | lucasagomes: correct, we've talked about doing it before | 14:41 |
jroll | really those are the two options I see | 14:41 |
jroll | imaging you build a new ramdisk | 14:41 |
jroll | need to figure out which have the old | 14:41 |
lucasagomes | yeah, this is good. Tho not sure if that would block that patch tho, would it? | 14:41 |
lucasagomes | I mean, we still can have both | 14:42 |
jroll | idk, I don't like the idea of blocking all of driver_info | 14:42 |
lucasagomes | it just blocks the fields that has *password* in the ke | 14:42 |
lucasagomes | key* | 14:42 |
lucasagomes | such as ipmi_password, ssh_password | 14:42 |
jroll | no, it blocks all of driver_info | 14:42 |
lucasagomes | everytihng? | 14:42 |
jroll | oh | 14:42 |
lucasagomes | wow, ok... I will give it go | 14:42 |
jroll | ohhhhh | 14:42 |
lucasagomes | yeah if it blocks everything it sucks | 14:42 |
lucasagomes | but I don't think it does | 14:42 |
* lucasagomes it's in my todo list to test that patch | 14:43 | |
jroll | hmm, mask_password takes a string https://github.com/openstack/ironic/blob/master/ironic/openstack/common/strutils.py#L264 | 14:44 |
jroll | yeah, guess we'll need to test that patch | 14:44 |
* jroll tries | 14:44 | |
lucasagomes | In [5]: strutils.mask_password(test_dict, '******') | 14:44 |
lucasagomes | Out[5]: u"{'test_password': '******', 'normal_field': 'bar'}" | 14:44 |
*** zhenzanz has quit IRC | 14:44 | |
jroll | oh huh | 14:44 |
lucasagomes | In [4]: test_dict = {'test_password': 'foo', 'normal_field': 'bar'} | 14:44 |
jroll | neat | 14:44 |
lucasagomes | yeah, it just mask the fields that contains password in the key | 14:45 |
jroll | nice one | 14:45 |
lucasagomes | so I think it's good | 14:45 |
lucasagomes | yeah, it's handy | 14:45 |
jroll | should we wait for the microversions | 14:45 |
jroll | I almost think not | 14:45 |
lucasagomes | yeah, I don't think we should | 14:45 |
jroll | k | 14:45 |
deva__ | why not? | 14:45 |
lucasagomes | we could have it both, I mean it's a policy | 14:45 |
deva__ | oh. policy | 14:45 |
lucasagomes | because it's not actually changing the API, it's a policy configuration | 14:46 |
lucasagomes | whether to mask some fields or not | 14:46 |
deva__ | right. that's fine | 14:46 |
deva__ | let's make sure the policy widths correctly | 14:46 |
lucasagomes | if we add the credentials field, yes we bump it | 14:46 |
lucasagomes | deva__, +1 | 14:46 |
deva__ | like, create two admin users,c and don't give one this access | 14:46 |
deva__ | driver internal info, and node name, both should get a version bump | 14:47 |
lucasagomes | indeed, yeah adding new fields | 14:47 |
deva__ | those are the only other ones on my radar | 14:47 |
lucasagomes | yeah can't think about any other off the top of my head | 14:49 |
*** rameshg87 has quit IRC | 14:49 | |
Shrews | deva__: so this microversioning stuff... forgive me if this has been discussed (catching up on some things), but it seems to go outside the spec of the state machine, no? | 14:51 |
Shrews | i mean, it looks like a very cool idea | 14:51 |
NobodyCam | good morning Ironicers | 14:51 |
NobodyCam | woo hoo its not 3 am today TGIF! | 14:52 |
*** anderbubble has joined #openstack-ironic | 14:52 | |
jroll | lol, good morning and happy friday NobodyCam :) | 14:52 |
Shrews | lol, morning NobodyCam | 14:52 |
NobodyCam | morning jroll Shrews lucasagomes and deva__ :) | 14:52 |
dtantsur | NobodyCam, morning, the biggest TGIF I even imagined!! | 14:54 |
NobodyCam | morning dtantsur :) | 14:55 |
lucasagomes | NobodyCam, hi there | 14:56 |
NobodyCam | :) | 14:57 |
* lucasagomes jumps in a call | 14:58 | |
jroll | Shrews: so, we had to think of a way to transition NOSTATE -> AVAILABLE in the API, without breaking things (tempest, user tools, etc), microversions solves that along with other problems | 14:59 |
*** zz_jgrimm is now known as jgrimm | 14:59 | |
* BadCub_ is off to conferencecallmageddon day 5 | 14:59 | |
NobodyCam | :O | 14:59 |
*** anderbubble has quit IRC | 14:59 | |
* jroll bbiab | 15:00 | |
Shrews | jroll: hrm, ok. still feels uncomfortable squeezing in this change along with another, but i understand the reason now at least. thx | 15:01 |
Shrews | i mean, it warranted its own spec for nova, and we're just throwing it in fairly casually | 15:02 |
Shrews | but i see the need for it | 15:02 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver https://review.openstack.org/146803 | 15:04 |
*** foexle has quit IRC | 15:05 | |
*** timbyr_ is now known as kbyrne | 15:10 | |
jroll | Shrews: right, we're basically following their spec, though some of it is specific to them like dealing with extensions | 15:10 |
jroll | Shrews: basically, we decided that this was worth an API rev, decided we should do API revs right from the start | 15:10 |
jroll | though if people are uncomfortable with doing it without consensus they should voice that and we should write a quick spec | 15:11 |
Shrews | jroll: i hear ya. going to spend some time trying to break this change | 15:15 |
Shrews | maybe i'll feel better about it then | 15:15 |
jroll | hehe | 15:15 |
*** naohirot has quit IRC | 15:15 | |
jroll | yeah I haven't actually tested it | 15:15 |
jroll | going to add a patch on top though | 15:16 |
Shrews | *tsk* *tsk* | 15:16 |
jroll | shh | 15:16 |
jroll | tempest tested it for me :P | 15:16 |
Shrews | :) | 15:16 |
NobodyCam | :O | 15:16 |
tchaypo | is it possible to find how many bugs times a bug has been mentioned in a “recheck bug xxxx” message? | 15:19 |
*** kbyrne has quit IRC | 15:20 | |
*** ndipanov has quit IRC | 15:20 | |
NobodyCam | tchaypo: I'd ask that in -infra | 15:21 |
NobodyCam | I am sure there is | 15:21 |
tchaypo | This isn’t -infra? | 15:22 |
tchaypo | oops | 15:22 |
* tchaypo was off by one | 15:22 | |
*** r-daneel has joined #openstack-ironic | 15:22 | |
Shrews | tchaypo: yes! | 15:22 |
Shrews | tchaypo: should be on the zuul rechecks page | 15:22 |
Shrews | tchaypo: http://status.openstack.org/elastic-recheck/ | 15:22 |
tchaypo | maybe this *was* the right place to ask ;) | 15:23 |
Shrews | tchaypo: we know all the things | 15:23 |
tchaypo | that page is showing empty for me | 15:23 |
tchaypo | oh no, it opens eventually | 15:23 |
Shrews | except for the things we don't know | 15:23 |
jroll | s/we/Shrews | 15:24 |
tchaypo | in any case - that shows me recent frequent issues. I was wondering if a particular 9-month-old bug had ever had even a single recheck | 15:24 |
Shrews | tchaypo: that might require some sort of direct magical elasticsearch query. that's one of the things we/me does not know | 15:26 |
tchaypo | i suspect elasticsearch doesn’t store 9 months worths of logs anyway | 15:27 |
Shrews | http://docs.openstack.org/infra/elastic-recheck/readme.html#adding-bug-signatures | 15:27 |
Shrews | tchaypo: possibly | 15:28 |
*** kbyrne has joined #openstack-ironic | 15:30 | |
*** kbyrne has quit IRC | 15:30 | |
*** DenisEgorenko1 has left #openstack-ironic | 15:31 | |
*** kbyrne has joined #openstack-ironic | 15:31 | |
tchaypo | no no no. Don’t get me started on that. | 15:34 |
tchaypo | My last attempt to add signatures just wasted everybody’s time | 15:34 |
tchaypo | hence https://review.openstack.org/#/c/151221/ | 15:34 |
*** jjohnson2 has joined #openstack-ironic | 15:34 | |
tchaypo | I have a note to myself to udpate the elastic-recheck docs as well, I think I’ve just figured out where to update them | 15:35 |
tchaypo | but actually that doc is probably okay already - it already starts by talking specifically about gate jobs | 15:35 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Use prolianutils module for ilo driver tests https://review.openstack.org/151674 | 15:36 |
*** achanda has joined #openstack-ironic | 15:37 | |
Shrews | jroll: fear not. it seems to work as expected | 15:38 |
jroll | woo | 15:39 |
jroll | see, I can test code with my brain | 15:39 |
Shrews | jroll: you iz the awesome! | 15:39 |
jroll | :P | 15:39 |
* Shrews sends all review requests telepathically to jroll | 15:39 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Use prolianutils module for ilo driver tests https://review.openstack.org/151674 | 15:39 |
jroll | dang | 15:39 |
jroll | I screwed myself over there | 15:40 |
*** dencaval has quit IRC | 15:40 | |
*** achanda has quit IRC | 15:43 | |
*** rainya_ is now known as rainya | 15:50 | |
*** kragniz is now known as zingark | 15:53 | |
*** anderbubble has joined #openstack-ironic | 15:55 | |
*** jcoufal_ has joined #openstack-ironic | 16:00 | |
*** pcrews has joined #openstack-ironic | 16:03 | |
*** jcoufal has quit IRC | 16:04 | |
*** EmilienM|afk is now known as EmilienM | 16:07 | |
*** nosleep77 has quit IRC | 16:07 | |
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 16:12 | |
-openstackstatus- NOTICE: zuul is running again and changes have been reenqueud. seehttp://status.openstack.org/zuul/ before rechecking if in doubt | 16:12 | |
lucasagomes | jroll, may be a dumb question... In the network-provider spec, when is configure/unconfigure_tenant_networks actually called? | 16:16 |
lucasagomes | it says, to be removed by Nova | 16:16 |
lucasagomes | I don't get that bit | 16:17 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Don't hardcode default minimum API version https://review.openstack.org/151686 | 16:17 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow minimum API version to be configured https://review.openstack.org/151687 | 16:17 |
jroll | ^ kinda hate these, suggestions welcome | 16:18 |
jroll | lucasagomes: nova does the create/delete of the port, ironic grabs that info and configures the switch | 16:18 |
lucasagomes | right, nova creates it, pass the vif to ironic | 16:19 |
lucasagomes | then we call configure_tenant_networks? | 16:19 |
jroll | yeah, then between deploy and instance boot ironic calls configure_...() | 16:19 |
*** david-lyle_afk is now known as david-lyle | 16:19 | |
jroll | because the node needs to be on the provisioning network during deploy | 16:19 |
lucasagomes | right, then on deletion, we unconfigure and nova will delete the port in neuton? | 16:20 |
lucasagomes | neutron* | 16:20 |
jroll | so deploy, power off, configure_tenant_networks, power on | 16:20 |
jroll | yep | 16:20 |
lucasagomes | and the add_ remove_ is going to be called at initialization? By using the CONF.provision_network | 16:20 |
Shrews | All: Isn't the "correct" thing to do for this review is to add "conductor_affinity" to NodePatchType.internal_attrs()? https://review.openstack.org/#/c/151262/1/ironic/tests/api/utils.py | 16:20 |
lucasagomes | and it will create the provision network on the switch, and that's the network that is going to be used on the deployment | 16:20 |
*** jerryz has quit IRC | 16:22 | |
lucasagomes | jroll, right... and the add/remove provision network. When is that called? initialization ^ ? | 16:22 |
* lucasagomes wants to understand better before approving it | 16:22 | |
jroll | lucasagomes: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1447-1449 etc | 16:22 |
jroll | lucasagomes: basically, at the same time | 16:22 |
jroll | we're switching between provisioning and tenant networks | 16:23 |
jroll | so after deploy, we remove provisioning networks, configure tenant | 16:23 |
jroll | at tear down, we deconfigure tenant networks, add provisioning network | 16:23 |
jroll | does that make sense? | 16:24 |
Shrews | ah, ignore my previous question | 16:24 |
vdrok | Shrews, API node does not have conductor_affinity attribute at all | 16:24 |
lucasagomes | right, it does | 16:24 |
vdrok | Shrews, and morning :) | 16:24 |
lucasagomes | jroll, reason why the name differences? I mean add and configure seems to do same stuff | 16:24 |
lucasagomes | but just diff networks | 16:24 |
dtantsur | jroll, still not sure why mixing names | 16:24 |
dtantsur | oh, lucasagomes ++ :) | 16:24 |
Shrews | vdrok: yeah. took me a bit to grok it | 16:24 |
jroll | lucasagomes: dtantsur: the difference is add/remove does an actual create/delete | 16:25 |
jroll | configure/deconfigure do not | 16:25 |
jroll | I mean, it's just names, I don't actually care that much | 16:25 |
lucasagomes | right, yeah docstrings can be expanded to explain that bit better | 16:25 |
lucasagomes | and that's implementation detail | 16:25 |
lucasagomes | the concept looks good | 16:25 |
jroll | sweet | 16:26 |
lucasagomes | I'm good with the spec | 16:26 |
dtantsur | aha, makes sense now | 16:26 |
jroll | now go review the code :P | 16:26 |
jroll | (it still needs tests but other than that I think it's close) | 16:26 |
lucasagomes | dtantsur, u want to review that first? or I will +a that | 16:26 |
lucasagomes | jroll, cool, yea sorry for that. I haven't had time to look at the code | 16:27 |
dtantsur | lucasagomes, I think I was it one and was pretty ok with it. Anyway, too tired to review specs now, go 4 it | 16:27 |
lucasagomes | cool | 16:27 |
jroll | nah, it's ok, thanks for reviewing :) | 16:27 |
lucasagomes | boom it's approved then | 16:27 |
jroll | woot | 16:28 |
jroll | ty! | 16:28 |
lucasagomes | now it's up to gerrit | 16:28 |
lucasagomes | :) | 16:28 |
openstackgerrit | Merged openstack/ironic-specs: Add pluggable network provider spec https://review.openstack.org/143480 | 16:32 |
jroll | so excited to se that land | 16:32 |
lucasagomes | :) | 16:32 |
lucasagomes | s/gerrit/jenkins | 16:33 |
jroll | has anyone thought about rescue mode yet? | 16:37 |
jroll | out of curiousity | 16:38 |
dtantsur | I try not to :) | 16:38 |
*** Marga__ has quit IRC | 16:39 | |
*** Marga_ has joined #openstack-ironic | 16:39 | |
jroll | dtantsur: well... we have it mostly working | 16:40 |
jroll | but it involves golang, and depends on this networking stuff somewhat (though I guess it would work on a flat network) | 16:40 |
dtantsur | golang... why? :) | 16:40 |
jroll | because we can't do everything we want to from within a container | 16:41 |
jroll | https://github.com/rackerlabs/onmetal-rescue-agent | 16:42 |
jroll | (and because coreos doesn't have python) | 16:42 |
jroll | also, having IPA running on public networks seems not ideal :P | 16:42 |
dtantsur | heh | 16:43 |
*** Marga_ has quit IRC | 16:43 | |
jroll | so trying to feel out how to work things into ironic, idk, we probably don't have time in kilo anyway | 16:43 |
*** Marga_ has joined #openstack-ironic | 16:44 | |
*** deva__ has quit IRC | 16:45 | |
dtantsur | yeah, I feel like we have enormous amount of things to do already... | 16:47 |
*** alexpilotti has joined #openstack-ironic | 16:48 | |
jroll | right | 16:48 |
*** rameshg87 has joined #openstack-ironic | 16:48 | |
jroll | we'll talk about it in vancouver then :) | 16:48 |
lucasagomes | yeah, haven't think about rescue either | 16:49 |
*** Marga_ has quit IRC | 16:49 | |
*** rwsu-afk is now known as rwsu | 16:49 | |
*** ChuckC has joined #openstack-ironic | 17:01 | |
*** ChuckC_ has quit IRC | 17:05 | |
*** yjiang5 is now known as yginger5 | 17:06 | |
*** ChuckC_ has joined #openstack-ironic | 17:06 | |
*** zingark is now known as krangiz | 17:07 | |
*** krangiz is now known as kragniz | 17:07 | |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Rework IO Worker thread behavior https://review.openstack.org/151706 | 17:07 |
openstackgerrit | Merged openstack/ironic: Add missing exceptions into function docstring https://review.openstack.org/151507 | 17:08 |
* NobodyCam wishes more devs would research a bug befor opening a ticket. at least to figure out the best group to open the ticket under | 17:08 | |
*** ChuckC has quit IRC | 17:10 | |
*** ChuckC has joined #openstack-ironic | 17:10 | |
*** jcoufal_ has quit IRC | 17:10 | |
*** ChuckC_ has quit IRC | 17:14 | |
*** ijw has joined #openstack-ironic | 17:14 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver https://review.openstack.org/146803 | 17:17 |
dtantsur | have a nice weekend everyone! | 17:17 |
*** dtantsur is now known as dtantsur|afk | 17:17 | |
NobodyCam | have a great weekend dtantsur|afk | 17:17 |
jroll | cheers dtantsur|afk :) | 17:17 |
lucasagomes | dtantsur|afk, see ya, enjoy the weekend | 17:19 |
*** alexpilotti has quit IRC | 17:20 | |
rameshg87 | lucasagomes, NobodyCam, got some time for https://review.openstack.org/#/c/135899/, please have a look :) | 17:22 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add documentation for VirtualBox drivers https://review.openstack.org/146307 | 17:23 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add put method to API to create or update a node https://review.openstack.org/130228 | 17:24 |
NobodyCam | do we have anything I can point folks to about ipmi bridging or even dbl bridging? | 17:25 |
*** derekh has quit IRC | 17:29 | |
jroll | NobodyCam: did you want to look at the state machine reviews? | 17:30 |
jroll | they all have at least 2x+2 | 17:30 |
* NobodyCam looks | 17:34 | |
openstackgerrit | Merged openstack/ironic-specs: Update the expose-configdrive spec https://review.openstack.org/151605 | 17:35 |
openstackgerrit | Merged openstack/ironic: Do not POST conductor_affinity in tests https://review.openstack.org/151262 | 17:39 |
NobodyCam | jroll: https://review.openstack.org/#/c/150821 lgtm +2'd waiting for Jenkins for +a | 17:39 |
jroll | oh right, jenkins | 17:39 |
NobodyCam | :-p | 17:40 |
openstackgerrit | Merged openstack/ironic: Address final comments of edf532db91 https://review.openstack.org/151589 | 17:40 |
jroll | I was wondering why they weren't +A | 17:40 |
*** Marga_ has joined #openstack-ironic | 17:40 | |
jroll | it all makes sense now | 17:40 |
lucasagomes | updated the spec in nova too for the config drive, in case someone here wants to review thathttps://review.openstack.org/#/c/151717/ | 17:40 |
NobodyCam | lol | 17:41 |
JayF | lucasagomes: I honestly don't know why this is supposed to be better; configdrives can be huge; that's going to be a big thing to have in API responses | 17:42 |
JayF | depending on what people embed | 17:42 |
lucasagomes | JayF, well... Nova doesn't want to have a hard dependency on swift | 17:42 |
JayF | Like I'm OK with it, because it's how our stuff works today; but just would be curious to know the reasoning | 17:42 |
JayF | (our stuff = downstream) | 17:42 |
jroll | JayF: there was already an option to store it in the db, and still is | 17:42 |
lucasagomes | the last time we argued about it, for the migration from nova bm to ironic for e.g | 17:42 |
lucasagomes | took over 6 months | 17:42 |
lucasagomes | to have a consensus | 17:43 |
JayF | I'm not saying you're wrong to give it :) | 17:43 |
lucasagomes | I also prefer the old way, but I don't think it worth arguing really | 17:43 |
JayF | was just generally curious | 17:43 |
JayF | s/it/in/ | 17:43 |
jroll | I think it's more like "let ironic handle all the configdrive things" | 17:43 |
lucasagomes | JayF, I see yeah | 17:43 |
lucasagomes | there's some discussion in the Nova patch, if u wanna see | 17:44 |
lucasagomes | https://review.openstack.org/#/c/144792/ | 17:44 |
lucasagomes | JayF, but yeah, to be very honest. I pretty much just want to get it done | 17:45 |
JayF | +1 to jfdi | 17:45 |
JayF | + | 17:45 |
*** harlowja_away is now known as harlowja_ | 17:50 | |
*** yginger5 has quit IRC | 17:51 | |
*** alex_xu has quit IRC | 17:52 | |
*** yjiang5 has joined #openstack-ironic | 17:54 | |
*** yjiang5 is now known as yginger5 | 17:54 | |
*** yginger5 has left #openstack-ironic | 17:54 | |
*** bnemec is now known as beekneemech | 17:55 | |
*** alex_xu has joined #openstack-ironic | 17:55 | |
*** anderbubble has quit IRC | 17:56 | |
lucasagomes | JayF, but anyway, last comment on that before I go for beers :D if we see that the config drive becomes too big in some cases that can't be passed in the request body | 17:57 |
lucasagomes | we can change and start doing what the old spec proposed in Nova | 17:58 |
lucasagomes | uploading it to swift etc... | 17:58 |
lucasagomes | and passing the URI via the same API we added for /provision | 17:58 |
jroll | mmm, beer | 17:58 |
lucasagomes | so interface-wise we still can do it if needed | 17:58 |
lucasagomes | +1 :) | 17:58 |
*** ChuckC has quit IRC | 17:58 | |
lucasagomes | it's cold, so guinness is it | 17:58 |
jroll | sounds great | 17:58 |
lucasagomes | alright, so have a good night everyone | 18:00 |
*** ChuckC has joined #openstack-ironic | 18:00 | |
lucasagomes | enjoy the weekend! | 18:00 |
lucasagomes | I talk to you guys later, and see some of you next week too | 18:00 |
*** eghobo has joined #openstack-ironic | 18:00 | |
NobodyCam | have a good weekend lucasagomes | 18:00 |
*** ChuckC_ has joined #openstack-ironic | 18:01 | |
*** lucasagomes is now known as lucas-beer | 18:02 | |
lucas-beer | NobodyCam, u too! | 18:02 |
*** datajerk has quit IRC | 18:02 | |
jroll | night lucas-beer :) | 18:03 |
*** harlowja_ has quit IRC | 18:04 | |
*** ChuckC has quit IRC | 18:05 | |
Shrews | ooh, four +2's on the microversion thingy | 18:06 |
*** harlowja has joined #openstack-ironic | 18:07 | |
*** early` has quit IRC | 18:07 | |
jroll | still want a spec? :P | 18:07 |
Shrews | jroll: no. but I demand a spec for your follow up patches | 18:08 |
jroll | D: | 18:08 |
NobodyCam | lol | 18:08 |
*** sirushti has quit IRC | 18:08 | |
*** dividehex has quit IRC | 18:08 | |
*** morgabra has quit IRC | 18:08 | |
*** lynxman has quit IRC | 18:09 | |
*** alex_xu has quit IRC | 18:09 | |
*** lynxman has joined #openstack-ironic | 18:10 | |
*** alex_xu has joined #openstack-ironic | 18:11 | |
*** dividehex has joined #openstack-ironic | 18:13 | |
*** sirushti has joined #openstack-ironic | 18:13 | |
*** morgabra has joined #openstack-ironic | 18:13 | |
*** rameshg87 has quit IRC | 18:14 | |
*** achanda has joined #openstack-ironic | 18:16 | |
*** early has joined #openstack-ironic | 18:16 | |
*** penick has joined #openstack-ironic | 18:17 | |
*** athomas has quit IRC | 18:17 | |
*** ChuckC_ has quit IRC | 18:18 | |
*** ChuckC has joined #openstack-ironic | 18:18 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Add :raises: for Version constructor docstring https://review.openstack.org/151727 | 18:20 |
*** anderbubble has joined #openstack-ironic | 18:20 | |
*** pensu has quit IRC | 18:22 | |
*** tchaypo has quit IRC | 18:23 | |
*** tchaypo has joined #openstack-ironic | 18:24 | |
*** yjiang5 has joined #openstack-ironic | 18:26 | |
*** yjiang5 is now known as yginger5 | 18:27 | |
JayF | Shrews: fwiw like all but one patchset in that series have +2x4 and +1 | 18:29 |
JayF | Shrews: if you wanted to land all the things | 18:29 |
* Shrews wonders why no one is bothering to +A then | 18:32 | |
JayF | Shrews: NobodyCam was waiting on jenkins | 18:32 |
jroll | s/nobodycam/everyone | 18:32 |
JayF | Shrews: which has passed now, and I would approve but I don't have the big hammer there | 18:32 |
Shrews | JayF: i still need to review them, but if other cores are ready, go for it | 18:32 |
NobodyCam | Shrews: tests had not finished | 18:32 |
* jroll will approve | 18:32 | |
jroll | I got 2/3 at least | 18:33 |
NobodyCam | jroll: :) nice | 18:36 |
*** spandhe has joined #openstack-ironic | 18:39 | |
JayF | this is a spec that could land if it got another review -> https://review.openstack.org/#/c/134865 | 18:40 |
JayF | been through the ringer with it so it should be right as rain | 18:40 |
*** Marga_ has quit IRC | 18:40 | |
JayF | this one too -> https://review.openstack.org/#/c/142401/7 | 18:41 |
JayF | JoshNang: ^ that's the iLO cleaning steps spec; so relevant to your interests | 18:41 |
JoshNang | JayF: kk i'll review | 18:42 |
JoshNang | i should do the iLO zapping one as well | 18:42 |
Shrews | wow. dehaan is leaving ansible | 18:42 |
JayF | 'leaving ansible' ? | 18:42 |
Shrews | http://michaeldehaan.net/post/109595670406/happy-trails-ansible | 18:42 |
anderbubble | I’m continuing to try to track down an error I’m getting when following the directions at http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 18:42 |
-openstackstatus- NOTICE: Gerrit and Zuul will be offline from 1900 to 1930 UTC for project renames | 18:42 | |
JayF | oh, dehaan | 18:42 |
JayF | I read that as 'debian' | 18:42 |
anderbubble | If my ml2_conf.ini/ml2/type_drivers = flat, then I get regular thrown exceptions: Invalid input for operation: network_type | 18:43 |
anderbubble | value 'vxlan' not supported. | 18:43 |
anderbubble | but if my ml2_conf.init/mp2/type_drivers = flat,vxlan, then the exceptions stop. | 18:44 |
*** achanda has quit IRC | 18:44 | |
anderbubble | Should I be including vxlan in my type_drivers, or is there something else that I could remove that is still referencing a vxlan network that I’m missing? | 18:44 |
anderbubble | (exceptions appear in neutron/server.log) | 18:45 |
*** zsmithnyc has quit IRC | 18:45 | |
*** achanda has joined #openstack-ironic | 18:45 | |
*** zsmithnyc has joined #openstack-ironic | 18:45 | |
JayF | Hm. I'm not super familiar with that specifically; but that seems like it's more likely to be something that #openstack-neutron could help with? IMBW | 18:45 |
anderbubble | full trace: http://pastebin.com/kJBZWaRa | 18:46 |
anderbubble | JayF, I’m happy to go there if you think it’d be a better target. | 18:46 |
JayF | I'm pretty sure you'd be more likely to get someone with specific knowledge of that; but I'd also hang out here in case one of the smart folks in here knows a thing :) | 18:47 |
anderbubble | JayF, I’m just afraid of them saying “meh: we don’t know anything about how *ironic* is supposed to work" | 18:50 |
anderbubble | but I’ve x-posted there. | 18:51 |
*** penick has quit IRC | 18:51 | |
*** MattMan has left #openstack-ironic | 18:52 | |
anderbubble | Well, I appear to have found it. | 19:03 |
JayF | nice! | 19:03 |
JayF | Config or code problem? | 19:03 |
anderbubble | ovs_neutron_plugin.ini (as configured by RDO/packstack) had vxlan tunneling configured | 19:04 |
anderbubble | (for the tenant network) | 19:04 |
JayF | aha | 19:04 |
anderbubble | but since the network is flat… | 19:04 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Rework IO Worker thread behavior https://review.openstack.org/151706 | 19:05 |
*** penick has joined #openstack-ironic | 19:05 | |
JayF | Makes sense to me | 19:05 |
JayF | Other than that Ironic seem to be OK? Or can you not tell yet :) | 19:05 |
-openstackstatus- NOTICE: Gerrit and Zuul are offline until 1930 UTC for project renames | 19:07 | |
*** ChanServ changes topic to "Gerrit and Zuul are offline until 1930 UTC for project renames" | 19:07 | |
*** jistr has quit IRC | 19:08 | |
*** penick has quit IRC | 19:11 | |
Shrews | hrm, we could probably target this spec to kilo if we really wanted to: https://blueprints.launchpad.net/ironic/+spec/ironic-virtualbox-webservice-support | 19:11 |
Shrews | code seems ready (+2 from me) | 19:11 |
* Shrews actually assumed (incorrectly) that it was | 19:12 | |
JayF | if a thing is done; I can hardly think of a good reason not to have it in | 19:14 |
* NobodyCam waits got Gerrit to come back up | 19:14 | |
Shrews | ikr? | 19:14 |
NobodyCam | s/got/for/ | 19:14 |
*** igordcard has joined #openstack-ironic | 19:15 | |
*** jjohnson2 has quit IRC | 19:23 | |
*** pelix has quit IRC | 19:27 | |
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 19:30 | |
-openstackstatus- NOTICE: Gerrit is back online | 19:30 | |
*** eghobo has quit IRC | 19:31 | |
* NobodyCam wishes virsh used a different exit code for each error | 19:34 | |
*** EmilienM is now known as EmilienM|afk | 19:47 | |
*** PaulCzar has joined #openstack-ironic | 19:59 | |
*** mrmoje has joined #openstack-ironic | 20:01 | |
*** eghobo has joined #openstack-ironic | 20:04 | |
NobodyCam | all I have a POC that I'd luv to get some feed back on before I actually code it.. http://paste.openstack.org/show/5iolH6yTtlE6hs5AVPy7 | 20:05 |
*** openstackgerrit has quit IRC | 20:06 | |
*** openstackgerrit has joined #openstack-ironic | 20:07 | |
*** penick has joined #openstack-ironic | 20:18 | |
*** nosleep77 has joined #openstack-ironic | 20:20 | |
*** david-lyle has quit IRC | 20:25 | |
*** harlowja is now known as harlowja_away | 20:29 | |
*** harlowja_away is now known as harlowja | 20:34 | |
*** alexpilotti has joined #openstack-ironic | 20:34 | |
*** alexpilotti has quit IRC | 20:34 | |
*** spandhe has quit IRC | 20:50 | |
*** penick has quit IRC | 20:50 | |
*** lucas-beer has quit IRC | 20:57 | |
*** spandhe has joined #openstack-ironic | 20:59 | |
*** mlsoule has quit IRC | 21:01 | |
openstackgerrit | Merged openstack/ironic: Add support for API microversions https://review.openstack.org/150821 | 21:05 |
openstackgerrit | Merged openstack/ironic: Rename NOSTATE to AVAILABLE https://review.openstack.org/149754 | 21:06 |
openstackgerrit | Merged openstack/ironic: Improve testing of the Node's REST API https://review.openstack.org/150603 | 21:06 |
openstackgerrit | Merged openstack/ironic: Add :raises: for Version constructor docstring https://review.openstack.org/151727 | 21:06 |
*** Haomeng has quit IRC | 21:07 | |
*** Haomeng has joined #openstack-ironic | 21:08 | |
JayF | victor_lowther: around? | 21:10 |
*** penick has joined #openstack-ironic | 21:10 | |
JayF | victor_lowther: cancel ping, I'm not the droid you're looking for | 21:11 |
*** dprince has quit IRC | 21:18 | |
*** mjturek has quit IRC | 21:21 | |
openstackgerrit | Merged stackforge/pyghmi: Rework IO Worker thread behavior https://review.openstack.org/151706 | 21:38 |
*** achanda has quit IRC | 21:39 | |
*** achanda has joined #openstack-ironic | 21:39 | |
*** achanda has quit IRC | 21:43 | |
*** jjohnson2 has joined #openstack-ironic | 21:44 | |
*** romcheg has quit IRC | 21:44 | |
*** achanda has joined #openstack-ironic | 21:48 | |
*** achanda has quit IRC | 21:49 | |
*** anderbubble has quit IRC | 21:51 | |
*** jgrimm is now known as zz_jgrimm | 21:52 | |
*** igordcard has quit IRC | 21:55 | |
*** achanda has joined #openstack-ironic | 22:00 | |
*** anderbubble has joined #openstack-ironic | 22:01 | |
openstackgerrit | Merged openstack/ironic: Do not pass PXE net config from bootloader to ramdisk https://review.openstack.org/141148 | 22:01 |
*** achanda has quit IRC | 22:10 | |
*** spandhe has quit IRC | 22:11 | |
*** jjohnson2 has quit IRC | 22:12 | |
*** achanda has joined #openstack-ironic | 22:12 | |
*** achanda has quit IRC | 22:45 | |
*** achanda has joined #openstack-ironic | 22:46 | |
*** achanda has quit IRC | 22:50 | |
*** ChuckC has quit IRC | 22:54 | |
NobodyCam | brb | 22:56 |
anderbubble | I am getting SO CLOSE to provisioning a baremetal node with Ironic… now the node is pulling an address during PXE, but then it says “Unable to locate configuration file” on the booting client. | 23:02 |
anderbubble | tftpboot looks sane, but I get these errors in syslog: http://pastebin.com/GGGkm3zm | 23:03 |
*** harlowja is now known as harlowja_away | 23:05 | |
*** BadCub_ has quit IRC | 23:09 | |
*** eghobo has quit IRC | 23:10 | |
anderbubble | Might have figured it out! The pxelinux.cfg/ file is a symlink with an absolute path, and it sounds like tftpd is chrooting to /tftpboot on startup. This would make the absolute paths break. | 23:12 |
anderbubble | This is probably a good feature request, actually: if possible, neutron should probably create relative symlinks. | 23:12 |
*** andreykurilin_ has joined #openstack-ironic | 23:12 | |
*** eghobo has joined #openstack-ironic | 23:14 | |
NobodyCam | anderbubble: do you have a tftp map file? | 23:17 |
anderbubble | like this? http://pastebin.com/jKuSaW0j | 23:18 |
anderbubble | I removed the -s from my tftpd startup script, and am getting a new error; but it went by too fast for me to see; so I’m disabling hdd boot now. ;) | 23:18 |
NobodyCam | yep like that | 23:19 |
NobodyCam | :) | 23:19 |
anderbubble | NobodyCam, SO CLOSE!!! | 23:19 |
NobodyCam | :) | 23:19 |
anderbubble | I’ve got lots of notes for documentation updates, though. | 23:19 |
NobodyCam | awesome!!!!! | 23:20 |
anderbubble | I had to go scold the RDO/packstack guys, too. :) | 23:20 |
NobodyCam | lol | 23:20 |
anderbubble | Their documentation still claims to provision nova-network by default in *many* places. | 23:20 |
anderbubble | (though it definitely does neutron) | 23:20 |
NobodyCam | hehehe | 23:21 |
anderbubble | So, now my PXE rom is saying (transcribing from cellphone picture of console output): PXE-T02: Only absolute filenames allowed / PXE-E3C: TFTP Error - Access Violation | 23:22 |
anderbubble | …and my research is indicating that that’s a result of *not* having the -s flag (which chroots)… :( | 23:25 |
anderbubble | (╯°_°)╯︵ ┻━┻ | 23:26 |
*** achanda has joined #openstack-ironic | 23:26 | |
*** harlowja_away is now known as harlowja | 23:27 | |
*** chuckC_ has joined #openstack-ironic | 23:28 | |
*** harlowja has quit IRC | 23:29 | |
*** harlowja has joined #openstack-ironic | 23:29 | |
NobodyCam | :( but you removed that -s no? | 23:32 |
*** penick has quit IRC | 23:32 | |
NobodyCam | oh not having the -s | 23:35 |
* NobodyCam can read | 23:35 | |
NobodyCam | anderbubble: do you have selinux running? | 23:36 |
anderbubble | what do you mean, “running”? | 23:36 |
anderbubble | I think I realized how people fix this. | 23:36 |
NobodyCam | enabled? | 23:36 |
anderbubble | ln -s . /tftpboot/tftpboot | 23:36 |
anderbubble | I’m pretty sure I’ve actually seen that in production before. | 23:37 |
anderbubble | Where would one enable it? I have it installed... | 23:37 |
NobodyCam | I haven't seen that, but its been a while for me and RH/Cent | 23:37 |
NobodyCam | youd want selinux disabled | 23:38 |
anderbubble | I tried copying the config file explicitely over the symlink, and it got further, but was trying to access files by explicit /tftpboot/ paths; so I’m pretty sure this is what I’ll have to do | 23:38 |
anderbubble | oh! sorry; I was reading syslinux | 23:38 |
anderbubble | This isn’t an selinux problem. | 23:38 |
NobodyCam | :) | 23:38 |
anderbubble | There’s no denials in /var/log/audit/audit.log | 23:38 |
*** achanda has quit IRC | 23:39 | |
NobodyCam | anything in syslog or tftp? | 23:39 |
anderbubble | Just multiple requests for pxelinux.0 | 23:40 |
*** datajerk has joined #openstack-ironic | 23:40 | |
anderbubble | It looks like rhel’s tftp server won’t allow you to request files with relative paths if you don’t specify -s (chroot) server-side | 23:40 |
anderbubble | but ironic is expecting to be able to specify absolute paths from actual root, like /tftpboot/blah | 23:41 |
NobodyCam | oh | 23:41 |
anderbubble | so creating a tftpboot symlink in /tftpboot that points at itself will fix this | 23:41 |
anderbubble | at least, that’s the theory | 23:41 |
anderbubble | and I think I’ve seen it that way before | 23:41 |
anderbubble | THIS HAS BEEN THE BIGGEST PAIN | 23:42 |
anderbubble | Even to the point of PXE being disabled on the box and Dell going the furthest out of their way possible to obscure how to enable it. | 23:42 |
anderbubble | IT’S BOOTING! | 23:42 |
anderbubble | …error | 23:43 |
NobodyCam | :-/ | 23:44 |
NobodyCam | but it booted? | 23:44 |
*** achanda has joined #openstack-ironic | 23:44 | |
anderbubble | It booted deploy, but not image | 23:45 |
*** chuckC_ has quit IRC | 23:45 | |
*** ijw has quit IRC | 23:45 | |
anderbubble | I didn’t see the error go by on the console; but I got a bunch of traceback in my ironic logs. | 23:45 |
*** ijw has joined #openstack-ironic | 23:45 | |
NobodyCam | ok thats progress | 23:46 |
NobodyCam | what error did you get in the ironic-conductor.log | 23:46 |
*** anderbubble_ has joined #openstack-ironic | 23:47 | |
anderbubble_ | insult to injury: my battery is dying | 23:47 |
*** anderbubble_ has quit IRC | 23:48 | |
*** anderbubble has quit IRC | 23:49 | |
NobodyCam | :( | 23:49 |
*** anderbubble has joined #openstack-ironic | 23:50 | |
openstackgerrit | Hadi Bannazadeh proposed openstack/ironic: region missing in endpoint selection https://review.openstack.org/148128 | 23:54 |
anderbubble | Looks like jroll has had this problem before: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2014-11-21.log | 23:58 |
*** achanda has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!