Tuesday, 2016-07-12

*** jaybeale has quit IRC00:07
*** thiagop has quit IRC00:09
*** PollyZ has joined #openstack-ironic00:18
*** hoangcx has quit IRC00:18
*** bapalm has quit IRC00:22
*** PollyZ has quit IRC00:22
*** bapalm has joined #openstack-ironic00:25
*** yibo has joined #openstack-ironic00:26
*** mbound has joined #openstack-ironic00:41
*** fragatina has quit IRC00:44
*** mbound has quit IRC00:46
*** ijw has quit IRC00:50
*** bvandewa has joined #openstack-ironic00:52
*** bvandewa has quit IRC00:54
*** PollyZ has joined #openstack-ironic00:54
*** bvandewa has joined #openstack-ironic00:58
*** PollyZ has quit IRC01:00
*** ijw has joined #openstack-ironic01:09
*** garthb has quit IRC01:17
*** mtanin___ has joined #openstack-ironic01:21
*** mtanino has quit IRC01:22
*** rbudden has joined #openstack-ironic01:41
openstackgerritMerged openstack/ironic: Centralize config options - [glance]  https://review.openstack.org/30482901:42
*** piet has quit IRC01:52
*** _milan_ has joined #openstack-ironic01:55
*** PixieBoots has quit IRC01:55
*** PixieBoots has joined #openstack-ironic01:57
openstackgerritYuiko Takada Mori proposed openstack/ironic: [WIP] Nova-compatible serial console: socat console_utils  https://review.openstack.org/32816801:57
openstackgerritYuiko Takada Mori proposed openstack/ironic: Mock global variables  https://review.openstack.org/33866501:57
openstackgerritYuiko Takada Mori proposed openstack/ironic: IPMITool: add IPMISocatConsole class  https://review.openstack.org/29387301:57
*** milan has quit IRC01:57
*** PollyZ has joined #openstack-ironic01:59
*** PollyZ has quit IRC02:03
*** joprovost has joined #openstack-ironic02:06
*** rama_y has quit IRC02:12
openstackgerritTan Lin proposed openstack/ironic: Add a doc about appending kernel parameters to boot instances  https://review.openstack.org/32084802:12
*** Sukhdev_ has quit IRC02:15
openstackgerrityuyafei proposed openstack/ironic-lib: Correct reraising of exception  https://review.openstack.org/33879702:16
openstackgerritTan Lin proposed openstack/ironic: Improve _check_depolying_status to handle nodes in maintenance  https://review.openstack.org/32426902:18
*** yuikotakadamori has joined #openstack-ironic02:26
*** amotoki has joined #openstack-ironic02:36
*** jaybeale has joined #openstack-ironic02:40
*** piet has joined #openstack-ironic02:43
*** mtanino has joined #openstack-ironic02:45
*** mtanin___ has quit IRC02:47
*** jaybeale has quit IRC02:50
*** mtanino has quit IRC02:50
*** krtaylor has joined #openstack-ironic02:51
*** piet has quit IRC02:54
*** bvandewa has quit IRC02:54
*** ijw has quit IRC02:54
*** piet has joined #openstack-ironic02:54
*** Nisha1 has joined #openstack-ironic02:55
*** joprovost has quit IRC02:58
*** joprovost has joined #openstack-ironic02:58
*** amotoki has quit IRC02:59
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for building ISO for deploy ramdisk  https://review.openstack.org/26459003:07
*** joprovost has quit IRC03:16
*** piet has quit IRC03:16
*** amotoki has joined #openstack-ironic03:21
*** rbudden has quit IRC03:21
*** jaybeale has joined #openstack-ironic03:22
*** amotoki has quit IRC03:26
*** amotoki has joined #openstack-ironic03:27
openstackgerritNisha Agarwal proposed openstack/ironic: Config variable to configure [glance] section  https://review.openstack.org/26680303:35
*** jaybeale has quit IRC03:36
*** ijw has joined #openstack-ironic03:45
*** PollyZ has joined #openstack-ironic03:45
*** amotoki has quit IRC03:49
*** ijw has quit IRC03:50
*** mtanino has joined #openstack-ironic03:51
*** rloo has quit IRC03:59
*** amotoki has joined #openstack-ironic03:59
*** links has joined #openstack-ironic04:02
*** Sukhdev has joined #openstack-ironic04:14
*** garthb has joined #openstack-ironic04:22
*** davidlenwell has quit IRC04:28
*** bvandewa has joined #openstack-ironic04:32
*** davidlenwell has joined #openstack-ironic04:38
*** d0ugal has quit IRC05:01
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging support - objs  https://review.openstack.org/23335705:02
*** bvandewa has quit IRC05:08
*** moshele has joined #openstack-ironic05:11
*** rama_y has joined #openstack-ironic05:19
openstackgerritYuiko Takada Mori proposed openstack/ironic: IPMITool: add IPMISocatConsole class  https://review.openstack.org/29387305:20
mosheledtantsur|afk: can you remove your  -2 from https://review.openstack.org/#/c/264263/ and https://review.openstack.org/#/c/264257/. the spec in now approved05:22
*** garthb has quit IRC05:23
*** sdake_ has joined #openstack-ironic05:25
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844005:27
*** PollyZ has quit IRC05:46
*** jvgrant_ has joined #openstack-ironic05:46
*** jvgrant has quit IRC05:49
*** Sukhdev has quit IRC05:53
*** ChubYann has quit IRC05:55
*** rama_y has quit IRC05:56
*** rcernin has joined #openstack-ironic05:58
*** edand has joined #openstack-ironic06:03
*** Nisha1 has quit IRC06:03
*** amotoki has quit IRC06:03
*** d0ugal has joined #openstack-ironic06:04
openstackgerritVasyl Saienko proposed openstack/ironic: Switch to Neutron Flat network.  https://review.openstack.org/34069506:07
*** bvandewa has joined #openstack-ironic06:08
*** mjura has joined #openstack-ironic06:11
*** bvandewa_ has joined #openstack-ironic06:12
*** bvandewa has quit IRC06:12
*** lennyb has quit IRC06:12
*** rajinir has quit IRC06:15
*** amotoki has joined #openstack-ironic06:17
*** mjura has quit IRC06:18
*** lennyb has joined #openstack-ironic06:19
*** mtanino has quit IRC06:24
*** Fdaisuke has quit IRC06:29
*** mjura has joined #openstack-ironic06:31
*** hoangcx has joined #openstack-ironic06:34
*** d0ugal has quit IRC06:43
*** Fdaisuke has joined #openstack-ironic06:47
*** sdake_ has quit IRC06:49
*** amotoki has quit IRC06:52
*** mkovacik__ has joined #openstack-ironic06:56
*** _milan_ has quit IRC06:57
*** _milan_ has joined #openstack-ironic06:59
*** tesseract- has joined #openstack-ironic07:00
*** pcaruana has joined #openstack-ironic07:00
*** mkovacik__ has quit IRC07:02
*** zer0c00l_ has quit IRC07:04
*** ljjjustin has joined #openstack-ironic07:08
openstackgerritDao Cong Tien proposed openstack/ironic: Nova-compatible serial console: socat console_utils  https://review.openstack.org/32816807:12
*** zer0c00l has joined #openstack-ironic07:13
*** amotoki has joined #openstack-ironic07:14
*** fragatina has joined #openstack-ironic07:17
*** fragatina has quit IRC07:18
*** fragatina has joined #openstack-ironic07:19
*** amotoki has quit IRC07:19
*** UForgotten has joined #openstack-ironic07:22
*** ifarkas has joined #openstack-ironic07:23
*** UForgotten has quit IRC07:26
*** UForgotten has joined #openstack-ironic07:26
openstackgerritZhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter  https://review.openstack.org/27262407:27
*** daemontool has joined #openstack-ironic07:53
*** Romanenko_K has joined #openstack-ironic07:54
*** jrist has joined #openstack-ironic07:55
_milan_#pixiesay morning Ironic! -m flexing07:57
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: morning Ironic!07:57
yuikotakadamorigood morning, _milan_ and PixieBoots!07:58
_milan_yuikotakadamori, good morning :)07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** athomas has joined #openstack-ironic08:03
vsaienk0Morning _milan_, yuikotakadamori, dtantsur and all Ironic'ers08:03
_milan_vsaienk0, good morning :)08:03
yuikotakadamorivsaienk0, morning :)08:04
*** amotoki has joined #openstack-ironic08:09
*** amotoki has quit IRC08:13
*** jtomasek_ has joined #openstack-ironic08:13
*** lucas|afk is now known as lucasagomes08:15
lucasagomesmorning all :-)08:15
*** yuikotakadamori has quit IRC08:15
*** jpich has joined #openstack-ironic08:15
Romanenko_KGood morning!08:17
*** MattMan has quit IRC08:19
*** MattMan has joined #openstack-ironic08:19
*** derekh has joined #openstack-ironic08:20
*** jistr has joined #openstack-ironic08:22
Romanenko_K<lucasagomes>, could you repeat your +2 on this patch? I was fixing Ruby's comments after your previous review there: https://review.openstack.org/#/c/322788/08:32
*** yogi_ has joined #openstack-ironic08:33
alinebmorning all!08:35
ifarkasmorning all!08:36
*** moshele has quit IRC08:38
*** edand has quit IRC08:39
*** Fdaisuke has quit IRC08:39
*** moshele has joined #openstack-ironic08:40
*** mgould|afk is now known as mgould08:49
mgouldmorning Ironic08:49
mgouldand in particular _milan_ dtantsur|afk alineb lucasagomes Romanenko_K ifarkas PixieBoots08:50
*** e0ne has joined #openstack-ironic08:51
_milan_morning mgould lucasagomes ifarkas Romanenko_K :)08:51
*** electrofelix has joined #openstack-ironic08:56
*** amotoki has joined #openstack-ironic09:03
*** ohamada has joined #openstack-ironic09:04
*** dtantsur|afk is now known as dtantsur09:08
dtantsurmorning _milan_, mgould, alineb, ifarkas, lucasagomes, vsaienk0 and everyone else :)09:08
_milan_dtantsur, good morning :)09:09
lucasagomesRomanenko_K, looking09:09
*** amotoki has quit IRC09:09
lucasagomesmgoddard, dtantsur _milan_ morning :-)09:09
_milan_lucasagomes, good morning :)09:09
mgoddardmorning lucasgomes!09:09
dtantsurmoshele, done09:10
_milan_mgoddard, good morning :)09:11
mgoddardmorning _milan_09:12
mgouldmordred: mgoddard09:13
mgouldbah09:13
mgouldmorning mgoddard09:13
mgouldmy client is not smart enough to autocomplete "morning" :-)09:13
mgoddardmordred mgould :-p09:13
_milan_:)09:15
mordredzomg. I'm becomming a greeting09:17
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Small refactor in the root device loop matching logic  https://review.openstack.org/33876309:17
*** moshele has quit IRC09:19
*** mbound has joined #openstack-ironic09:21
*** moshele has joined #openstack-ironic09:22
Romanenko_Klucasagomes, thanks!09:22
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs specs: Change the file name template  https://review.openstack.org/33610809:23
lucasagomesRomanenko_K, np :-)09:24
mosheledtantsur: thanks09:24
*** ccamacho has joined #openstack-ironic09:28
mgouldmordred: sorry!09:31
mgouldalso, good morning :-)09:31
mordredmorning mgould ! and no worries ;)09:36
openstackgerritMerged openstack/python-ironicclient: Negative tests for testing chassis-create command  https://review.openstack.org/32278809:44
*** mbound has quit IRC09:56
*** amotoki has joined #openstack-ironic09:57
*** athomas has quit IRC10:01
*** d0ugal has joined #openstack-ironic10:02
*** amotoki has quit IRC10:02
*** ohamada has quit IRC10:03
*** ohamada has joined #openstack-ironic10:03
vdrokmorning ironic!10:09
*** athomas has joined #openstack-ironic10:10
*** hoangcx has quit IRC10:12
mgouldmorning vdrok!10:16
vdrokgood morning mgould :)10:18
*** ljjjustin has left #openstack-ironic10:21
*** ljjjustin has joined #openstack-ironic10:21
* mgould uses "Perl" and "principled" in the same comment on https://review.openstack.org/#/c/339596; I hope they don't take my Pythonista badge away10:28
*** ohamada has quit IRC10:32
*** ohamada has joined #openstack-ironic10:32
lucasagomesmgould, and haskell :-)10:33
mgouldmuahahaha10:42
*** sborkows has joined #openstack-ironic10:58
*** sdake has joined #openstack-ironic11:07
*** sdake_ has joined #openstack-ironic11:08
*** sdake has quit IRC11:13
*** bvandewa_ has quit IRC11:20
*** amotoki has joined #openstack-ironic11:25
*** yibo has quit IRC11:43
*** sambetts|afk is now known as sambetts11:44
sambettsMorning all11:44
vsaienk0morning sambetts11:48
dtantsurmorning sambetts11:48
vsaienk0sambetts, have you seen https://github.com/openstack/neutron/commit/b672c26cb42ad3d9a17ed049b506b5622601e891 ?11:48
sambettsvsaienk0: I hadn't but thats awesome, and we should hook into it some how :D11:50
sambettsI've just read your email11:50
vsaienk0sambetts, we can always do like nova :)11:51
sambettsyeah, I think that might be pretty cool, I still think its a shame we have to do it by API, instead of a message bus publish subscribe msg queue that we can subscribe too for notifications...11:54
*** Nisha1 has joined #openstack-ironic12:00
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Collect deployment logs from IPA  https://review.openstack.org/33610212:04
*** lucasagomes is now known as lucas-hungry12:04
*** Nisha1 has quit IRC12:04
openstackgerritYatin Kumbhare proposed openstack/ironic: Add Python 3.5 classifier and venv  https://review.openstack.org/34090212:05
sambettsdtantsur: Morning, did you see my response to deva's write up?12:10
dtantsurnot yet, just moving to it12:11
dtantsurand this it's more or less in agreement with what he wrote, no?12:12
dtantsurs/and this/I think/12:12
* dtantsur has no clue what he's typing12:12
TheJuliadtantsur: coffee?12:12
sambettsthere are 2 major corrections to his write up in my email12:12
dtantsuryeah12:12
dtantsurTheJulia, oh yeah, I definitely need more (even though it's 2pm already)12:13
TheJuliaheh12:13
sambettsjust wanted to make sure you saw them because your email touched on the same parts12:13
*** amotoki_ has joined #openstack-ironic12:13
*** amotoki has quit IRC12:13
dtantsuryeah, I think we're in agreement here too12:13
*** ccamacho is now known as ccamacho|lunch12:15
*** sdake_ has quit IRC12:16
sborkowsHi, I have a problem with testing my local change to ironic python agent. I followed the instructions in https://github.com/openstack/ironic-python-agent/tree/master/imagebuild/coreos and built cpio.gz and vmlinuz files. Now I do not know how to substitute it with actual image to my node. In general I want to check that some code from agent is called by ironic inspector during inspection process. How can I check/debug such thing?12:16
*** rbudden has joined #openstack-ironic12:17
sambettssborkows: you need to add it to glance like specified in 3. here: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements12:18
sambettssborkows: then update the node with the new UUIDs as specified in 4. here: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#flavor-creation12:18
*** mbound has joined #openstack-ironic12:21
*** piet has joined #openstack-ironic12:22
sborkowssambetts: Thanks, I will try it out.'12:23
*** caiobo has joined #openstack-ironic12:26
*** mbound has quit IRC12:26
*** trown|outtypewww is now known as trown12:26
*** dprince has joined #openstack-ironic12:27
caiobogood morning Ironic :)12:29
sambettsmorning caiobo12:29
caiobomorning sambetts!12:30
*** jrist has quit IRC12:31
mgouldmorning caiobo sambetts12:32
caiobomorning mgould12:32
* mgould wonders if he can make coffee without scalding himself this time12:33
mgouldgotta be worth a try12:33
openstackgerritVladyslav Drok proposed openstack/ironic: Expose node's network_interface field in API  https://review.openstack.org/31739212:36
*** baoli has joined #openstack-ironic12:39
*** piet has quit IRC12:40
*** piet has joined #openstack-ironic12:40
*** amotoki_ has quit IRC12:41
*** Goneri has joined #openstack-ironic12:46
*** ccamacho|lunch is now known as ccamacho12:46
*** thiagop has joined #openstack-ironic12:47
*** baoli_ has joined #openstack-ironic12:47
thiagopGood morning folks12:48
sambettsMorning thiagop12:49
*** baoli has quit IRC12:49
*** amotoki has joined #openstack-ironic12:50
*** jrist has joined #openstack-ironic12:51
*** moshele has quit IRC12:53
*** edand has joined #openstack-ironic12:53
mjturek1morning thiagop, you were looking for me yesterday12:54
mjturek1?12:54
sborkowssambetts: I have another question: am I right that the python agent image is still used during the hardware inspection? I mean, if I start inspection on a given node with modified ramdisk, will the inspector call the agent on that ramdisk?12:55
*** rloo has joined #openstack-ironic12:56
*** lucas-hungry is now known as lucasagomes12:56
sambettssborkows: no, the ironic inspector even though it uses IPA uses a separate ramdisk to the one configured in Ironic, to change the one configured for inspector you need to copy your ramdisk and kernel into your TFTP location, under the names ironic-agent.kernel and ironic-agent.initramfs12:57
* mgould has now booked his transport to the summit!12:58
*** caiobo is now known as caiobo-afk12:58
sambettsmgould: \o/12:58
sborkowssambetts: Do you mean something like 'tftpboot' or 'httpboot' directory?12:58
mgouldlooking forward to putting some faces to names :-)12:58
sambettssborkows: whichever location is configured in the ironic-inspecotor dnsmasq.conf as the tftp-root12:59
*** makowals has joined #openstack-ironic13:00
*** mbound has joined #openstack-ironic13:02
*** amotoki has quit IRC13:03
*** Goneri has quit IRC13:07
*** loki_ has joined #openstack-ironic13:07
thiagopmjturek1: Hi! Yes, I was. krtaylor said that you or him would like to participate on tha panel about 3rd party CI13:08
thiagopmjturek1: we are drafting the resume right now and I would like to check if you are still interested in joining13:09
sborkowssambetts: I found that there are ramdisk and kernel inside httpboot, not tftpboot. The node's driver is agent-ssh. Also, the whole dnsmasq.conf file is commented. Does it change anything?13:10
sambettssborkows: how are you deploying inspector devstack or is this a production system?13:12
sambettslucasagomes: do you know if we use ironic-specs for IPA???13:14
lucasagomessambetts, hi there, I don't think so if the change is only in IPA13:15
sborkowssambetts: It is a devstack deployment13:15
*** Goneri has joined #openstack-ironic13:16
sambettssborkows: in a devstack deployment normally the tftp root location is in /opt/stack/data/ironic13:16
sambettslucasagomes: hmm, I have a TODO to write a spec for doing IPA versioning stuff, and I don't know where to put it, I guess that touches Ironic too13:17
sborkowssambetts: yes, these directories I have under that location13:17
lucasagomessambetts, oh ironic-specs I think, because it will require changes to ironic no?13:17
sambettslucasagomes: yeah I think so13:17
lucasagomessambetts, wasn't the idea to query IPA about the version it has and have the ironic code to adapt to it ?13:18
jrollmorning y'all, I'm actually here today13:18
lucasagomesor something like that, so ironic-specs seems fine since the change affects both projects13:18
lucasagomesjroll, morning13:18
sambettslucasagomes: yeah I think that makes sense13:18
lucasagomesjroll, feeling better?13:18
jrollsambetts: I'd use ironic specs for IPA API versioning13:18
jrolllucasagomes: yeah, all good today, thanks :)13:19
sambettsjroll: cool :)13:19
lucasagomesgood good :-)13:19
mgouldmorning jroll, glad to hear you're feeling better13:19
jroll:)13:19
dtantsurmorning jroll13:19
sborkowssambetts: So inspector will load ramdisk image located in /opt/stack/data/ironic/httpboot/. So I cannot test my changes in python agent with inspection?13:22
sambettssborkows: yup, it should do13:23
*** amotoki has joined #openstack-ironic13:23
sborkowssambetts: So is it any possibility to test on my environment changes in python agent?13:26
sambettsyou just need copy your generated ramdisk into httpboot and replace the existing ones13:26
sambettsand it'll serve that ramdisk for inspection13:27
sborkowssambetts: even if it is an agent ramdisk?13:27
*** jcoufal has joined #openstack-ironic13:28
sambettsinspector uses an agent ramdisk, just not the one uploaded to glance13:28
sborkowssambetts: Oh, I understand now :) thanks13:28
*** amotoki has quit IRC13:28
*** jpich_ has joined #openstack-ironic13:29
*** jrist_ has joined #openstack-ironic13:29
*** jtomasek__ has joined #openstack-ironic13:29
*** d0ugal_ has joined #openstack-ironic13:30
*** jrist has quit IRC13:32
*** jpich has quit IRC13:32
*** d0ugal has quit IRC13:33
*** jtomasek_ has quit IRC13:33
*** ChrisAusten has joined #openstack-ironic13:33
*** baoli_ has quit IRC13:34
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: Fix Quick-start example syntax.  https://review.openstack.org/34094413:34
*** jrist_ has quit IRC13:34
*** d0ugal_ is now known as d0ugal13:35
*** mtanino has joined #openstack-ironic13:36
*** d0ugal is now known as Guest6571513:36
*** Guest65715 has quit IRC13:36
*** d0ugal_ has joined #openstack-ironic13:36
*** baoli has joined #openstack-ironic13:37
*** xhku has quit IRC13:38
*** xhku has joined #openstack-ironic13:39
*** baoli_ has joined #openstack-ironic13:39
*** sdake_ has joined #openstack-ironic13:41
*** baoli has quit IRC13:42
*** mtanino has quit IRC13:43
*** mgoddard_ has joined #openstack-ironic13:44
*** d0ugal_ has quit IRC13:44
vsaienk0Morning jroll, lucasagomes, thiagop and all!13:44
lucasagomesvsaienk0, hey there morning13:44
*** d0ugal has joined #openstack-ironic13:44
thiagopmorning vsaienk013:44
*** amotoki has joined #openstack-ironic13:45
*** jrist_ has joined #openstack-ironic13:45
vsaienk0folks could you please review: https://review.openstack.org/#/c/334382, the patch adds flat network support to our devstack plugin13:46
sborkowssambetts: I tried to do that and received kernel panic -> I think replacing initramfs image with cpio.gz was not correct. When building image I just ran 'make' inside ironic-python-agent/imagebuild/coreos location13:47
*** mgoddard has quit IRC13:47
*** clenimar has quit IRC13:48
*** e0ne has quit IRC13:49
*** Goneri has quit IRC13:49
sambettssborkows: I think the reason that you are seeing a kernel panic is most likely because of not having enough ram in the BM VMs to boot coreos13:50
*** PollyZ has joined #openstack-ironic13:50
sborkowssambetts: Actually it says that it cannot open device "(null)" or unknown-block(0,0)13:51
thiagopsambetts: sborkows don't know about your problem exactly, but I think I found a bug on the coreos ramdisk on master. I'm confirming it now, but appears that it can't detect disks on my hardware13:51
sambetts:/13:52
thiagopstable-mitaka is fine though13:52
thiagopYou can try to download and try it13:52
*** e0ne has joined #openstack-ironic13:53
thiagopsborkows: on devstack, you can just use the property IPA_DOWNLOAD_BRANCH=stable/mitaka on local.conf13:53
mgouldmorning vsaienk0 thiagop13:54
thiagopmorning mgould13:54
sborkowsthiagop: I actually need to check my modifications to python agent and wanted to di this with inspection13:55
thiagopsborkows: own, so you're building it13:55
thiagop:/13:55
jrollthiagop: "stable/mitaka is fine", I'd bet money on the coreos version. hrm. any special kernel mods needed for your hardware or is it just plane sata13:56
vsaienk0o/ mgould13:56
sborkowsthiagop: yes, I'm building it locally13:56
sambettssborkows: have you tried building a tinyipa version instead of coreos/13:57
sambetts?13:57
sborkowssambetts: Now I'm trying with disk-image-builder13:57
*** Goneri has joined #openstack-ironic13:57
loki_review please https://review.openstack.org/#/c/272658/13:58
sborkowsIf this won't help, I will switch to tinyipa13:58
sambettsif its an up to date master IPA then it requires more RAM to run coreos than specified in the Ironic devstack script13:59
sambettsthe default RAM amout is 1024 and it now requires 1280 at least14:00
* sambetts is going to create a bug and change that in our devstack script I think14:01
*** piet has quit IRC14:04
*** Nakato has quit IRC14:15
*** edand has quit IRC14:15
*** links has quit IRC14:15
*** Nakato has joined #openstack-ironic14:16
*** wajdi has joined #openstack-ironic14:24
openstackgerritMerged openstack/ironic: Add a doc about appending kernel parameters to boot instances  https://review.openstack.org/32084814:27
vgadirajMorning everyone, wanted to introduce myself on this channel. My name is Varun Gadiraju, I am a new OSIC member working on Ironic and I'm very excited to get started and work with all of you :)14:27
jrollvgadiraj: welcome to the party :D14:28
rloowelcome vgadiraj!14:28
*** piet has joined #openstack-ironic14:28
TheJuliait's a piet! :)  Good morning14:29
pietHi!14:29
TheJulia:)14:29
thiagopvgadiraj: Be welcome to the Ironic community, the most prone to memes and sci-fi quotes on openstack14:29
vgadirajthanks all :) I have looked through the summary of the midcycle that jroll provided me and am going through the notes from midcycle that is on etherpad to try to get up to speed14:29
vgadirajhahah i look forward to it14:30
TheJuliathiagop: I love that description!14:30
sambettshehe14:30
* rloo wonders if what thiagop said is true...14:30
thiagop#pixiesay -m happy welcome vgadiraj14:30
PixieBootsʕ^ᴥ^ʔ: welcome vgadiraj14:30
*** mtanino has joined #openstack-ironic14:32
*** rama_y has joined #openstack-ironic14:32
openstackgerritCarol Bouchard proposed openstack/ironic: Increase devstack BM VM RAM for coreos to boot  https://review.openstack.org/34097214:35
thiagopWho is Carol Bouchard here on the channel?14:36
*** sabeen has joined #openstack-ironic14:37
jrollcisco, sambetts might know :)14:37
TheJuliaHer launchpad profile doesn't list an irc handle :(14:38
thiagopsambetts: Maybe you (and/or her) should look at this too: https://github.com/openstack-infra/devstack-gate/blob/3ef48cec70c771242d743467768f22d1ecb55ecb/devstack-vm-gate.sh#L35914:38
mgouldmorning rloo TheJulia vgadiraj14:39
sambettsthiagop: woah?! We already have an override for that I thoguht!14:39
*** mgoddard has joined #openstack-ironic14:40
sambettsShe's one of my co-workers, I handed this bug off to her as she's not had much time to work on upstream this cycle but still wanted to get involved :)14:40
*** mgoddard_ has quit IRC14:40
rloohi mgould, sambetts, thiagop. Good morning TheJulia, jroll, and everyone else.14:40
thiagoprloo: morning14:40
sambettsthiagop: we inject it into devstack local config in our jobs which I guess overrides anything set by devstack-gate https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml14:41
sambettshttps://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L70314:42
sambettsincluding line number ^14:42
TheJuliagood morning mgould, rloo14:42
mariojvmorning Ironic14:42
thiagopsambetts: oww, yeah. Right.14:42
mariojvlucasagomes: I had two questions about L620 here if you have a second: https://review.openstack.org/#/c/338763/5/ironic_python_agent/hardware.py14:42
mariojv1. will hint_value implicitly be converted to a string while logging?14:43
mariojv2. does IPA use _("log message) for i18n?14:43
*** clenimar has joined #openstack-ironic14:43
sambettsthiagop: /me hates all the extra logic like that devstack gate... its a nightmare for working out whats actually going on14:43
*** clenimar_ has joined #openstack-ironic14:43
*** clenimar has quit IRC14:43
*** clenimar has joined #openstack-ironic14:44
sambettsthiagop: don't actually bother with it in my CI, I just have a local.conf file I define that I inject some extra values into14:44
thiagopsambetts: you tell me... And some options are hardcoded, I had a problem to discover how to increase deploywait_timeout on tempest14:44
thiagopsambetts: and how to influence active_timeout through other completely different variable14:45
mgouldmorning mariojv14:45
sambettsthiagop: its these sort of things that I'd love to cover in the panel14:46
thiagopsambetts: we will :)14:46
sambettsthiagop: like a "put your hand up if you use devstack-gate"14:46
thiagopthiagop: I use because we have a full infra here (zuul, jjb, jenkins, nodepool) and I think it's kind of easier from future newcomers on our project to have a similar structure to manage here and in the community14:48
*** edand has joined #openstack-ironic14:48
thiagop(and it kind of scales, but shhhh...)14:48
*** rama_y has quit IRC14:48
mariojvlucasagomes: i commented on the line in question, took away the -114:49
lucasagomesmariojv, thanks a lot will take a look14:50
mariojvnp thank you14:50
sambettsthiagop: I have a slightly modfied infra ;-P zuul + jenkins + nodepool + manually defined jobs + custom test code to run devstack and tempest with my settings14:50
lucasagomesmariojv, sorry for the delay was looking at some downstream stuff14:51
lucasagomesmariojv, I will reply inline on that patch14:51
mariojvnp14:51
mariojvthanks14:51
sambettsbut my custom test code is literally just a python test utils wrapper around setting up the local.conf + starting devstack + running tempest14:51
thiagopsambetts: jjb will start to get useful in the next month, we have brand new mitaka cloud being put in place and I plan to move the CI there :)14:51
thiagopsambetts: YEEEAHH, tempest tests running!14:52
thiagopo/\o/\o/\o/\o\/14:52
thiagopTheJulia: your guess was right. Apparently, the problem is on the last coreos build14:53
sambettsthiagop: whats the issue? missing drivers or something/14:53
sambetts?14:53
thiagopsambetts: Don't know exactly. lsblk wasn't discovering the disks correctly14:54
thiagopsambetts: I have a screenshot, let me discover how to paste it somewhere14:54
TheJuliathiagop: awesome /o\14:54
sambettshmm, what did you change to make it work? roll back the version or something?14:55
jrollheads up, v2 api meeting in 5 minutes14:55
TheJuliayes, screen shot would be awesome14:55
jrollhttps://wiki.openstack.org/wiki/Meetings/Ironic_v2_API14:55
jroll#openstack-meeting-314:55
thiagopsambetts: I just set the IPA_DOWNLOAD_BRANCH to stable/mitaka14:55
lucasagomesmariojv, replied, thanks for the review14:55
*** loki_ has quit IRC14:56
sambettsthiagop: hmm, I wonder if it because of this: https://review.openstack.org/#/c/327891/14:56
sambettsTheJulia: ^14:56
*** edand has quit IRC14:57
mariojvah, i did not catch the s after the %(...)14:57
TheJuliasambetts: that is the link I sent thiagop yesterday :)14:57
thiagopsambetts TheJulia I don't speak IPA enough to confirm anything, but...14:58
sambettsTheJulia, thiagop: could you try cloning IPA, roll back to before that change merged and try manually building a coreos image ??14:58
sambettsthiagop: ^14:58
*** jaybeale has joined #openstack-ironic14:59
TheJuliathiagop: I would be curious to see what the kernel and manual command output is between your stable/mitaka ramdisk and your master branch ramdisk15:00
jrollapi v2 meeting now in -meeting-315:00
TheJuliaand yes, sorry, that means booting the image up with an ssh key defined and collecting data :(15:00
dtantsurjroll, now? Oo15:00
jrolldtantsur: yes, that's what the email said, right?15:00
dtantsurthen I seriously misunderstood it15:00
dtantsurjroll, 1800 UTC is now?15:01
jrolldtantsur: yeah, I'm wrong15:01
jrollsomehow my local calendar was wrong15:01
sambetts:-P15:01
jrollsorry :(15:01
sambettsI was excited for a meeting that was not in the middle of me eating dinner :-P15:02
TheJuliaheh15:02
dtantsurthat's pity, I already got glad I can attend it actually :)15:02
*** mjura has quit IRC15:02
TheJuliaany meeting not in the middle of lunch is awesome to me15:02
*** jaybeale has quit IRC15:03
jrollyeah, finding a slot was hard :(15:04
TheJuliayeah :(15:04
thiagopTheJulia sambetts the output during cleaning was this -> https://owncloud.lsd.ufcg.edu.br/index.php/s/G024Uuq5eBsU93I15:05
thiagopsorry, during deployment15:05
*** jaybeale has joined #openstack-ironic15:05
sambettsI can only think that the new CoreOS version is missing some drivers from the kernel15:06
sambettsso its no longer finding your disks15:07
thiagopsambetts: I'll try to rollback the change to see what I got15:07
*** jaybeale has quit IRC15:09
*** vishwanathj has joined #openstack-ironic15:19
*** ametts has joined #openstack-ironic15:21
JayFsambetts: thiagop: Like, I don't think CoreOS dropping drivers is a valid reason to rollback that change.15:23
JayFsambetts: thiagop: We can't ship out of date software; it may just mean a CoreOS ramdisk isn't well-suited for that use case.15:23
thiagopJayF: I'll rollback locally just to check if the problem exists without the change15:24
thiagopJayF: Do you have another suggestion of ramdisk to use? TinyIPA doesn't have network drivers for our blades either...15:24
* thiagop feels like in a deadlock15:25
JayFThere are DIB ramdisks that supposedly work but are not gated15:25
JayFand those allow for more customization15:25
JayFI'm not an expert on those but there are folks in here who know things about it15:25
JayFalso, CoreOS uses stock kernels, so you can build custom modules and inject them into your image, just by building them out of vanilla kernels of the same version15:26
*** e0ne has quit IRC15:26
*** harlowja_at_home has joined #openstack-ironic15:27
thiagopJayF: the setup here is for CI, I rather not use out of date software too...15:28
JayFoh man :(15:28
JayFI would suggest going down the DIB path then15:28
JayFI think cinerama and maybe TheJulia know things about that ^ but it's not well documented yet15:28
TheJuliacinerama was beginning to look at what would be required to move the ironic-agent element into the IPA repo so we could build a gate job around it that we control, and then deprecate the old element.  I don't know the status of that because it raised a bug in diskimage-builder15:30
JayFTheJulia: I think what would help thiagop would just be docs on how to build an ironic-agent image using DIB and/or customize it if needed (althoug h I suspect a DIB ramdisk may work out of the box)15:31
TheJuliathiagop: So the ilo CI team had to do the same, so the devstack variables allow definition of other pre-built images15:33
TheJuliathiagop: I feel like we've discussed this before though.15:34
sean-k-mooneyTheJulia: hi. quick question15:35
TheJuliathiagop: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#buildingdibbaseddeployramdisk15:35
TheJuliasean-k-mooney: sure15:35
sean-k-mooneyTheJulia: does ironic observer the normal milestone 3 code freeze and milestone 2 spec approval deadline as other project such as nova?15:35
*** rajinir has joined #openstack-ironic15:35
JayFsean-k-mooney: no.15:35
sean-k-mooneyoh ok15:36
*** sdake_ has quit IRC15:36
TheJuliathiagop: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L14515:36
JayFsean-k-mooney: We do indepedent releases; we sorta view the milestones as a "good spot" to think of doing a release15:36
JayFsean-k-mooney: we generally release an X.0.0 around final release freeze, which ends up being our stable branch15:36
TheJuliaJayF: Thank you :)15:36
*** sdake_ has joined #openstack-ironic15:36
dtantsurleaving earlier today, see you tomorrow15:36
JayFTheJulia: I honestly didn't even see that was pointed at you. lol15:36
TheJuliasean-k-mooney: are you at the midcycle in Durham?15:36
sean-k-mooneyJayF: oh ok i was looking at http://releases.openstack.org/newton/schedule.html and noticed ironic follows http://governance.openstack.org/reference/tags/release_cycle-with-intermediary.html15:37
sean-k-mooneyTheJulia: am no sorry.15:37
thiagopThanks TheJulia15:37
thiagopnight dtantsur15:37
TheJuliasean-k-mooney: okay, I was going to suggest going and having a beer if you were :)15:37
*** dtantsur is now known as dtantsur|afk15:37
*** makowals has quit IRC15:38
thiagopbtw, the manual doesn't even cite TinyIPA...15:38
JayFsean-k-mooney: the way you're talking about before, with the explicit milestones, is cycle-with-milestones15:38
TheJuliathiagop: :(15:39
JayFthiagop: if you'll file a bug about that, I'm sure someone will get it in place15:39
thiagoplow-hanging-fruit15:39
JayFthiagop: yep, exactly15:39
JayFlow-hanging-fruit == contributors magicaly drop from the ceiling and fix your bug15:39
*** rloo has quit IRC15:39
JayFit's glorious :P15:39
*** rloo has joined #openstack-ironic15:40
TheJuliaThat is not always true though ;)15:40
sambettsJayF: https://review.openstack.org/#/c/34097215:40
JayFsambetts: should we only change that in the case where it's a coreos ramdisk?15:41
thiagopJayF: LOL15:41
JayFsambetts: changing that default seems like a big hammer15:41
JayFsambetts: especially since the default, tinyipa, works in 1g15:41
sean-k-mooneyJayF: yes i just did not see any newton specific timeline on the ironic wiki so i taught i would ask as im not particalary familar with the intermediary process vs milestone approch.15:41
* thiagop would love to build a tinyipa image with the drivers he needs, but..15:42
JayFsean-k-mooney: a good way to look at that would be to check our release notes15:42
JayFsean-k-mooney: we release point releases reasonably often, at least compared to many other projects15:42
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add and document the "rotational" root device hint  https://review.openstack.org/33826415:42
sambettsJayF: problem is that its hard to change once you've stacked, and people like to replace their ramdisks with thier own so I think supporting the biggest ramdisk by default is ok15:44
sean-k-mooneyJayF: thanks for the advice. ya ill check them out. just one other question then interms of accpeting new feautre is it a best effort e.g. if its merged it will be in the next dot release or is there still a deadline for inclution in a major release?15:44
*** fragatina has quit IRC15:44
JayFsambetts: I don't really agree :/ I think 1.25G of ram is bigger enough to make a difference15:45
sambettsJayF: then when someone stacks they are locked to a particular ramdisk type15:49
*** fragatina has joined #openstack-ironic15:49
*** ohamada has quit IRC15:49
*** z- has quit IRC15:50
*** z- has joined #openstack-ironic15:51
JayFsambetts: unless they specify a larger amount of ram15:51
JayFsambetts: unstack/restack does raise the ram amount15:51
JayFsambetts: which I usually do when changing ramdisks anyway15:51
rloojroll: so when is the ironic-api meeting? did it just happen, or will it be in 2 hours?15:51
*** penick has joined #openstack-ironic15:53
openstackgerritIgor Kalnitsky proposed openstack/ironic: Fix fake.FakeBoot.prepare_ramdisk() signature  https://review.openstack.org/34105615:54
*** sborkows has quit IRC15:54
vdrokrloo: 1800 utc :)15:55
rloovdrok: ok, as scheduled then. thx!15:56
TheJuliasean-k-mooney: We make a final release at the end of the cycle, so for that in a sense there are some deadlines where we simply cannot merge major changes at the end of the cycle.  Does that make sense?  Does it answer your question?15:56
sambettsJayF: then they need to know if they are going to use a different ramdisk before they stack?15:56
*** adu has joined #openstack-ironic15:57
JayFsambetts: I've personally always viewed a ramdisk change as needing an unstack/restack15:57
JayFsambetts: I know jroll uses the same pattern; I'm a little surprised to hear it isn't normal15:58
JayFbut maybe we should have that discussion in the review?15:58
*** ifarkas has quit IRC15:58
sambettsI've often uploaded my own ramdisk build to glance and then node-updated in Ironic15:59
JayFand you're not willing, as someone using that pattern, to set the VM size manually in your ironic.conf?15:59
thiagopJayF: ^this way we don't have to wait 30min to re-stack15:59
JayFI mean, if I'm wrong someone can land it over me, or convince me otherwise, I'm just not sure it's a great idea to bump that value16:00
JayFwhen msot folks probably don't have more than 8g to give to a devstack16:00
JayFyou're dropping the number of instances they can run by 1 or 216:00
*** pcaruana has quit IRC16:00
JayFbut I guess the same argument of "why can't you set it manually in that case" goes against that too16:01
*** rcernin has quit IRC16:03
sambettsas someone who has the knowledge that CoreOS has a higher ram requirement then I can make that decision but as someone who doesn't I'd be very confused to why when I uploaded a CoreOS image to my devstack install it won't boot16:03
*** ChrisAusten has quit IRC16:04
JayFI mean, sure, but there are other factors into if the ram is large enough16:04
JayFlike how large the image you're deploying is16:04
*** ChrisAusten has joined #openstack-ironic16:04
JayF(assuming it's not raw)16:04
devanandamorning, all16:06
sambettsmorning devananda16:06
mgouldmorning devananda16:06
*** daemontool has quit IRC16:06
*** dprince has quit IRC16:06
thiagopmorning devananda16:06
*** harlowja_at_home has quit IRC16:06
mariojvhi deva16:07
JayFsambetts: after reflection, going to change the vote, for this reason: we broke existing devstack configurations16:09
*** sdake_ has quit IRC16:09
JayFsambetts: when we pushed out the new coreos change16:09
*** tesseract- has quit IRC16:09
*** sdake_ has joined #openstack-ironic16:10
sambettsJayF: +1 and we can think about adding better logic after, adding better logic would also remove a big chunk of conditional from project-config16:10
sambettsJayF: I have no idea what a sane default is for a DIB based ramdisk16:11
sambetts:/16:11
JayFsambetts: we have to get DIB into the gate16:11
JayFsambetts: before we worry about it at all16:11
JayFsambetts: AFAI am concerned, it doesn't exist until/unless we're testing on it regularly. Just too much mental load to worry about something that's not tested.16:11
openstackgerritVladyslav Drok proposed openstack/ironic: Add 'neutron' network interface  https://review.openstack.org/31739316:12
openstackgerritVladyslav Drok proposed openstack/ironic: Expose node's network_interface field in API  https://review.openstack.org/31739216:12
openstackgerritVladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326216:12
openstackgerritVladyslav Drok proposed openstack/ironic: Add network interface to base driver class  https://review.openstack.org/28585216:12
*** david-lyle has joined #openstack-ironic16:13
*** joprovost has joined #openstack-ironic16:13
sambettsJayF: you might be interested in: https://review.openstack.org/#/c/340567/16:13
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add and document the "rotational" root device hint  https://review.openstack.org/33826416:14
jrollsambetts: JayF: the bummer is that someone not re-stacking can't just update a config file to bump VM ram amount, you actually have to edit or re-create the VMs in qemu16:16
sambettsyeah that sucks :(16:16
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing node-create command  https://review.openstack.org/26205516:17
Romanenko_KReduced test code size in half16:17
thiagopRomanenko_K: w00t16:18
*** daemontool has joined #openstack-ironic16:19
*** jaybeale has joined #openstack-ironic16:19
*** david-lyle has quit IRC16:21
*** rama_y has joined #openstack-ironic16:22
*** sdake_ has quit IRC16:24
*** d0ugal has quit IRC16:24
*** garthb has joined #openstack-ironic16:24
*** jpich_ has quit IRC16:24
TheJuliaJayF: sambetts: imho, just a base debian build + ironic-agent element in dib is fairly sane imho16:26
sambettsTheJulia: thats what devstack does now, but whats the RAM requirement on that?16:26
*** jtomasek__ has quit IRC16:26
TheJuliasambetts: 1GB I think, maybe debian-minimal + ironic-agent16:27
TheJuliasambetts: but I need to check, too much context switching at the moment16:27
sambettssure :)16:27
devanandaJayF: did I take that out of context, or are you suggesting we gate ironic on DIB ?16:28
*** catintheroof has joined #openstack-ironic16:28
sambettsdevananda: we're suggesting adding a job to test a DIB based IPA ramdisk on Ironic16:28
JayFdevananda: other way around. We need to test IPA aganist DIB ramdisk if we're going to "support" it16:28
sambettslike we have for the coreos / tinyipa ramdisks16:28
devanandaJayF: gotcha. and yes :)16:29
TheJuliacinerama: what is the status with dib w/r/t/ the failure upon duplicate elements being defind?16:29
*** ijw has joined #openstack-ironic16:30
*** makowals has joined #openstack-ironic16:30
*** jrist_ has quit IRC16:30
TheJuliacinerama: well, duplicate elements across paths16:31
*** fragatina has quit IRC16:31
cineramaTheJulia: so we have a change that addresses that failure more generally but i'm not sure it will land soon16:31
cineramagrabbing the link16:31
cineramahttps://review.openstack.org/#/c/33478516:33
*** trown is now known as trown|lunch16:33
cineramathere's a bunch of related changes as well16:33
*** yogi_ has quit IRC16:34
cineramamy expectation was that we could have the discussion about moving the element into the ipa tree in parallel with trying to get that to land16:34
cineramasince i anticipate that both will take a while16:34
*** e0ne has joined #openstack-ironic16:36
*** edand has joined #openstack-ironic16:36
*** Romanenko_K has quit IRC16:37
cineramaanyway the current failure is that svc-map dies on having duplicated services. ian's changes more specifically address override behaviour16:39
*** sdake_ has joined #openstack-ironic16:40
*** lucasagomes is now known as lucas|afk16:42
cineramafor the record my change is up at https://review.openstack.org/#/c/335583/ & still pending me writing an email to the list (working on that now fwiw)16:43
devanandaso, there were a couple responses to my email about network_interface. I said we'd start talking about it at 1700UTC (15min from now)16:44
devanandasambetts: thanks for the clarification in your response. i'm glad to learn that you were thinking the same thing I was :)16:45
devanandaand it was just a matter of typing too quickly in a meeting16:45
sambettsyup :)16:45
sambettshehe16:46
devanandadtantsur raised the same point in his reply, so we're all in agreement on that16:46
TheJuliaare we going to do it as a call on conference.openstack.org?16:47
*** athomas has quit IRC16:48
devanandaTheJulia: yep16:51
devanandaI am just preparing my notes16:52
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for IPA API versioning support  https://review.openstack.org/34108616:52
NobodyCamgood Morning Ironicers16:53
* devananda is having laptop audio problems16:54
thiagopmorning NobodyCam16:56
*** derekh has quit IRC16:56
*** dprince has joined #openstack-ironic16:57
rloodevananda, sambetts, TheJulia: I just posted a response. I can't attend the meeting. My contribution is this patch: https://review.openstack.org/#/c/341084/16:57
devanandarloo: thanks. I'll take a look16:58
NobodyCammorning thiagop rloo devananda TheJulia jlvillal sambetts jroll JayF dtantsur|afk vdrok and all others not directly listed here16:58
rloohi NobodyCam16:58
devanandafolks who want to talk about network_interfaces, let's use +1 512-808-5750   7777#16:58
NobodyCam:)16:58
devanandaI've also set up my notes here: https://etherpad.openstack.org/p/ironic_network_interface_discussion16:59
vdrokugh, no conferencing?16:59
sambettsdevananda: jumping on now16:59
JayFjroll: wanna land https://review.openstack.org/#/c/327437/ ?16:59
JayFNobodyCam: ^16:59
thiagoprajinir: thanks for editing the doc. Looks good to me16:59
devanandavdrok: there is a SIP interface as well, details are here: https://wiki.openstack.org/wiki/Infrastructure/Conferencing16:59
vdrokdevananda: thanks!16:59
jrollJayF: I should probably read it first17:00
rajinirthiagop: :)17:00
NobodyCamoh . I too need to read it first17:04
JayFif one or both of you read it I achieved my goal :P17:04
*** ChubYann has joined #openstack-ironic17:10
*** edand has quit IRC17:10
*** jrist_ has joined #openstack-ironic17:10
* mgould -> home; good night!17:12
*** mgould is now known as mgould|afk17:12
*** PollyZ has quit IRC17:13
*** jtomasek__ has joined #openstack-ironic17:17
*** sdake_ has quit IRC17:17
*** bvandewa has joined #openstack-ironic17:18
*** jrist_ has quit IRC17:21
*** jtomasek__ has quit IRC17:22
NobodyCamnight mgould|afk17:30
*** makowals has quit IRC17:34
*** makowals has joined #openstack-ironic17:34
*** ijw has quit IRC17:35
*** makowals has quit IRC17:35
*** PollyZ has joined #openstack-ironic17:35
*** makowals has joined #openstack-ironic17:36
*** ijw has joined #openstack-ironic17:38
*** edand has joined #openstack-ironic17:39
*** ijw has quit IRC17:39
*** PollyZ has quit IRC17:41
*** dprince has quit IRC17:41
*** trown|lunch is now known as trown17:42
*** dprince has joined #openstack-ironic17:43
*** adu has quit IRC17:44
jroll'v2 api meeting in 15 minutes, as a reminder17:44
*** bvandewa has quit IRC17:44
*** jaybeale has quit IRC17:48
*** PollyZ has joined #openstack-ironic17:49
*** edand has quit IRC17:54
*** PollyZ has quit IRC17:54
*** ijw has joined #openstack-ironic17:56
sambettshttps://review.openstack.org/#/c/341084/1/specs/approved/driver-composition-reform.rst17:56
sambettsTheJulia: ^17:56
*** electrofelix has quit IRC17:57
* vdrok leaves17:57
vdrokgood night!17:57
*** e0ne has quit IRC17:58
rloobye vdrok17:58
sambettscya vdrok17:58
jrollv2 api meeting in 1 minute17:58
thiagopnight vdrok17:59
devanandajroll: what channel?17:59
TheJuliasambetts: thank you17:59
jrolldevananda: -318:00
jrolldevananda: #openstack-meeting-3, that is18:01
*** PollyZ has joined #openstack-ironic18:07
*** amotoki has quit IRC18:07
*** sdake_ has joined #openstack-ironic18:11
*** bvandewa has joined #openstack-ironic18:14
rloodevananda, TheJulia, others: sorry I couldn't attend the network meeting. Are we good to continue with the network patches or are changes needed?18:17
*** PollyZ has quit IRC18:17
sambettsrloo: I think we're good with contining with the network patches, I'm just leaving some comments on your patch with what we discussed in the meeting18:18
rloosambetts: thx. you or anyone else are welcome to update that patch; or i can do it later today.18:19
*** bvandewa has quit IRC18:19
openstackgerritMerged openstack/ironic: Increase devstack BM VM RAM for coreos to boot  https://review.openstack.org/34097218:19
*** PollyZ has joined #openstack-ironic18:23
*** PollyZ has quit IRC18:29
*** adu has joined #openstack-ironic18:31
*** sdake_ has quit IRC18:32
*** Sukhdev has joined #openstack-ironic18:33
*** sdake_ has joined #openstack-ironic18:33
*** jaybeale has joined #openstack-ironic18:34
*** bvandewa has joined #openstack-ironic18:40
*** bvandewa has quit IRC18:41
*** bvandewa has joined #openstack-ironic18:41
*** Goneri has quit IRC18:46
mjturek1thiagop: sorry I just saw your ping from earlier.. not sure what happened there. Anyways, yes I'm definitely interested in the panel (in regards to ironic 3rd party CI)18:48
*** piet has quit IRC18:48
*** PollyZ has joined #openstack-ironic18:48
thiagopmjturek1: we are currently agreeing on content and finishing the resume to send18:48
thiagopwhat is your e-mail? (you can send it on pvt if you like)18:49
mjturek1mjturek@linux.vnet.ibm.com18:49
mjturek1thiagop: ^18:49
mjturek1thiagop: where's the discussion happening?? in an email thread?18:52
thiagope-mail/gdocs18:52
mjturek1cool cool18:52
thiagopmjturek1: I just cc'ed you on both18:53
mjturek1thiagop: great, watching for it18:53
thiagopmjturek1: please, take a look on the history of the e-mail to get up to speed on what we discussed18:55
mjturek1will do18:55
*** piet has joined #openstack-ironic18:57
*** daemontool has quit IRC19:04
sambettsdevananda: The more I dig into this the more I realise having default implementations for any interface specified in the hardware_type is pretty horrible19:05
sambettsbecause you have to provide all the driver_info for your interfaces anyway, I don't see why we need defaults, node-create creates a node in enroll state, then you should be able to syncronously update all the nodes info, e.g. interfaces and driver_info etc then move it to availible via validation19:07
*** sdake_ has quit IRC19:08
sambettswe don't have any interfaces that just magically work out of the box without providing some driver_info, which is normally unique per node, so I have to update the node anyway, so why not just be explicit about which interfaces you want at the same time?19:09
rloosambetts: but why explicitly specify the interface per node? or am i missing something?19:11
sambettswhats making me think this is that say I have a hardware_type X, and that specfies to use ipmitool power by default and also supports redfish, but in my datacenter I just want to use redfish power, I'd have to override it for every node anyway19:12
rloosambetts: not all interfaces require something to be set in driver_info?19:12
*** vishwanathj has quit IRC19:12
*** vishwanathj has joined #openstack-ironic19:13
rloosambetts: right, that's true. but if you have hardware_type Y and that specifies to use ipmitool power by default and also supports redfish, but in your datacenter you just want to use ipmitool, then no need to override.19:13
rloosambetts: maybe i don't understand, but i don't see the reason for *not* allowing for defaults.19:14
sambettsrloo: but I have to set ipmitool credtials an IP address on the node anyway so why not just set the interface too at that point19:14
rloosambetts: if you want to, go ahead and set the interface too. we allow that.19:15
sambettsrloo: problem is in my case with hardware_type X, the ipmitool interface would be loaded and enabled whether I want to enable users to use it or not19:15
rloosambetts: i don't see why we should make it mandatory to specify the interface per node. that seems too restrictive.19:15
*** PollyZ has quit IRC19:17
sambettsI think the thing thats niggling at me is that I'm not sure I like that an operator can not disable the default_X_interface for a hardware_type19:19
rloosambetts: I'm not sure what you mean by that.19:20
rloosambetts: oh. hmm...19:21
sambettse.g. because ipmitool power is the default for hardware_type X I can not disable ipmitool power in my Ironic install because its the default19:21
sambettsfor that hardware_type19:21
rloosambetts: unless we add some new 'disable' value for a node.*_interface?19:21
rloosambetts: does it make sense to disable an interface, vs specifying which interface implementation to use?19:22
sambettsthats not exactly what I mean, I mean that interfaces specified as default_x_interface in a hardware_type are automagically added to enabled_X_interfaces19:22
rloosambetts: i mean, eg does it make sense for an operator to want to disable the power_interface for a particular node?19:22
*** ijw has quit IRC19:23
rloosambetts: oh. sorry, i misunderstood.19:23
sambettsso as a deployer if I want to support hardware_type X but only on redfish, I can't disable IPMItool to prevent users to creating nodes with that interface19:24
rloosambetts: you're a mean deployer19:25
sambettswithout specified defaults on the hardware_types I'd just remove it from enabled_X_interfaces and it would be disabled and a user couldn't select it19:25
sambettsrloo: heh19:25
rloosambetts: so if you only enabled your hardware type, wouldn't that work?19:25
jrollI'm curious what the business structure looks like for a business that has deployers that have strong opinions about power management like that, and users enrolling nodes that aren't in sync with those opinions19:26
rloosambetts: anyway. i don't know if what you have is a corner case or something that certain hw vendors might want. worth writing up i guess. i am just glad it doesn't block the network patches :)19:26
*** sdake_ has joined #openstack-ironic19:26
*** jaybeale has quit IRC19:27
* sambetts needs to go and unwind, and needs to come back to this with a fresh brain and more coffee in the morning 19:28
rloobye sambetts, go unwind! :)19:29
jrollnight sambetts \o19:29
sambettso/ cya tomorrow19:29
*** sambetts is now known as sambetts|afk19:29
*** fragatina has joined #openstack-ironic19:33
*** PollyZ has joined #openstack-ironic19:38
*** PollyZ has quit IRC19:43
*** fragatina has quit IRC19:44
*** daemontool has joined #openstack-ironic19:50
*** PollyZ has joined #openstack-ironic19:52
*** ijw has joined #openstack-ironic19:54
*** PollyZ has quit IRC19:57
*** ijw has quit IRC19:59
mariojvis there a reason agent_last_heartbeat should not be a datetime in driver_internal_info in ironic? https://github.com/openstack/ironic/blob/8006ec8a519fdbb34e310e785d7647ee94f68c49/ironic/drivers/modules/agent_base_vendor.py#L46420:01
*** sdake_ has quit IRC20:01
mariojvother date-like fields are datetime fields it seems, not epoch seconds https://github.com/openstack/ironic/blob/master/ironic/objects/node.py#L9120:02
*** ijw has joined #openstack-ironic20:03
*** jcoufal has quit IRC20:05
*** PollyZ has joined #openstack-ironic20:05
mariojvperhaps it has something to do with the fact that driver_internal_info is just json, but it would be nice to have that be something more readable.20:06
* TheJulia starts reading what she missed when she took her cats to the vet20:06
*** ijw has quit IRC20:07
rloomariojv: dunno, but I think those rackspace folks did that :)20:08
jrollwe would never!20:08
JayFOh, we totally would.20:09
rloomariojv, jroll: I am sure there is a way to find out. But regardless... seems a bit late to change. or if we want to, we'll have to deal with bakcwards compatibility.20:09
jrollfwiw, that isn't actually used today20:10
rloomariojv: is it only an inconsistency issue, or is there another reason you ask?20:10
rloojroll: in that case, it can be removed.20:10
JayFit's not used /by ironic/ today20:10
jrollright20:10
JayFI can attest we use it downstream in dashboard and the like.20:10
rlooJayF: damn those downstream folks20:10
*** PollyZ has quit IRC20:10
JayFI mean, not like in downstream patches, just in dashboards :)20:10
*** ElCoyote_ has quit IRC20:11
rlooerr I mean, JayF, awesome that downstream folks are so creative20:11
TheJuliaand truthfully, having it there and understanding what it represents does make it easier when troubleshooting odd issues20:11
jrollrloo: I mean, we should keep it there because it is useful for downstreams, and also will be useful for long-running ramdisks20:11
jrollrloo: but I wouldn't be afraid of changing it without deprecation, personally (though it doesn't hurt)20:11
JayFjroll: It would be an API break, right? Unless we displayed it as epoch to older version-requesting clients?20:12
rloojroll: :) It is true that it is in driver_internal_info so that doesn't require deprecation, etc. unless it is in ironic/older versions in tree20:12
jrollJayF: we don't promise compatibility for data within json fields, only around how a json field works20:12
JayFooh, okay20:12
rlooJayF: no, not API change for driver_internal_info.20:12
jrollbut, it wouldn't be hard to deprecate right20:13
* JayF files that distinction away20:13
jrollmove to a new name, have both for a cycle20:13
*** abashmak_ has joined #openstack-ironic20:13
mariojvrloo: just an inconsistency issue, and like TheJulia said, making troubleshooting easier20:14
mariojvi always go to epochconverter.com or whatever when debugging something wrong with a node with that field20:14
mariojvor fire up a python interpreter20:14
rloomariojv: ok, i like consistency. feel free to fix as suggested by JayF20:15
*** ElCoyote_ has joined #openstack-ironic20:15
rloomariojv: or open a bug about it :)20:15
* mariojv opens the bug first20:15
TheJuliamariojv: ++ on converting the time :)20:15
mariojvi like the move to a new name have both for a cycle idea20:16
TheJuliabut yeah, converting seconds from epoch is annoying at the same time20:16
TheJuliaSeems ideal20:16
*** rama_y has quit IRC20:16
*** rama_y has joined #openstack-ironic20:16
*** PollyZ has joined #openstack-ironic20:19
*** PollyZ has quit IRC20:24
*** Sukhdev has quit IRC20:27
*** lucas|afk has quit IRC20:28
mariojvhttps://bugs.launchpad.net/ironic/+bug/160241020:30
openstackLaunchpad bug 1602410 in Ironic "agent_last_heartbeat type not consistent with other date fields" [Undecided,New] - Assigned to Mario Villaplana (mario-villaplana-j)20:30
*** PollyZ has joined #openstack-ironic20:30
mariojvrama_y: thank you for the review on https://review.openstack.org/#/c/337885/, good docs make me happy :)20:31
rama_ymariojv: you are welcome :)20:34
*** lucasagomes has joined #openstack-ironic20:35
devanandaif any spec reviewers have a few minutes, I'd be delighted if we can get another +2 on the keystone policy support spec: https://review.openstack.org/#/c/327437/20:38
*** wajdi_ has joined #openstack-ironic20:40
*** wajdi_ has quit IRC20:41
NobodyCamI'm looking now devananda :)20:42
devanandatyvm20:42
*** ccamacho has quit IRC20:42
devanandaI'll resume coding that once the spec is approved :)20:42
JayFdevananda: I shilled that to chris and jim this morning too :P20:43
*** wajdi has quit IRC20:43
devanandaJayF: hehe20:44
*** priteau has joined #openstack-ironic20:47
*** joprovost has quit IRC20:52
*** ijw has joined #openstack-ironic20:54
*** ijw has quit IRC20:55
*** ijw has joined #openstack-ironic20:55
NobodyCamJayF: devananda: +2 with a couple of nits. perfect for a follow up.20:56
*** piet has quit IRC20:56
JayFNobodyCam: let me take a quick look, I might can just turn it around real quick and then land it20:56
NobodyCamjroll: did you want to have a read over before I +a?20:56
NobodyCamJayF: comments are all minor20:57
NobodyCamI am happy to land as is and toss up a follow up.20:57
JayFif you give me 5m I'll knock out your comments + miles comments20:57
JayFand then you and deva can land?20:57
NobodyCameither way!20:58
*** jaybeale has joined #openstack-ironic20:58
*** ccamacho has joined #openstack-ironic20:58
*** adu has quit IRC20:58
* jroll +W20:59
JayFno wait21:00
JayFjroll: wait21:00
jrollbefore reading what JayF said21:00
devanandaheh21:00
JayFjroll: I have the patchset like 90% done21:00
jrollJayF: then upload it21:00
jrollit's a race!21:00
devanandaJayF: uploading before the merge bot ... or upload hte diff as a follow on :p21:00
* NobodyCam is happy either way21:00
openstackgerritJay Faulkner proposed openstack/ironic-specs: Implement Keystone Policy Support  https://review.openstack.org/32743721:01
devanandaheh21:01
jrollgg21:01
JayFman making me sweat21:01
JayFI always work faster on a deadline :P21:01
* JayF in b4 fails tests21:01
jrollsuper minor changes21:01
jrollre-approved21:01
jrollfeel free to hit me next time you see me if you think that's wrong :P21:02
NobodyCamlol21:02
* devananda re+2's21:02
devanandathanks. now I need to get back into that code ;)21:03
NobodyCam:) ++++21:03
*** piet has joined #openstack-ironic21:07
*** fragatina has joined #openstack-ironic21:08
rloojroll, devananda: wrt the network isolation patches, did we (you) decide that there should be a -2 on some patch that exposes <something?> in the API?21:08
*** trown is now known as trown|outtypewww21:09
jrolllong ago we decided all the API stuff should land at once21:09
jrollI care less about that now21:09
JayFI think the concern was to not land an api that did nothing at all21:09
rloojroll: OK, so I'm going to ignore and if anyone cares, they can -2.21:09
jrollrloo: okay21:10
openstackgerritMerged openstack/ironic-specs: Implement Keystone Policy Support  https://review.openstack.org/32743721:10
rloojroll: I asked, so I am not purposely sneaking around it :)21:10
jroll:)21:11
JayF\o/21:11
*** adu has joined #openstack-ironic21:12
*** mbound has quit IRC21:14
*** fragatina has quit IRC21:15
*** adu has quit IRC21:16
TheJuliavdrok: https://review.openstack.org/#/c/213262/125 appears to have merged with another patch?  Is that intentional21:17
*** adu has joined #openstack-ironic21:17
*** adu has quit IRC21:20
*** rcernin has joined #openstack-ironic21:21
*** PollyZ has quit IRC21:27
TheJuliaI guess it is kind of intended looking at the dependent patch21:29
*** priteau has quit IRC21:32
*** PollyZ has joined #openstack-ironic21:32
*** priteau has joined #openstack-ironic21:33
*** dprince has quit IRC21:36
openstackgerritJulia Kreger proposed openstack/bifrost: Do Not Merge: Canery test for ironic networking changes  https://review.openstack.org/33013821:38
*** ametts has quit IRC21:49
*** clenimar_ has joined #openstack-ironic21:54
*** ijw has quit IRC21:54
*** Sukhdev has joined #openstack-ironic21:57
JayFjroll: others; question:22:06
JayFfor https://review.openstack.org/#/c/301923/22:06
JayFaren't I going to have to bump the minimum version of ironic_lib in requirements/22:06
JayFbecause that code isn't even going to import on the wrong version of ironic_lib.22:06
*** clenimar_ has quit IRC22:06
*** _milan_ has quit IRC22:07
*** priteau has quit IRC22:09
*** priteau has joined #openstack-ironic22:10
openstackgerritJay Faulkner proposed openstack/ironic: Metric chassis, driver, node, and port API calls  https://review.openstack.org/30192322:10
JayF^ put a -1 on my own patch and a question about how to handle that requirements.txt22:12
*** priteau has quit IRC22:13
*** _milan_ has joined #openstack-ironic22:14
*** mbound has joined #openstack-ironic22:14
*** clenimar_ has joined #openstack-ironic22:19
*** mbound has quit IRC22:20
JayFIsn't there someone in the community who is already running IPA on PPC?22:22
JayFFor some reason I wanna think some of the IBM 3rd party CI is running on power?22:22
JayFcc: natorious ^22:22
JayFkrtaylor: ^ do you know?22:23
*** ijw has joined #openstack-ironic22:26
natoriousJayF: so to get it as close as our current coreos image, I could go with say Fedora or Debian and use a docker container22:27
JayFnatorious: well, our dib and tinycorelinux images don't use that pattern, so we don't neccessarily have to follow it. Honestly, I see deprecation of the CoreOS image in the future for upstream, because of some of the difficulties we've had around ... restraining systemd away from the tenant disk22:28
*** Sukhdev has quit IRC22:29
natoriousk, good to know22:29
natoriousso I can get all the deps in the main image then and we'd run the agent service from that22:29
natoriousdo you know if there was an agent user in other ppls setups?22:30
JayFthe dib image isn't officially supported/gated, but basically that's ssetup with a mix-and-match kinda deal22:31
JayFso you can setup access with ssh, or users, by mixing in those elements22:31
JayFthe TinyIPA image is basically a minimal setup that is pretty much designed for CI, AFAIK nobody runs it in production (now that I've said that, someone will come and prove me wrong :P)22:31
*** rcernin has quit IRC22:35
natoriousohhh, whats the pxe deploy driver?22:36
JayFso both drivers use IPA now to deploy22:37
JayFbut "pxe" driver, the agent exports all the local filesystems as iscsi targets, which are then mounted on the conductor and manipulated from there22:37
JayFit tends to be better for doing a large number of parallel installs on a fast network, since the conductor acts as a cache for the image22:38
JayFnot everyone gets a free giant fast swift cluster like we do :P22:38
natoriousbig data might not be so good though right?22:38
JayFI mean, it's only during deploy phase22:38
natoriouslike high io22:38
JayFit doesn't impact anything once booted, if you're using localboot22:39
natoriousoh, so the iscsi data is used as the image data22:39
JayF(vs non-local boot, where the server will pxe everytime, but into the OS on disk.)22:39
JayFyes, exactly22:39
natoriousnice22:39
JayFlike think of it as the conductor just mounts it up during deploy22:39
JayFpartitions, writes image out, sets up bootloader if localboot+partition image22:39
JayFthen unmounts, reboots22:39
natoriousI'm surprised there isn't an ipxe option22:40
JayFthere is, the driver is just badly named22:40
natoriouslike chain load remote kernel and initrd to automate the deploy22:40
natoriousoh, ok22:40
JayFthe "pxe" driver would be better called the "iscsi" driver22:40
JayFand it can be booted over pxe, ipxe, virtual media22:40
JayFI don't think there's a deploy driver that utilizes ipxe as part of the deploy mechanism, just as a means to get a ramdisk on the node to start booting, or a means to give a kernel/initrd for non-localboot after deploy22:41
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Fix docstring warnings  https://review.openstack.org/33985222:41
natoriousJayF: yeah, maybe thats not needed right? Like what gain is there from remote booting to chain loading to copying vs just downloading and dding22:42
JayFI don't really think there is one, yeah22:43
*** Fdaisuke has joined #openstack-ironic22:45
natoriousJayF: apparently someone has done it.  Found this under 3rd party drivers - pxe_ipmitool (ppc64)22:46
natoriousUses the pxe_ipmitool driver on IBM OpenPOWER Servers22:46
JayFnatorious: yeah; I think that's being used by IBM third party ci22:46
JayFyou could maybe find the contact email for that third party ci and ask them about how they build their image?22:46
natoriousthere is also an agent_ipmitool(ppc64)22:47
JayFyeah. honestly don't put much credence in the driver names22:48
JayFthey're kinda badly named, and driver composition is being changed this cycle to make it more sensible22:48
natoriousnp.  Got some good dig starting points.  Thnx for the infos :)22:49
JayFnp22:49
JayFnatorious: for bonus points; when you figure it out, you could docs patch how to build a PPC DIB ramdisk22:49
natoriousindeed22:50
*** vishwanathj has quit IRC22:57
*** vishwanathj has joined #openstack-ironic22:58
*** vishwanathj has quit IRC23:02
*** vishwanathj has joined #openstack-ironic23:03
*** Sukhdev has joined #openstack-ironic23:03
*** vishwanathj has quit IRC23:07
*** PollyZ has quit IRC23:07
*** vishwanathj has joined #openstack-ironic23:08
*** ijw has quit IRC23:08
*** vishwanathj has quit IRC23:08
*** vishwanathj has joined #openstack-ironic23:09
*** catintheroof has quit IRC23:09
*** penick has quit IRC23:11
*** Sukhdev has quit IRC23:14
*** jaybeale has quit IRC23:17
*** sabeen has quit IRC23:20
*** fragatina has joined #openstack-ironic23:21
*** ijw has joined #openstack-ironic23:25
*** fragatina has quit IRC23:28
*** vishwanathj has quit IRC23:30
*** vishwanathj has joined #openstack-ironic23:31
*** ccamacho has quit IRC23:34
*** baoli_ has quit IRC23:35
*** vishwanathj has quit IRC23:36
*** vishwanathj has joined #openstack-ironic23:36
*** ijw has quit IRC23:36
*** Sukhdev has joined #openstack-ironic23:38
openstackgerritXavier proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619223:58

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