*** dsneddon has quit IRC | 00:21 | |
*** uzumaki has joined #openstack-ironic | 00:55 | |
hjensas | TheJulia: around? I'm seeing ironic.common.exception.AgentAPIError: Agent API for node f1a36f47-ddfd-442c-9c81-a47f5a3dca22 returned HTTP status code 401 with error: 401 Unauthorized: Token invalid. | 00:56 |
---|---|---|
hjensas | TheJulia: in tripleo, we do not set require_agent_token afict. | 00:57 |
*** uzumaki has quit IRC | 01:05 | |
*** gyee has quit IRC | 01:07 | |
*** yedongcan has joined #openstack-ironic | 01:13 | |
*** rloo has quit IRC | 01:20 | |
*** uzumaki has joined #openstack-ironic | 01:28 | |
*** ricolin has joined #openstack-ironic | 01:58 | |
*** k_mouza has joined #openstack-ironic | 02:31 | |
*** k_mouza has quit IRC | 02:35 | |
*** mkrai has joined #openstack-ironic | 02:35 | |
*** Goneri has quit IRC | 02:40 | |
*** rh-jelabarre has quit IRC | 03:00 | |
*** Qianbiao has joined #openstack-ironic | 03:11 | |
Qianbiao | hello | 03:20 |
Qianbiao | Good morning | 03:21 |
Qianbiao | Is stein using python3 by default now? | 03:22 |
uzumaki | There has been work going on to jump things to py3, not sure about stein though | 03:26 |
*** jdandrea has quit IRC | 03:34 | |
*** yaawang__ has quit IRC | 03:53 | |
*** cdearborn has quit IRC | 03:59 | |
*** mkrai has quit IRC | 04:01 | |
*** mkrai has joined #openstack-ironic | 04:02 | |
*** Qianbiao has quit IRC | 04:47 | |
*** ociuhandu has joined #openstack-ironic | 04:47 | |
*** zaneb has quit IRC | 04:48 | |
*** zaneb has joined #openstack-ironic | 04:49 | |
*** ociuhandu has quit IRC | 04:51 | |
*** igordc has quit IRC | 05:32 | |
iurygregory | stein have support for py3 and so train | 05:39 |
iurygregory | good morning Ironic | 05:39 |
*** markguz_ has quit IRC | 05:40 | |
iurygregory | by default i think it only started in Stein for most of the projects | 05:41 |
iurygregory | Ussuri will be python3 only | 05:41 |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 05:42 |
uzumaki | morning iurygregory o/ | 05:47 |
uzumaki | my patch^ is ready for reviews now, if anybody wants to take a look | 05:47 |
*** yaawang has joined #openstack-ironic | 05:58 | |
*** iurygregory has quit IRC | 06:25 | |
*** iurygregory has joined #openstack-ironic | 06:27 | |
iurygregory | hey uzumaki o/ | 06:28 |
iurygregory | sorry my notebook went crazy (the screen went black and didn't show anything) O.o I had to restart by pressing power button... | 06:28 |
uzumaki | iurygregory, ah, that sucks! | 06:29 |
iurygregory | yeah, I was like wtf?! | 06:29 |
* kaifeng think iurygregory is lucky enough to not have partition table destroyed | 06:30 | |
iurygregory | kaifeng, ouch! that would be a problem D: | 06:31 |
* uzumaki has back in the day messed around with drives with corrupted partition tables/file systems, attempting to restore data | 06:35 | |
*** Qianbiao has joined #openstack-ironic | 06:38 | |
arne_wiebalck | Good morning, ironic! | 06:49 |
iurygregory | morning arne_wiebalck o/ | 06:49 |
arne_wiebalck | hey iurygregory o/ | 06:49 |
*** yaawang has quit IRC | 07:04 | |
*** yaawang has joined #openstack-ironic | 07:05 | |
arne_wiebalck | mgoddard: at the midcycle I whined once more about the difficulty to manage projects with many baremetal flavors and the lack of per flavor quota :-) ... is os-capacity sth I should give a try (or is this abandoned)? | 07:13 |
arne_wiebalck | mgoddard: good morning :) | 07:13 |
*** mkrai has quit IRC | 07:14 | |
*** belmoreira has joined #openstack-ironic | 07:16 | |
*** rpittau|afk is now known as rpittau | 07:17 | |
rpittau | good morning ironic! o/ | 07:17 |
iurygregory | morning rpittau o/ | 07:18 |
rpittau | hey iurygregory :) | 07:20 |
iurygregory | rpittau, I've found something interesting in ironic when using unittest.mock with py36 https://review.opendev.org/#/c/717979/ | 07:25 |
patchbot | patch 717979 - ironic - Switch to unittest mock - 2 patch sets | 07:25 |
rpittau | lots of red there :) | 07:28 |
iurygregory | only because it's py36 | 07:28 |
iurygregory | =( | 07:28 |
iurygregory | py37 38 is fine D: | 07:28 |
iurygregory | https://bugs.python.org/issue23078 | 07:29 |
iurygregory | seems like they only fixed for Python3.7 and 3.8 XD | 07:30 |
rpittau | it's interesting that's happening only in ironic and not other projects | 07:31 |
rpittau | iurygregory: you said locally was working ? | 07:31 |
iurygregory | rpittau, yep because I have 3.7 locally XD | 07:31 |
iurygregory | I just checked =X | 07:31 |
iurygregory | I think it's the structure of the test it's only happening in the virtualmediaredfish | 07:32 |
iurygregory | I will try to run with 3.6 locally and change things in the tests | 07:32 |
rpittau | iurygregory: you mean you noly have py3.7 installed ? cause tox should enforce the correct version, if you have the binary installed | 07:33 |
iurygregory | I mean if I run tox -r -vv -e py3 (it will use python3.7) | 07:34 |
rpittau | use py36 | 07:34 |
*** mkrai has joined #openstack-ironic | 07:34 | |
rpittau | otherwise it will use the default py3 binary | 07:34 |
iurygregory | yeah, normally I only run py3 | 07:35 |
iurygregory | =) | 07:35 |
iurygregory | hehehe py36 fail =D | 07:37 |
*** dtantsur|afk is now known as dtantsur | 07:41 | |
dtantsur | morning ironic | 07:41 |
dtantsur | can someone approve https://review.opendev.org/#/c/718067/ please? arne_wiebalck or mgoddard if you're around? | 07:41 |
patchbot | patch 718067 - python-ironicclient - Standalone CLI: optional support for ironic-inspector - 1 patch set | 07:41 |
dtantsur | we need it for the release | 07:41 |
iurygregory | morning dtantsur | 07:41 |
dtantsur | etingof: hey, do we need anything for sushy-cli before a release? | 07:43 |
etingof | o/ I think we are good | 07:43 |
*** ociuhandu has joined #openstack-ironic | 07:44 | |
dtantsur | ok, thanks | 07:45 |
*** mkrai has quit IRC | 07:54 | |
*** mkrai has joined #openstack-ironic | 07:57 | |
*** dmellado has quit IRC | 08:01 | |
*** aedc_ has joined #openstack-ironic | 08:02 | |
*** ociuhandu has quit IRC | 08:02 | |
*** ociuhandu has joined #openstack-ironic | 08:03 | |
*** dmellado has joined #openstack-ironic | 08:05 | |
*** aedc has quit IRC | 08:05 | |
*** lucasagomes has joined #openstack-ironic | 08:08 | |
*** ociuhandu has quit IRC | 08:08 | |
*** aedc_ has quit IRC | 08:11 | |
*** mkrai has quit IRC | 08:15 | |
*** aedc has joined #openstack-ironic | 08:17 | |
*** mkrai has joined #openstack-ironic | 08:20 | |
Qianbiao | hello, ironic | 08:21 |
Qianbiao | i fould eventlet is used by ironic. | 08:21 |
openstackgerrit | Pierre Riteau proposed openstack/tenks stable/1.0: Pin virtualbmc to <2 when using Python 2 https://review.opendev.org/718341 | 08:22 |
Qianbiao | but it seems eventlet lib has an open issue | 08:22 |
Qianbiao | https://github.com/eventlet/eventlet/issues/240 | 08:22 |
Qianbiao | the issue is described here: https://github.com/eventlet/eventlet/issues/240 | 08:22 |
Qianbiao | when i try to install ironic, it will throw 'select' object has no attribute 'poll'" | 08:23 |
Qianbiao | which is throw by eventlet lib | 08:23 |
rpittau | Qianbiao: that issue is quite old, are you sure it's still active ? | 08:24 |
dtantsur | the issue you post doesn't seem exactly relevant, we monkey-patch explicitly | 08:24 |
Qianbiao | Hmm. that's wierd. I try to install stein in ubuntu 18.04 | 08:24 |
Qianbiao | it does throw that error | 08:24 |
Qianbiao | If i install eventlet by pip3 install eventlet==0.18.4 before `apt install ironic-api`, everything goes well then | 08:26 |
Qianbiao | It indicates that an old version eventlet works. | 08:26 |
openstackgerrit | shenxindi proposed openstack/ironic master: Release reservation when stoping the ironic-conductor service https://review.opendev.org/718342 | 08:27 |
*** rcernin has quit IRC | 08:27 | |
rpittau | Qianbiao: shouldn't the dependencies of ironic-api overwrite the precedent installation of eventlet ? | 08:28 |
Qianbiao | how to at someone like rpittau? | 08:29 |
mgoddard | dtantsur: looking at cli | 08:29 |
Qianbiao | If i install eventlet manually, then ironic-api will not install eventlet then | 08:29 |
*** uzumaki has quit IRC | 08:32 | |
*** derekh has joined #openstack-ironic | 08:33 | |
rpittau | Qianbiao: there are a couple of things that are not clear to me, you say you're installing stein, AFAIK by default bionic installs 10.1.1 which is Queens, if you install it using the pkg manager apt | 08:34 |
Qianbiao | i install repo munually. | 08:34 |
rpittau | Qianbiao: then, if you install eventlet using pip, apt won't care about that and install its dependencies | 08:34 |
Qianbiao | loading order maybe? | 08:35 |
rpittau | Qianbiao: if the location is the same, and it should be if you use the same python version, then the installation is overwritten | 08:36 |
openstackgerrit | Merged openstack/ironic master: Improve `redfish` set-boot-mode implementation https://review.opendev.org/710846 | 08:36 |
openstackgerrit | Merged openstack/ironic-lib master: Convert jobs to dib https://review.opendev.org/716998 | 08:36 |
Qianbiao | not the same i think. | 08:37 |
Qianbiao | apt will install to /usr/lib | 08:37 |
Qianbiao | pip will install to /usr/local/lib | 08:37 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Standalone CLI: fix a minor issue in the docs https://review.opendev.org/718344 | 08:38 |
dtantsur | mgoddard: thanks for review, this ^^ is a follow-up | 08:38 |
mgoddard | lightening fast | 08:38 |
rpittau | Qianbiao: the ubuntu distribution of Python will look at libs under /usr/lib/{python-ver} first by default | 08:42 |
Qianbiao | rpittau not sure, when i run `pip3 show eventlet`, it shows eventlet | 08:45 |
Qianbiao | Version: 0.18.4 | 08:45 |
*** ociuhandu has joined #openstack-ironic | 08:45 | |
Qianbiao | if i uninstall `pip3 uninstall eventlet`. then pip3 show eventlet shows: Version: 0.20.0 | 08:45 |
Qianbiao | so i think, i will load 0.18.4 by default | 08:45 |
openstackgerrit | Pierre Riteau proposed openstack/tenks stable/1.0: Pin virtualbmc to <2 when using Python 2 https://review.opendev.org/718341 | 08:47 |
*** yaawang has quit IRC | 08:48 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Set default boot option to netboot for jobs using cirros https://review.opendev.org/716613 | 08:49 |
iurygregory | dtantsur, sorry I didn't see that ironic-inspector-tempest-discovery used wholedisk =) fixed | 08:50 |
*** yaawang has joined #openstack-ironic | 08:50 | |
*** ociuhandu has quit IRC | 08:50 | |
*** tkajinam has quit IRC | 08:51 | |
*** ociuhandu has joined #openstack-ironic | 08:52 | |
dtantsur | thanks | 08:57 |
iurygregory | np o/ | 08:58 |
*** alexmcleod has joined #openstack-ironic | 09:03 | |
*** ociuhandu has quit IRC | 09:04 | |
*** ociuhandu has joined #openstack-ironic | 09:04 | |
openstackgerrit | Mahendra Kamble proposed x/sushy-oem-idrac master: Export System Configuration https://review.opendev.org/710327 | 09:05 |
*** ociuhandu has quit IRC | 09:09 | |
*** k_mouza has joined #openstack-ironic | 09:11 | |
*** yaawang has quit IRC | 09:14 | |
*** yaawang has joined #openstack-ironic | 09:16 | |
openstackgerrit | Pierre Riteau proposed openstack/tenks stable/1.0: Pin virtualbmc to <2 when using Python 2 https://review.opendev.org/718341 | 09:23 |
openstackgerrit | Pierre Riteau proposed openstack/tenks stable/1.0: Update .gitreview for stable/1.0 https://review.opendev.org/718220 | 09:23 |
openstackgerrit | Pierre Riteau proposed openstack/tenks stable/1.0: Update TOX/UPPER_CONSTRAINTS_FILE for stable/1.0 https://review.opendev.org/718221 | 09:23 |
openstackgerrit | Merged openstack/tenks master: Update master for stable/1.0 https://review.opendev.org/718222 | 09:25 |
*** ricolin_ has quit IRC | 09:29 | |
*** ricolin_ has joined #openstack-ironic | 09:30 | |
*** ricolin has quit IRC | 09:30 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Store an explicit flag while running an in-band step https://review.opendev.org/718358 | 09:30 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Support executing in-band deploy steps https://review.opendev.org/698768 | 09:32 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step https://review.opendev.org/698309 | 09:32 |
*** k_mouza has quit IRC | 09:32 | |
*** yaawang has quit IRC | 09:35 | |
*** k_mouza has joined #openstack-ironic | 09:35 | |
*** yaawang has joined #openstack-ironic | 09:36 | |
*** ociuhandu has joined #openstack-ironic | 09:40 | |
openstackgerrit | Andreas Jaeger proposed openstack/ironic-specs master: Cleanup py27 support https://review.opendev.org/718365 | 09:51 |
*** ricolin_ has quit IRC | 09:57 | |
*** ricolin_ has joined #openstack-ironic | 09:57 | |
*** ricolin has joined #openstack-ironic | 10:00 | |
openstackgerrit | Merged openstack/ironic master: DRAC: Fix a failure to create virtual disk bug https://review.opendev.org/710852 | 10:01 |
openstackgerrit | Merged openstack/python-ironicclient master: Add node lessee https://review.opendev.org/716011 | 10:01 |
openstackgerrit | Merged openstack/ironic master: Fix gunicorn name on Py3@CentOS7 in devstack https://review.opendev.org/718173 | 10:01 |
openstackgerrit | Merged openstack/python-ironicclient master: Standalone CLI: optional support for ironic-inspector https://review.opendev.org/718067 | 10:01 |
openstackgerrit | Merged openstack/python-ironicclient master: Standalone CLI: fix a minor issue in the docs https://review.opendev.org/718344 | 10:01 |
*** ricolin_ has quit IRC | 10:02 | |
openstackgerrit | Merged openstack/tenks stable/1.0: Pin virtualbmc to <2 when using Python 2 https://review.opendev.org/718341 | 10:04 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/718373 | 10:08 |
openstackgerrit | Merged openstack/tenks stable/1.0: Update .gitreview for stable/1.0 https://review.opendev.org/718220 | 10:08 |
openstackgerrit | Merged openstack/tenks stable/1.0: Update TOX/UPPER_CONSTRAINTS_FILE for stable/1.0 https://review.opendev.org/718221 | 10:08 |
dtantsur | mgoddard: I'm intending to update https://review.opendev.org/#/c/718250/ with the recently merged changes. any objections? | 10:21 |
patchbot | patch 718250 - releases - Release final python-ironicclient for ussuri - 1 patch set | 10:21 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Remove unused version parameter in version header function https://review.opendev.org/718378 | 10:23 |
mgoddard | dtantsur: no, go for it | 10:23 |
*** rpittau is now known as rpittau|bbl | 10:24 | |
*** ricolin has quit IRC | 10:39 | |
*** k_mouza has quit IRC | 10:47 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Support burning configdrive into boot ISO https://review.opendev.org/687250 | 10:48 |
*** k_mouza has joined #openstack-ironic | 10:55 | |
*** jtomasek has joined #openstack-ironic | 10:55 | |
openstackgerrit | Harald Jensås proposed openstack/ironic master: Fix AttributeError in check allowed port fields https://review.opendev.org/718127 | 11:00 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.opendev.org/718391 | 11:01 |
hjensas | good morning. | 11:07 |
iurygregory | morning hjensas | 11:09 |
*** mkrai has quit IRC | 11:30 | |
*** ociuhandu has quit IRC | 11:34 | |
*** ociuhandu has joined #openstack-ironic | 11:35 | |
*** ociuhandu has quit IRC | 11:39 | |
Qianbiao | rpittau check the log here: https://justpaste.it/39uhj | 11:40 |
Qianbiao | rpittau has been offline ? | 11:41 |
*** rh-jelabarre has joined #openstack-ironic | 11:45 | |
openstackgerrit | Mark Goddard proposed openstack/tenks master: Fix issue with VirtualBMC client path https://review.opendev.org/718404 | 11:46 |
*** ociuhandu has joined #openstack-ironic | 11:51 | |
iurygregory | Qianbiao, he probably went for lunch =) | 11:58 |
iurygregory | Qianbiao, his nick changed to rpittau|bbl | 11:59 |
*** Qianbiao has quit IRC | 12:05 | |
*** uzumaki has joined #openstack-ironic | 12:06 | |
*** ociuhandu has quit IRC | 12:10 | |
*** ociuhandu has joined #openstack-ironic | 12:11 | |
*** tkajinam has joined #openstack-ironic | 12:14 | |
*** ociuhandu has quit IRC | 12:16 | |
*** Goneri has joined #openstack-ironic | 12:16 | |
*** rpittau|bbl is now known as rpittau | 12:23 | |
*** rloo has joined #openstack-ironic | 12:40 | |
*** ricolin has joined #openstack-ironic | 12:57 | |
TheJulia | good morning all | 13:07 |
rpittau | good morning TheJulia :) | 13:08 |
TheJulia | So, what does everyone think of https://review.opendev.org/#/c/718436/ for cycle highlights | 13:08 |
patchbot | patch 718436 - releases - Cycle highlights for ironic - 1 patch set | 13:08 |
uzumaki | morning TheJulia o/ | 13:09 |
uzumaki | Looks cool! TheJulia | 13:11 |
* uzumaki wonders if his work can make it in the highlights (blushes) | 13:11 | |
*** cdearborn has joined #openstack-ironic | 13:13 | |
arne_wiebalck | TheJulia: how about "UEFI and device selection is now available for Software RAID deployments." ? | 13:17 |
*** uzumaki has quit IRC | 13:18 | |
arne_wiebalck | TheJulia: Also, will there be separate highlights for the inspector? I am thinking of the scoped introspection rules which should be very helpful for operators using the rules feature. | 13:19 |
arne_wiebalck | TheJulia: I'd also think the deploy step work should be added, no? | 13:21 |
openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.opendev.org/718391 | 13:21 |
* arne_wiebalck has maybe no idea what qualifies a feature to become a cycle highlight | 13:21 | |
rpittau | arne_wiebalck: https://docs.openstack.org/project-team-guide/release-management.html#cycle-highlights | 13:23 |
arne_wiebalck | rpittau: thanks! | 13:25 |
rpittau | np :) | 13:25 |
arne_wiebalck | After reading this, I think the features I raised qualify :) | 13:25 |
rpittau | TheJulia, dtantsur, and others! Considering that newton is gone since a while, should we increase the minimum ironic api version in ipa to 1.31 (latest ocata) ? | 13:28 |
iurygregory | rpittau, | 13:29 |
iurygregory | ++ | 13:29 |
*** jdandrea has joined #openstack-ironic | 13:29 | |
iurygregory | lol the ++ didn't went on the first | 13:29 |
*** ociuhandu has joined #openstack-ironic | 13:33 | |
TheJulia | arne_wiebalck: ohh I like | 13:34 |
TheJulia | arne_wiebalck: we've never submitted more than the one scope but I guess we could submit some for the inspector | 13:35 |
TheJulia | rpittau: propse the patch... quickly :) | 13:35 |
rpittau | incoming in 3.. 2.. 1.. | 13:36 |
rpittau | ok maybe more | 13:36 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Move minimum ironic version to latest ocata https://review.opendev.org/718450 | 13:36 |
rpittau | there it is! | 13:36 |
rpittau | maybe should've added a release note | 13:37 |
TheJulia | rpittau: I was just going to say reno | 13:37 |
rpittau | yeah.... | 13:37 |
arne_wiebalck | TheJulia: I guess we could also simply add it to the Ironic one, not sure anyone would complain ...? | 13:38 |
TheJulia | arne_wiebalck: they prefer only 3 entries per project, so I think if one is submitted for the ironic-inspector repo directly it would be better | 13:38 |
TheJulia | arne_wiebalck: go ahead and submit one :) | 13:38 |
arne_wiebalck | TheJulia: ah ... I suspected we cannot list *all* the work :) | 13:39 |
arne_wiebalck | TheJulia: will do | 13:39 |
*** tzumainn has joined #openstack-ironic | 13:39 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Move minimum ironic version to latest ocata https://review.opendev.org/718450 | 13:40 |
dtantsur | morning TheJulia | 13:48 |
* dtantsur -> short walk | 13:48 | |
*** dtantsur is now known as dtantsur|brb | 13:50 | |
arne_wiebalck | TheJulia: https://review.opendev.org/#/c/718456/ | 13:53 |
patchbot | patch 718456 - releases - Cycle highlights for ironic-inspector - 1 patch set | 13:53 |
TheJulia | Any last comments for https://review.opendev.org/#/c/718436/ | 13:55 |
patchbot | patch 718436 - releases - Cycle highlights for ironic - 1 patch set | 13:55 |
TheJulia | tzumainn: I guess explicit docs for multitenancy are still in the air? | 13:58 |
tzumainn | TheJulia, oh, I submitted this yesterday: https://review.opendev.org/#/c/718174/ | 13:59 |
patchbot | patch 718174 - ironic - Added node multitenancy doc - 2 patch sets | 13:59 |
TheJulia | lgtm, I'm going to submit the cycle highlights without the link and once it is up we should amend the release repo yaml file for the ussuri cycyle | 14:01 |
tzumainn | okay! | 14:01 |
*** belmoreira has quit IRC | 14:02 | |
TheJulia | mgoddard: dtantsur|brb I'm stepping away for a couple days. If someone could push up releases that would be <3 for most things except that we don't have any changes queued up for | 14:05 |
arne_wiebalck | TheJulia: there seems to be a formatting issue now, no? | 14:05 |
TheJulia | oO | 14:06 |
TheJulia | fixed | 14:06 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Add sushy-cli to client libraries release list https://review.opendev.org/718465 | 14:09 |
mgoddard | TheJulia: Sean pushed client release patches already. Are you suggesting we release the remaining projects too? | 14:09 |
arne_wiebalck | TheJulia: you want to add the doc link for the device selection as well? IIRC, dtantsur's patches included docs as well. | 14:09 |
TheJulia | mgoddard: anything that seems like it is not going to have more patches for the cycle seems reasonable to me | 14:09 |
TheJulia | arne_wiebalck: I didn't find those docs when I was looking, if you have the link handy please feel free to ammend the patch | 14:10 |
* arne_wiebalck has a look | 14:10 | |
mgoddard | TheJulia: ack | 14:10 |
* TheJulia goes and pours more coffee into her mug and grabs a book to go read | 14:11 | |
rpittau | TheJulia: enjoy :) | 14:12 |
mgoddard | +1 | 14:12 |
arne_wiebalck | tzumainn: reading through the multi-tenancy doc, it seems that I could have two different projects using the same flavor/resource class but still be able to control a) how many and b) which nodes are instantiated ... is that correct? | 14:25 |
tzumainn | arne_wiebalck, you'd still be able to control which projects own/lease which nodes, and allocations would only match the correct nodes with the correct projects | 14:28 |
tzumainn | let me know if I misunderstood the question! | 14:28 |
arne_wiebalck | tzumainn: from what I understand I would have more control not less | 14:29 |
arne_wiebalck | tzumainn: I have often machines of a certain type which are accessible by more than one project | 14:29 |
tzumainn | arne_wiebalck, as an administrator, yes - you'd be able to control which nodes could get allocated to which project | 14:30 |
tzumainn | but each project could still use the same flavor/resource classes | 14:30 |
arne_wiebalck | tzumainn: and this eliminates the quota problem where one project could eat into the nodes for another project | 14:30 |
tzumainn | arne_wiebalck, it should! | 14:31 |
arne_wiebalck | tzumainn: (a problem which I cannot solve with quotas as these are not per flavor/resource class) | 14:31 |
arne_wiebalck | tzumainn: at which stage would an allocation fail in such a case? | 14:31 |
arne_wiebalck | tzumainn: like, a project still has quota, but there is no more node it owns | 14:32 |
arne_wiebalck | tzumainn: placement is not aware | 14:32 |
arne_wiebalck | tzumainn: I guess | 14:32 |
tzumainn | arne_wiebalck, the allocation will say something like "ReservationFailed: Failed to reserve a node: Allocation f0dd4abe-5d4e-4535-9ebd-5f53ce4bc12b failed: Failed to process allocation f0dd4abe-5d4e-4535-9ebd-5f53ce4bc12b: no available nodes match the resource class baremetal." | 14:33 |
arne_wiebalck | tzumainn: so placement will provide nova with allocation candidates and nova will fail on the reservation? | 14:34 |
tzumainn | arne_wiebalck, hm, I haven't actually done this with nova yet - does placement use allocations? | 14:35 |
arne_wiebalck | tzumainn: I'm getting a bit out of my depth here concerning the details I have to admit ... | 14:37 |
arne_wiebalck | tzumainn: I was just wondering what the user sees and how quickly things would fail. | 14:38 |
arne_wiebalck | tzumainn: anyway, looks really interesting :) | 14:38 |
arne_wiebalck | tzumainn: I left one comment on the doc patch as well | 14:38 |
tzumainn | arne_wiebalck, thanks! the link is almost certainly incorrect :) | 14:40 |
*** uzumaki has joined #openstack-ironic | 14:42 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Added node multitenancy doc https://review.opendev.org/718174 | 14:43 |
*** igordc has joined #openstack-ironic | 14:45 | |
*** hoonetorg has quit IRC | 14:56 | |
openstackgerrit | Merged openstack/ironic master: Drop netaddr - use netutils.is_valid_ipv6() https://review.opendev.org/718113 | 14:58 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic master: [doc] Remove the device selection limitation for Software RAID https://review.opendev.org/718485 | 15:05 |
*** priteau has joined #openstack-ironic | 15:06 | |
*** gyee has joined #openstack-ironic | 15:13 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Move logic for raid start sector to raid_utils https://review.opendev.org/718489 | 15:24 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Move logic for raid start sector to raid_utils https://review.opendev.org/718489 | 15:32 |
openstackgerrit | Merged openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/718373 | 15:42 |
*** dtantsur|brb is now known as dtantsur | 15:48 | |
dtantsur | TheJulia: I'll help mgoddard with whatever is needed | 15:49 |
dtantsur | TheJulia: I've got a question about boot-from-ceph. Have you heard anything from cinder about it? | 15:49 |
iurygregory | wow | 15:49 |
dtantsur | arne_wiebalck: I highly doubt that what tzumainn is talking about involves placement at all | 15:50 |
dtantsur | it was a plan to have it as bare metal allocation backend, but nobody has got to it yet | 15:50 |
dtantsur | so bare metal allocations and placement allocations are currently unrelated | 15:51 |
* dtantsur hopes it clarifies anything | 15:51 | |
tzumainn | dtantsur, that actually clarifies it for me, I was unsure how placement and baremetal allocations were related | 15:51 |
tzumainn | and now I understand it is "not at all" :) | 15:51 |
dtantsur | at least not yet | 15:51 |
dtantsur | "no available nodes match resource class" should be taken quite literally | 15:52 |
dtantsur | it's a bit more verbose analogue of "no valid host found" from nova | 15:52 |
dtantsur | (or was it even a question? maybe I didn't read the scrollback carefully enough) | 15:54 |
*** mkrai has joined #openstack-ironic | 15:55 | |
arne_wiebalck | dtantsur: thanks! so a node allocated via nova/placement would fully ignore the owner and the deployment would happen, even if there are no more nodes owned by the corresponding project? | 15:57 |
dtantsur | arne_wiebalck: if nove is accessing ironic with a non-admin user, it won't see nodes that don't belong to it | 15:58 |
arne_wiebalck | dtantsur: it would be ok if it fails late, if it fails at all | 15:58 |
dtantsur | * nova | 15:58 |
dtantsur | if nova is using an admin user (like it's the case now), it will see all nodes and use any of them | 15:58 |
arne_wiebalck | hm, ok | 15:58 |
arne_wiebalck | thanks! | 15:59 |
*** hoonetorg has joined #openstack-ironic | 16:02 | |
*** lucasagomes has quit IRC | 16:03 | |
*** jcoufal has joined #openstack-ironic | 16:04 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Added node multitenancy doc https://review.opendev.org/718174 | 16:12 |
*** ociuhandu has quit IRC | 16:16 | |
*** ociuhandu has joined #openstack-ironic | 16:17 | |
*** ociuhandu has quit IRC | 16:22 | |
rpittau | bye, good night! o/ | 16:23 |
*** rpittau is now known as rpittau|afk | 16:23 | |
*** mkrai has quit IRC | 16:40 | |
jlvillal | Good morning Ironic :) | 16:43 |
dtantsur | oh wow, morning jlvillal | 16:48 |
dtantsur | tell me either or both of you didn't end up stuck in Russian :) | 16:49 |
jlvillal | Hey dtantsur :) | 16:49 |
jlvillal | Hehe. No we left on 18-March. | 16:49 |
dtantsur | good timing | 16:49 |
jlvillal | We are back in Oregon now. | 16:49 |
jlvillal | Thanks. I assume you are somewhere in Germany? | 16:50 |
dtantsur | Düsseldorf, yes | 16:50 |
jlvillal | Seems like Germany is one of the best situations at the moment. If the stats are to be believed. | 16:50 |
dtantsur | It doesn't look too bad indeed, at least for now | 16:50 |
jlvillal | Strange situation. Spending a LOT of time at home. Basically only leave to go shopping for food. | 16:51 |
dtantsur | Same. Well, we do venture outside of short walks or jogging. Otherwise even such introverts as we will go insane. | 16:51 |
jlvillal | Yeah. Well hopefully things get back to normal in a few months! | 16:52 |
dtantsur | ++ | 16:52 |
* etingof experiences tick attack | 16:55 | |
dtantsur | again? Oo | 16:55 |
etingof | and found a baby-bat in the woods, what a scary creature it is | 16:56 |
dtantsur | make sure at least that doesn't bite you. may not end up well. | 16:56 |
etingof | no-no, ticks it my thing | 16:56 |
dtantsur | a bit better. but just a bit :) | 16:57 |
etingof | that baby bat has quite impressive teeth | 16:57 |
dtantsur | did you make a picture? | 16:57 |
etingof | my daughter did | 16:57 |
* dtantsur is interested | 16:58 | |
etingof | I held the bat | 16:58 |
dtantsur | you're adventurous for sure :) I'd not risk it. | 16:58 |
etingof | like this https://andreas-rose.org/wp-content/uploads/2019/12/Tonatia_saurophila.jpg | 16:59 |
*** derekh has quit IRC | 16:59 | |
dtantsur | a serious thing :) | 16:59 |
etingof | yeah, brr | 16:59 |
*** yedongcan has left #openstack-ironic | 17:03 | |
*** dtantsur is now known as dtantsur|afk | 17:07 | |
dtantsur|afk | g'night! | 17:07 |
*** ociuhandu has joined #openstack-ironic | 17:10 | |
*** priteau has quit IRC | 17:15 | |
*** ociuhandu has quit IRC | 17:16 | |
arne_wiebalck | bye everyone o/ | 17:19 |
*** jcoufal has quit IRC | 17:34 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Added node multitenancy doc https://review.opendev.org/718174 | 17:45 |
openstackgerrit | Harald Jensås proposed openstack/ironic stable/train: Additional IP addresses to IPv6 stateful ports https://review.opendev.org/717205 | 17:48 |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 17:49 |
openstackgerrit | Verification of a change to openstack/ironic failed: Allow INSPECTWAIT state for lookup https://review.opendev.org/718017 | 17:56 |
*** ociuhandu has joined #openstack-ironic | 18:26 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Support burning configdrive into boot ISO https://review.opendev.org/687250 | 18:36 |
*** k_mouza has quit IRC | 18:46 | |
*** k_mouza has joined #openstack-ironic | 18:47 | |
*** k_mouza has quit IRC | 18:59 | |
*** dsneddon has joined #openstack-ironic | 19:09 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Fix AttributeError in check allowed port fields https://review.opendev.org/718127 | 19:13 |
*** k_mouza has joined #openstack-ironic | 19:27 | |
*** alexmcleod has quit IRC | 19:34 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `network_data` field to ironic node object https://review.opendev.org/687910 | 19:36 |
*** k_mouza has quit IRC | 19:36 | |
*** k_mouza has joined #openstack-ironic | 19:37 | |
*** k_mouza has quit IRC | 19:41 | |
*** Lucas_Gray has joined #openstack-ironic | 19:50 | |
*** igordc has quit IRC | 20:17 | |
*** Lucas_Gray has quit IRC | 20:18 | |
*** igordc has joined #openstack-ironic | 20:18 | |
*** Lucas_Gray has joined #openstack-ironic | 20:19 | |
*** uzumaki has quit IRC | 20:32 | |
*** Lucas_Gray has quit IRC | 20:33 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Added node multitenancy doc https://review.opendev.org/718174 | 20:47 |
*** Lucas_Gray has joined #openstack-ironic | 21:01 | |
*** k_mouza has joined #openstack-ironic | 21:05 | |
*** k_mouza has quit IRC | 21:06 | |
*** k_mouza has joined #openstack-ironic | 21:06 | |
*** k_mouza has quit IRC | 21:19 | |
*** jtomasek has quit IRC | 21:25 | |
*** jdandrea has quit IRC | 21:46 | |
*** jdandrea has joined #openstack-ironic | 21:47 | |
*** tzumainn has quit IRC | 21:53 | |
*** Lucas_Gray has quit IRC | 22:01 | |
*** iurygregory has quit IRC | 22:09 | |
*** k_mouza has joined #openstack-ironic | 22:30 | |
*** k_mouza has quit IRC | 22:31 | |
*** rcernin has joined #openstack-ironic | 22:32 | |
*** gyee has quit IRC | 22:49 | |
*** threestrands has joined #openstack-ironic | 23:10 | |
openstackgerrit | Merged openstack/ironic master: Allow INSPECTWAIT state for lookup https://review.opendev.org/718017 | 23:41 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!