| *** hshiina has joined #openstack-ironic | 00:46 | |
| *** ptx0 has quit IRC | 01:13 | |
| *** ptx0 has joined #openstack-ironic | 01:18 | |
| *** zshi has joined #openstack-ironic | 01:50 | |
| *** links has joined #openstack-ironic | 02:11 | |
| *** r-daneel has joined #openstack-ironic | 02:24 | |
| *** yangyapeng has quit IRC | 02:30 | |
| *** r-daneel has quit IRC | 02:32 | |
| *** r-daneel has joined #openstack-ironic | 03:40 | |
| *** yangyapeng has joined #openstack-ironic | 03:44 | |
| *** dpawar has joined #openstack-ironic | 03:49 | |
| *** dpawar_ has joined #openstack-ironic | 03:58 | |
| *** dpawar has quit IRC | 04:02 | |
| *** dpawar_ has quit IRC | 04:09 | |
| *** khyr0n has joined #openstack-ironic | 04:37 | |
| *** olivierbourdon38 has joined #openstack-ironic | 05:04 | |
| *** dpawar has joined #openstack-ironic | 05:11 | |
| *** dpawar has quit IRC | 05:12 | |
| *** dpawar has joined #openstack-ironic | 05:12 | |
| openstackgerrit | Merged openstack/ironic master: Allow sqalchemy filtering by id and uuid https://review.openstack.org/545041 | 05:24 |
|---|---|---|
| *** dpawar has quit IRC | 05:31 | |
| *** rbartal has joined #openstack-ironic | 06:02 | |
| *** threestrands has quit IRC | 06:10 | |
| *** rbartal has quit IRC | 06:18 | |
| *** threestrands has joined #openstack-ironic | 06:23 | |
| *** jaganathan has joined #openstack-ironic | 06:46 | |
| *** actatux has quit IRC | 06:46 | |
| *** rbartal has joined #openstack-ironic | 06:48 | |
| *** dpawar has joined #openstack-ironic | 06:52 | |
| *** threestrands has quit IRC | 07:00 | |
| *** mjura has joined #openstack-ironic | 07:02 | |
| *** rcernin has quit IRC | 07:11 | |
| *** e0ne has joined #openstack-ironic | 07:11 | |
| *** vsaienk0 has joined #openstack-ironic | 07:18 | |
| *** e0ne has quit IRC | 07:29 | |
| *** hoonetorg has quit IRC | 07:30 | |
| *** vsaienk0 has quit IRC | 07:32 | |
| *** vsaienk0 has joined #openstack-ironic | 07:34 | |
| *** e0ne has joined #openstack-ironic | 07:35 | |
| *** hoonetorg has joined #openstack-ironic | 07:46 | |
| *** pcaruana has joined #openstack-ironic | 07:47 | |
| *** e0ne has quit IRC | 07:47 | |
| olivierbourdon38 | Hello everyone. Any known regression lately ? I was configuring ironic with keystone using Bifrost and everything was fine end of last week and today I get 2018-02-19 08:45:17.718 22458 ERROR keystone ImportError: (u'Unable to find %(name)r driver in %(namespace)r.', {'namespace': 'keystone.token.provider', 'name': 'uuid'}) | 07:48 |
| *** vsaienk0 has quit IRC | 07:51 | |
| *** links has quit IRC | 07:55 | |
| *** dpawar has quit IRC | 07:56 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:56 | |
| *** dpawar has joined #openstack-ironic | 07:56 | |
| pas-ha | olivierbourdon38: I suspect it might have been removed from keystone, together with v2 API. lemme check | 07:58 |
| olivierbourdon38 | pas-ha thx for your help and time | 07:58 |
| *** dpawar_ has joined #openstack-ironic | 07:59 | |
| *** dpawar has quit IRC | 07:59 | |
| pas-ha | yep, it is gone https://review.openstack.org/#/c/543060/ | 08:00 |
| patchbot | patch 543060 - keystone - Remove the sql token driver and uuid token provider (MERGED) | 08:00 |
| pas-ha | olivierbourdon38: ^ | 08:00 |
| *** MattMan has quit IRC | 08:02 | |
| *** MattMan has joined #openstack-ironic | 08:03 | |
| *** vsaienk0 has joined #openstack-ironic | 08:04 | |
| mkrai | Hi, I setup a latest devstack setup for Ironic. I tried to boot from Nova but it failed with "No valid host found" error | 08:06 |
| mkrai | Logs http://paste.openstack.org/show/677483/ | 08:06 |
| mkrai | I am not sure how the placement api works. | 08:06 |
| *** e0ne has joined #openstack-ironic | 08:06 | |
| mkrai | Can anyone help me with this? | 08:06 |
| olivierbourdon38 | pas-ha | 08:07 |
| *** links has joined #openstack-ironic | 08:07 | |
| olivierbourdon38 | pas-ha got it but this is still strange as I am pretty sure I was able to deploy stuff after Feb 13th .... | 08:08 |
| *** e0ne has quit IRC | 08:08 | |
| *** tesseract has joined #openstack-ironic | 08:15 | |
| *** racedo has joined #openstack-ironic | 08:17 | |
| *** mrunge has quit IRC | 08:17 | |
| mkrai | pas-ha: TheJulia ^^ | 08:21 |
| olivierbourdon38 | currently digging out what could be responsible for still calling this removed driver | 08:24 |
| pas-ha | olivierbourdon38: I think bifrost sets up keystone using uuid tokens | 08:30 |
| pas-ha | olivierbourdon38: https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-keystone-install/templates/keystone.conf.j2#L26 | 08:31 |
| *** mrunge has joined #openstack-ironic | 08:31 | |
| pas-ha | I suspect this is due to the fact that fernets are a bit more envolved to set up correctly, but now it seems there's no other way :-/ | 08:32 |
| pas-ha | involved | 08:32 |
| olivierbourdon38 | thx, you were quicker than I was on this ;-) | 08:33 |
| *** stendulker has joined #openstack-ironic | 08:33 | |
| *** krtaylor has quit IRC | 08:38 | |
| olivierbourdon38 | currently trying a fix for this issue in bifrost. Thx again for the help | 08:44 |
| *** dpawar_ has quit IRC | 08:52 | |
| *** dpawar has joined #openstack-ironic | 08:53 | |
| mkrai | Hi pas-ha | 08:53 |
| pas-ha | mkrai: me neither (sure how placement API works) | 08:56 |
| mkrai | pas-ha: oh ok no problem. Who can help with this? | 09:01 |
| *** jtomasek has joined #openstack-ironic | 09:03 | |
| olivierbourdon38 | so trying my patch for using fernets instead of uuid, I ended up in another issue | 09:11 |
| olivierbourdon38 | "2018-02-19 08:37:33.325 21884 ERROR keystone.notifications [-] Failed to construct notifier: ImportError: No module named pyngus", | 09:11 |
| olivierbourdon38 | "2018-02-19 08:37:33.325 21884 ERROR keystone.notifications import pyngus", | 09:11 |
| olivierbourdon38 | "2018-02-19 08:37:33.325 21884 ERROR keystone.notifications ImportError: No module named pyngus", | 09:11 |
| olivierbourdon38 | which I solved by adding pyngus to the keystone requirements.txt file but I do not understand why it is not there by default | 09:12 |
| *** derekh has joined #openstack-ironic | 09:21 | |
| *** mrch has joined #openstack-ironic | 09:22 | |
| *** mgoddard_ has joined #openstack-ironic | 09:23 | |
| *** vsaienk0 has quit IRC | 09:23 | |
| *** r-daneel has quit IRC | 09:28 | |
| *** links has quit IRC | 09:32 | |
| *** vsaienk0 has joined #openstack-ironic | 09:39 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: WIP: Add check for (i)PXE and Ironic location coherence https://review.openstack.org/545711 | 09:40 |
| *** dpawar has quit IRC | 09:42 | |
| *** links has joined #openstack-ironic | 09:45 | |
| *** r-daneel has joined #openstack-ironic | 09:46 | |
| *** mrch has quit IRC | 09:51 | |
| *** e0ne has joined #openstack-ironic | 09:54 | |
| *** vsaienk0 has quit IRC | 09:55 | |
| *** vsaienk0 has joined #openstack-ironic | 09:57 | |
| *** serlex has joined #openstack-ironic | 10:05 | |
| *** serlex has quit IRC | 10:05 | |
| *** r-daneel has quit IRC | 10:06 | |
| *** vsaienk0 has quit IRC | 10:07 | |
| *** vsaienk0 has joined #openstack-ironic | 10:08 | |
| *** r-daneel has joined #openstack-ironic | 10:22 | |
| *** mrch has joined #openstack-ironic | 10:26 | |
| *** vsaienk0 has quit IRC | 10:34 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: WIP: Add check for (i)PXE and Ironic location coherence https://review.openstack.org/545711 | 10:35 |
| *** hshiina has quit IRC | 10:50 | |
| *** mrch has quit IRC | 10:54 | |
| *** vsaienk0 has joined #openstack-ironic | 10:57 | |
| *** zshi has quit IRC | 10:59 | |
| *** AlexeyAbashkin has quit IRC | 11:07 | |
| *** stendulker has quit IRC | 11:18 | |
| *** lucas-afk is now known as lucasagomes | 11:18 | |
| *** priteau has joined #openstack-ironic | 11:19 | |
| *** krtaylor has joined #openstack-ironic | 11:23 | |
| *** dtantsur|afk is now known as dtantsur | 11:23 | |
| pas-ha | olivierbourdon38: looks like pungus is a test requirement for keystone http://codesearch.openstack.org/?q=pyngus&i=nope&files=requirements.txt&repos= | 11:23 |
| pas-ha | s/keystone/oslo.messaging | 11:24 |
| pas-ha | so probably keystone is set up with this backend (as default for testing) for notifications | 11:24 |
| dtantsur | morning ironic | 11:24 |
| pas-ha | morning dtantsur | 11:25 |
| pas-ha | Queens is definitely a fun release for Keystone - no more V2 API, no more UUID tokens... slim and clean :) | 11:26 |
| *** dpawar has joined #openstack-ironic | 11:26 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:32 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Allow sqalchemy filtering by id and uuid https://review.openstack.org/545806 | 11:34 |
| dtantsur | pas-ha: oh cool :) | 11:36 |
| *** bnemec is now known as bnemec-pto | 11:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Rework exception handling on deploy failures in conductor https://review.openstack.org/545807 | 11:44 |
| olivierbourdon38 | pas-ha however when bifrost deployment calls https://github.com/openstack/bifrost/blob/12ef3cbbadb638e363a73f3c269b3138de740ef0/playbooks/roles/bifrost-keystone-install/tasks/bootstrap.yml#L138 it "fails" even though the deployment just goes on | 11:47 |
| *** vsaienk0 has quit IRC | 11:53 | |
| *** vsaienk0 has joined #openstack-ironic | 11:57 | |
| *** masber has joined #openstack-ironic | 11:58 | |
| *** mrch has joined #openstack-ironic | 12:01 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Build instance PXE options for unrescue https://review.openstack.org/544278 | 12:05 |
| *** slagle has joined #openstack-ironic | 12:20 | |
| *** dmellado has quit IRC | 12:26 | |
| *** vsaienk0 has quit IRC | 12:28 | |
| *** mrch has quit IRC | 12:28 | |
| *** ricardoas has quit IRC | 12:33 | |
| *** vsaienk0 has joined #openstack-ironic | 12:33 | |
| *** nicodemos has quit IRC | 12:35 | |
| *** links has quit IRC | 12:37 | |
| *** milan_ has joined #openstack-ironic | 12:50 | |
| *** rbartal has quit IRC | 12:53 | |
| *** dprince has joined #openstack-ironic | 12:57 | |
| *** jlabarre has joined #openstack-ironic | 13:03 | |
| *** deadnull has joined #openstack-ironic | 13:04 | |
| *** dmellado has joined #openstack-ironic | 13:07 | |
| *** deadnull has quit IRC | 13:10 | |
| *** bfournie has quit IRC | 13:17 | |
| *** bfournie has joined #openstack-ironic | 13:18 | |
| *** mrch has joined #openstack-ironic | 13:21 | |
| *** bfournie has quit IRC | 13:22 | |
| *** skazi has quit IRC | 13:23 | |
| *** skazi has joined #openstack-ironic | 13:24 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM Test disabling .pyc files for Grenade https://review.openstack.org/545089 | 13:24 |
| *** dpawar has quit IRC | 13:25 | |
| TheJulia | Good morning ironic | 13:25 |
| dtantsur | morning TheJulia | 13:28 |
| *** mrch has quit IRC | 13:32 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Register traits on nodes in devstack https://review.openstack.org/545370 | 13:32 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Support nested objects and object lists in as_dict https://review.openstack.org/545830 | 13:32 |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Validate scheduling fields in basic ops scenario https://review.openstack.org/545297 | 13:33 |
| *** dmellado has quit IRC | 13:38 | |
| *** mgoddard_ has quit IRC | 13:39 | |
| *** trown|outtypewww is now known as trown | 13:42 | |
| *** ricardoas has joined #openstack-ironic | 13:42 | |
| *** jcoufal has joined #openstack-ironic | 13:44 | |
| * dtantsur reached 0 emails in his inbox, w00t! | 13:46 | |
| dtantsur | lucasagomes: hey! I guess you won't mind if I take over https://bugs.launchpad.net/ironic/+bug/1614094 will you? | 13:48 |
| openstack | Launchpad bug 1614094 in Ironic "Updating the port address for an active instance does not regenerate the PXE configuration" [High,Triaged] - Assigned to Lucas Alvares Gomes (lucasagomes) | 13:48 |
| dtantsur | lucasagomes: ditt for https://bugs.launchpad.net/ironic/+bug/1588901 | 13:48 |
| openstack | Launchpad bug 1588901 in Ironic "Nodes stuck on power state transitions" [High,Triaged] - Assigned to Lucas Alvares Gomes (lucasagomes) | 13:48 |
| *** rbartal has joined #openstack-ironic | 13:48 | |
| lucasagomes | dtantsur, not all all, go for it :D | 13:49 |
| dtantsur | cool | 13:49 |
| *** bfournie has joined #openstack-ironic | 13:51 | |
| jroll | morning y'all \o | 13:53 |
| lucasagomes | jroll, morning | 13:53 |
| *** masuberu has joined #openstack-ironic | 13:54 | |
| *** vsaienk0 has quit IRC | 13:56 | |
| *** masber has quit IRC | 13:58 | |
| dtantsur | morning jroll | 14:00 |
| *** vsaienk0 has joined #openstack-ironic | 14:01 | |
| jlvillal | Good morning Ironic | 14:02 |
| jlvillal | dtantsur, If you have time to look at: https://review.openstack.org/#/c/524321/ | 14:02 |
| patchbot | patch 524321 - ironic - Fix issue with double mocking of utils.execute fun... | 14:02 |
| jlvillal | An old patch... | 14:02 |
| dtantsur | morning jlvillal, putting on my list | 14:03 |
| jlvillal | dtantsur, Thanks :) | 14:03 |
| jlvillal | dtantsur, I'll give you my secret ~/.gitconfig alias | 14:05 |
| jlvillal | frbi = !git rebase -i $(git log --pretty=oneline --color=always | fzf --ansi | cut -d ' ' -f1)^ | 14:05 |
| jlvillal | :) | 14:05 |
| *** rbartal has quit IRC | 14:05 | |
| *** etingof has quit IRC | 14:12 | |
| *** mrch has joined #openstack-ironic | 14:12 | |
| *** mrch has left #openstack-ironic | 14:13 | |
| * dtantsur tries | 14:14 | |
| dtantsur | heh nice! | 14:15 |
| jlvillal | dtantsur, :) | 14:15 |
| jlvillal | dtantsur, 'rbi' = rebase --interactive | 14:16 |
| jlvillal | dtantsur, so 'frbi' is with fzf | 14:16 |
| jlvillal | Now I rarely use 'rbi' since so easy with 'frbi' | 14:17 |
| dtantsur | yeah, that's cool. especially since I always do interactive rebases | 14:17 |
| dtantsur | unrelated, but can someone please review https://review.openstack.org/545069 ? | 14:17 |
| patchbot | patch 545069 - ironic - Fix rare HTTP 400 from port list API | 14:17 |
| jlvillal | dtantsur, Done. I reviewed it over the weekend and left a comment | 14:18 |
| jlvillal | dtantsur, But not that important of a comment | 14:18 |
| dtantsur | jlvillal: yeah, but good to know, thanks | 14:18 |
| jlvillal | You're welcome | 14:18 |
| *** rbartal has joined #openstack-ironic | 14:19 | |
| *** Goneri has joined #openstack-ironic | 14:20 | |
| *** mjturek has joined #openstack-ironic | 14:24 | |
| *** etingof has joined #openstack-ironic | 14:26 | |
| *** mgoddard_ has joined #openstack-ironic | 14:26 | |
| *** jistr is now known as jistr|mtg | 14:31 | |
| *** baha has joined #openstack-ironic | 14:33 | |
| *** bfournie1 has joined #openstack-ironic | 14:38 | |
| *** bfournie has quit IRC | 14:39 | |
| *** links has joined #openstack-ironic | 14:41 | |
| *** dmellado has joined #openstack-ironic | 14:44 | |
| *** jroll has quit IRC | 14:47 | |
| *** jroll has joined #openstack-ironic | 14:48 | |
| *** links has quit IRC | 14:49 | |
| *** hjensas has quit IRC | 14:50 | |
| *** dtantsur is now known as dtantsur|brb | 15:01 | |
| *** links has joined #openstack-ironic | 15:02 | |
| *** dmellado has quit IRC | 15:02 | |
| *** jistr|mtg is now known as jistr | 15:02 | |
| *** rbartal has quit IRC | 15:06 | |
| -openstackstatus- NOTICE: Zuul has been restarted to pick up latest memory fixes. Queues were saved however patches uploaded after 14:40UTC may have been missed. Please recheck if needed. | 15:15 | |
| *** sovsianikov has joined #openstack-ironic | 15:17 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic stable/queens: Allow sqalchemy filtering by id and uuid https://review.openstack.org/545806 | 15:31 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic stable/pike: Allow sqalchemy filtering by id and uuid https://review.openstack.org/545854 | 15:32 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic stable/ocata: Allow sqalchemy filtering by id and uuid https://review.openstack.org/545855 | 15:32 |
| jroll | woo, three clean cherry picks | 15:33 |
| jroll | dtantsur|brb: ^ probably want to at least hit queens when you have a minute | 15:33 |
| *** lucasagomes is now known as lucas-hungry | 15:34 | |
| *** devananda has joined #openstack-ironic | 15:46 | |
| *** deadnull has joined #openstack-ironic | 15:47 | |
| sovsianikov | Hi there, Guys, I need your help/advice. | 15:47 |
| sovsianikov | I deploy Overcloud on my test environment and specify the following serail number of the disk: "serial": "WD-WMC1U5346656", but probably Nova can't use the disk and I have the Error No suitable device was found for deployment using these hints {u'serial': u'WD-WMC1U5346656'}", u'code': 404, u'type': u'DeviceNotFound', u'details': | 15:47 |
| sovsianikov | u"No suitable device was found for deployment using these hints {u'serial': u'WD-WMC1U5346656'}"} | 15:47 |
| sovsianikov | More logs can be found here http://paste.openstack.org/show/677599/ | 15:47 |
| sovsianikov | Could you please take a look at the Error and let me know which parameter should I use instead of serial number of the disk in order to deploy Overcloud? | 15:48 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 15:50 |
| *** dtantsur|brb is now known as dtantsur | 15:54 | |
| dtantsur | jroll: oh, I also proposed queens :) anyway, I'm fine with merging yours | 15:54 |
| dtantsur | sovsianikov: hi! which version of openstack is it? I remember the way some problem (/cc derekh) with lower vs upper case of serial | 15:55 |
| *** hjensas has joined #openstack-ironic | 15:55 | |
| sovsianikov | dtantsur: Hi, it's Newton | 15:56 |
| dtantsur | sovsianikov: wild case: try lower case serial in the root hints | 15:56 |
| dtantsur | s/case/guess/ | 15:56 |
| dtantsur | meh, I'm too sleepy today :( | 15:56 |
| sovsianikov | Ok, got it. Thanks a lot for your help! | 15:56 |
| * TheJulia hands dtantsur coffee | 15:56 | |
| dtantsur | thnx <3 | 15:57 |
| derekh | dtantsur: sovsianikov iirc the problem with case was fixed when the device hints matching logic was switched to ironic-lib, what ever version that was | 15:58 |
| sovsianikov | python-ironic-lib.noarch 2.1.3-1.el7ost | 15:59 |
| dtantsur | derekh: I recall is patching some version downstream, do you remember what it was? | 15:59 |
| derekh | dtantsur: I'm looking for it now | 15:59 |
| dtantsur | s/is/us/ (just pardon all typos from now on) | 15:59 |
| derekh | dtantsur: sovsianikov it was OSP 10 (Newton) | 16:00 |
| sovsianikov | this one? https://bugs.launchpad.net/ironic-python-agent/+bug/1679726 | 16:01 |
| openstack | Launchpad bug 1679726 in ironic-python-agent "discovery does not include /dev/disk/by-path device name" [Wishlist,Fix released] - Assigned to Ilya Etingof (etingof) | 16:01 |
| dtantsur | no, there was another one | 16:03 |
| *** deadnull has quit IRC | 16:05 | |
| derekh | sovsianikov: this was the downstream bug, iirc I didn't send the fix upstream as the newton branch is gone https://bugzilla.redhat.com/show_bug.cgi?id=1452226 | 16:05 |
| sovsianikov | Thank you derekh and dtantsur fro your help! | 16:06 |
| derekh | np | 16:06 |
| *** pcaruana has quit IRC | 16:07 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Update comment and mock about autospec not working on staticmethods https://review.openstack.org/545866 | 16:11 |
| jlvillal | dtantsur, ^^^ related to your HTTP 400 patch | 16:11 |
| dtantsur | ack thanks | 16:12 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Update comment and mock about autospec not working on staticmethods https://review.openstack.org/545866 | 16:14 |
| jlvillal | dtantsur, ^^ I put the 'import types' in the wrong section the first time :( | 16:14 |
| *** mjura has quit IRC | 16:17 | |
| jlvillal | mgoddard, FYI: https://review.openstack.org/545370 seems to be failing py35 unit tests | 16:18 |
| patchbot | patch 545370 - ironic - Register traits on nodes in devstack | 16:18 |
| *** masuberu has quit IRC | 16:21 | |
| jroll | dtantsur: oops! I didn't even realize it, even with the second patchset :| | 16:21 |
| dtantsur | no problem | 16:21 |
| *** r-daneel has quit IRC | 16:23 | |
| openstackgerrit | Michael Bayer proposed openstack/ironic master: Use oslo_db.sqlalchemy.test_fixtures https://review.openstack.org/545874 | 16:27 |
| *** lucas-hungry is now known as lucasagomes | 16:30 | |
| *** sovsianikov has quit IRC | 16:34 | |
| mgoddard_ | jlvillal: hmm, thanks | 16:36 |
| *** stendulker has joined #openstack-ironic | 16:36 | |
| mgoddard_ | jlvillal: not seeing any py35 unit test failures on that change | 16:40 |
| jlvillal | mgoddard_, I looked at zuul.o.o | 16:40 |
| jlvillal | mgoddard_, http://logs.openstack.org/70/545370/2/check/openstack-tox-py35/b8c8957/ | 16:41 |
| mgoddard_ | ah, I'll check there for the preview | 16:41 |
| *** vsaienk0 has quit IRC | 16:47 | |
| * TheJulia hopes http://zuul.openstack.org/stream.html?uuid=b79bd5917ee14a999c6fb9bbed4fb058&logfile=console.log | 16:50 | |
| *** rpioso|afk is now known as rpioso | 16:50 | |
| rpioso | Good morning | 16:50 |
| *** AlexeyAbashkin has quit IRC | 16:50 | |
| *** e0ne has quit IRC | 16:52 | |
| *** hshiina has joined #openstack-ironic | 16:53 | |
| TheJulia | good morning | 16:56 |
| rpioso | :) | 16:57 |
| *** hjensas has quit IRC | 16:59 | |
| TheJulia | #startmeeting ironic | 17:00 |
| openstack | Meeting started Mon Feb 19 17:00:10 2018 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 17:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 17:00 |
| TheJulia | o/ | 17:00 |
| dtantsur | o/ | 17:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 17:00 | |
| openstack | The meeting name has been set to 'ironic' | 17:00 |
| jroll | ohai | 17:00 |
| baha | o/ | 17:00 |
| rpioso | o/ | 17:00 |
| etingof | o/ | 17:00 |
| stendulker | o/ | 17:00 |
| hshiina | o/ | 17:00 |
| mgoddard_ | /o\ | 17:01 |
| TheJulia | Our agenda is on the wiki as always | 17:01 |
| TheJulia | Hopefully this meeting will be relatively quick given that next week is the PTG. | 17:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 17:02 |
| TheJulia | #topic Announcements/Reminders | 17:02 |
| *** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 17:02 | |
| TheJulia | #info Rocky PTG is next week, and as such next week's IRC meeting is cancelled. | 17:02 |
| TheJulia | #info An initial schedule is up on the PTG planning etherpad. https://etherpad.openstack.org/p/ironic-rocky-ptg | 17:03 |
| *** khyr0n has quit IRC | 17:03 | |
| TheJulia | #info The at PTG evening gathering will be on Tuesday at 7 PM. Information is inside the planning etherpad. | 17:03 |
| TheJulia | Also dtantsur, I believe we will be doing our final queens release for this cycle later this week? | 17:04 |
| dtantsur | TheJulia: yep, by Thu | 17:04 |
| TheJulia | #info Final queens release this week. | 17:04 |
| TheJulia | Anyone have anything else? | 17:04 |
| mjturek | o/ | 17:05 |
| TheJulia | On, on a side note, expect me to be sporadically available Wednesday-Friday. dtantsur will be likely be the one submitting the final queens release. | 17:05 |
| dtantsur | a mere shadow of former powers :D | 17:06 |
| TheJulia | And less stress :) | 17:07 |
| TheJulia | Anyway, I guess we can move on! | 17:07 |
| ricardoas | o/ | 17:07 |
| dtantsur | ++ move on | 17:08 |
| TheJulia | #topic Review Action Items | 17:08 |
| *** openstack changes topic to "Review Action Items (Meeting topic: ironic)" | 17:08 | |
| TheJulia | Last week, we had two action items. (1) Everyone triage bugs and (2) For myself to try and reproduce the crash of nova-conductor locally. | 17:08 |
| TheJulia | I was unable to do the latter, but we determined that disabling bytecode from being stored on disk prevents the crash created by library incompatibilities in grenade. | 17:09 |
| *** links has quit IRC | 17:09 | |
| * dtantsur triaged a few things and revives some older bugs | 17:10 | |
| TheJulia | Moving on! | 17:10 |
| TheJulia | dtantsur: thanks! | 17:10 |
| dtantsur | :) | 17:10 |
| *** dmellado has joined #openstack-ironic | 17:10 | |
| TheJulia | #topic Review sub-team status reports | 17:10 |
| *** openstack changes topic to "Review sub-team status reports (Meeting topic: ironic)" | 17:10 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 17:10 |
| TheJulia | I doubt there are any updates worth discussing, since last week was a fairly quiet week. | 17:11 |
| dtantsur | #action dtantsur add networking-baremetal and networking-generic-switch to his bug dashboard | 17:12 |
| * dtantsur dunno if #action works for non-chairs | 17:12 | |
| TheJulia | I don't believe so | 17:13 |
| TheJulia | #action dtantsur add networking-baremetal and networking-generic-switch to his bug dashboard | 17:13 |
| TheJulia | Looks like there is nothing to really discuss with regards to status, anyone have anything to bring up before we move on? | 17:14 |
| dtantsur | it seems like we have 2 bugs in rescue | 17:14 |
| dtantsur | to finish and backport | 17:14 |
| dtantsur | https://review.openstack.org/#/c/544278/ and https://review.openstack.org/#/c/545186/ | 17:15 |
| patchbot | patch 544278 - ironic - Build instance PXE options for unrescue | 17:15 |
| patchbot | patch 545186 - ironic - Set 'initrd' to 'rescue_ramdisk' for rescue with iPXE | 17:15 |
| * dtantsur is ready to move on | 17:15 | |
| TheJulia | dtantsur: please add those to the priorities list so we're tracking them (if they are not aleady there) | 17:15 |
| TheJulia | already | 17:15 |
| dtantsur | I think I did | 17:16 |
| TheJulia | #topic Priorities for the coming week | 17:16 |
| *** openstack changes topic to "Priorities for the coming week (Meeting topic: ironic)" | 17:16 | |
| TheJulia | \o/ | 17:16 |
| mgoddard_ | There is a bug when using cleaning with traits set: https://bugs.launchpad.net/ironic/+bug/1750027 | 17:17 |
| openstack | Launchpad bug 1750027 in Ironic "Traits & cleaning issue" [High,In progress] - Assigned to Mark Goddard (mgoddard) | 17:17 |
| openstackgerrit | Merged openstack/ironic master: Conductor version cannot be null in Rocky https://review.openstack.org/544382 | 17:17 |
| openstackgerrit | Merged openstack/ironic master: Clean nodes stuck in CLEANING state when ir-cond restarts https://review.openstack.org/349971 | 17:17 |
| *** trown is now known as trown|lunch | 17:17 | |
| openstackgerrit | Merged openstack/ironic master: Fix rare HTTP 400 from port list API https://review.openstack.org/545069 | 17:18 |
| * dtantsur cherry-picks ^^^ | 17:18 | |
| mgoddard_ | Would be nice to get the fix merged before the queens release | 17:18 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Fix rare HTTP 400 from port list API https://review.openstack.org/545892 | 17:19 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Clean nodes stuck in CLEANING state when ir-cond restarts https://review.openstack.org/545893 | 17:19 |
| TheJulia | I think that is good, I cleaned up the lists and removed prior items | 17:19 |
| dtantsur | mgoddard_: I keep an eye on it, just waiting for the CI | 17:19 |
| mgoddard_ | dtantsur: thanks. jlvillal pointed out there is a py3.5 unit test failure so there will be at least another revision | 17:20 |
| TheJulia | I agree we should likely try and merge/backport | 17:20 |
| jroll | there's also that bug we're seeing in grenade that I'm hacking on - tl;dr explosive problems with placement if ironic goes down for too long | 17:21 |
| dtantsur | oh | 17:21 |
| TheJulia | jroll: has a bug been opened for that? | 17:21 |
| jroll | here's the wip patch https://review.openstack.org/#/c/545479/ | 17:21 |
| patchbot | patch 545479 - nova - ironic: stop lying to the RT when ironic is down | 17:21 |
| jroll | TheJulia: I should open a separate bug if there's not yet | 17:22 |
| * jroll will do today | 17:22 | |
| TheJulia | jroll: thanks, please add to the priorites list for the week since we will need ironic eyes on it | 17:22 |
| TheJulia | err | 17:22 |
| TheJulia | wip patch, I'll add that | 17:22 |
| jroll | thanks | 17:22 |
| TheJulia | Okay, added | 17:23 |
| TheJulia | Anything else? | 17:23 |
| dtantsur | not from me | 17:24 |
| TheJulia | Everyone happy with the list? | 17:24 |
| * jroll is good | 17:24 | |
| TheJulia | Moving to Open Discussion since we have no discussion items on the agenda... | 17:24 |
| TheJulia | #topic Open discussion | 17:24 |
| *** openstack changes topic to "Open discussion (Meeting topic: ironic)" | 17:24 | |
| * jroll thanks derekh for putting together the thing | 17:25 | |
| TheJulia | Everyone going to remember to bring an ironic t-shirt for the team photo? | 17:25 |
| TheJulia | Yes, thanks derekh! | 17:25 |
| derekh | no prob | 17:25 |
| * jroll has one | 17:25 | |
| jroll | team photo is on thursday, for the curious | 17:26 |
| TheJulia | \o/ | 17:27 |
| TheJulia | Does anyone have anything else to discuss? | 17:27 |
| TheJulia | Otherwise, I think we can wrap this short meeting up for today. | 17:27 |
| dtantsur | cri... | 17:28 |
| devananda | I'd be ever so thrilled to join some of the shenanigans remotely, maybe do a video call in at some point | 17:28 |
| dtantsur | devananda: we'll call you from the bar ;) | 17:28 |
| TheJulia | ++++ | 17:28 |
| devananda | davidlenwell offered to pin a phone to his shirt while we use hangouts ;) | 17:28 |
| dtantsur | ... or when we discuss converting to a CMDB finally? | 17:28 |
| jroll | LOL | 17:28 |
| devananda | dtantsur: yes pls! I've got a nice bottle of whisky here - I would love to raise a glass with ya'll | 17:28 |
| dtantsur | :D | 17:29 |
| devananda | dtantsur: bwahaha! | 17:29 |
| TheJulia | So sounds like we have some plans! | 17:29 |
| TheJulia | :) | 17:29 |
| * dtantsur seems to hear crickets again | 17:30 | |
| TheJulia | Anyway, the crickets as a service have returned with a beautiful cricket sound. I think we can call this meeting a wrap | 17:30 |
| TheJulia | Thanks everyone! | 17:30 |
| dtantsur | ++ thanks TheJulia | 17:30 |
| TheJulia | Oh, one final thing, I'll break the etherpad up after I have lunch in a little bit, so expect to see text moving around in relation to the schedule | 17:31 |
| TheJulia | #endmeeting | 17:31 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 17:31 | |
| openstack | Meeting ended Mon Feb 19 17:31:43 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:31 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-19-17.00.html | 17:31 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-19-17.00.txt | 17:31 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-19-17.00.log.html | 17:31 |
| *** etingof has quit IRC | 17:32 | |
| dtantsur | TheJulia: where we ended up with https://bugs.launchpad.net/ironic/+bug/1596421 do you remember? | 17:32 |
| openstack | Launchpad bug 1596421 in Ironic "[RFE] Stop storing configdrive in the database" [Wishlist,In progress] - Assigned to Dmitry Tantsur (divius) | 17:32 |
| TheJulia | ugh | 17:33 |
| TheJulia | that one | 17:33 |
| dtantsur | yes, got a customer complaining about that | 17:34 |
| dtantsur | given that providing configdrive in rebuild is microversioned.. I'm not sure we can just stop saving it | 17:34 |
| dtantsur | though maybe we should strip it if we get an error from the database? | 17:35 |
| dtantsur | like, it's still better than failing completely? or not? | 17:35 |
| * dtantsur thinks | 17:35 | |
| TheJulia | I think the solution that we came to that would kind of be ideal... was to add a table for it specifically and store it in there | 17:35 |
| TheJulia | but... that presents other issues | 17:35 |
| TheJulia | I mean... maybe not if we also begin wiping it out... | 17:36 |
| * TheJulia thinks that would be a fair microversion change to have | 17:36 | |
| dtantsur | well, in theory we should not store blobs in database, but rather use something like swift | 17:36 |
| dtantsur | none of this is backportable | 17:36 |
| TheJulia | no, none of it would be | 17:36 |
| TheJulia | swift would be ideal, but then all deployments require swift. :\ | 17:37 |
| dtantsur | for reference, people asking about it on Newton: https://bugzilla.redhat.com/show_bug.cgi?id=1433088 | 17:37 |
| pas-ha | dtantsur: what about take over happening during deploy/rebuild when config drive is not in the DB? | 17:37 |
| *** ijw has joined #openstack-ironic | 17:37 | |
| pas-ha | and w/o swift I mean | 17:38 |
| *** stendulker has quit IRC | 17:38 | |
| TheJulia | Well, we could store blobs in the db as long as their content is not indexed and the table is kept clean | 17:38 |
| dtantsur | pas-ha: I'm not sure take over is affected here. if config drive is not in DB and is not supplied to rebuild, we won't have a config drive.. | 17:38 |
| TheJulia | which would be "new table" territory | 17:38 |
| *** ijw has quit IRC | 17:38 | |
| dtantsur | or we should merge that change that makes the column larger on mysql | 17:39 |
| *** ijw has joined #openstack-ironic | 17:39 | |
| * TheJulia mulls that | 17:39 | |
| pas-ha | or the bug is about 'not storing configdrive after deploy is done for active node'? I may be confused.. | 17:39 |
| TheJulia | Not great... but we also know some folks have done just that in their deployments | 17:40 |
| pas-ha | I mean we still need it during deploy/rebuild as part if instance_info in the DB so that takeover may happen before the configdrive is written | 17:40 |
| dtantsur | pas-ha: we're trying to git the config drive as part of a JSON into a text column. which is something just too small. | 17:40 |
| *** baha has quit IRC | 17:40 | |
| dtantsur | I mean, putting it in instance_info simply does not always work | 17:40 |
| dtantsur | if you have a lot of metadata - ooops | 17:41 |
| pas-ha | yeah, that's not great... | 17:41 |
| dtantsur | it's not even about rebuild, you won't deploy in the first place | 17:41 |
| pas-ha | but we do need it either in DB or in the external source. | 17:41 |
| dtantsur | pas-ha: we don't strictly need it, since starting with queens nova passes it to us IIRC | 17:42 |
| dtantsur | but we cannot probably rely on it | 17:42 |
| dtantsur | e.g. standalone people might not do it still | 17:42 |
| TheJulia | yeah | 17:42 |
| dtantsur | but standalone people don't have many options as well: they don't have swift | 17:42 |
| TheJulia | I think the "cleanest" thing to do is likely expand the column width, and if we need to migrate it out to another table, then we actually have some advantages there to try and keep that quick for most deployments | 17:43 |
| dtantsur | so, https://review.openstack.org/334967 | 17:43 |
| patchbot | patch 334967 - ironic - Fix non-deployed nodes due to short nodes.instance... | 17:43 |
| TheJulia | yeah | 17:43 |
| dtantsur | which we won't be able to backport past queens? | 17:43 |
| TheJulia | hey, jroll, any thoughts on ^^ | 17:43 |
| pas-ha | still not get it why we do not need to store it w/o swift being available. nova posts it to instance_info -> ironic starts deploy -> ironic-conductor fails even before first heartbeat received -> another conductor takes over -> it needs to recreate the node's instance_info from DB | 17:44 |
| dtantsur | pas-ha: if we have swift, we can try storing it in swift instead | 17:44 |
| jroll | TheJulia: on backporting db changes or the configdrive thing? | 17:44 |
| TheJulia | jroll: in general | 17:44 |
| TheJulia | the configdrive thing | 17:44 |
| jroll | ah, give me a few to finish a nova conversation? | 17:44 |
| dtantsur | TheJulia: well, we can try backporting it further, but we'll have to carefully adjust the migration | 17:45 |
| pas-ha | yeah, but that would the require swift, which I thought we were trying to avoid (and me personally too). | 17:45 |
| TheJulia | jroll: sure | 17:45 |
| dtantsur | pas-ha: yes, this is my point: swift could save us, but we cannot rely on it :) | 17:45 |
| * dtantsur updates the patch | 17:46 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Set 'initrd' to 'rescue_ramdisk' for rescue with iPXE https://review.openstack.org/545186 | 17:46 |
| * dtantsur responds to himself: no, we cannot backport it past queens, it will break later updates | 17:46 | |
| pas-ha | a bitter side note - if we currently start to require swift, we loose Py3 support as we won't be able to test it on gates... | 17:49 |
| TheJulia | dtantsur: that patch as is? | 17:49 |
| dtantsur | yep | 17:49 |
| dtantsur | pas-ha: ugh, true | 17:49 |
| dtantsur | anyway, we're not giving up standalone mode | 17:49 |
| * TheJulia hasn't even gotten to that point mentally | 17:50 | |
| * TheJulia needs to eat something | 17:50 | |
| dtantsur | mmm, should we use MEDIUMTEXT or LONGTEXT? a configdrive can be up to 64MiB IIRC? | 17:52 |
| dtantsur | while MEDIUMTEXT is 16 MiB | 17:52 |
| * dtantsur votes for LONGTEXT and API check on len(instance_info['configdrive']) <= 64 MiB | 17:53 | |
| TheJulia | I kind of like that idea | 17:54 |
| jroll | note the configdrive is gzipped and b64 encoded, so that length check may not work | 17:54 |
| jroll | (as expected) | 17:55 |
| TheJulia | yeah... | 17:55 |
| *** racedo has quit IRC | 17:59 | |
| dtantsur | hmm, should we even check actually? | 18:00 |
| *** derekh has quit IRC | 18:00 | |
| *** e0ne has joined #openstack-ironic | 18:01 | |
| jroll | idk | 18:01 |
| * jroll reads back for context | 18:01 | |
| *** mgoddard_ has quit IRC | 18:02 | |
| *** hshiina has quit IRC | 18:02 | |
| dtantsur | well, in PostreSQL TEXT is unlimited, so for consistency we probably should not check | 18:02 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Increase the instance_info column size to LONGTEXT on MySQL/MariaDB https://review.openstack.org/334967 | 18:03 |
| dtantsur | here goes the patch ^^^ | 18:03 |
| jroll | dtantsur: TheJulia: so we're storing it simply so that we can do take_over properly during a build? | 18:14 |
| * jroll would rather just not store it, people love to put secrets there | 18:14 | |
| dtantsur | jroll: for rebuilds as well | 18:14 |
| jroll | dtantsur: rebuilds get a new configdrive now, though, right | 18:15 |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:15 | |
| * jroll sees we decided a long time ago to pass configdrive on rebuild, then stop storing it | 18:17 | |
| *** rwsu has quit IRC | 18:17 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| jroll | I would be fine with failing builds where a conductor dies during the build, btw, to be able to keep the configdrive out of the db | 18:18 |
| *** AlexeyAbashkin has quit IRC | 18:20 | |
| *** e0ne has quit IRC | 18:21 | |
| *** hjensas has joined #openstack-ironic | 18:23 | |
| TheJulia | jroll: I know of at least one group that expected it to continue to be able to deploy with a conductor failure and was in their test plan as such..... :\ | 18:23 |
| TheJulia | Although, we've not explicitly stated that is something we do, it is by side-effect | 18:24 |
| TheJulia | I would really prefer to keep it out of the db, but where do we house it while waiting for the machine? | 18:24 |
| *** hjensas has quit IRC | 18:27 | |
| jroll | um, uh, good question. | 18:28 |
| jroll | dang. | 18:28 |
| *** jrist is now known as jrist-afk | 18:28 | |
| *** jrist-afk is now known as jrist | 18:28 | |
| *** rwsu has joined #openstack-ironic | 18:29 | |
| * jroll wonders wtf we were planning to do when we decided that | 18:30 | |
| *** tesseract has quit IRC | 18:30 | |
| dtantsur | swift when it's present otherwise a separate blob field... | 18:31 |
| * dtantsur does not remember | 18:31 | |
| jroll | yeah, we must have been assuming we'd keep it in ram | 18:31 |
| * jroll wishes we just had a thread-ish thing that polled the agent, rather than waiting for the callback | 18:32 | |
| TheJulia | I really think it was another table | 18:33 |
| TheJulia | jroll: soonish? ;) | 18:33 |
| jroll | I don't think so: "since bug #1575935 is fixed, we no longer need to store the configdrive in the database at all." | 18:33 |
| openstack | bug 1575935 in python-ironicclient "Rebuild should also accept a configdrive" [Medium,Fix released] https://launchpad.net/bugs/1575935 - Assigned to Mathieu Gagné (mgagne) | 18:33 |
| dtantsur | jroll: even with polling the agent, you have to be able to survive a conductor restart | 18:35 |
| jroll | dtantsur: I'd be willing to let builds fail if the conductor restarts, to keep user secrets out of the db | 18:35 |
| dtantsur | jroll: this is not what everyone would prefer, I'd suspect.. | 18:35 |
| jroll | dtantsur: --i-am-a-dummy, then :( | 18:36 |
| dtantsur | on a busy environment it make it essentially impossible to restart conductor without breaking a few (dozens?) of deployments | 18:36 |
| jroll | idk, even though people shouldn't pass secrets in nova metadata, I feel like we should be secure about user data always (or at least by default) | 18:36 |
| jroll | dtantsur: we have soft shutdown for that | 18:37 |
| dtantsur | I'm on the "people should not pass secrets" side | 18:37 |
| jroll | stop taking new builds, finish the existing | 18:37 |
| dtantsur | ehmm, how will soft shutdown help here? | 18:37 |
| dtantsur | ah, I think I see it | 18:37 |
| jroll | it solves "on a busy environment it make it essentially impossible to restart conductor without breaking a few (dozens?) of deployments" | 18:37 |
| jroll | just slows down the rolling restart | 18:37 |
| dtantsur | yep, substantially | 18:37 |
| dtantsur | I think currently we don't wait for all nodes in DEPLOY WAIT | 18:38 |
| jroll | this is true, good point | 18:38 |
| * dtantsur feels like we've found another topic for TheJulia to squeeze into the PTG schedule | 18:38 | |
| jroll | anyway, I don't have the time to fight against storing it at all | 18:38 |
| *** r-daneel has joined #openstack-ironic | 18:39 | |
| jroll | we're okay with continuing to use instance_info, rather than a new column? | 18:39 |
| *** trown|lunch is now known as trown | 18:39 | |
| *** e0ne has joined #openstack-ironic | 18:40 | |
| * jroll wonders how drastically a large configdrive slows down un/serialization there | 18:40 | |
| dtantsur | okay short-term | 18:40 |
| dtantsur | like a fix for Queens | 18:41 |
| jroll | ah yes, we can backport to queens, but only if we do it quickly | 18:41 |
| dtantsur | yep | 18:41 |
| jroll | we can't ever go back to MEDIUMTEXT thoug :( | 18:41 |
| *** khyr0n has joined #openstack-ironic | 18:42 | |
| jroll | or TEXT, sorry | 18:42 |
| dtantsur | well, I'm not sure how crazy mysql is, but I hope it does not allocate too much for LONGTEXT | 18:42 |
| dtantsur | like, postresql only has LONGTEXT | 18:42 |
| * dtantsur wishes we used a more serious database, sigh... | 18:42 | |
| jroll | heh | 18:42 |
| * jroll throws mongodb in and runs far far away | 18:43 | |
| dtantsur | haha | 18:43 |
| dtantsur | devnull as a service | 18:43 |
| persia | That's an excellent place to store config drives :) | 18:43 |
| *** mgoddard_ has joined #openstack-ironic | 18:44 | |
| jroll | heh | 18:44 |
| dtantsur | jroll: okay, so the biggest problem is that a new node field will require a new RPC version | 18:44 |
| jroll | dtantsur: yeah, that also isn't awesome for backporting | 18:44 |
| dtantsur | which, if I get it right, we cannot backport | 18:44 |
| jroll | I'm good with changing the size for now, let's do it | 18:44 |
| *** hjensas has joined #openstack-ironic | 18:44 | |
| * jroll +1'd | 18:44 | |
| dtantsur | yep, let's do it.. | 18:44 |
| *** rwsu has quit IRC | 18:46 | |
| jroll | [openstack-dev] [oslo.db] [all] please DO NOT IMPORT from oslo_db.tests.* ! projects doing this need to revert ASAP | 18:47 |
| jroll | ouch | 18:47 |
| jroll | ah, he fixed it for us, cool | 18:47 |
| *** lucasagomes is now known as lucas-brb | 18:57 | |
| *** baha has joined #openstack-ironic | 18:58 | |
| *** baha1 has joined #openstack-ironic | 18:58 | |
| * TheJulia is +1 for size and having a real fix this cycle | 18:59 | |
| *** baha has quit IRC | 19:01 | |
| *** rwsu has joined #openstack-ironic | 19:01 | |
| *** harlowja has joined #openstack-ironic | 19:03 | |
| *** baha1 is now known as baha | 19:04 | |
| * TheJulia watches her first sand storm move in | 19:07 | |
| *** dtantsur is now known as dtantsur|afk | 19:10 | |
| dtantsur|afk | o/ | 19:10 |
| *** openstackgerrit has joined #openstack-ironic | 19:10 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: WIP: Add check for (i)PXE and Ironic location coherence https://review.openstack.org/545711 | 19:10 |
| *** ijw has quit IRC | 19:23 | |
| *** mgoddard_ has quit IRC | 19:25 | |
| *** mgoddard_ has joined #openstack-ironic | 19:56 | |
| *** Goneri has quit IRC | 19:57 | |
| *** lucas-brb is now known as lucasagomes | 19:58 | |
| TheJulia | Okay folks, I think I have the etherpad split up | 20:02 |
| TheJulia | for the PTG that is | 20:02 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Support nested objects and object lists in as_dict https://review.openstack.org/545830 | 20:10 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Register traits on nodes in devstack https://review.openstack.org/545370 | 20:10 |
| *** lucasagomes is now known as lucas-afk | 20:14 | |
| *** etingof has joined #openstack-ironic | 20:31 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate instance_info.traits against node traits https://review.openstack.org/543461 | 20:34 |
| *** mgoddard has quit IRC | 20:35 | |
| *** devananda has quit IRC | 20:37 | |
| *** mgoddard has joined #openstack-ironic | 20:42 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM Test disabling .pyc files for Grenade https://review.openstack.org/545089 | 20:44 |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Add check for (i)PXE and Ironic location coherence https://review.openstack.org/545711 | 20:45 |
| *** mgoddard_ has quit IRC | 20:53 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM Test disabling .pyc files for Grenade https://review.openstack.org/545089 | 20:59 |
| *** AlexeyAbashkin has joined #openstack-ironic | 21:26 | |
| *** AlexeyAbashkin has quit IRC | 21:31 | |
| *** dprince has quit IRC | 21:33 | |
| *** rcernin has joined #openstack-ironic | 21:36 | |
| *** threestrands has joined #openstack-ironic | 21:37 | |
| jroll | TheJulia: how do we feel about nova-compute failing to start if ironic-api is not up? | 21:40 |
| * TheJulia ponders | 21:40 | |
| jroll | my self-healing hat says no, but my don't-want-to-get-into-the-guts-of-nova hat says it's fine | 21:40 |
| TheJulia | I think it is fine in that the compute service has then visibily failed | 21:41 |
| TheJulia | if we say we're good and not really working, then that in it's self is a disservice to an operator | 21:41 |
| TheJulia | So on start-up, I think it is fine for us to fail, the conundrum might be grenade... but that might be the motivation to retool finally if we can't restart the process due to library compatability issues across releases | 21:42 |
| jroll | good point | 21:42 |
| jroll | it'll keep running if it's already up, fwiw | 21:42 |
| *** Goneri has joined #openstack-ironic | 21:42 | |
| jroll | just with outdated resource tracking | 21:42 |
| TheJulia | which it... ideally... shoudl deal with eventually | 21:43 |
| TheJulia | should() | 21:43 |
| TheJulia | I think that returns false | 21:43 |
| jroll | heh | 21:43 |
| jroll | or it hangs forever | 21:44 |
| TheJulia | woot! | 21:44 |
| TheJulia | will it stop heartbeating inside nova for service status? | 21:44 |
| * jroll adds a release note about it | 21:44 | |
| jroll | that's a good question, I don't think so? | 21:44 |
| TheJulia | We should put a release note on each side | 21:44 |
| TheJulia | ohhh... hmmmmmm | 21:44 |
| jroll | I mean, we could | 21:45 |
| TheJulia | known issue I guess? We have a solid hour without topics with nova for the schedule right now | 21:45 |
| jroll | it'd be nice to sit down and think this through fully (as there's lots of hacks around this right now), but would also be nice to get this bug fixed now | 21:45 |
| TheJulia | Agreed | 21:46 |
| jroll | it'd be trivial to mark self._ironic_down = True in the driver | 21:46 |
| TheJulia | Hence why I was thinking about the schedule | 21:46 |
| jroll | yeah | 21:46 |
| TheJulia | or _compute_down | 21:46 |
| jroll | yeah | 21:46 |
| * jroll has no clue how the heartbeat works O_o | 21:48 | |
| jroll | well, still not sure where the heartbeat is | 21:52 |
| jroll | but we can do this: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1707-L1711 | 21:52 |
| TheJulia | perfect | 21:53 |
| *** priteau has quit IRC | 21:53 | |
| *** priteau has joined #openstack-ironic | 21:54 | |
| *** jcoufal has quit IRC | 21:55 | |
| *** trown is now known as trown|outtypewww | 21:56 | |
| * jroll will leave this for another day | 21:57 | |
| TheJulia | Okay | 21:58 |
| *** priteau has quit IRC | 21:59 | |
| *** e0ne has quit IRC | 22:06 | |
| *** hamzy has quit IRC | 22:07 | |
| * jroll files https://bugs.launchpad.net/nova/+bug/1750450 in the meantime | 22:09 | |
| openstack | Launchpad bug 1750450 in OpenStack Compute (nova) "ironic: n-cpu fails to recover after losing connection to ironic-api and placement-api" [Undecided,New] | 22:09 |
| *** hamzy has joined #openstack-ironic | 22:16 | |
| *** openstackgerrit has quit IRC | 22:18 | |
| *** openstackgerrit has joined #openstack-ironic | 22:20 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM Test disabling .pyc files for Grenade https://review.openstack.org/545089 | 22:20 |
| *** priteau has joined #openstack-ironic | 22:22 | |
| *** masber has joined #openstack-ironic | 22:23 | |
| TheJulia | jroll: thanks! | 22:24 |
| jroll | :) | 22:24 |
| *** baha has quit IRC | 22:27 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Disable .pyc files for grenade multinode https://review.openstack.org/544750 | 22:32 |
| *** priteau has quit IRC | 22:38 | |
| *** bfournie1 has quit IRC | 22:46 | |
| *** bfournie has joined #openstack-ironic | 22:46 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add docs for ansible deploy interface https://review.openstack.org/525501 | 22:48 |
| *** bfournie has quit IRC | 22:53 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM Test disabling .pyc files for Grenade https://review.openstack.org/545089 | 22:59 |
| *** AlexeyAbashkin has joined #openstack-ironic | 23:06 | |
| *** AlexeyAbashkin has quit IRC | 23:10 | |
| openstackgerrit | Merged openstack/networking-baremetal master: Update reno for stable/queens https://review.openstack.org/542331 | 23:13 |
| jlvillal | TheJulia, I replied to your comment: https://review.openstack.org/#/c/528303/2/playbooks/legacy/tempest-dsvm-ironic-inspector/run.yaml@a97 | 23:18 |
| patchbot | patch 528303 - ironic - zuul: Remove duplicated TEMPEST_PLUGIN entry | 23:18 |
| jlvillal | And thanks for reviewing :) | 23:18 |
| *** pmannidi has joined #openstack-ironic | 23:19 | |
| *** masber has quit IRC | 23:34 | |
| *** r-daneel has quit IRC | 23:36 | |
| *** masber has joined #openstack-ironic | 23:37 | |
| *** bfournie has joined #openstack-ironic | 23:52 | |
| *** masber has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!