Friday, 2015-01-30

*** jgrimm is now known as zz_jgrimm00:09
tteggel_anyone fancy a nice easy stable/juno +W ;) https://review.openstack.org/#/c/14995900:11
openstackgerritChris Krelle proposed openstack/ironic: Fix typos in commit I68c9f9f86f5f113bb111c0f4fd83216ae0659d36  https://review.openstack.org/15141200:12
*** tteggel_ is now known as tteggel00:14
*** tteggel has quit IRC00:14
*** tteggel has joined #openstack-ironic00:14
NobodyCam+W?00:15
NobodyCamwow more changes in conf.sample then I expected?00:16
NobodyCamoh I know00:18
*** zer0c00l has quit IRC00:21
*** zer0c00l has joined #openstack-ironic00:22
*** penick has quit IRC00:24
openstackgerritChris Krelle proposed openstack/ironic: Fix typos in commit I68c9f9f86f5f113bb111c0f4fd83216ae0659d36  https://review.openstack.org/15141200:25
*** jjohnson2 has joined #openstack-ironic00:29
*** smoriya has joined #openstack-ironic00:30
*** david-lyle is now known as david-lyle_afk00:31
*** ijw has quit IRC00:35
*** derekh has joined #openstack-ironic00:39
*** zhenzanz has joined #openstack-ironic00:39
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic  https://review.openstack.org/13486500:41
*** penick has joined #openstack-ironic00:41
*** penick has quit IRC00:42
*** Haomeng|2 has joined #openstack-ironic00:44
*** Haomeng has quit IRC00:45
*** spandhe has quit IRC00:46
*** Nisha has quit IRC00:48
*** bradjones has quit IRC00:56
*** penick has joined #openstack-ironic00:57
*** penick has quit IRC00:58
*** chenleji has quit IRC00:58
jrolldevananda: didn't get around to those patches today, sorry00:58
*** chenleji has joined #openstack-ironic00:58
jiangfeidevananda: hi00:59
*** bradjones has joined #openstack-ironic01:00
*** Marga_ has joined #openstack-ironic01:05
*** derekh has quit IRC01:10
*** igordcard has joined #openstack-ironic01:19
*** achanda_ has joined #openstack-ironic01:19
*** achanda has quit IRC01:23
*** jjohnson2 has quit IRC01:23
*** achanda_ has quit IRC01:24
*** igordcard has quit IRC01:27
*** Guest63943 has quit IRC01:38
*** Marga_ has quit IRC01:44
*** Marga_ has joined #openstack-ironic01:45
*** ChuckC_ has quit IRC01:48
*** ChuckC_ has joined #openstack-ironic01:49
*** chenglch has joined #openstack-ironic01:50
*** bradjones has quit IRC02:00
*** bradjones has joined #openstack-ironic02:01
*** nosleep77 has quit IRC02:07
*** nosleep77 has joined #openstack-ironic02:11
*** Marga_ has quit IRC02:12
*** jerryz has joined #openstack-ironic02:13
*** Marga_ has joined #openstack-ironic02:13
openstackgerritNaohiro Tamura proposed openstack/ironic: Add missing exceptions into function docstring  https://review.openstack.org/15150702:19
*** Marga__ has joined #openstack-ironic02:26
*** Marga_ has quit IRC02:26
*** Marga_ has joined #openstack-ironic02:27
*** penick has joined #openstack-ironic02:27
*** Marga__ has quit IRC02:31
*** yog_ has joined #openstack-ironic02:40
*** penick has quit IRC02:40
*** ramineni has joined #openstack-ironic02:43
*** rwsu is now known as rwsu-afk02:48
*** zhenzanz_ has joined #openstack-ironic02:50
*** zhenzanz has quit IRC02:52
*** zhenzanz_ is now known as zhenzanz02:52
*** anderbubble has joined #openstack-ironic02:53
*** rloo has quit IRC02:53
*** harlowja_ is now known as harlowja_away02:58
*** bradjones has quit IRC03:00
*** bradjones has joined #openstack-ironic03:03
*** yog_ has quit IRC03:12
*** adam_g_out is now known as adam_g03:29
*** bradjones has quit IRC03:30
*** bradjones has joined #openstack-ironic03:36
*** Marga_ has quit IRC03:41
*** Marga_ has joined #openstack-ironic03:49
*** penick has joined #openstack-ironic03:54
*** ChuckC_ has quit IRC03:58
*** ChuckC_ has joined #openstack-ironic03:59
*** coolsvap|afk is now known as coolsvap04:03
*** lazy_prince has quit IRC04:06
*** killer_prince has joined #openstack-ironic04:07
*** killer_prince is now known as lazy_prince04:07
*** pensu has joined #openstack-ironic04:08
*** yog_ has joined #openstack-ironic04:21
*** rameshg87 has joined #openstack-ironic04:24
openstackgerritSirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images  https://review.openstack.org/15014204:28
*** anderbubble has quit IRC04:33
*** achanda has joined #openstack-ironic04:36
*** achanda has quit IRC04:40
*** penick_ has joined #openstack-ironic04:42
*** penick has quit IRC04:45
*** penick_ is now known as penick04:45
*** achanda has joined #openstack-ironic04:49
*** pcrews has quit IRC04:56
*** rakesh_hs has joined #openstack-ironic05:04
*** anderbubble has joined #openstack-ironic05:04
*** anderbubble has quit IRC05:06
*** penick has quit IRC05:17
*** pensu has quit IRC05:17
*** achanda has quit IRC05:20
*** chenleji has quit IRC05:29
*** chenleji has joined #openstack-ironic05:30
*** anderbubble has joined #openstack-ironic05:30
*** achanda has joined #openstack-ironic05:31
*** chenleji has quit IRC05:34
*** chenleji has joined #openstack-ironic05:35
*** pradipta_away is now known as pradipta05:38
*** achanda has quit IRC05:41
*** pensu has joined #openstack-ironic05:42
openstackgerritZhenzan Zhou proposed openstack/ironic: Add config option to control password mask for node-show  https://review.openstack.org/15068805:51
*** Marga_ has quit IRC05:53
*** Marga_ has joined #openstack-ironic05:54
openstackgerritRakesh H S proposed openstack/python-ironicclient: ironicclient node-set-maintenance to accept true/false  https://review.openstack.org/15069205:54
*** chenleji has quit IRC05:55
*** chenleji has joined #openstack-ironic05:56
*** achanda has joined #openstack-ironic05:58
openstackgerritZhenzan Zhou proposed openstack/ironic: Add policy mask_password to mask passwords in driver_info  https://review.openstack.org/15068806:21
*** achanda has quit IRC06:24
*** achanda has joined #openstack-ironic06:25
*** EmilienM is now known as EmilienM|afk06:29
*** eghobo has joined #openstack-ironic06:35
openstackgerritMerged openstack/ironic: Add 'irmc_' prefix to optional properties  https://review.openstack.org/15084906:37
*** achanda has quit IRC06:54
*** andreykurilin_ has joined #openstack-ironic07:03
*** eghobo has quit IRC07:10
*** andreykurilin|ho has joined #openstack-ironic07:11
*** andreykurilin_ has quit IRC07:12
*** anderbubble has quit IRC07:34
*** ndipanov has joined #openstack-ironic07:34
*** penick_ has joined #openstack-ironic07:34
*** ifarkas has joined #openstack-ironic07:36
*** romcheg has joined #openstack-ironic07:58
*** penick_ has quit IRC08:01
*** romcheg has quit IRC08:03
*** chlong has quit IRC08:08
*** dtantsur|afk is now known as dtantsur08:24
*** jmank has quit IRC08:24
dtantsurMorning Ironic08:24
dtantsurfinally TGIF08:24
*** jmank has joined #openstack-ironic08:24
*** erwan_taf has joined #openstack-ironic08:26
*** romcheg has joined #openstack-ironic08:29
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680308:41
*** yjiang5 has quit IRC08:47
*** yjiang5 has joined #openstack-ironic08:48
*** dtantsur is now known as dtantsur|brb08:52
*** pensu has quit IRC08:52
openstackgerritNisha Agarwal proposed openstack/ironic: Add states required for node-inspection  https://review.openstack.org/14785708:52
*** Nisha has joined #openstack-ironic08:52
*** athomas has joined #openstack-ironic08:52
*** pradipta is now known as pradipta_away08:54
*** jcoufal has joined #openstack-ironic08:57
*** zhenzanz has quit IRC09:01
*** ndipanov has quit IRC09:08
*** ndipanov has joined #openstack-ironic09:12
*** MattMan has quit IRC09:15
*** derekh has joined #openstack-ironic09:19
*** MattMan has joined #openstack-ironic09:24
openstackgerritNisha Agarwal proposed openstack/ironic: Add states required for node-inspection  https://review.openstack.org/14785709:26
openstackgerritNisha Agarwal proposed openstack/ironic: Add InspectInterface for node-introspection  https://review.openstack.org/14981709:26
*** dlpartain has joined #openstack-ironic09:26
*** jiangfei has quit IRC09:30
*** jistr has joined #openstack-ironic09:31
*** lucasagomes has joined #openstack-ironic09:33
openstackgerritNisha Agarwal proposed openstack/ironic: Hardware Inspection  https://review.openstack.org/14982309:41
openstackgerritNisha Agarwal proposed openstack/ironic: Add states required for node-inspection  https://review.openstack.org/14785709:41
openstackgerritNisha Agarwal proposed openstack/ironic: Add InspectInterface for node-introspection  https://review.openstack.org/14981709:41
openstackgerritImre Farkas proposed openstack/ironic-specs: DRAC vendor passthru for RAID management  https://review.openstack.org/10798109:46
openstackgerritFrederic Lepied proposed stackforge/ironic-discoverd: Add eDeploy plugin  https://review.openstack.org/14659909:46
*** coolsvap is now known as coolsvap|afk09:58
openstackgerritsandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool  https://review.openstack.org/13486610:01
*** Haomeng has joined #openstack-ironic10:04
*** Haomeng|2 has quit IRC10:04
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Address final comments of edf532db91  https://review.openstack.org/15158910:05
*** dlpartain has quit IRC10:15
*** pelix has joined #openstack-ironic10:20
openstackgerritMerged openstack/ironic: Fix typos in commit I68c9f9f86f5f113bb111c0f4fd83216ae0659d36  https://review.openstack.org/15141210:28
sambettsmorning dtantsur10:31
sambettsand I agree TGIF10:32
*** igordcard has joined #openstack-ironic10:40
openstackgerritNisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection  https://review.openstack.org/15159610:42
*** chlong has joined #openstack-ironic10:43
*** chenglch has left #openstack-ironic10:44
*** Nisha_away has joined #openstack-ironic10:44
openstackgerritsandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool  https://review.openstack.org/13486610:47
*** Nisha has quit IRC10:48
lucasagomessambetts, yo morning10:52
*** Nisha_away has quit IRC10:53
*** Haomeng has quit IRC10:55
*** Haomeng has joined #openstack-ironic10:56
lucasagomesit's only here or review.openstack.org is down?10:56
*** ramineni has quit IRC10:59
sambettslucasagomes: its seems a bit unstable this morning11:05
lucasagomesyeah11:05
*** athomas has quit IRC11:11
*** athomas has joined #openstack-ironic11:18
*** romcheg has quit IRC11:21
*** dmellado has joined #openstack-ironic11:21
dmelladoHi all here, I'm having issues downloading python-ironicclient from  pypi11:22
dmelladodo you know if there's any kind of issue?11:22
dmelladohttp://paste.openstack.org/show/164109/11:23
*** vdrok_afk is now known as vdrok11:23
*** dtantsur|brb is now known as dtantsur11:25
dtantsurdmellado, some DNS problems on your side?11:26
dtantsur(or maybe not your...)11:26
*** rameshg87 has quit IRC11:33
lucasagomesdmellado, 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
dmelladoit might be some issue with my side and pip, that was I was checking...11:34
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Update the expose-configdrive spec  https://review.openstack.org/15160511:36
dmelladothanks anyway dtantsur lucasagomes11:37
lucasagomesdmellado, np11:37
*** igordcard has quit IRC11:39
*** ifarkas has quit IRC11:48
*** dlpartain has joined #openstack-ironic11:56
*** andreykurilin|ho has quit IRC12:00
*** zhenzanz has joined #openstack-ironic12:12
*** yog_ has quit IRC12:22
*** DenisEgorenko1 has joined #openstack-ironic12:24
DenisEgorenko1Hi 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
DenisEgorenko1http://paste.openstack.org/show/164136/12:30
DenisEgorenko1Execution of dd command is comleted. Any ideas?12:30
DenisEgorenko1If i try to run base ubuntu again - it's ok.12:30
dtantsurhmm... lucasagomes ideas on ^^^12:31
* lucasagomes looks12:32
lucasagomesDenisEgorenko1, 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
DenisEgorenko1i use next elements: base vm ubuntu hadoop oozie mysql hive12:34
lucasagomesthat blkid is trying find the fs UUID that will later passed to the pxe config file as root=UUID12:34
lucasagomesDenisEgorenko1, right, that's why then12:34
lucasagomesthe VM element creates a full disk image12:35
DenisEgorenko1lucasagomes: should i use without VM?12:35
lucasagomesand 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
lucasagomesDenisEgorenko1, yes12:35
DenisEgorenko1lucasagomes: thank you, i will try.12:36
lucasagomesjust remove the vm element and try it again, perhaps we should document it better12:36
lucasagomesbut yeah, I hope the full disk image support merges soon12:36
lucasagomesDenisEgorenko1, np12:36
lucasagomeslemme know if that works for u12:36
DenisEgorenko1lucasagomes: ok, i will tell you, works it or not12:37
lucasagomescool :) thakns12:37
lucasagomesthanks*12:37
*** smoriya has quit IRC12:45
*** Marga_ has quit IRC12:52
*** Marga_ has joined #openstack-ironic12:53
*** lucasagomes is now known as lucas-hungry12:59
* lucas-hungry lunch brb12:59
*** Marga__ has joined #openstack-ironic13:04
*** Marga_ has quit IRC13:04
*** dlpartain has left #openstack-ironic13:06
openstackgerritDevananda van der Veen proposed openstack/ironic: Improve testing of the Node's REST API  https://review.openstack.org/15060313:10
openstackgerritDevananda van der Veen proposed openstack/ironic: Rename NOSTATE to AVAILABLE  https://review.openstack.org/14975413:10
openstackgerritDevananda van der Veen proposed openstack/ironic: Add MANAGEABLE state and associated transitions  https://review.openstack.org/15007313:10
openstackgerritDevananda van der Veen proposed openstack/ironic: Add support for API microversions  https://review.openstack.org/15082113:10
*** rameshg87 has joined #openstack-ironic13:14
devanandajroll: I believe ^ are done now13:14
*** zhenzanz has quit IRC13:16
*** romcheg has joined #openstack-ironic13:16
rameshg87devananda, can you please have a look at raid configuration spec: https://review.openstack.org/#/c/135899/13:25
jrolldevananda: you rock, are you at FOSDEM?13:25
jrollmorning all :)13:25
rameshg87morning jroll13:26
jrollheya rameshg8713:26
dtantsurdevananda, rameshg87, jroll, morning13:27
jroll\o dtantsur13:28
rameshg87dtantsur, o/13:28
*** EmilienM|afk is now known as EmilienM13:28
*** zhenzanz has joined #openstack-ironic13:29
*** foexle has joined #openstack-ironic13:30
*** dprince has joined #openstack-ironic13:32
DenisEgorenko1lucas-hungry: i rebuild image, without vm element, it works fine now! Thank you so much :-)13:37
jrolldevananda: 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
jrolldtantsur: did you see the merge conflict here? https://review.openstack.org/#/c/147523/13:39
dtantsurjroll, hell, I missed it... thanks for noting13:40
jrollyeah no13:40
jrollnp*13:40
* dtantsur things that git needs a special case for Python imports13:42
dtantsuranyone here remembers oldschool way of setting unix password without /etc/shadow and PAM?13:42
openstackgerritDmitry Tantsur proposed openstack/ironic: Add support for driver-specific periodic tasks  https://review.openstack.org/14752313:43
jrolldtantsur: do you see why I refuse to use DIB yet?13:43
dtantsurjroll, 100% >_<13:44
jrollread the code at the first midcycle I went to (last march) and kicked and screamed13:44
jrollsame for people trying to convince us to use os-*-config and rabbit as a transport layer for IPA13:45
*** rakesh_hs has quit IRC13:45
*** deva__ has joined #openstack-ironic13:45
deva__jroll: sure. seems reasonable. however13:46
deva__if downstream folks start adding features and raising their version independently13:47
deva__that'd just break all sanity13:47
deva__so13:47
jrollthat's something they'll need to deal with themselves, and not having a config option won't stop that13:47
jrolldeva__: for example, we've bumped the rpcapi version downstream, it's a pain point we deal with when rebasing13:48
deva__I'd suggest we don't do something which makes it look like we endorse that kind our change13:48
jrollso just make min_version configurable13:48
deva__that makes cross cloud compatibility totally broken13:48
jrolldoes it?13:49
deva__if two services both user the same version number13:49
deva__and provide something divergent13:49
deva__.....13:49
jrolloh, downstream api bumping, right13:50
deva__right13:50
deva__that idea makes me so sad13:50
jrollso 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
jrollmaybe we just don't do a bump for that, and break anyone depending on creds being in the response13:51
jrollindeed13:51
* jroll wonders when nova is going to start running into that problem13:51
deva__they haven't finished landing micro versions, I think13:51
jrollright13:52
jrollso presumably it will happen within a couple months of landing it13:52
jrollwhich is really lame13:52
jrollunless they still support extensions, I guess they probably will for a while13:52
deva__that's so far beyond lame13:52
deva__well. both probably13:53
jrollI'd rather not support extensions in ironic simply because being a downstream consumer is really hard13:53
jrollon the other hand, I question how much ironic needs cross-cloud compatibility, being an admin api13:54
deva__I would be more ragey at these ideas if I wasn't able to take a nap13:54
jrollwho is an admin on multiple clouds with different codebases?13:54
deva__sure. not as much an issue for us13:54
jrolls/with different codebases/by different downstream consumers/13:54
deva__this just kills users of nova13:54
jrollsure13:55
deva__which really is the main entry point for users of ironic13:55
jrollbut to that end, should *we* worry about "looking like we endorse that kind of change"13:55
jrollI guess the big question is: should we allow downstream consumers to decide when to deprecate an api version, rather than only developers?13:56
jrollor put differently: do we want to allow that via a config option or a code change13:57
*** pensu has joined #openstack-ironic13:57
*** Haomeng has quit IRC14:05
*** Haomeng has joined #openstack-ironic14:05
lucas-hungryDenisEgorenko1, np :)14:05
*** lucas-hungry is now known as lucasagomes14:06
jrollheya lucasagomes \o14:06
lucasagomesjroll, hey! morning!14:06
lucasagomesdeva__, afternoon :)14:07
jrolllucasagomes: any thoughts on this stuff? ^14:08
openstackgerritZhenzan Zhou proposed openstack/ironic: Add policy mask_password to mask passwords in driver_info  https://review.openstack.org/15068814:08
* lucasagomes will read, 1 sec14:08
jrollyeah no rush14:09
BadCub_morning Ironic14:09
jrollmorning BadCub_ :)14:09
BadCub_heya jroll14: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 numbers14:11
jrolldeva__: yeah, that's my goal with that, though we may end up with deployers allowing deprecated versions (might be ok?)14:13
jrolldeva__: or we could allow them to only set min_version higher than the code does14:13
deva__option two14:14
jrollyeah14:14
jrollwe can do this on top of the rest of this stuff14:14
* jroll keeps reviewing14: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__oh14:15
deva__there a bug then14:15
deva__if you set min version >0, but don't specify a header, it's going to be broken right now14:16
deva__I'm assuming you'll do the config thing in a follow on patch. should fix that then14:16
jrolldeva__: right, I just commented on that in the review14:17
deva__heh14:17
jrollwe should build the default from '1.%d' % MIN_VERSION14:17
deva__yes14:17
jrolleven without a config thing, it's going to trip us up later14:17
deva__I would also love to see our tempest API testing pulled in to our tree14:19
jroll+114:20
jrollhave other projects done that yet? how does it work?14:20
jrollI assume we just have tempest import from ironic or vice versa?14:20
*** openstackstatus has joined #openstack-ironic14:20
*** ChanServ sets mode: +v openstackstatus14:20
dtantsurjroll, 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 now14:21
jrolldtantsur: when I tested it, it required 3-4gb of RAM just to boot14:21
deva__or. really close14:21
jrolldtantsur: it also writes the image to /tmp so need to account for that too14:21
dtantsurjroll, hell... so don't like I have any working option for my ssh ramdisk idea...14:21
jrolldeva__: cool, might be a good hacking project for one of these midcycles14:21
deva__yah14:22
jrolldtantsur: use the coreos builder :D14:22
jrollI don't know why everything has to depend on DIB, that's silly14:22
deva__I'm also thinking about moving all the non production drivers into a separate namespace14:22
jrollmmm, that should be a good battle as to what's production and what isn't :P14:22
deva__but no idea what that looks like yet14:23
deva__heh14:23
deva__ssh and fake => bye bye14:23
jrollnaturally14:23
dtantsurjroll, 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
jrollare people running ilo/drac in production yet?14:24
openstackgerritRamakrishnan G proposed openstack/ironic: Add VirtualBox drivers and its modules  https://review.openstack.org/14630514:24
deva__that's the one that made me take want to split it14:24
Shrewsdeva__: ooh, ++ for pulling tempest tests into our tree during sprint14:24
-openstackstatus- NOTICE: zuul isn't running jobs since ~10:30 utc, investigation underway14:24
*** ChanServ changes topic to "zuul isn't running jobs since ~10:30 utc, investigation underway"14:24
deva__vbox14:24
jrolldtantsur: I kind of hate to say this, let other people build on it for their use case14:24
jrollah yeah14:25
deva__oh and reminder for folks14:25
deva__next week is kilo 214:26
deva__review all the things, please. prioritize by the wisdom that is launch pad14:26
jrolldang14:26
*** lazy_prince has quit IRC14:27
deva__out use your own wisdom. because launchpad sucks. even though I try to use it14:27
deva__*or14:27
jrollthis needs to be bumped, I assume https://blueprints.launchpad.net/ironic/+spec/implement-cleaning-states14:28
deva__yah14:28
jrollmrda-away: still working on https://blueprints.launchpad.net/ironic/+spec/logical-names ?14:28
openstackgerritVictor Lowther proposed openstack/ironic: Add DRAC BIOS config vendor passthru API -- DO NOT MERGE  https://review.openstack.org/15048814:28
jrolloh, that was updated last week, hmm14:28
deva__I review it yesterday14:28
jrollyeah14:29
deva__very close. but wasn't working for me14:29
jrolljust haven't noticed action on it in a while14:29
deva__also, now that we have a hammer, by which I mean micro versioning, I see nails14:29
*** killer_prince has joined #openstack-ironic14:29
*** killer_prince is now known as lazy_prince14:30
deva__but adding new API fields -- definitely a version bump14:30
Shrewsdeva__: 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 busy14:32
lucasagomesyeah 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 ideas14:32
lucasagomesthe idea of not setting a version < than the code is good14:32
jrolllucasagomes: this is what made me think of it: https://review.openstack.org/15068814:33
jrollas a deployer, I want to deprecate the version that allowed viewing credentials before upstream does14:34
*** erwan_taf has quit IRC14:35
* jroll makes a note about api versioning there as well14:35
*** EmilienM is now known as EmilienM|afl14:35
*** EmilienM|afl is now known as EmilienM|afk14:35
lucasagomesjroll, right, so 1) would be a new attribute at nodes ?14:40
jrolllucasagomes: correct, we've talked about doing it before14:41
jrollreally those are the two options I see14:41
jrollimaging you build a new ramdisk14:41
jrollneed to figure out which have the old14:41
lucasagomesyeah, this is good. Tho not sure if that would block that patch tho, would it?14:41
lucasagomesI mean, we still can have both14:42
jrollidk, I don't like the idea of blocking all of driver_info14:42
lucasagomesit just blocks the fields that has *password* in the ke14:42
lucasagomeskey*14:42
lucasagomessuch as ipmi_password, ssh_password14:42
jrollno, it blocks all of driver_info14:42
lucasagomeseverytihng?14:42
jrolloh14:42
lucasagomeswow, ok... I will give it go14:42
jrollohhhhh14:42
lucasagomesyeah if it blocks everything it sucks14:42
lucasagomesbut I don't think it does14:42
* lucasagomes it's in my todo list to test that patch14:43
jrollhmm, mask_password takes a string https://github.com/openstack/ironic/blob/master/ironic/openstack/common/strutils.py#L26414:44
jrollyeah, guess we'll need to test that patch14:44
* jroll tries14:44
lucasagomesIn [5]: strutils.mask_password(test_dict, '******')14:44
lucasagomesOut[5]: u"{'test_password': '******', 'normal_field': 'bar'}"14:44
*** zhenzanz has quit IRC14:44
jrolloh huh14:44
lucasagomesIn [4]: test_dict = {'test_password': 'foo', 'normal_field': 'bar'}14:44
jrollneat14:44
lucasagomesyeah, it just mask the fields that contains password in the key14:45
jrollnice one14:45
lucasagomesso I think it's good14:45
lucasagomesyeah, it's handy14:45
jrollshould we wait for the microversions14:45
jrollI almost think not14:45
lucasagomesyeah, I don't think we should14:45
jrollk14:45
deva__why not?14:45
lucasagomeswe could have it both, I mean it's a policy14:45
deva__oh. policy14:45
lucasagomesbecause it's not actually changing the API, it's a policy configuration14:46
lucasagomeswhether to mask some fields or not14:46
deva__right. that's fine14:46
deva__let's make sure the policy widths correctly14:46
lucasagomesif we add the credentials field, yes we bump it14:46
lucasagomesdeva__, +114:46
deva__like, create two admin users,c and don't give one this access14:46
deva__driver internal info, and node name, both should get a version bump14:47
lucasagomesindeed, yeah adding new fields14:47
deva__those are the only other ones on my radar14:47
lucasagomesyeah can't think about any other off the top of my head14:49
*** rameshg87 has quit IRC14:49
Shrewsdeva__: 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
Shrewsi mean, it looks like a very cool idea14:51
NobodyCamgood morning Ironicers14:51
NobodyCamwoo hoo its not 3 am today TGIF!14:52
*** anderbubble has joined #openstack-ironic14:52
jrolllol, good morning and happy friday NobodyCam :)14:52
Shrewslol, morning NobodyCam14:52
NobodyCammorning jroll Shrews lucasagomes and deva__ :)14:52
dtantsurNobodyCam, morning, the biggest TGIF I even imagined!!14:54
NobodyCammorning dtantsur :)14:55
lucasagomesNobodyCam, hi there14:56
NobodyCam:)14:57
* lucasagomes jumps in a call14:58
jrollShrews: 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 problems14:59
*** zz_jgrimm is now known as jgrimm14:59
* BadCub_ is off to conferencecallmageddon day 514:59
NobodyCam:O14:59
*** anderbubble has quit IRC14:59
* jroll bbiab15:00
Shrewsjroll: hrm, ok. still feels uncomfortable squeezing in this change along with another, but i understand the reason now at least. thx15:01
Shrewsi mean, it warranted its own spec for nova, and we're just throwing it in fairly casually15:02
Shrewsbut i see the need for it15:02
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680315:04
*** foexle has quit IRC15:05
*** timbyr_ is now known as kbyrne15:10
jrollShrews: right, we're basically following their spec, though some of it is specific to them like dealing with extensions15:10
jrollShrews: basically, we decided that this was worth an API rev, decided we should do API revs right from the start15:10
jrollthough if people are uncomfortable with doing it without consensus they should voice that and we should write a quick spec15:11
Shrewsjroll: i hear ya. going to spend some time trying to break this change15:15
Shrewsmaybe i'll feel better about it then15:15
jrollhehe15:15
*** naohirot has quit IRC15:15
jrollyeah I haven't actually tested it15:15
jrollgoing to add a patch on top though15:16
Shrews*tsk*  *tsk*15:16
jrollshh15:16
jrolltempest tested it for me :P15:16
Shrews:)15:16
NobodyCam:O15:16
tchaypois it possible to find how many bugs times a bug has been mentioned in a “recheck bug xxxx” message?15:19
*** kbyrne has quit IRC15:20
*** ndipanov has quit IRC15:20
NobodyCamtchaypo: I'd ask that in -infra15:21
NobodyCamI am sure there is15:21
tchaypoThis isn’t -infra?15:22
tchaypooops15:22
* tchaypo was off by one15:22
*** r-daneel has joined #openstack-ironic15:22
Shrewstchaypo: yes!15:22
Shrewstchaypo: should be on the zuul rechecks page15:22
Shrewstchaypo: http://status.openstack.org/elastic-recheck/15:22
tchaypomaybe this *was* the right place to ask ;)15:23
Shrewstchaypo: we know all the things15:23
tchaypothat page is showing empty for me15:23
tchaypooh no, it opens eventually15:23
Shrewsexcept for the things we don't know15:23
jrolls/we/Shrews15:24
tchaypoin any case - that shows me recent frequent issues. I was wondering if a particular 9-month-old bug had ever had even a single recheck15:24
Shrewstchaypo: that might require some sort of direct magical elasticsearch query. that's one of the things we/me does not know15:26
tchaypoi suspect elasticsearch doesn’t store 9 months worths of logs anyway15:27
Shrewshttp://docs.openstack.org/infra/elastic-recheck/readme.html#adding-bug-signatures15:27
Shrewstchaypo: possibly15:28
*** kbyrne has joined #openstack-ironic15:30
*** kbyrne has quit IRC15:30
*** DenisEgorenko1 has left #openstack-ironic15:31
*** kbyrne has joined #openstack-ironic15:31
tchaypono no no. Don’t get me started on that.15:34
tchaypoMy last attempt to add signatures just wasted everybody’s time15:34
tchaypohence https://review.openstack.org/#/c/151221/15:34
*** jjohnson2 has joined #openstack-ironic15:34
tchaypoI have a note to myself to udpate the elastic-recheck docs as well, I think I’ve just figured out where to update them15:35
tchaypobut actually that doc is probably okay already - it already starts by talking specifically about gate jobs15:35
openstackgerritRamakrishnan G proposed openstack/ironic: Use prolianutils module for ilo driver tests  https://review.openstack.org/15167415:36
*** achanda has joined #openstack-ironic15:37
Shrewsjroll: fear not. it seems to work as expected15:38
jrollwoo15:39
jrollsee, I can test code with my brain15:39
Shrewsjroll: you iz the awesome!15:39
jroll:P15:39
* Shrews sends all review requests telepathically to jroll15:39
openstackgerritRamakrishnan G proposed openstack/ironic: Use prolianutils module for ilo driver tests  https://review.openstack.org/15167415:39
jrolldang15:39
jrollI screwed myself over there15:40
*** dencaval has quit IRC15:40
*** achanda has quit IRC15:43
*** rainya_ is now known as rainya15:50
*** kragniz is now known as zingark15:53
*** anderbubble has joined #openstack-ironic15:55
*** jcoufal_ has joined #openstack-ironic16:00
*** pcrews has joined #openstack-ironic16:03
*** jcoufal has quit IRC16:04
*** EmilienM|afk is now known as EmilienM16:07
*** nosleep77 has quit IRC16: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 doubt16:12
lucasagomesjroll, may be a dumb question... In the network-provider spec, when is configure/unconfigure_tenant_networks actually called?16:16
lucasagomesit says, to be removed by Nova16:16
lucasagomesI don't get that bit16:17
openstackgerritJim Rollenhagen proposed openstack/ironic: Don't hardcode default minimum API version  https://review.openstack.org/15168616:17
openstackgerritJim Rollenhagen proposed openstack/ironic: Allow minimum API version to be configured  https://review.openstack.org/15168716:17
jroll^ kinda hate these, suggestions welcome16:18
jrolllucasagomes: nova does the create/delete of the port, ironic grabs that info and configures the switch16:18
lucasagomesright, nova creates it, pass the vif to ironic16:19
lucasagomesthen we call configure_tenant_networks?16:19
jrollyeah, then between deploy and instance boot ironic calls configure_...()16:19
*** david-lyle_afk is now known as david-lyle16:19
jrollbecause the node needs to be on the provisioning network during deploy16:19
lucasagomesright, then on deletion, we unconfigure and nova will delete the port in neuton?16:20
lucasagomesneutron*16:20
jrollso deploy, power off, configure_tenant_networks, power on16:20
jrollyep16:20
lucasagomesand the add_ remove_ is going to be called at initialization? By using the CONF.provision_network16:20
ShrewsAll: 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.py16:20
lucasagomesand it will create the provision network on the switch, and that's the network that is going to be used on the deployment16:20
*** jerryz has quit IRC16:22
lucasagomesjroll, right... and the add/remove provision network. When is that called? initialization ^ ?16:22
* lucasagomes wants to understand better before approving it16:22
jrolllucasagomes: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1447-1449 etc16:22
jrolllucasagomes: basically, at the same time16:22
jrollwe're switching between provisioning and tenant networks16:23
jrollso after deploy, we remove provisioning networks, configure tenant16:23
jrollat tear down, we deconfigure tenant networks, add provisioning network16:23
jrolldoes that make sense?16:24
Shrewsah, ignore my previous question16:24
vdrokShrews, API node does not have conductor_affinity attribute at all16:24
lucasagomesright, it does16:24
vdrokShrews, and morning :)16:24
lucasagomesjroll, reason why the name differences? I mean add and configure seems to do same stuff16:24
lucasagomesbut just diff networks16:24
dtantsurjroll, still not sure why mixing names16:24
dtantsuroh, lucasagomes ++ :)16:24
Shrewsvdrok: yeah. took me a bit to grok it16:24
jrolllucasagomes: dtantsur: the difference is add/remove does an actual create/delete16:25
jrollconfigure/deconfigure do not16:25
jrollI mean, it's just names, I don't actually care that much16:25
lucasagomesright, yeah docstrings can be expanded to explain that bit better16:25
lucasagomesand that's implementation detail16:25
lucasagomesthe concept looks good16:25
jrollsweet16:26
lucasagomesI'm good with the spec16:26
dtantsuraha, makes sense now16:26
jrollnow go review the code :P16:26
jroll(it still needs tests but other than that I think it's close)16:26
lucasagomesdtantsur, u want to review that first? or I will +a that16:26
lucasagomesjroll, cool, yea sorry for that. I haven't had time to look at the code16:27
dtantsurlucasagomes, I think I was it one and was pretty ok with it. Anyway, too tired to review specs now, go 4 it16:27
lucasagomescool16:27
jrollnah, it's ok, thanks for reviewing :)16:27
lucasagomesboom it's approved then16:27
jrollwoot16:28
jrollty!16:28
lucasagomesnow it's up to gerrit16:28
lucasagomes:)16:28
openstackgerritMerged openstack/ironic-specs: Add pluggable network provider spec  https://review.openstack.org/14348016:32
jrollso excited to se that land16:32
lucasagomes:)16:32
lucasagomess/gerrit/jenkins16:33
jrollhas anyone thought about rescue mode yet?16:37
jrollout of curiousity16:38
dtantsurI try not to :)16:38
*** Marga__ has quit IRC16:39
*** Marga_ has joined #openstack-ironic16:39
jrolldtantsur: well... we have it mostly working16:40
jrollbut it involves golang, and depends on this networking stuff somewhat (though I guess it would work on a flat network)16:40
dtantsurgolang... why? :)16:40
jrollbecause we can't do everything we want to from within a container16:41
jrollhttps://github.com/rackerlabs/onmetal-rescue-agent16:42
jroll(and because coreos doesn't have python)16:42
jrollalso, having IPA running on public networks seems not ideal :P16:42
dtantsurheh16:43
*** Marga_ has quit IRC16:43
jrollso trying to feel out how to work things into ironic, idk, we probably don't have time in kilo anyway16:43
*** Marga_ has joined #openstack-ironic16:44
*** deva__ has quit IRC16:45
dtantsuryeah, I feel like we have enormous amount of things to do already...16:47
*** alexpilotti has joined #openstack-ironic16:48
jrollright16:48
*** rameshg87 has joined #openstack-ironic16:48
jrollwe'll talk about it in vancouver then :)16:48
lucasagomesyeah, haven't think about rescue either16:49
*** Marga_ has quit IRC16:49
*** rwsu-afk is now known as rwsu16:49
*** ChuckC has joined #openstack-ironic17:01
*** ChuckC_ has quit IRC17:05
*** yjiang5 is now known as yginger517:06
*** ChuckC_ has joined #openstack-ironic17:06
*** zingark is now known as krangiz17:07
*** krangiz is now known as kragniz17:07
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Rework IO Worker thread behavior  https://review.openstack.org/15170617:07
openstackgerritMerged openstack/ironic: Add missing exceptions into function docstring  https://review.openstack.org/15150717: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 under17:08
*** ChuckC has quit IRC17:10
*** ChuckC has joined #openstack-ironic17:10
*** jcoufal_ has quit IRC17:10
*** ChuckC_ has quit IRC17:14
*** ijw has joined #openstack-ironic17:14
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680317:17
dtantsurhave a nice weekend everyone!17:17
*** dtantsur is now known as dtantsur|afk17:17
NobodyCamhave a great weekend dtantsur|afk17:17
jrollcheers dtantsur|afk :)17:17
lucasagomesdtantsur|afk, see ya, enjoy the weekend17:19
*** alexpilotti has quit IRC17:20
rameshg87lucasagomes, NobodyCam, got some time for https://review.openstack.org/#/c/135899/, please have a look :)17:22
openstackgerritRamakrishnan G proposed openstack/ironic: Add documentation for VirtualBox drivers  https://review.openstack.org/14630717:23
openstackgerritVladyslav Drok proposed openstack/ironic: Add put method to API to create or update a node  https://review.openstack.org/13022817:24
NobodyCamdo we have anything I can point folks to about ipmi bridging or even dbl bridging?17:25
*** derekh has quit IRC17:29
jrollNobodyCam: did you want to look at the state machine reviews?17:30
jrollthey all have at least 2x+217:30
* NobodyCam looks17:34
openstackgerritMerged openstack/ironic-specs: Update the expose-configdrive spec  https://review.openstack.org/15160517:35
openstackgerritMerged openstack/ironic: Do not POST conductor_affinity in tests  https://review.openstack.org/15126217:39
NobodyCamjroll: https://review.openstack.org/#/c/150821 lgtm +2'd waiting for Jenkins for +a17:39
jrolloh right, jenkins17:39
NobodyCam:-p17:40
openstackgerritMerged openstack/ironic: Address final comments of edf532db91  https://review.openstack.org/15158917:40
jrollI was wondering why they weren't +A17:40
*** Marga_ has joined #openstack-ironic17:40
jrollit all makes sense now17:40
lucasagomesupdated the spec in nova too for the config drive, in case someone here wants to review thathttps://review.openstack.org/#/c/151717/17:40
NobodyCamlol17:41
JayFlucasagomes: 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 responses17:42
JayFdepending on what people embed17:42
lucasagomesJayF, well... Nova doesn't want to have a hard dependency on swift17:42
JayFLike I'm OK with it, because it's how our stuff works today; but just would be curious to know the reasoning17:42
JayF(our stuff = downstream)17:42
jrollJayF: there was already an option to store it in the db, and still is17:42
lucasagomesthe last time we argued about it, for the migration from nova bm to ironic for e.g17:42
lucasagomestook over 6 months17:42
lucasagomesto have a consensus17:43
JayFI'm not saying you're wrong to give it :)17:43
lucasagomesI also prefer the old way, but I don't think it worth arguing really17:43
JayFwas just generally curious17:43
JayFs/it/in/17:43
jrollI think it's more like "let ironic handle all the configdrive things"17:43
lucasagomesJayF, I see yeah17:43
lucasagomesthere's some discussion in the Nova patch, if u wanna see17:44
lucasagomeshttps://review.openstack.org/#/c/144792/17:44
lucasagomesJayF, but yeah, to be very honest. I pretty much just want to get it done17:45
JayF+1 to jfdi17:45
JayF+17:45
*** harlowja_away is now known as harlowja_17:50
*** yginger5 has quit IRC17:51
*** alex_xu has quit IRC17:52
*** yjiang5 has joined #openstack-ironic17:54
*** yjiang5 is now known as yginger517:54
*** yginger5 has left #openstack-ironic17:54
*** bnemec is now known as beekneemech17:55
*** alex_xu has joined #openstack-ironic17:55
*** anderbubble has quit IRC17:56
lucasagomesJayF, 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 body17:57
lucasagomeswe can change and start doing what the old spec proposed in Nova17:58
lucasagomesuploading it to swift etc...17:58
lucasagomesand passing the URI via the same API we added for /provision17:58
jrollmmm, beer17:58
lucasagomesso interface-wise we still can do it if needed17:58
lucasagomes+1 :)17:58
*** ChuckC has quit IRC17:58
lucasagomesit's cold, so guinness is it17:58
jrollsounds great17:58
lucasagomesalright, so have a good night everyone18:00
*** ChuckC has joined #openstack-ironic18:00
lucasagomesenjoy the weekend!18:00
lucasagomesI talk to you guys later, and see some of you next week too18:00
*** eghobo has joined #openstack-ironic18:00
NobodyCamhave a good weekend lucasagomes18:00
*** ChuckC_ has joined #openstack-ironic18:01
*** lucasagomes is now known as lucas-beer18:02
lucas-beerNobodyCam, u too!18:02
*** datajerk has quit IRC18:02
jrollnight lucas-beer :)18:03
*** harlowja_ has quit IRC18:04
*** ChuckC has quit IRC18:05
Shrewsooh, four +2's on the microversion thingy18:06
*** harlowja has joined #openstack-ironic18:07
*** early` has quit IRC18:07
jrollstill want a spec? :P18:07
Shrewsjroll: no. but I demand a spec for your follow up patches18:08
jrollD:18:08
NobodyCamlol18:08
*** sirushti has quit IRC18:08
*** dividehex has quit IRC18:08
*** morgabra has quit IRC18:08
*** lynxman has quit IRC18:09
*** alex_xu has quit IRC18:09
*** lynxman has joined #openstack-ironic18:10
*** alex_xu has joined #openstack-ironic18:11
*** dividehex has joined #openstack-ironic18:13
*** sirushti has joined #openstack-ironic18:13
*** morgabra has joined #openstack-ironic18:13
*** rameshg87 has quit IRC18:14
*** achanda has joined #openstack-ironic18:16
*** early has joined #openstack-ironic18:16
*** penick has joined #openstack-ironic18:17
*** athomas has quit IRC18:17
*** ChuckC_ has quit IRC18:18
*** ChuckC has joined #openstack-ironic18:18
openstackgerritJay Faulkner proposed openstack/ironic: Add :raises: for Version constructor docstring  https://review.openstack.org/15172718:20
*** anderbubble has joined #openstack-ironic18:20
*** pensu has quit IRC18:22
*** tchaypo has quit IRC18:23
*** tchaypo has joined #openstack-ironic18:24
*** yjiang5 has joined #openstack-ironic18:26
*** yjiang5 is now known as yginger518:27
JayFShrews: fwiw like all but one patchset in that series have +2x4 and +118:29
JayFShrews: if you wanted to land all the things18:29
* Shrews wonders why no one is bothering to +A then18:32
JayFShrews: NobodyCam was waiting on jenkins18:32
jrolls/nobodycam/everyone18:32
JayFShrews: which has passed now, and I would approve but I don't have the big hammer there18:32
ShrewsJayF: i still need to review them, but if other cores are ready, go for it18:32
NobodyCamShrews: tests had not finished18:32
* jroll will approve18:32
jrollI got 2/3 at least18:33
NobodyCamjroll: :) nice18:36
*** spandhe has joined #openstack-ironic18:39
JayFthis is a spec that could land if it got another review -> https://review.openstack.org/#/c/13486518:40
JayFbeen through the ringer with it so it should be right as rain18:40
*** Marga_ has quit IRC18:40
JayFthis one too -> https://review.openstack.org/#/c/142401/718:41
JayFJoshNang: ^ that's the iLO cleaning steps spec; so relevant to your interests18:41
JoshNangJayF: kk i'll review18:42
JoshNangi should do the iLO zapping one as well18:42
Shrewswow. dehaan is leaving ansible18:42
JayF'leaving ansible' ?18:42
Shrewshttp://michaeldehaan.net/post/109595670406/happy-trails-ansible18:42
anderbubbleI’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.html18:42
-openstackstatus- NOTICE: Gerrit and Zuul will be offline from 1900 to 1930 UTC for project renames18:42
JayFoh, dehaan18:42
JayFI read that as 'debian'18:42
anderbubbleIf my ml2_conf.ini/ml2/type_drivers = flat, then I get regular thrown exceptions: Invalid input for operation: network_type18:43
anderbubble value 'vxlan' not supported.18:43
anderbubblebut if my ml2_conf.init/mp2/type_drivers = flat,vxlan, then the exceptions stop.18:44
*** achanda has quit IRC18:44
anderbubbleShould 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 IRC18:45
*** achanda has joined #openstack-ironic18:45
*** zsmithnyc has joined #openstack-ironic18:45
JayFHm. 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? IMBW18:45
anderbubblefull trace: http://pastebin.com/kJBZWaRa18:46
anderbubbleJayF, I’m happy to go there if you think it’d be a better target.18:46
JayFI'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
anderbubbleJayF, I’m just afraid of them saying “meh: we don’t know anything about how *ironic* is supposed to work"18:50
anderbubblebut I’ve x-posted there.18:51
*** penick has quit IRC18:51
*** MattMan has left #openstack-ironic18:52
anderbubbleWell, I appear to have found it.19:03
JayFnice!19:03
JayFConfig or code problem?19:03
anderbubbleovs_neutron_plugin.ini (as configured by RDO/packstack) had vxlan tunneling configured19:04
anderbubble(for the tenant network)19:04
JayFaha19:04
anderbubblebut since the network is flat…19:04
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Rework IO Worker thread behavior  https://review.openstack.org/15170619:05
*** penick has joined #openstack-ironic19:05
JayFMakes sense to me19:05
JayFOther 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 renames19:07
*** ChanServ changes topic to "Gerrit and Zuul are offline until 1930 UTC for project renames"19:07
*** jistr has quit IRC19:08
*** penick has quit IRC19:11
Shrewshrm, we could probably target this spec to kilo if we really wanted to: https://blueprints.launchpad.net/ironic/+spec/ironic-virtualbox-webservice-support19:11
Shrewscode seems ready (+2 from me)19:11
* Shrews actually assumed (incorrectly) that it was19:12
JayFif a thing is done; I can hardly think of a good reason not to have it in19:14
* NobodyCam waits got Gerrit to come back up19:14
Shrewsikr?19:14
NobodyCams/got/for/19:14
*** igordcard has joined #openstack-ironic19:15
*** jjohnson2 has quit IRC19:23
*** pelix has quit IRC19: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 online19:30
*** eghobo has quit IRC19:31
* NobodyCam wishes virsh used a different exit code for each error 19:34
*** EmilienM is now known as EmilienM|afk19:47
*** PaulCzar has joined #openstack-ironic19:59
*** mrmoje has joined #openstack-ironic20:01
*** eghobo has joined #openstack-ironic20:04
NobodyCamall I have a POC that I'd luv to get some feed back on before I actually code it.. http://paste.openstack.org/show/5iolH6yTtlE6hs5AVPy720:05
*** openstackgerrit has quit IRC20:06
*** openstackgerrit has joined #openstack-ironic20:07
*** penick has joined #openstack-ironic20:18
*** nosleep77 has joined #openstack-ironic20:20
*** david-lyle has quit IRC20:25
*** harlowja is now known as harlowja_away20:29
*** harlowja_away is now known as harlowja20:34
*** alexpilotti has joined #openstack-ironic20:34
*** alexpilotti has quit IRC20:34
*** spandhe has quit IRC20:50
*** penick has quit IRC20:50
*** lucas-beer has quit IRC20:57
*** spandhe has joined #openstack-ironic20:59
*** mlsoule has quit IRC21:01
openstackgerritMerged openstack/ironic: Add support for API microversions  https://review.openstack.org/15082121:05
openstackgerritMerged openstack/ironic: Rename NOSTATE to AVAILABLE  https://review.openstack.org/14975421:06
openstackgerritMerged openstack/ironic: Improve testing of the Node's REST API  https://review.openstack.org/15060321:06
openstackgerritMerged openstack/ironic: Add :raises: for Version constructor docstring  https://review.openstack.org/15172721:06
*** Haomeng has quit IRC21:07
*** Haomeng has joined #openstack-ironic21:08
JayFvictor_lowther: around?21:10
*** penick has joined #openstack-ironic21:10
JayFvictor_lowther: cancel ping, I'm not the droid you're looking for21:11
*** dprince has quit IRC21:18
*** mjturek has quit IRC21:21
openstackgerritMerged stackforge/pyghmi: Rework IO Worker thread behavior  https://review.openstack.org/15170621:38
*** achanda has quit IRC21:39
*** achanda has joined #openstack-ironic21:39
*** achanda has quit IRC21:43
*** jjohnson2 has joined #openstack-ironic21:44
*** romcheg has quit IRC21:44
*** achanda has joined #openstack-ironic21:48
*** achanda has quit IRC21:49
*** anderbubble has quit IRC21:51
*** jgrimm is now known as zz_jgrimm21:52
*** igordcard has quit IRC21:55
*** achanda has joined #openstack-ironic22:00
*** anderbubble has joined #openstack-ironic22:01
openstackgerritMerged openstack/ironic: Do not pass PXE net config from bootloader to ramdisk  https://review.openstack.org/14114822:01
*** achanda has quit IRC22:10
*** spandhe has quit IRC22:11
*** jjohnson2 has quit IRC22:12
*** achanda has joined #openstack-ironic22:12
*** achanda has quit IRC22:45
*** achanda has joined #openstack-ironic22:46
*** achanda has quit IRC22:50
*** ChuckC has quit IRC22:54
NobodyCambrb22:56
anderbubbleI 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
anderbubbletftpboot looks sane, but I get these errors in syslog: http://pastebin.com/GGGkm3zm23:03
*** harlowja is now known as harlowja_away23:05
*** BadCub_ has quit IRC23:09
*** eghobo has quit IRC23:10
anderbubbleMight 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
anderbubbleThis is probably a good feature request, actually: if possible, neutron should probably create relative symlinks.23:12
*** andreykurilin_ has joined #openstack-ironic23:12
*** eghobo has joined #openstack-ironic23:14
NobodyCamanderbubble: do you have a tftp map file?23:17
anderbubblelike this? http://pastebin.com/jKuSaW0j23:18
anderbubbleI 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
NobodyCamyep like that23:19
NobodyCam:)23:19
anderbubbleNobodyCam, SO CLOSE!!!23:19
NobodyCam:)23:19
anderbubbleI’ve got lots of notes for documentation updates, though.23:19
NobodyCamawesome!!!!!23:20
anderbubbleI had to go scold the RDO/packstack guys, too. :)23:20
NobodyCamlol23:20
anderbubbleTheir documentation still claims to provision nova-network by default in *many* places.23:20
anderbubble(though it definitely does neutron)23:20
NobodyCamhehehe23:21
anderbubbleSo, now my PXE rom is saying (transcribing from cellphone picture of console output): PXE-T02: Only absolute filenames allowed / PXE-E3C: TFTP Error - Access Violation23: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-ironic23:26
*** harlowja_away is now known as harlowja23:27
*** chuckC_ has joined #openstack-ironic23:28
*** harlowja has quit IRC23:29
*** harlowja has joined #openstack-ironic23:29
NobodyCam:( but you removed that -s no?23:32
*** penick has quit IRC23:32
NobodyCamoh not having the -s23:35
* NobodyCam can read23:35
NobodyCamanderbubble: do you have selinux running?23:36
anderbubblewhat do you mean, “running”?23:36
anderbubbleI think I realized how people fix this.23:36
NobodyCamenabled?23:36
anderbubbleln -s . /tftpboot/tftpboot23:36
anderbubbleI’m pretty sure I’ve actually seen that in production before.23:37
anderbubbleWhere would one enable it? I have it installed...23:37
NobodyCamI haven't seen that, but its been a while for me and RH/Cent23:37
NobodyCamyoud want selinux disabled23:38
anderbubbleI 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 do23:38
anderbubbleoh! sorry; I was reading syslinux23:38
anderbubbleThis isn’t an selinux problem.23:38
NobodyCam:)23:38
anderbubbleThere’s no denials in /var/log/audit/audit.log23:38
*** achanda has quit IRC23:39
NobodyCamanything in syslog or tftp?23:39
anderbubbleJust multiple requests for pxelinux.023:40
*** datajerk has joined #openstack-ironic23:40
anderbubbleIt looks like rhel’s tftp server won’t allow you to request files with relative paths if you don’t specify -s (chroot) server-side23:40
anderbubblebut ironic is expecting to be able to specify absolute paths from actual root, like /tftpboot/blah23:41
NobodyCamoh23:41
anderbubbleso creating a tftpboot symlink in /tftpboot that points at itself will fix this23:41
anderbubbleat least, that’s the theory23:41
anderbubbleand I think I’ve seen it that way before23:41
anderbubbleTHIS HAS BEEN THE BIGGEST PAIN23:42
anderbubbleEven 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
anderbubbleIT’S BOOTING!23:42
anderbubble…error23:43
NobodyCam:-/23:44
NobodyCambut it booted?23:44
*** achanda has joined #openstack-ironic23:44
anderbubbleIt booted deploy, but not image23:45
*** chuckC_ has quit IRC23:45
*** ijw has quit IRC23:45
anderbubbleI 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-ironic23:45
NobodyCamok thats progress23:46
NobodyCamwhat error did you get in the ironic-conductor.log23:46
*** anderbubble_ has joined #openstack-ironic23:47
anderbubble_insult to injury: my battery is dying23:47
*** anderbubble_ has quit IRC23:48
*** anderbubble has quit IRC23:49
NobodyCam:(23:49
*** anderbubble has joined #openstack-ironic23:50
openstackgerritHadi Bannazadeh proposed openstack/ironic: region missing in endpoint selection  https://review.openstack.org/14812823:54
anderbubbleLooks like jroll has had this problem before: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2014-11-21.log23:58
*** achanda has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!