| *** cdearborn has quit IRC | 00:00 | |
| eandersson | https://github.com/jriguera/ansible-ironic-standalone/wiki/Cloud-Init-and-Config-Drive | 00:03 |
|---|---|---|
| *** dsneddon has quit IRC | 00:05 | |
| eandersson | Haven't read the above yet but looks like what I was looing for ^ | 00:06 |
| *** dsneddon has joined #openstack-ironic | 00:06 | |
| *** ijw has joined #openstack-ironic | 00:16 | |
| *** mbeierl has quit IRC | 00:17 | |
| *** mbeierl has joined #openstack-ironic | 00:23 | |
| TheJulia | eandersson: yeah, I guess you could | 00:24 |
| *** dsneddon has quit IRC | 00:31 | |
| *** dsneddon has joined #openstack-ironic | 00:54 | |
| *** dsneddon has quit IRC | 00:59 | |
| *** dsneddon has joined #openstack-ironic | 01:02 | |
| *** dsneddon has quit IRC | 01:07 | |
| eandersson | TheJulia, just reverse engineerd what nova does was the path :p | 01:27 |
| *** hwoarang has quit IRC | 01:28 | |
| *** hwoarang has joined #openstack-ironic | 01:28 | |
| eandersson | *forward for us | 01:28 |
| *** bfournie has joined #openstack-ironic | 01:38 | |
| *** rloo has quit IRC | 01:43 | |
| *** keekz has joined #openstack-ironic | 01:45 | |
| *** whoami-rajat has joined #openstack-ironic | 01:53 | |
| *** kiseok7 has joined #openstack-ironic | 01:57 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-specs master: Support error log history https://review.openstack.org/652811 | 02:02 |
| *** ijw has quit IRC | 02:04 | |
| *** andrein has quit IRC | 02:15 | |
| *** hwoarang has quit IRC | 02:37 | |
| *** hwoarang has joined #openstack-ironic | 02:39 | |
| *** dsneddon has joined #openstack-ironic | 03:03 | |
| *** dsneddon has quit IRC | 03:12 | |
| *** dsneddon has joined #openstack-ironic | 03:41 | |
| TheJulia | eandersson: \o/ | 03:42 |
| eandersson | Was thinking that it might be worth making a plugin or driver for this | 03:44 |
| eandersson | because Ironic will have all the data we need already | 03:44 |
| eandersson | or maybe just a tool to automatically render the config | 03:45 |
| *** dsneddon has quit IRC | 03:47 | |
| *** dsneddon has joined #openstack-ironic | 04:12 | |
| *** dsneddon has quit IRC | 04:17 | |
| *** hwoarang has quit IRC | 04:21 | |
| *** dsneddon has joined #openstack-ironic | 04:22 | |
| *** hwoarang has joined #openstack-ironic | 04:26 | |
| *** dsneddon has quit IRC | 04:27 | |
| *** e0ne has joined #openstack-ironic | 04:53 | |
| *** e0ne has quit IRC | 04:56 | |
| *** dsneddon has joined #openstack-ironic | 05:02 | |
| *** gyee has quit IRC | 05:08 | |
| *** dsneddon has quit IRC | 05:15 | |
| *** dsneddon has joined #openstack-ironic | 05:15 | |
| TheJulia | eandersson: hmm, interesting idea. | 05:18 |
| *** dsneddon has quit IRC | 05:20 | |
| *** e0ne has joined #openstack-ironic | 05:21 | |
| *** e0ne has quit IRC | 05:25 | |
| *** dsneddon has joined #openstack-ironic | 05:49 | |
| *** dsneddon has quit IRC | 05:54 | |
| *** andrein has joined #openstack-ironic | 06:12 | |
| *** dsneddon has joined #openstack-ironic | 06:20 | |
| *** dsneddon has quit IRC | 06:25 | |
| *** pcaruana has joined #openstack-ironic | 06:26 | |
| *** rpittau|afk has quit IRC | 06:28 | |
| *** vdrok has quit IRC | 06:28 | |
| *** TheJulia has quit IRC | 06:29 | |
| *** hogepodge has quit IRC | 06:32 | |
| *** nishagb has quit IRC | 06:32 | |
| *** johnsom has quit IRC | 06:32 | |
| *** nishagb has joined #openstack-ironic | 06:36 | |
| *** dsneddon has joined #openstack-ironic | 06:40 | |
| *** nishagb has quit IRC | 06:42 | |
| *** TheJulia has joined #openstack-ironic | 06:43 | |
| *** dsneddon has quit IRC | 06:45 | |
| *** NobodyCam has quit IRC | 06:46 | |
| *** TheJulia has quit IRC | 06:47 | |
| *** nishagb has joined #openstack-ironic | 06:52 | |
| *** TheJulia has joined #openstack-ironic | 06:52 | |
| *** rpittau|afk has joined #openstack-ironic | 06:53 | |
| *** NobodyCam has joined #openstack-ironic | 06:53 | |
| *** johnsom has joined #openstack-ironic | 06:55 | |
| mgoddard | tonyb[m]: I expect you're right about needing to install udevadm in the ironic-conductor image | 06:55 |
| *** hogepodge has joined #openstack-ironic | 06:56 | |
| *** vdrok has joined #openstack-ironic | 06:58 | |
| *** e0ne has joined #openstack-ironic | 07:06 | |
| *** rpittau|afk is now known as rpittau | 07:06 | |
| rpittau | good morning ironic! o/ | 07:06 |
| *** e0ne has quit IRC | 07:09 | |
| *** e0ne has joined #openstack-ironic | 07:16 | |
| *** dsneddon has joined #openstack-ironic | 07:17 | |
| *** rcernin has quit IRC | 07:20 | |
| *** andrein has quit IRC | 07:22 | |
| *** tssurya has joined #openstack-ironic | 07:29 | |
| *** dsneddon has quit IRC | 07:41 | |
| xXraphXx | hello everyone. I had a question regarding: https://github.com/openstack/networking-baremetal/blob/stable/queens/requirements.txt | 07:50 |
| *** andrein has joined #openstack-ironic | 07:51 | |
| xXraphXx | it looks like the dependency python-ironicclient>=2.2.0 is conflicting with current queens upper constraints: | 07:51 |
| xXraphXx | Collecting 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 |
| xXraphXx | openstacksdk 0.27.0 has requirement keystoneauth1>=3.13.0, but you'll have keystoneauth1 3.4.0 which is incompatible. | 07:51 |
| xXraphXx | osc-lib 1.12.1 has requirement keystoneauth1>=3.7.0, but you'll have keystoneauth1 3.4.0 which is incompatible. | 07:51 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/652935 | 07:52 |
| *** iurygregory has joined #openstack-ironic | 07:52 | |
| *** e0ne has quit IRC | 07:52 | |
| iurygregory | morning all | 07:53 |
| *** dsneddon has joined #openstack-ironic | 07:53 | |
| xXraphXx | this being said all seems to work correctly with these conflicting libs. But I was wondering if we should fix it | 07:54 |
| *** diga has joined #openstack-ironic | 07:58 | |
| *** e0ne has joined #openstack-ironic | 07:58 | |
| *** e0ne has quit IRC | 07:59 | |
| *** johnsom has quit IRC | 08:04 | |
| *** johnsom has joined #openstack-ironic | 08:05 | |
| *** rpittau_ has joined #openstack-ironic | 08:05 | |
| *** rpittau_ is now known as rpittua | 08:05 | |
| *** rpittua is now known as rpittau_ | 08:05 | |
| rpittau_ | heh bouncers | 08:06 |
| *** rpittau has quit IRC | 08:06 | |
| *** rpittau_ is now known as rpittau | 08:06 | |
| *** dsneddon has quit IRC | 08:07 | |
| *** dsneddon has joined #openstack-ironic | 08:08 | |
| *** priteau has joined #openstack-ironic | 08:09 | |
| *** dougsz has joined #openstack-ironic | 08:09 | |
| *** gkadam has joined #openstack-ironic | 08:09 | |
| iurygregory | morning rpittau o/ | 08:09 |
| rpittau | hey iurygregory :) | 08:10 |
| *** gkadam has quit IRC | 08:10 | |
| *** e0ne has joined #openstack-ironic | 08:10 | |
| *** dsneddon has quit IRC | 08:13 | |
| *** andrein has quit IRC | 08:24 | |
| *** Lucas_Gray has joined #openstack-ironic | 08:26 | |
| *** lucasagomes has joined #openstack-ironic | 08:34 | |
| diga | Good Morning rpittau iurygregory | 08:44 |
| iurygregory | morning diga | 08:44 |
| *** livelace has joined #openstack-ironic | 08:45 | |
| rpittau | hi diga :) | 08:45 |
| *** dsneddon has joined #openstack-ironic | 08:45 | |
| *** stendulker has joined #openstack-ironic | 08:47 | |
| *** andrein has joined #openstack-ironic | 08:49 | |
| *** dsneddon has quit IRC | 08:50 | |
| *** dsneddon has joined #openstack-ironic | 08:57 | |
| *** livelace has quit IRC | 09:00 | |
| *** dsneddon has quit IRC | 09:02 | |
| *** dsneddon has joined #openstack-ironic | 09:04 | |
| *** dsneddon has quit IRC | 09:09 | |
| *** livelace has joined #openstack-ironic | 09:11 | |
| diga | iurygregory: rpittau TheJulia I want to discuss related management interface cleaning steps | 09:14 |
| diga | We are working on adding reset cleaning steps to management interface in drac | 09:14 |
| diga | normal time of reset idrac call takes around 5-10 mins | 09:15 |
| diga | I added that call in management interface and its working fine | 09:16 |
| diga | Question here is "Is this a right design to add reset call in management interface if its taking 5 mins to execute" | 09:16 |
| diga | it locks the thread till the completion | 09:17 |
| diga | You can correct me if this is right approach or not | 09:17 |
| * iurygregory think clean steps can take a huge time.. | 09:18 | |
| diga | okay | 09:18 |
| iurygregory | you want to stop clean steps if the time is above a specific limit or something? | 09:19 |
| diga | yes | 09:19 |
| *** dsneddon has joined #openstack-ironic | 09:19 | |
| diga | s/yes/no | 09:19 |
| diga | once reset_idrac steps is called, we should let it execute otherwise node goes into bad state | 09:20 |
| * iurygregory not sure if this would be good O.o | 09:20 | |
| iurygregory | stop the cleaning in the middle of an operation | 09:20 |
| diga | hmm | 09:20 |
| iurygregory | makes sense if you call the reset_clean to move the node to bad state | 09:21 |
| diga | okay | 09:22 |
| iurygregory | I'm just not sure about the consequences of stopping the cleaning | 09:22 |
| iurygregory | .-. | 09:22 |
| diga | * thinking of finding exact time | 09:22 |
| diga | it all depends on it, or assign specific time if it goes beyond that then move into bad state | 09:23 |
| iurygregory | if we know in how much time each step in cleaning can take it makes sense.. | 09:23 |
| diga | yes | 09:23 |
| diga | I am also thinking on the same lines | 09:24 |
| diga | iurygregory: I dont see any bad state here, can be put into ironic.common.states.ERROR = 'error' state | 09:27 |
| iurygregory | ERROR is a good one XD | 09:28 |
| diga | but once node goes into error state, what will be another cleaning step performed on it to bring it online | 09:28 |
| diga | which we can mention in the warning | 09:28 |
| iurygregory | tbh, no idea XD | 09:28 |
| diga | ok | 09:28 |
| iurygregory | except if you know the list of steps and like (1,2,3,4,5) and you stopped on 3 | 09:29 |
| iurygregory | maybe will be necessary run 2 3 4 5 not only 3 4 5 | 09:29 |
| rpittau | diga: from error state coming from clean, it'll be clean_failed | 09:30 |
| diga | rpittau: okk | 09:30 |
| diga | got it | 09:30 |
| diga | iurygregory: rpittau Thank you :) | 09:33 |
| rpittau | diga: yw | 09:34 |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:34 | |
| *** dtantsur|afk is now known as dtantsur | 09:35 | |
| dtantsur | morning ironic | 09:35 |
| dtantsur | eandersson: 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_Agarwal | dtantsur, morning | 09:42 |
| dtantsur | morning Nisha_Agarwal | 09:43 |
| Nisha_Agarwal | dtantsur, we received one bug in proliantutils which is there in ironic as well because now we uncap jsonschema https://bugs.launchpad.net/proliantutils/+bug/1824084 | 09:43 |
| openstack | Launchpad bug 1824084 in proliantutils "testsuite incompatible with jsonschema 3.0.1" [Undecided,New] | 09:43 |
| Nisha_Agarwal | i am working on this but just wante dto bring this to the notice of ironic community | 09:44 |
| dtantsur | okay, thanks for the heads up | 09:44 |
| Nisha_Agarwal | i am planning to fix this in ironic as well. | 09:44 |
| dtantsur | mgoddard: ^^^ may affect your capping of proliantutils in bifrost | 09:44 |
| dtantsur | if we need to pull in the fix | 09:45 |
| dtantsur | Nisha_Agarwal: is anything broken in ironic itself? | 09:45 |
| Nisha_Agarwal | should i open the bug in ironic with same details | 09:45 |
| Nisha_Agarwal | dtantsur, yes | 09:45 |
| Nisha_Agarwal | the same schema file exists in ironic as well | 09:45 |
| Nisha_Agarwal | where we are seeing SchemaError | 09:45 |
| dtantsur | ugh | 09:45 |
| Nisha_Agarwal | the code works fine with jsonschema 2.6 | 09:46 |
| dtantsur | it would be ideal if you could make your change compatible with both jsonschema 2 and 3 | 09:46 |
| dtantsur | then we can backport it | 09:46 |
| Nisha_Agarwal | but with jsonschema 3.0.1 | 09:46 |
| mgoddard | Nisha_Agarwal: will the fix be backported to proliantutils 2.7.x? | 09:46 |
| Nisha_Agarwal | sure i would propose the change to ironci soon | 09:46 |
| Nisha_Agarwal | mgoddard, proliantutils doesnt has any backporting | 09:46 |
| Nisha_Agarwal | we will release proliantutils with the fix | 09:47 |
| Nisha_Agarwal | mgoddard, we dont have any branching in proliantutils | 09:47 |
| dtantsur | thanks Nisha_Agarwal! | 09:47 |
| Nisha_Agarwal | dtantsur, wc :) | 09:48 |
| *** rcernin has joined #openstack-ironic | 10:01 | |
| diga | dtantsur: Morning :) | 10:08 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [POC][WIP] Fetch baremetal endpoint from mdns https://review.openstack.org/652017 | 10:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [DNM] Test the CI with mDNS resolving https://review.openstack.org/652962 | 10:10 |
| dtantsur | morning diga | 10:10 |
| openstackgerrit | Digambar proposed openstack/ironic stable/rocky: Remove nic method dependencies from dracclient==1.5.0 https://review.openstack.org/648360 | 10:18 |
| diga | dtantsur: I just proposed patch for removing list_nic_dependency for older releases | 10:18 |
| diga | dtantsur: Also request you to take a look at this one - https://review.openstack.org/#/c/634903/ | 10:20 |
| patchbot | patch 634903 - ironic - Modify the iDRAC driver to use realtime RAID creation - 16 patch sets | 10:20 |
| dtantsur | it's still on my list, and I still have no idea when I get to it. busy time, sorry. | 10:20 |
| diga | No Problem | 10:20 |
| diga | TheJulia: rpittau: iurygregory: I also request you guys to take a look at realtime patch - https://review.openstack.org/#/c/634903/ | 10:21 |
| patchbot | patch 634903 - ironic - Modify the iDRAC driver to use realtime RAID creation - 16 patch sets | 10:21 |
| *** Nisha_Agarwal has quit IRC | 10:31 | |
| *** diga has quit IRC | 10:51 | |
| *** andrein has quit IRC | 11:04 | |
| *** Lucas_Gray has quit IRC | 11:09 | |
| *** Lucas_Gray has joined #openstack-ironic | 11:12 | |
| *** andrein has joined #openstack-ironic | 11:16 | |
| *** pcaruana has quit IRC | 11:17 | |
| *** andrein has quit IRC | 11:27 | |
| *** stendulker has quit IRC | 11:29 | |
| dtantsur | TheJulia: 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_601597 | 11:40 |
| *** dtantsur is now known as dtantsur|brb | 11:45 | |
| iurygregory | ubuntu-xenial =X | 11:47 |
| *** Lucas_Gray has quit IRC | 11:52 | |
| jroll | tonyb[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 guess | 11:53 |
| jroll | morning | 11:53 |
| *** pcaruana has joined #openstack-ironic | 12:04 | |
| *** jcoufal has joined #openstack-ironic | 12:07 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:10 | |
| *** andrein has joined #openstack-ironic | 12:16 | |
| *** priteau has quit IRC | 12:17 | |
| *** priteau has joined #openstack-ironic | 12:19 | |
| iurygregory | morning jroll o/ | 12:40 |
| *** andrein has quit IRC | 12:48 | |
| *** andrein has joined #openstack-ironic | 12:48 | |
| *** baha has joined #openstack-ironic | 12:57 | |
| *** mjturek has joined #openstack-ironic | 12:58 | |
| *** rloo has joined #openstack-ironic | 13:02 | |
| *** pcaruana has quit IRC | 13:07 | |
| *** Goneri has joined #openstack-ironic | 13:20 | |
| *** yaawang has quit IRC | 13:22 | |
| *** yaawang has joined #openstack-ironic | 13:23 | |
| *** dtantsur|brb is now known as dtantsur | 13:24 | |
| *** mjturek has quit IRC | 13:27 | |
| dtantsur | morning jroll | 13:32 |
| *** lennyb has quit IRC | 13:32 | |
| *** pcaruana has joined #openstack-ironic | 13:33 | |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/652935 | 13:39 |
| TheJulia | good morning everyone | 13:43 |
| iurygregory | good morning | 13:43 |
| *** ijpascual5 has joined #openstack-ironic | 13:44 | |
| *** sthussey has joined #openstack-ironic | 13:44 | |
| *** lennyb has joined #openstack-ironic | 13:45 | |
| *** ijpascual has quit IRC | 13:46 | |
| rpittau | hi TheJulia :) | 13:47 |
| dtantsur | morning TheJulia | 13:55 |
| *** cdearborn has joined #openstack-ironic | 14:00 | |
| *** baha has quit IRC | 14:06 | |
| *** Lucas_Gray has joined #openstack-ironic | 14:13 | |
| TheJulia | Does anyone have any questions for people that may be viewing the project update? | 14:13 |
| TheJulia | dtantsur: will you be available to run the weekly meeting on May 13th? | 14:16 |
| TheJulia | Or does someone else want to? | 14:16 |
| dtantsur | TheJulia: I think so, yes | 14:16 |
| TheJulia | #open-to-volunteers | 14:16 |
| rpittau | btw do we have meeting next monday ? | 14:17 |
| TheJulia | We didn't decide to cancel it.... Typically we cancel the week after | 14:17 |
| TheJulia | I 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 27th | 14:18 |
| rpittau | and I think next monday is bank holiday in a lot of countries | 14:19 |
| *** priteau has quit IRC | 14:19 | |
| dtantsur | yeah, including here | 14:19 |
| rpittau | maybe postpone it to tuesday? | 14:19 |
| TheJulia | Oh right... I guess it would be good to cancel it | 14:19 |
| TheJulia | Nah, we don't want to try and postpone an upstream meeting | 14:20 |
| TheJulia | No need to create confusion for those that are less plugged into emails | 14:20 |
| TheJulia | rloo jroll arne_wiebalck: Any thoughts on cancelling the meeting next week? | 14:21 |
| rpittau | right, still thinking in terms of "company meetings" :P | 14:21 |
| jroll | TheJulia: I'm fine either way | 14:21 |
| * jroll is fairly disconnected from ironic upstream recently :( | 14:21 | |
| TheJulia | I 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 updated | 14:22 |
| TheJulia | jroll: it happens :( | 14:22 |
| rloo | TheJulia: 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 employees | 14:22 | |
| rloo | TheJulia: guess we should decide about the meeting after the open whatever whatever. PTG. | 14:23 |
| TheJulia | Traditionally we cancel it and let people recoup | 14:23 |
| rloo | TheJulia: we should cancel that one, since the PTG goes til Sat. | 14:23 |
| TheJulia | yeah | 14:23 |
| TheJulia | Apparently friday is a holiday for me.... | 14:24 |
| etingof | perhaps for EU folks as well | 14:25 |
| *** iurygregory_ has joined #openstack-ironic | 14:25 | |
| rpittau | etingof: AFAIK is just monday | 14:26 |
| TheJulia | etingof: eh... .cz page says friday and monday | 14:26 |
| iurygregory_ | rpittau, in czech republic friday and monday | 14:26 |
| rpittau | :/ | 14:27 |
| *** rcernin has quit IRC | 14:27 | |
| TheJulia | In france, just Monday :( | 14:27 |
| arne_wiebalck | TheJulia: I'll be still on holidays next Monday :) | 14:27 |
| rpittau | yeah | 14:27 |
| *** iurygregory has quit IRC | 14:27 | |
| rpittau | well, we have a bunch of days off in May so I guess it's fair :) | 14:27 |
| *** dtantsur has quit IRC | 14:27 | |
| *** iurygregory_ is now known as iurygregory | 14:27 | |
| TheJulia | sounds like there are no objections | 14:28 |
| * arne_wiebalck sends greetings from Italy to ironic o/ | 14:29 | |
| TheJulia | arne_wiebalck: enjoy your vacation in Italy :) | 14:29 |
| rpittau | arne_wiebalck: enjoying some good food? :) | 14:29 |
| arne_wiebalck | rpittau: the best | 14:29 |
| rpittau | xD | 14:29 |
| arne_wiebalck | TheJulia: thx :) | 14:29 |
| *** dtantsur has joined #openstack-ironic | 14:31 | |
| TheJulia | dtantsur: 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 |
| dtantsur | TheJulia: cool, yeah :) | 14:32 |
| *** iurygregory_ has joined #openstack-ironic | 14:32 | |
| *** iurygregory has quit IRC | 14:33 | |
| *** iurygregory_ is now known as iurygregory | 14:33 | |
| *** priteau has joined #openstack-ironic | 14:37 | |
| *** mjturek has joined #openstack-ironic | 14:41 | |
| *** baha has joined #openstack-ironic | 14:42 | |
| *** ajya[m] has joined #openstack-ironic | 14:49 | |
| *** iurygregory has quit IRC | 15:05 | |
| openstackgerrit | Merged openstack/ironic master: devstack: Remove syslinux dependency https://review.openstack.org/651285 | 15:15 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Ironic python agent does not extract correct available memory https://review.openstack.org/648096 | 15:16 |
| openstackgerrit | Merged openstack/ironic master: Place upper bound on python-dracclient version https://review.openstack.org/651408 | 15:16 |
| *** hamzy has quit IRC | 15:22 | |
| *** pcaruana has quit IRC | 15:27 | |
| *** pcaruana has joined #openstack-ironic | 15:30 | |
| *** e0ne has quit IRC | 15:31 | |
| *** iurygregory has joined #openstack-ironic | 15:31 | |
| *** pcaruana has quit IRC | 15:40 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/stein: Ironic python agent does not extract correct available memory https://review.openstack.org/653030 | 15:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/rocky: Ironic python agent does not extract correct available memory https://review.openstack.org/653031 | 15:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/queens: Ironic python agent does not extract correct available memory https://review.openstack.org/653032 | 15:45 |
| dtantsur | TheJulia: do we need to do any stable/stein releases? | 15:45 |
| * dtantsur can check | 15:45 | |
| dtantsur | I guess we need the ansible fixes to merge for ironic | 15:46 |
| *** pcaruana has joined #openstack-ironic | 15:46 | |
| openstackgerrit | Vlad Sorokin proposed openstack/networking-generic-switch master: Add Mellanox MLNX-OS Switch support https://review.openstack.org/642565 | 15:46 |
| *** gyee has joined #openstack-ironic | 15:50 | |
| *** pcaruana has quit IRC | 15:55 | |
| *** bfournie has quit IRC | 15:58 | |
| TheJulia | Yeah, we likely should release those | 16:00 |
| *** pcaruana has joined #openstack-ironic | 16:02 | |
| *** lucasagomes has quit IRC | 16:07 | |
| *** pcaruana has quit IRC | 16:08 | |
| eandersson | Maybe we could put some of the nova code used to generate config-drive into a shared library. | 16:19 |
| *** S4ren has joined #openstack-ironic | 16:20 | |
| *** Lucas_Gray has quit IRC | 16:20 | |
| dtantsur | eandersson: like https://github.com/openstack/openstacksdk/blob/master/openstack/baremetal/configdrive.py ? | 16:22 |
| eandersson | Yep - pretty close | 16:23 |
| rpittau | hi eandersson ;) | 16:26 |
| eandersson | o/ | 16:26 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:31 | |
| *** pcaruana has joined #openstack-ironic | 16:36 | |
| *** dtruong has quit IRC | 16:40 | |
| *** problem_v has quit IRC | 16:40 | |
| *** problem_v has joined #openstack-ironic | 16:41 | |
| *** dtruong has joined #openstack-ironic | 16:41 | |
| rpittau | good night! o/ | 16:43 |
| *** rpittau is now known as rpittau|afk | 16:43 | |
| *** tssurya has quit IRC | 16:47 | |
| Nisha_Agarwal | dtantsur, hi | 16:54 |
| *** yolanda_ has quit IRC | 16:57 | |
| *** dougsz has quit IRC | 16:59 | |
| *** ijw has joined #openstack-ironic | 17:00 | |
| dtantsur | hi Nisha_Agarwal | 17:09 |
| *** priteau has quit IRC | 17:11 | |
| Nisha_Agarwal | dtantsur, regarding the bug for jsonschema it will require a different schema file to support darft-06 and draft-07 of json schema | 17:11 |
| dtantsur | Nisha_Agarwal: oh, so no way to support both versions? | 17:12 |
| Nisha_Agarwal | dtantsur, we can have probably two schema files with same properties one for draft-04 and one for draft-6 and 07 | 17:12 |
| Nisha_Agarwal | and in code we can validate schemas against Draft4-6 Validators | 17:13 |
| dtantsur | Nisha_Agarwal: probably yes, especially since the CI still runs with 2.6.0 right now | 17:13 |
| Nisha_Agarwal | yeah | 17:14 |
| Nisha_Agarwal | i will propose the fix today in some time or might be by tomorrow | 17:14 |
| Nisha_Agarwal | for raising the bug can i just clone proliantutils bug report in the bug? | 17:14 |
| dtantsur | Nisha_Agarwal: if proliantutils uses storyboard, you can just add a task for ironic | 17:21 |
| dtantsur | and thanks! | 17:21 |
| Nisha_Agarwal | dtantsur, still the bugs on proliantutils are raised on launchpad | 17:22 |
| Nisha_Agarwal | dtantsur, CI may not break if it is not testing raid feature | 17:23 |
| Nisha_Agarwal | the issue is only for raid json schema | 17:23 |
| Nisha_Agarwal | but yes py27 and py25 tests may fail | 17:23 |
| dtantsur | right | 17:26 |
| dtantsur | okay, then you can copy the bug description, I guess | 17:26 |
| *** hamzy has joined #openstack-ironic | 17:27 | |
| Nisha_Agarwal | dtantsur, there is one more point i dont know if community will want to support that | 17:27 |
| *** Nisha_Agarwal has quit IRC | 17:28 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 17:30 | |
| *** S4ren has quit IRC | 17:30 | |
| dtantsur | Nisha_Agarwal: support what, both versions of jsonschema? | 17:30 |
| Nisha_Agarwal | dtantsur, 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 error | 17:30 |
| Nisha_Agarwal | jsonschema 2.6 support draft 3 and 4 | 17:30 |
| dtantsur | and 3.0? | 17:31 |
| Nisha_Agarwal | jsonschema 3.0 supports all 4 | 17:31 |
| Nisha_Agarwal | i.e 3,4,6 and 7 | 17:31 |
| *** e0ne has joined #openstack-ironic | 17:32 | |
| dtantsur | pardon my ignorance, can we somehow declare using e.g. draft 4? | 17:32 |
| dtantsur | so that we're compatible with all versions? | 17:32 |
| Nisha_Agarwal | yes we can define schema as draft04 | 17:33 |
| Nisha_Agarwal | and then it can be used for both 2.6 and 3.0.1 | 17:33 |
| dtantsur | this sounds like the least invasive change, right? | 17:33 |
| Nisha_Agarwal | yes | 17:33 |
| Nisha_Agarwal | but dont know if shud support old schema given draft-07 is already out | 17:33 |
| dtantsur | we can start with this solution to solve the immediate problem (plus backport it) | 17:34 |
| dtantsur | then we can bump requirements.txt to jsonschema>=3.0.1 and use draft-07 explicitly | 17:34 |
| Nisha_Agarwal | ok | 17:34 |
| dtantsur | the latter will have to wait until upper-constraints are updated | 17:34 |
| Nisha_Agarwal | yes | 17:34 |
| Nisha_Agarwal | ok then thats the most easy fix for this issue | 17:35 |
| Nisha_Agarwal | :) | 17:35 |
| dtantsur | yeah :) | 17:35 |
| Nisha_Agarwal | thanks :) | 17:35 |
| *** dtantsur is now known as dtantsur|afk | 17:36 | |
| dtantsur|afk | g'night, ironic | 17:36 |
| Nisha_Agarwal | dtantsur, can u create the story for this bug in storyboard taking it from proliantutils bug....i am unable to login to storyboard for some reason | 17:37 |
| dtantsur|afk | Nisha_Agarwal: please send me the link to the bug | 17:37 |
| * TheJulia gets off a call and needs a coffee IV | 17:37 | |
| Nisha_Agarwal | https://bugs.launchpad.net/proliantutils/+bug/1824084 | 17:37 |
| openstack | Launchpad bug 1824084 in proliantutils "testsuite incompatible with jsonschema 3.0.1" [Undecided,New] | 17:37 |
| Nisha_Agarwal | thanks dtantsur|afk | 17:37 |
| dtantsur|afk | Nisha_Agarwal: https://storyboard.openstack.org/#!/story/2005464 | 17:39 |
| Nisha_Agarwal | dtantsur|afk, thank u so much | 17:39 |
| dtantsur|afk | np | 17:39 |
| Nisha_Agarwal | good night | 17:39 |
| * dtantsur|afk goes for real | 17:39 | |
| TheJulia | goodnight | 17:39 |
| *** ijw has quit IRC | 17:40 | |
| *** ijw has joined #openstack-ironic | 17:42 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Jsonschema 3.0.1: Binding the schema to draft-04 https://review.openstack.org/653060 | 17:49 |
| *** priteau has joined #openstack-ironic | 17:49 | |
| *** e0ne has quit IRC | 17:49 | |
| openstackgerrit | raphael.glon proposed openstack/ironic master: Truncate node text fields when too long https://review.openstack.org/650307 | 18:04 |
| *** iurygregory has quit IRC | 18:07 | |
| *** Nisha_Agarwal has quit IRC | 18:08 | |
| *** ijw has quit IRC | 18:15 | |
| *** ijw has joined #openstack-ironic | 18:16 | |
| *** absubram has joined #openstack-ironic | 18:20 | |
| *** ijw has quit IRC | 18:22 | |
| *** priteau has quit IRC | 18:33 | |
| *** ijw has joined #openstack-ironic | 18:35 | |
| *** irclogbot_2 has quit IRC | 18:39 | |
| *** irclogbot_0 has joined #openstack-ironic | 18:40 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add indicators REST API endpoints https://review.openstack.org/651785 | 18:57 |
| *** andrein has quit IRC | 19:02 | |
| *** andrein has joined #openstack-ironic | 19:12 | |
| *** andrein has quit IRC | 19:23 | |
| *** andrein has joined #openstack-ironic | 19:31 | |
| *** andrein has quit IRC | 19:33 | |
| *** andrein has joined #openstack-ironic | 19:34 | |
| *** pcaruana has quit IRC | 20:19 | |
| *** priteau has joined #openstack-ironic | 20:23 | |
| *** priteau has quit IRC | 20:27 | |
| *** ijw has quit IRC | 20:42 | |
| *** ijw has joined #openstack-ironic | 20:47 | |
| *** ijw has quit IRC | 20:54 | |
| *** hamzy has quit IRC | 20:54 | |
| *** seba has joined #openstack-ironic | 20:56 | |
| *** Goneri has quit IRC | 21:00 | |
| *** jcoufal has quit IRC | 21:07 | |
| *** baha has quit IRC | 21:08 | |
| eandersson | TheJulia, you here by any chance? | 21:17 |
| eandersson | https://github.com/openstack/openstacksdk/blob/master/openstack/baremetal/configdrive.py#L74 | 21:17 |
| eandersson | Is this just me? | 21:17 |
| eandersson | or is network_data just never passed on :p | 21:17 |
| TheJulia | eandersson: braaaains | 21:18 |
| TheJulia | yes, I'm around. I swear I'm not a zombie | 21:18 |
| eandersson | :D | 21:18 |
| TheJulia | NICE | 21:19 |
| TheJulia | WOW that is a nice bug | 21:19 |
| TheJulia | eandersson: You are not crazy, you win today's prize | 21:19 |
| eandersson | Glad I looked at the code before we started to... go crazy | 21:19 |
| TheJulia | seems like a super quick fix | 21:20 |
| eandersson | Yep | 21:21 |
| eandersson | We can probably push one later if no one beat us to it :p | 21:21 |
| TheJulia | I might be able to, but I'm bouncing between four things at the moment | 21:22 |
| eandersson | Sounds good - no worries | 21:23 |
| *** ijw has joined #openstack-ironic | 21:25 | |
| TheJulia | eandersson: thanks! | 21:33 |
| *** Goneri has joined #openstack-ironic | 21:48 | |
| eandersson | TheJulia, I don't think regions works in nova-ironic in Rocky | 21:54 |
| eandersson | unless you set it manually using CONF.ironic.api_endpoint | 21:54 |
| TheJulia | That doesn't surprise me at all | 21:55 |
| eandersson | kwargs is supposed to pass on region_name | 21:55 |
| *** ijw has quit IRC | 21:55 | |
| eandersson | it isn't | 21:55 |
| *** ijw has joined #openstack-ironic | 21:55 | |
| TheJulia | Your finding some really fun bugs then.... | 21:56 |
| TheJulia | I 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 |
| eandersson | or nvm get_endpoint | 21:57 |
| eandersson | is missing region_name | 21:57 |
| eandersson | https://github.com/openstack/nova/blob/master/nova/virt/ironic/client_wrapper.py#L122 | 22:00 |
| TheJulia | looks like before that it was purely url config based | 22:04 |
| TheJulia | well, before the change to put the adapter in | 22:05 |
| TheJulia | looks like efried did it | 22:05 |
| *** ccstone has joined #openstack-ironic | 22:07 | |
| *** andrein has quit IRC | 22:16 | |
| *** livelace has quit IRC | 22:24 | |
| eandersson | Actually this is odd | 22:25 |
| eandersson | that code path is working | 22:25 |
| eandersson | it's something after that is failing | 22:25 |
| TheJulia | Oo | 22:25 |
| eandersson | > self._cached_client | 22:26 |
| eandersson | I assume this is just in-memory cache right? | 22:26 |
| TheJulia | already instansiated | 22:26 |
| TheJulia | so yeah, basically | 22:26 |
| TheJulia | why? | 22:26 |
| eandersson | because I printed the url it gives and it's correct, but the call after fails with the wrong region | 22:26 |
| TheJulia | my brain's deep storage robot is currently scanning barcodes to find the tape to load | 22:28 |
| eandersson | unless I am missing something, it looks like region just does not work at all | 22:30 |
| eandersson | *region override | 22:30 |
| TheJulia | maybe we should ping efried | 22:30 |
| eandersson | looks like an issue with the client tbh | 22:31 |
| eandersson | because the endpoint is correct | 22:31 |
| TheJulia | got a indicator of where at you see it correct and then wrong? | 22:31 |
| eandersson | but the ironicclient does not repsect it | 22:31 |
| eandersson | So I added a print(ironic_url) | 22:31 |
| eandersson | just before get_client | 22:31 |
| eandersson | and it prints the correct endpoint | 22:31 |
| eandersson | and in the next line it shows Error contacting Ironic server: Unable to establish connection to https://wrong-region.exe | 22:32 |
| TheJulia | ugh | 22:32 |
| eandersson | ok was able to fix it by adding 'region_name': 'myregion' | 22:35 |
| eandersson | to the ironicclient | 22:35 |
| eandersson | https://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/client.py#L130 | 22:35 |
| eandersson | oh | 22:37 |
| eandersson | > ironic_url=ironic_url, | 22:37 |
| eandersson | this does nothing in rocky | 22:38 |
| eandersson | https://github.com/openstack/python-ironicclient/blob/stable/queens/ironicclient/client.py#L20 | 22:38 |
| eandersson | https://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/client.py#L20 | 22:38 |
| eandersson | So it's a rocky bug | 22:38 |
| *** rcernin has joined #openstack-ironic | 22:38 | |
| eandersson | https://github.com/openstack/nova/commit/35f49f403534e174578dcd1b9ab33daf6f14c3e8 | 22:39 |
| eandersson | This needs to get back ported to nova | 22:40 |
| eandersson | regions just does not work at all in Rocky | 22:40 |
| *** Goneri has quit IRC | 22:50 | |
| eandersson | It is odd though as convert_keystoneauth_opts should fix this | 22:55 |
| *** andrein has joined #openstack-ironic | 22:59 | |
| TheJulia | eandersson: problem is, that is a version bump on the library which while non-breaking is not going to be possible with stable branch policy | 23:00 |
| eandersson | Yea - actually I might be wrong about that being the issue | 23:00 |
| eandersson | Gonna reproduce it in the lab | 23:00 |
| *** whoami-rajat has quit IRC | 23:02 | |
| TheJulia | k | 23:05 |
| *** Goneri has joined #openstack-ironic | 23:06 | |
| * TheJulia hunts for someplace to gather the peoples in Denver | 23:14 | |
| *** Goneri has quit IRC | 23:14 | |
| *** jcoufal has joined #openstack-ironic | 23:15 | |
| TheJulia | How does everyone feel about walking ~1 Mile? | 23:17 |
| *** andrein has quit IRC | 23:20 | |
| *** dsneddon has quit IRC | 23:25 | |
| *** dsneddon has joined #openstack-ironic | 23:27 | |
| *** dsneddon has quit IRC | 23:29 | |
| *** dsneddon has joined #openstack-ironic | 23:30 | |
| eandersson | So actually looks like a ironic client bug | 23:35 |
| TheJulia | \o/ | 23:36 |
| *** rh-jelabarre has quit IRC | 23:36 | |
| eandersson | but of course my lab only has one region :p | 23:37 |
| *** jcoufal has quit IRC | 23:41 | |
| eandersson | This code is so complicated | 23:47 |
| eandersson | https://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/common/http.py#L101 | 23:48 |
| eandersson | https://github.com/openstack/python-ironicclient/blob/stable/rocky/ironicclient/common/http.py#L613 | 23:49 |
| eandersson | Pretty sure this code path does not respect endpoint, only region_name | 23:49 |
| *** hwoarang has quit IRC | 23:57 | |
| eandersson | ok got it failing in the lab :D | 23:57 |
| eandersson | don't think it's working in Stein either tbh | 23:58 |
| *** hwoarang has joined #openstack-ironic | 23:58 | |
| eandersson | Oh 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!