Tuesday, 2019-04-16

*** cdearborn has quit IRC00:00
eanderssonhttps://github.com/jriguera/ansible-ironic-standalone/wiki/Cloud-Init-and-Config-Drive00:03
*** dsneddon has quit IRC00:05
eanderssonHaven't read the above yet but looks like what I was looing for ^00:06
*** dsneddon has joined #openstack-ironic00:06
*** ijw has joined #openstack-ironic00:16
*** mbeierl has quit IRC00:17
*** mbeierl has joined #openstack-ironic00:23
TheJuliaeandersson: yeah, I guess you could00:24
*** dsneddon has quit IRC00:31
*** dsneddon has joined #openstack-ironic00:54
*** dsneddon has quit IRC00:59
*** dsneddon has joined #openstack-ironic01:02
*** dsneddon has quit IRC01:07
eanderssonTheJulia, just reverse engineerd what nova does was the path :p01:27
*** hwoarang has quit IRC01:28
*** hwoarang has joined #openstack-ironic01:28
eandersson*forward for us01:28
*** bfournie has joined #openstack-ironic01:38
*** rloo has quit IRC01:43
*** keekz has joined #openstack-ironic01:45
*** whoami-rajat has joined #openstack-ironic01:53
*** kiseok7 has joined #openstack-ironic01:57
openstackgerritKaifeng Wang proposed openstack/ironic-specs master: Support error log history  https://review.openstack.org/65281102:02
*** ijw has quit IRC02:04
*** andrein has quit IRC02:15
*** hwoarang has quit IRC02:37
*** hwoarang has joined #openstack-ironic02:39
*** dsneddon has joined #openstack-ironic03:03
*** dsneddon has quit IRC03:12
*** dsneddon has joined #openstack-ironic03:41
TheJuliaeandersson: \o/03:42
eanderssonWas thinking that it might be worth making a plugin or driver for this03:44
eanderssonbecause Ironic will have all the data we need already03:44
eanderssonor maybe just a tool to automatically render the config03:45
*** dsneddon has quit IRC03:47
*** dsneddon has joined #openstack-ironic04:12
*** dsneddon has quit IRC04:17
*** hwoarang has quit IRC04:21
*** dsneddon has joined #openstack-ironic04:22
*** hwoarang has joined #openstack-ironic04:26
*** dsneddon has quit IRC04:27
*** e0ne has joined #openstack-ironic04:53
*** e0ne has quit IRC04:56
*** dsneddon has joined #openstack-ironic05:02
*** gyee has quit IRC05:08
*** dsneddon has quit IRC05:15
*** dsneddon has joined #openstack-ironic05:15
TheJuliaeandersson: hmm, interesting idea.05:18
*** dsneddon has quit IRC05:20
*** e0ne has joined #openstack-ironic05:21
*** e0ne has quit IRC05:25
*** dsneddon has joined #openstack-ironic05:49
*** dsneddon has quit IRC05:54
*** andrein has joined #openstack-ironic06:12
*** dsneddon has joined #openstack-ironic06:20
*** dsneddon has quit IRC06:25
*** pcaruana has joined #openstack-ironic06:26
*** rpittau|afk has quit IRC06:28
*** vdrok has quit IRC06:28
*** TheJulia has quit IRC06:29
*** hogepodge has quit IRC06:32
*** nishagb has quit IRC06:32
*** johnsom has quit IRC06:32
*** nishagb has joined #openstack-ironic06:36
*** dsneddon has joined #openstack-ironic06:40
*** nishagb has quit IRC06:42
*** TheJulia has joined #openstack-ironic06:43
*** dsneddon has quit IRC06:45
*** NobodyCam has quit IRC06:46
*** TheJulia has quit IRC06:47
*** nishagb has joined #openstack-ironic06:52
*** TheJulia has joined #openstack-ironic06:52
*** rpittau|afk has joined #openstack-ironic06:53
*** NobodyCam has joined #openstack-ironic06:53
*** johnsom has joined #openstack-ironic06:55
mgoddardtonyb[m]: I expect you're right about needing to install udevadm in the ironic-conductor image06:55
*** hogepodge has joined #openstack-ironic06:56
*** vdrok has joined #openstack-ironic06:58
*** e0ne has joined #openstack-ironic07:06
*** rpittau|afk is now known as rpittau07:06
rpittaugood morning ironic! o/07:06
*** e0ne has quit IRC07:09
*** e0ne has joined #openstack-ironic07:16
*** dsneddon has joined #openstack-ironic07:17
*** rcernin has quit IRC07:20
*** andrein has quit IRC07:22
*** tssurya has joined #openstack-ironic07:29
*** dsneddon has quit IRC07:41
xXraphXxhello everyone. I had a question regarding: https://github.com/openstack/networking-baremetal/blob/stable/queens/requirements.txt07:50
*** andrein has joined #openstack-ironic07:51
xXraphXxit looks like the dependency python-ironicclient>=2.2.0 is conflicting with current queens upper constraints:07:51
xXraphXxCollecting os-service-types>=1.2.0 (from openstacksdk>=0.15.0->osc-lib>=1.10.0->python-ironicclient>=2.2.0->networking-baremetal)07:51
xXraphXxopenstacksdk 0.27.0 has requirement keystoneauth1>=3.13.0, but you'll have keystoneauth1 3.4.0 which is incompatible.07:51
xXraphXxosc-lib 1.12.1 has requirement keystoneauth1>=3.7.0, but you'll have keystoneauth1 3.4.0 which is incompatible.07:51
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/65293507:52
*** iurygregory has joined #openstack-ironic07:52
*** e0ne has quit IRC07:52
iurygregorymorning all07:53
*** dsneddon has joined #openstack-ironic07:53
xXraphXxthis being said all seems to work correctly with these conflicting libs. But I was wondering if we should fix it07:54
*** diga has joined #openstack-ironic07:58
*** e0ne has joined #openstack-ironic07:58
*** e0ne has quit IRC07:59
*** johnsom has quit IRC08:04
*** johnsom has joined #openstack-ironic08:05
*** rpittau_ has joined #openstack-ironic08:05
*** rpittau_ is now known as rpittua08:05
*** rpittua is now known as rpittau_08:05
rpittau_heh bouncers08:06
*** rpittau has quit IRC08:06
*** rpittau_ is now known as rpittau08:06
*** dsneddon has quit IRC08:07
*** dsneddon has joined #openstack-ironic08:08
*** priteau has joined #openstack-ironic08:09
*** dougsz has joined #openstack-ironic08:09
*** gkadam has joined #openstack-ironic08:09
iurygregorymorning rpittau o/08:09
rpittauhey iurygregory :)08:10
*** gkadam has quit IRC08:10
*** e0ne has joined #openstack-ironic08:10
*** dsneddon has quit IRC08:13
*** andrein has quit IRC08:24
*** Lucas_Gray has joined #openstack-ironic08:26
*** lucasagomes has joined #openstack-ironic08:34
digaGood Morning rpittau iurygregory08:44
iurygregorymorning diga08:44
*** livelace has joined #openstack-ironic08:45
rpittauhi diga :)08:45
*** dsneddon has joined #openstack-ironic08:45
*** stendulker has joined #openstack-ironic08:47
*** andrein has joined #openstack-ironic08:49
*** dsneddon has quit IRC08:50
*** dsneddon has joined #openstack-ironic08:57
*** livelace has quit IRC09:00
*** dsneddon has quit IRC09:02
*** dsneddon has joined #openstack-ironic09:04
*** dsneddon has quit IRC09:09
*** livelace has joined #openstack-ironic09:11
digaiurygregory: rpittau TheJulia I want to discuss related management interface cleaning steps09:14
digaWe are working on adding reset cleaning steps to management interface in drac09:14
diganormal time of reset idrac call takes around 5-10 mins09:15
digaI added that call in management interface and its working fine09:16
digaQuestion here is "Is this a right design to add reset call in management interface if its taking 5 mins to execute"09:16
digait locks the thread till the completion09:17
digaYou can correct me if this is right approach or not09:17
* iurygregory think clean steps can take a huge time..09:18
digaokay09:18
iurygregoryyou want to stop clean steps if the time is above a specific limit or something?09:19
digayes09:19
*** dsneddon has joined #openstack-ironic09:19
digas/yes/no09:19
digaonce reset_idrac steps is called, we should let it execute otherwise node goes into bad state09:20
* iurygregory not sure if this would be good O.o 09:20
iurygregorystop the cleaning in the middle of an operation09:20
digahmm09:20
iurygregorymakes sense if you call the reset_clean to move the node to bad state09:21
digaokay09:22
iurygregoryI'm just not sure about the consequences of stopping the cleaning09:22
iurygregory.-.09:22
diga* thinking of finding exact time09:22
digait all depends on it, or assign specific time if it goes beyond that then move into bad state09:23
iurygregoryif we know in how much time each step in cleaning can take it makes sense..09:23
digayes09:23
digaI am also thinking on the same lines09:24
digaiurygregory: I dont see any bad state here, can be put into ironic.common.states.ERROR = 'error' state09:27
iurygregoryERROR is a good one XD09:28
digabut once node goes into error state, what will be another cleaning step performed on it to bring it online09:28
digawhich we can mention in the warning09:28
iurygregorytbh, no idea XD09:28
digaok09:28
iurygregoryexcept if you know the list of steps and like (1,2,3,4,5) and you stopped on 309:29
iurygregorymaybe will be necessary run 2 3 4 5 not only 3 4 509:29
rpittaudiga: from error state coming from clean, it'll be clean_failed09:30
digarpittau: okk09:30
digagot it09:30
digaiurygregory: rpittau Thank you :)09:33
rpittaudiga: yw09:34
*** Nisha_Agarwal has joined #openstack-ironic09:34
*** dtantsur|afk is now known as dtantsur09:35
dtantsurmorning ironic09:35
dtantsureandersson: I agree, we could populate bonding information, if we know what has to be put there (and personally don't, that's why I haven't implemented it).09:36
Nisha_Agarwaldtantsur, morning09:42
dtantsurmorning Nisha_Agarwal09:43
Nisha_Agarwaldtantsur, we received one bug in proliantutils which is there in ironic as well because now we uncap jsonschema https://bugs.launchpad.net/proliantutils/+bug/182408409:43
openstackLaunchpad bug 1824084 in proliantutils "testsuite incompatible with jsonschema 3.0.1" [Undecided,New]09:43
Nisha_Agarwali am working on this but just wante dto bring this to the notice of ironic community09:44
dtantsurokay, thanks for the heads up09:44
Nisha_Agarwali am planning to fix this in ironic as well.09:44
dtantsurmgoddard: ^^^ may affect your capping of proliantutils in bifrost09:44
dtantsurif we need to pull in the fix09:45
dtantsurNisha_Agarwal: is anything broken in ironic itself?09:45
Nisha_Agarwalshould i open the bug in ironic with same details09:45
Nisha_Agarwaldtantsur, yes09:45
Nisha_Agarwalthe same schema file exists in ironic as well09:45
Nisha_Agarwalwhere we are seeing SchemaError09:45
dtantsurugh09:45
Nisha_Agarwalthe code works fine with jsonschema 2.609:46
dtantsurit would be ideal if you could make your change compatible with both jsonschema 2 and 309:46
dtantsurthen we can backport it09:46
Nisha_Agarwalbut with jsonschema 3.0.109:46
mgoddardNisha_Agarwal: will the fix be backported to proliantutils 2.7.x?09:46
Nisha_Agarwalsure i would propose the change to ironci soon09:46
Nisha_Agarwalmgoddard, proliantutils doesnt has any backporting09:46
Nisha_Agarwalwe will release proliantutils with the fix09:47
Nisha_Agarwalmgoddard, we dont have any branching in proliantutils09:47
dtantsurthanks Nisha_Agarwal!09:47
Nisha_Agarwaldtantsur, wc :)09:48
*** rcernin has joined #openstack-ironic10:01
digadtantsur: Morning :)10:08
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [POC][WIP] Fetch baremetal endpoint from mdns  https://review.openstack.org/65201710:09
openstackgerritDmitry Tantsur proposed openstack/bifrost master: [DNM] Test the CI with mDNS resolving  https://review.openstack.org/65296210:10
dtantsurmorning diga10:10
openstackgerritDigambar proposed openstack/ironic stable/rocky: Remove nic method dependencies from dracclient==1.5.0  https://review.openstack.org/64836010:18
digadtantsur: I just proposed patch for removing list_nic_dependency for older releases10:18
digadtantsur: Also request you to take a look at this one - https://review.openstack.org/#/c/634903/10:20
patchbotpatch 634903 - ironic - Modify the iDRAC driver to use realtime RAID creation - 16 patch sets10:20
dtantsurit's still on my list, and I still have no idea when I get to it. busy time, sorry.10:20
digaNo Problem10:20
digaTheJulia: rpittau: iurygregory: I also request you guys to take a look at realtime patch - https://review.openstack.org/#/c/634903/10:21
patchbotpatch 634903 - ironic - Modify the iDRAC driver to use realtime RAID creation - 16 patch sets10:21
*** Nisha_Agarwal has quit IRC10:31
*** diga has quit IRC10:51
*** andrein has quit IRC11:04
*** Lucas_Gray has quit IRC11:09
*** Lucas_Gray has joined #openstack-ironic11:12
*** andrein has joined #openstack-ironic11:16
*** pcaruana has quit IRC11:17
*** andrein has quit IRC11:27
*** stendulker has quit IRC11:29
dtantsurTheJulia: biforst dib jobs seem broken: http://logs.openstack.org/62/652962/1/check/bifrost-integration-dibipa-debian-ubuntu-xenial/0bc8fcc/job-output.txt.gz#_2019-04-16_10_34_08_60159711:40
*** dtantsur is now known as dtantsur|brb11:45
iurygregoryubuntu-xenial =X11:47
*** Lucas_Gray has quit IRC11:52
jrolltonyb[m]: yeah, I guess so. that's used in ironic-conductor (for iscsi deploy) and ironic-python-agent (for direct deploy), so you'll need to put it in the right one, I guess11:53
jrollmorning11:53
*** pcaruana has joined #openstack-ironic12:04
*** jcoufal has joined #openstack-ironic12:07
*** rh-jelabarre has joined #openstack-ironic12:10
*** andrein has joined #openstack-ironic12:16
*** priteau has quit IRC12:17
*** priteau has joined #openstack-ironic12:19
iurygregorymorning jroll o/12:40
*** andrein has quit IRC12:48
*** andrein has joined #openstack-ironic12:48
*** baha has joined #openstack-ironic12:57
*** mjturek has joined #openstack-ironic12:58
*** rloo has joined #openstack-ironic13:02
*** pcaruana has quit IRC13:07
*** Goneri has joined #openstack-ironic13:20
*** yaawang has quit IRC13:22
*** yaawang has joined #openstack-ironic13:23
*** dtantsur|brb is now known as dtantsur13:24
*** mjturek has quit IRC13:27
dtantsurmorning jroll13:32
*** lennyb has quit IRC13:32
*** pcaruana has joined #openstack-ironic13:33
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/65293513:39
TheJuliagood morning everyone13:43
iurygregorygood morning13:43
*** ijpascual5 has joined #openstack-ironic13:44
*** sthussey has joined #openstack-ironic13:44
*** lennyb has joined #openstack-ironic13:45
*** ijpascual has quit IRC13:46
rpittauhi TheJulia :)13:47
dtantsurmorning TheJulia13:55
*** cdearborn has joined #openstack-ironic14:00
*** baha has quit IRC14:06
*** Lucas_Gray has joined #openstack-ironic14:13
TheJuliaDoes anyone have any questions for people that may be viewing the project update?14:13
TheJuliadtantsur: will you be available to run the weekly meeting on May 13th?14:16
TheJuliaOr does someone else want to?14:16
dtantsurTheJulia: I think so, yes14:16
TheJulia#open-to-volunteers14:16
rpittaubtw do we have meeting next monday ?14:17
TheJuliaWe didn't decide to cancel it.... Typically we cancel the week after14:17
TheJuliaI wouldn't mind cancelling it next week, I've got tons of things I need to take care of before I fly out on the 27th14:18
rpittauand I think next monday is bank holiday in a lot of countries14:19
*** priteau has quit IRC14:19
dtantsuryeah, including here14:19
rpittaumaybe postpone it to tuesday?14:19
TheJuliaOh right... I guess it would be good to cancel it14:19
TheJuliaNah, we don't want to try and postpone an upstream meeting14:20
TheJuliaNo need to create confusion for those that are less plugged into emails14:20
TheJuliarloo jroll arne_wiebalck: Any thoughts on cancelling the meeting next week?14:21
rpittauright, still thinking in terms of "company meetings" :P14:21
jrollTheJulia: I'm fine either way14:21
* jroll is fairly disconnected from ironic upstream recently :(14:21
TheJuliaI guess cancelling for three weeks is not a big deal in the grand scheme as long as people review stuff and occassionally the list of things is updated14:22
TheJuliajroll: it happens :(14:22
rlooTheJulia: I'm good with cancelling. (either way, not sure yet if I'll be here on Monday.)14:22
* TheJulia goes and looks up if it is a holiday for RH US employees14:22
rlooTheJulia: guess we should decide about the meeting after the open whatever whatever. PTG.14:23
TheJuliaTraditionally we cancel it and let people recoup14:23
rlooTheJulia: we should cancel that one, since the PTG goes til Sat.14:23
TheJuliayeah14:23
TheJuliaApparently friday is a holiday for me....14:24
etingofperhaps for EU folks as well14:25
*** iurygregory_ has joined #openstack-ironic14:25
rpittauetingof: AFAIK is just monday14:26
TheJuliaetingof: eh... .cz page says friday and monday14:26
iurygregory_rpittau, in czech republic friday and monday14:26
rpittau:/14:27
*** rcernin has quit IRC14:27
TheJuliaIn france, just Monday :(14:27
arne_wiebalckTheJulia: I'll be still on holidays next Monday :)14:27
rpittauyeah14:27
*** iurygregory has quit IRC14:27
rpittauwell, we have a bunch of days off in May so I guess it's fair :)14:27
*** dtantsur has quit IRC14:27
*** iurygregory_ is now known as iurygregory14:27
TheJuliasounds like there are no objections14:28
* arne_wiebalck sends greetings from Italy to ironic o/14:29
TheJuliaarne_wiebalck: enjoy your vacation in Italy :)14:29
rpittauarne_wiebalck: enjoying some good food? :)14:29
arne_wiebalckrpittau: the best14:29
rpittauxD14:29
arne_wiebalckTheJulia: thx :)14:29
*** dtantsur has joined #openstack-ironic14:31
TheJuliadtantsur: I'm just going to send an email and cancel the next 3 meetings and you or whoever wants to be the victim can run the meeting when I'm on vacation :)14:32
dtantsurTheJulia: cool, yeah :)14:32
*** iurygregory_ has joined #openstack-ironic14:32
*** iurygregory has quit IRC14:33
*** iurygregory_ is now known as iurygregory14:33
*** priteau has joined #openstack-ironic14:37
*** mjturek has joined #openstack-ironic14:41
*** baha has joined #openstack-ironic14:42
*** ajya[m] has joined #openstack-ironic14:49
*** iurygregory has quit IRC15:05
openstackgerritMerged openstack/ironic master: devstack: Remove syslinux dependency  https://review.openstack.org/65128515:15
openstackgerritMerged openstack/ironic-python-agent master: Ironic python agent does not extract correct available memory  https://review.openstack.org/64809615:16
openstackgerritMerged openstack/ironic master: Place upper bound on python-dracclient version  https://review.openstack.org/65140815:16
*** hamzy has quit IRC15:22
*** pcaruana has quit IRC15:27
*** pcaruana has joined #openstack-ironic15:30
*** e0ne has quit IRC15:31
*** iurygregory has joined #openstack-ironic15:31
*** pcaruana has quit IRC15:40
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/stein: Ironic python agent does not extract correct available memory  https://review.openstack.org/65303015:44
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/rocky: Ironic python agent does not extract correct available memory  https://review.openstack.org/65303115:45
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/queens: Ironic python agent does not extract correct available memory  https://review.openstack.org/65303215:45
dtantsurTheJulia: do we need to do any stable/stein releases?15:45
* dtantsur can check15:45
dtantsurI guess we need the ansible fixes to merge for ironic15:46
*** pcaruana has joined #openstack-ironic15:46
openstackgerritVlad Sorokin proposed openstack/networking-generic-switch master: Add Mellanox MLNX-OS Switch support  https://review.openstack.org/64256515:46
*** gyee has joined #openstack-ironic15:50
*** pcaruana has quit IRC15:55
*** bfournie has quit IRC15:58
TheJuliaYeah, we likely should release those16:00
*** pcaruana has joined #openstack-ironic16:02
*** lucasagomes has quit IRC16:07
*** pcaruana has quit IRC16:08
eanderssonMaybe we could put some of the nova code used to generate config-drive into a shared library.16:19
*** S4ren has joined #openstack-ironic16:20
*** Lucas_Gray has quit IRC16:20
dtantsureandersson: like https://github.com/openstack/openstacksdk/blob/master/openstack/baremetal/configdrive.py ?16:22
eanderssonYep - pretty close16:23
rpittauhi eandersson ;)16:26
eanderssono/16:26
*** Nisha_Agarwal has joined #openstack-ironic16:31
*** pcaruana has joined #openstack-ironic16:36
*** dtruong has quit IRC16:40
*** problem_v has quit IRC16:40
*** problem_v has joined #openstack-ironic16:41
*** dtruong has joined #openstack-ironic16:41
rpittaugood night! o/16:43
*** rpittau is now known as rpittau|afk16:43
*** tssurya has quit IRC16:47
Nisha_Agarwaldtantsur, hi16:54
*** yolanda_ has quit IRC16:57
*** dougsz has quit IRC16:59
*** ijw has joined #openstack-ironic17:00
dtantsurhi Nisha_Agarwal17:09
*** priteau has quit IRC17:11
Nisha_Agarwaldtantsur, regarding the bug for jsonschema it will require a different schema file to support darft-06 and draft-07 of json schema17:11
dtantsurNisha_Agarwal: oh, so no way to support both versions?17:12
Nisha_Agarwaldtantsur, we can have probably two schema files with same properties one for draft-04 and one for draft-6 and 0717:12
Nisha_Agarwaland in code we can validate schemas against Draft4-6 Validators17:13
dtantsurNisha_Agarwal: probably yes, especially since the CI still runs with 2.6.0 right now17:13
Nisha_Agarwalyeah17:14
Nisha_Agarwali will propose the fix today in some time or might be by tomorrow17:14
Nisha_Agarwalfor raising the bug can i just clone proliantutils bug report in the bug?17:14
dtantsurNisha_Agarwal: if proliantutils uses storyboard, you can just add a task for ironic17:21
dtantsurand thanks!17:21
Nisha_Agarwaldtantsur, still the bugs on proliantutils are raised on launchpad17:22
Nisha_Agarwaldtantsur, CI may not break if it is not testing raid feature17:23
Nisha_Agarwalthe issue is only for raid json schema17:23
Nisha_Agarwalbut yes py27 and py25 tests may fail17:23
dtantsurright17:26
dtantsurokay, then you can copy the bug description, I guess17:26
*** hamzy has joined #openstack-ironic17:27
Nisha_Agarwaldtantsur, there is one more point i dont know if community will want to support that17:27
*** Nisha_Agarwal has quit IRC17:28
*** Nisha_Agarwal has joined #openstack-ironic17:30
*** S4ren has quit IRC17:30
dtantsurNisha_Agarwal: support what, both versions of jsonschema?17:30
Nisha_Agarwaldtantsur, sorry i got disconnected.....the schema fails for draft-3 also in jsonschema 2.6 also. But since the function always takes the latest validator it doesnt report error17:30
Nisha_Agarwaljsonschema 2.6 support draft 3 and 417:30
dtantsurand 3.0?17:31
Nisha_Agarwaljsonschema 3.0 supports all 417:31
Nisha_Agarwali.e 3,4,6 and 717:31
*** e0ne has joined #openstack-ironic17:32
dtantsurpardon my ignorance, can we somehow declare using e.g. draft 4?17:32
dtantsurso that we're compatible with all versions?17:32
Nisha_Agarwalyes we can define schema as draft0417:33
Nisha_Agarwaland then it can be used for both 2.6 and 3.0.117:33
dtantsurthis sounds like the least invasive change, right?17:33
Nisha_Agarwalyes17:33
Nisha_Agarwalbut dont know if shud support old schema given draft-07 is already out17:33
dtantsurwe can start with this solution to solve the immediate problem (plus backport it)17:34
dtantsurthen we can bump requirements.txt to jsonschema>=3.0.1 and use draft-07 explicitly17:34
Nisha_Agarwalok17:34
dtantsurthe latter will have to wait until upper-constraints are updated17:34
Nisha_Agarwalyes17:34
Nisha_Agarwalok then thats the most easy fix for this issue17:35
Nisha_Agarwal:)17:35
dtantsuryeah :)17:35
Nisha_Agarwalthanks :)17:35
*** dtantsur is now known as dtantsur|afk17:36
dtantsur|afkg'night, ironic17:36
Nisha_Agarwaldtantsur, can u create the story for this bug in storyboard taking it from proliantutils bug....i am unable to login to storyboard for some reason17:37
dtantsur|afkNisha_Agarwal: please send me the link to the bug17:37
* TheJulia gets off a call and needs a coffee IV17:37
Nisha_Agarwalhttps://bugs.launchpad.net/proliantutils/+bug/182408417:37
openstackLaunchpad bug 1824084 in proliantutils "testsuite incompatible with jsonschema 3.0.1" [Undecided,New]17:37
Nisha_Agarwalthanks dtantsur|afk17:37
dtantsur|afkNisha_Agarwal: https://storyboard.openstack.org/#!/story/200546417:39
Nisha_Agarwaldtantsur|afk, thank u so much17:39
dtantsur|afknp17:39
Nisha_Agarwalgood night17:39
* dtantsur|afk goes for real17:39
TheJuliagoodnight17:39
*** ijw has quit IRC17:40
*** ijw has joined #openstack-ironic17:42
openstackgerritNisha Agarwal proposed openstack/ironic master: Jsonschema 3.0.1: Binding the schema to draft-04  https://review.openstack.org/65306017:49
*** priteau has joined #openstack-ironic17:49
*** e0ne has quit IRC17:49
openstackgerritraphael.glon proposed openstack/ironic master: Truncate node text fields when too long  https://review.openstack.org/65030718:04
*** iurygregory has quit IRC18:07
*** Nisha_Agarwal has quit IRC18:08
*** ijw has quit IRC18:15
*** ijw has joined #openstack-ironic18:16
*** absubram has joined #openstack-ironic18:20
*** ijw has quit IRC18:22
*** priteau has quit IRC18:33
*** ijw has joined #openstack-ironic18:35
*** irclogbot_2 has quit IRC18:39
*** irclogbot_0 has joined #openstack-ironic18:40
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicators REST API endpoints  https://review.openstack.org/65178518:57
*** andrein has quit IRC19:02
*** andrein has joined #openstack-ironic19:12
*** andrein has quit IRC19:23
*** andrein has joined #openstack-ironic19:31
*** andrein has quit IRC19:33
*** andrein has joined #openstack-ironic19:34
*** pcaruana has quit IRC20:19
*** priteau has joined #openstack-ironic20:23
*** priteau has quit IRC20:27
*** ijw has quit IRC20:42
*** ijw has joined #openstack-ironic20:47
*** ijw has quit IRC20:54
*** hamzy has quit IRC20:54
*** seba has joined #openstack-ironic20:56
*** Goneri has quit IRC21:00
*** jcoufal has quit IRC21:07
*** baha has quit IRC21:08
eanderssonTheJulia, you here by any chance?21:17
eanderssonhttps://github.com/openstack/openstacksdk/blob/master/openstack/baremetal/configdrive.py#L7421:17
eanderssonIs this just me?21:17
eanderssonor is network_data just never passed on :p21:17
TheJuliaeandersson: braaaains21:18
TheJuliayes, I'm around. I swear I'm not a zombie21:18
eandersson:D21:18
TheJuliaNICE21:19
TheJuliaWOW that is a  nice bug21:19
TheJuliaeandersson: You are not crazy, you win today's prize21:19
eanderssonGlad I looked at the code before we started to... go crazy21:19
TheJuliaseems like a super quick fix21:20
eanderssonYep21:21
eanderssonWe can probably push one later if no one beat us to it :p21:21
TheJuliaI might be able to, but I'm bouncing between four things at the moment21:22
eanderssonSounds good - no worries21:23
*** ijw has joined #openstack-ironic21:25
TheJuliaeandersson: thanks!21:33
*** Goneri has joined #openstack-ironic21:48
eanderssonTheJulia, I don't think regions works in nova-ironic in Rocky21:54
eanderssonunless you set it manually using CONF.ironic.api_endpoint21:54
TheJuliaThat doesn't surprise me at all21:55
eanderssonkwargs is supposed to pass on region_name21:55
*** ijw has quit IRC21:55
eanderssonit isn't21:55
*** ijw has joined #openstack-ironic21:55
TheJuliaYour finding some really fun bugs then....21:56
TheJuliaI expect most operators in such a config likely directly state the endpoints and have never thought about just letting the catalog be used.21:56
eanderssonor nvm get_endpoint21:57
eanderssonis missing region_name21:57
eanderssonhttps://github.com/openstack/nova/blob/master/nova/virt/ironic/client_wrapper.py#L12222:00
TheJulialooks like before that it was purely url config based22:04
TheJuliawell, before the change to put the adapter in22:05
TheJulialooks like efried did it22:05
*** ccstone has joined #openstack-ironic22:07
*** andrein has quit IRC22:16
*** livelace has quit IRC22:24
eanderssonActually this is odd22:25
eanderssonthat code path is working22:25
eanderssonit's something after that is failing22:25
TheJuliaOo22:25
eandersson> self._cached_client22:26
eanderssonI assume this is just in-memory cache right?22:26
TheJuliaalready instansiated22:26
TheJuliaso yeah, basically22:26
TheJuliawhy?22:26
eanderssonbecause I printed the url it gives and it's correct, but the call after fails with the wrong region22:26
TheJuliamy brain's deep storage robot is currently scanning barcodes to find the tape to load22:28
eanderssonunless I am missing something, it looks like region just does not work at all22:30
eandersson*region override22:30
TheJuliamaybe we should ping efried22:30
eanderssonlooks like an issue with the client tbh22:31
eanderssonbecause the endpoint is correct22:31
TheJuliagot a indicator of where at you see it correct and then wrong?22:31
eanderssonbut the ironicclient does not repsect it22:31
eanderssonSo I added a print(ironic_url)22:31
eanderssonjust before get_client22:31
eanderssonand it prints the correct endpoint22:31
eanderssonand in the next line it shows Error contacting Ironic server: Unable to establish connection to https://wrong-region.exe22:32
TheJuliaugh22:32
eanderssonok was able to fix it by adding 'region_name': 'myregion'22:35
eanderssonto the ironicclient22:35
eanderssonhttps://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/client.py#L13022:35
eanderssonoh22:37
eandersson> ironic_url=ironic_url,22:37
eanderssonthis does nothing in rocky22:38
eanderssonhttps://github.com/openstack/python-ironicclient/blob/stable/queens/ironicclient/client.py#L2022:38
eanderssonhttps://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/client.py#L2022:38
eanderssonSo it's a rocky bug22:38
*** rcernin has joined #openstack-ironic22:38
eanderssonhttps://github.com/openstack/nova/commit/35f49f403534e174578dcd1b9ab33daf6f14c3e822:39
eanderssonThis needs to get back ported to nova22:40
eanderssonregions just does not work at all in Rocky22:40
*** Goneri has quit IRC22:50
eanderssonIt is odd though as convert_keystoneauth_opts should fix this22:55
*** andrein has joined #openstack-ironic22:59
TheJuliaeandersson: problem is, that is a version bump on the library which while non-breaking is not going to be possible with stable branch policy23:00
eanderssonYea - actually I might be wrong about that being the issue23:00
eanderssonGonna reproduce it in the lab23:00
*** whoami-rajat has quit IRC23:02
TheJuliak23:05
*** Goneri has joined #openstack-ironic23:06
* TheJulia hunts for someplace to gather the peoples in Denver23:14
*** Goneri has quit IRC23:14
*** jcoufal has joined #openstack-ironic23:15
TheJuliaHow does everyone feel about walking ~1 Mile?23:17
*** andrein has quit IRC23:20
*** dsneddon has quit IRC23:25
*** dsneddon has joined #openstack-ironic23:27
*** dsneddon has quit IRC23:29
*** dsneddon has joined #openstack-ironic23:30
eanderssonSo actually looks like a ironic client bug23:35
TheJulia\o/23:36
*** rh-jelabarre has quit IRC23:36
eanderssonbut of course my lab only has one region :p23:37
*** jcoufal has quit IRC23:41
eanderssonThis code is so complicated23:47
eanderssonhttps://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/common/http.py#L10123:48
eanderssonhttps://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/common/http.py#L61323:49
eanderssonPretty sure this code path does not respect endpoint, only region_name23:49
*** hwoarang has quit IRC23:57
eanderssonok got it failing in the lab :D23:57
eanderssondon't think it's working in Stein either tbh23:58
*** hwoarang has joined #openstack-ironic23:58
eanderssonOh and I see why no one has noticed this before.23:59

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