| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Provide a path to set explicit ipxe bootloaders https://review.opendev.org/728315 | 00:51 |
|---|---|---|
| TheJulia | dtantsur|afk: could you take a look at ^. I'm afraid I did something horribly stupid and... yeah. I also don't like the explicit interface value check. The task is there so I think the capability values are too, so that seems easy. Anyway, take a look, the commit message should explain it all | 00:52 |
| *** tzumainn has quit IRC | 00:59 | |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/train: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/727906 | 01:14 |
| *** Qianbiao has joined #openstack-ironic | 01:53 | |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/train: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/727906 | 01:54 |
| *** gyee has quit IRC | 02:18 | |
| *** michchap has joined #openstack-ironic | 02:41 | |
| *** ociuhandu has joined #openstack-ironic | 03:05 | |
| *** ociuhandu has quit IRC | 03:09 | |
| *** xiaolin has quit IRC | 03:11 | |
| *** stevebaker has quit IRC | 03:12 | |
| *** xiaolin has joined #openstack-ironic | 03:15 | |
| *** stevebaker has joined #openstack-ironic | 03:34 | |
| *** xiaolin has quit IRC | 03:51 | |
| *** xiaolin has joined #openstack-ironic | 03:51 | |
| *** stevebaker has quit IRC | 04:00 | |
| *** xiaolin has quit IRC | 04:01 | |
| *** Qianbiao is now known as qianbiao|afk | 04:16 | |
| *** zzzeek has quit IRC | 04:23 | |
| *** zzzeek has joined #openstack-ironic | 04:24 | |
| *** ociuhandu has joined #openstack-ironic | 04:33 | |
| *** jdandrea has quit IRC | 04:38 | |
| *** ociuhandu has quit IRC | 04:43 | |
| *** ociuhandu has joined #openstack-ironic | 04:44 | |
| *** qianbiao|afk has quit IRC | 04:47 | |
| *** ociuhandu has quit IRC | 04:50 | |
| *** uzumaki has quit IRC | 05:38 | |
| *** michchap has quit IRC | 05:45 | |
| *** ociuhandu has joined #openstack-ironic | 05:52 | |
| *** ociuhandu has quit IRC | 05:59 | |
| *** xiaolin has joined #openstack-ironic | 06:00 | |
| arne_wiebalck | Good morning, ironic! | 06:02 |
| *** qianbiao|afk has joined #openstack-ironic | 06:15 | |
| *** uzumaki has joined #openstack-ironic | 06:19 | |
| iurygregory | good morning arne_wiebalck and Ironic o/ | 06:25 |
| *** xiaolin has quit IRC | 06:26 | |
| uzumaki | morning iurygregory arne_wiebalck o/ | 06:26 |
| iurygregory | ironic cores please review https://review.opendev.org/#/c/728150/ so we can unblock the CI for ironic =) | 06:26 |
| patchbot | patch 728150 - ironic - Pin the python-ibmcclient upper bounds due to six ... - 1 patch set | 06:26 |
| iurygregory | hey uzumaki o/ | 06:27 |
| uzumaki | how's it going iurygregory ? | 06:27 |
| iurygregory | doing good =) | 06:28 |
| iurygregory | time for breakfast | 06:28 |
| uzumaki | great! | 06:34 |
| qianbiao|afk | hi iurygregory | 06:39 |
| *** qianbiao|afk is now known as qianbiao | 06:39 | |
| qianbiao | I find that patch limit the python-ibmcclient version to 0.1.0 due to six lib dependency. | 06:40 |
| qianbiao | Does it means six is not allowed in latest ironic version? | 06:41 |
| qianbiao | even used in OEM drived? | 06:41 |
| qianbiao | even used in OEM driver? | 06:41 |
| *** ociuhandu has joined #openstack-ironic | 06:45 | |
| arne_wiebalck | hey iurygregory uzumaki qianbiao o/ | 06:58 |
| qianbiao | o/ morning | 06:58 |
| uzumaki | hey arne_wiebalck o/ how's it going | 06:59 |
| arne_wiebalck | all good over here, thanks! | 06:59 |
| arne_wiebalck | and you? | 07:00 |
| uzumaki | i'm great | 07:00 |
| *** rpittau|afk is now known as rpittau | 07:12 | |
| rpittau | good morning ironic! o/ | 07:12 |
| rpittau | qianbiao: the six library was removed some time ago from ironic | 07:13 |
| rpittau | I do find weird that it's not installed as dependency from python-ibmcclient | 07:15 |
| qianbiao | <rpittau> i do not understand. why "ironic remove it" would effect oem driver. | 07:15 |
| qianbiao | only when user use oem driver, they will install client-lib. | 07:16 |
| qianbiao | it should not effect ironic at all | 07:16 |
| rpittau | qianbiao: it doesn't affect ironic, it's just a ci job that is impacted | 07:16 |
| qianbiao | oh, i see. | 07:16 |
| qianbiao | basiclly, our client-lib must support python27 even python27 is not support for now. | 07:17 |
| qianbiao | Our customer has all kind of env.. | 07:17 |
| rpittau | qianbiao: yeah, I understand that, I'm trying to understand why it complains because I see six begin installed | 07:18 |
| *** yolanda has joined #openstack-ironic | 07:32 | |
| rpittau | qianbiao: the extension of it is way bigger, if you try to install python-ibmcclient in a virtualenv it won't work | 07:38 |
| rpittau | qianbiao: I think the dependencies of the package are a bit messed up | 07:39 |
| qianbiao | I am using venv local | 07:39 |
| rpittau | me too :) | 07:40 |
| qianbiao | it does not have any problem | 07:40 |
| qianbiao | ah, ok. i will recheck. | 07:40 |
| rpittau | qianbiao: my suggestion is to build a wheel distribution and publish that, that should include the dependencies correctly | 07:45 |
| qianbiao | ok | 07:45 |
| *** ociuhandu has quit IRC | 07:51 | |
| iurygregory | hey qianbiao o/ | 07:56 |
| iurygregory | hey rpittau o/ | 07:56 |
| rpittau | hey iurygregory :) | 07:56 |
| qianbiao | hey iurygregory o/ | 08:05 |
| *** qianbiao has quit IRC | 08:13 | |
| *** tkajinam has quit IRC | 08:14 | |
| *** dtantsur|afk is now known as dtantsur | 08:17 | |
| dtantsur | morning ironic | 08:17 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Follow up to update functest-requirements https://review.opendev.org/728128 | 08:19 |
| *** k_mouza has joined #openstack-ironic | 08:25 | |
| iurygregory | morning dtantsur | 08:27 |
| *** Qianbiao has joined #openstack-ironic | 08:34 | |
| Qianbiao | rpittau can not upload wheel format. because it does not support markdown format long description | 08:35 |
| Qianbiao | Checking dist/python_ibmcclient-0.2.0-py2.py3-none-any.whl: FAILED | 08:35 |
| Qianbiao | `long_description` has syntax errors in markup and would not be rendered on PyPI. | 08:35 |
| Qianbiao | line 11: Error: Unexpected indentation. | 08:35 |
| Qianbiao | warning: `long_description_content_type` missing. defaulting to `text/x-rst`. | 08:35 |
| Qianbiao | Checking dist/python-ibmcclient-0.2.0.tar.gz: PASSED | 08:35 |
| Qianbiao | I will upload a version without six for now. | 08:35 |
| Qianbiao | I am sorry that python-ibmcclient 0.2.0 blocks CI ;（ | 08:36 |
| iurygregory | Qianbiao, we have a fix for now so it's fine =) | 08:36 |
| rpittau | Qianbiao: no worries, we just capped it for now | 08:36 |
| iurygregory | take your time and try to figure out a way that will work for you =) | 08:37 |
| rpittau | Qianbiao: I would also doublecheck your setup config, it looks like it's not installing dependencies when you run setup.py in a virtualenv and so it doesn't build, it complains because six is missing | 08:37 |
| Qianbiao | thanks ；) | 08:37 |
| Qianbiao | rpittau thanks, code repo is here: https://github.com/IamFive/python-ibmcclient | 08:38 |
| rpittau | Qianbiao: yeah, I cloned that to test it and the setup.py didn't work :) | 08:38 |
| Qianbiao | python setup.py install not work? | 08:39 |
| rpittau | Qianbiao: exactly | 08:39 |
| Qianbiao | I can install it on ubuntu 18.04 | 08:39 |
| Qianbiao | @ rpittau http://paste.openstack.org/show/793643/ | 08:40 |
| rpittau | Qianbiao: http://paste.openstack.org/show/793644/ | 08:41 |
| Qianbiao | hmm... that is really wierd... | 08:41 |
| rpittau | Qianbiao: you're not using a virtualenv there | 08:41 |
| Qianbiao | i see | 08:41 |
| rpittau | Qianbiao: you're usingthe system packages | 08:41 |
| Qianbiao | <rpittau> you are right, ican reproduce.. | 08:45 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Convert jobs to dib https://review.opendev.org/718627 | 08:47 |
| rpittau | Qianbiao: take your time, it's ok on our side :) | 08:47 |
| *** Qianbiao has quit IRC | 08:48 | |
| *** Qianbiao has joined #openstack-ironic | 08:49 | |
| *** ociuhandu has joined #openstack-ironic | 08:49 | |
| openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Convert job to dib https://review.opendev.org/718698 | 08:52 |
| openstackgerrit | Merged openstack/ironic master: Pin the python-ibmcclient upper bounds due to six usage https://review.opendev.org/728150 | 08:54 |
| iurygregory | ouch ironic-staging-drivers don't have the pdf-docs env XD | 08:56 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/06905e5a8f02469c88d64ba283241a20/log/sphinx-build-pdf.log | 08:56 |
| * iurygregory working on it | 08:56 | |
| dtantsur | do we even need it there? | 08:57 |
| dtantsur | given that nobody has complained so far.. | 08:57 |
| iurygregory | dtantsur, the CI did complain .-. | 08:58 |
| dtantsur | lol | 08:58 |
| iurygregory | check the link I sent ^ | 08:58 |
| dtantsur | ah, we have a job but no tox env for it? | 08:58 |
| dtantsur | sweeet | 08:58 |
| iurygregory | dtantsur, yep | 08:58 |
| dtantsur | okay, whatever makes us green again | 08:58 |
| iurygregory | well seems we can try to avoid building the pdfs | 08:59 |
| rpittau | iurygregory: weird, can't reprodce it locally | 08:59 |
| dtantsur | the ibmc upper bound patch, should we backport it? or are the branches fine? | 08:59 |
| iurygregory | https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L438 | 08:59 |
| rpittau | dtantsur: I was going to backport it to be sure | 08:59 |
| dtantsur | rpittau: train seems broken too: https://zuul.opendev.org/t/openstack/build/4c536cd1fb8842239632c9e79858d355 | 09:00 |
| dtantsur | so yes, please do | 09:00 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/ussuri: Pin the python-ibmcclient upper bounds due to six usage https://review.opendev.org/728388 | 09:00 |
| rpittau | ofc train is in conflict, a moment | 09:01 |
| iurygregory | rpittau, well we don't have the env in tox.ini so CI complains, locally if you run tox -e docs it works fine | 09:02 |
| iurygregory | but on CI it will also run pdf-docs | 09:02 |
| rpittau | iurygregory: yep, I just realized :) | 09:03 |
| iurygregory | seems we can also avoid to build the pdfshttps://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L438 | 09:03 |
| iurygregory | but the job is inside a template we use .-. | 09:04 |
| iurygregory | so I will go ahead and add the requirements so we can build pdf | 09:04 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/train: Pin the python-ibmcclient upper bounds due to six usage https://review.opendev.org/728389 | 09:05 |
| rpittau | pdfs are good, let's build them all | 09:05 |
| iurygregory | build all the pdfs! | 09:06 |
| openstackgerrit | Merged openstack/sushy-tools master: Fix pdf build https://review.opendev.org/728048 | 09:06 |
| rpittau | dtantsur: backports are there up to train | 09:06 |
| *** dougsz has joined #openstack-ironic | 09:08 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native ubuntu ovmf package https://review.opendev.org/716889 | 09:13 |
| openstackgerrit | wangwei1 proposed openstack/ironic-prometheus-exporter master: Add coverage configuration for tox https://review.opendev.org/728391 | 09:14 |
| openstackgerrit | Merged openstack/sushy master: Fix pdf build https://review.opendev.org/728039 | 09:15 |
| openstackgerrit | wangwei1 proposed openstack/ironic-prometheus-exporter master: Add coverage configuration for tox https://review.opendev.org/728392 | 09:16 |
| dtantsur | thanks rpittau | 09:23 |
| Qianbiao | dtantsur rpittau You may like to know that i have upload python-ibmcclient 0.2.1 which fixed the six module does not imported issue. | 09:24 |
| rpittau | Qianbiao: great :) | 09:24 |
| rpittau | Qianbiao: don't be mad if I'm going to test it first :D | 09:24 |
| dtantsur | Qianbiao: train seems broken for another reason: https://zuul.opendev.org/t/openstack/build/4c536cd1fb8842239632c9e79858d355 | 09:24 |
| Qianbiao | sure :) | 09:25 |
| Qianbiao | <dtantsur> checking | 09:25 |
| dtantsur | typing is not a thing in python 2 | 09:26 |
| dtantsur | we may be fine with upper capping ibmcclient on stable branches though | 09:26 |
| rpittau | dtantsur: the cap should work | 09:26 |
| Qianbiao | <dtantsur> yes, you can keep a upper capping on old version. | 09:26 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Proposal for a new release model https://review.opendev.org/725547 | 09:27 |
| rpittau | ok setup works, let's see pip | 09:27 |
| rpittau | looks fine, last test | 09:27 |
| *** uzumaki has quit IRC | 09:28 | |
| *** Qianbiao has quit IRC | 09:29 | |
| *** Qianbiao has joined #openstack-ironic | 09:30 | |
| Qianbiao | <dtantsur> that is wierd. how unittest passes. | 09:34 |
| dtantsur | Qianbiao: this is a job with driver libraries installed | 09:35 |
| dtantsur | (running on python 2) | 09:35 |
| Qianbiao | i know. i mean python-ibmcclient has python2 unittests. | 09:35 |
| Qianbiao | It all passes.. if an import not exists, it should not pass | 09:35 |
| iurygregory | dtantsur, where is the magic irc channel where it would show the ironic-staging-drivers patches? | 09:36 |
| Qianbiao | But you are right, typing not exists in py27 | 09:37 |
| dtantsur | iurygregory: none probably? | 09:37 |
| dtantsur | Qianbiao: this is weird indeed | 09:38 |
| iurygregory | now magic makes more sense \o/ | 09:38 |
| iurygregory | and also alternative realities | 09:38 |
| Qianbiao | lol | 09:39 |
| dtantsur | Qianbiao: unrelated, but this is redundant: https://github.com/IamFive/python-ibmcclient/blob/master/tox.ini#L6-L11 | 09:39 |
| iurygregory | Qianbiao, normally people in this channel is a bit crazy XD | 09:40 |
| Qianbiao | But very nice . iurygregory | 09:40 |
| iurygregory | yeah \o/ | 09:40 |
| dtantsur | Qianbiao: okay, I see the problem, I think | 09:41 |
| dtantsur | something in your test-requirements pulls in "typing" (a separate backport of the python 3 module to python 2) | 09:41 |
| dtantsur | so your tests pass happily, but when ibmcclient is pulled by ironic, test-requirements are not installed | 09:42 |
| dtantsur | Qianbiao: a solution: add "typing" to your requirements.txt conditional on python 2 | 09:42 |
| Qianbiao | i see. | 09:42 |
| Qianbiao | you mean the mock lib | 09:42 |
| dtantsur | Qianbiao: something like https://github.com/IamFive/python-ibmcclient/pull/1 (test it before merging) | 09:45 |
| Qianbiao | <dtantsur> thanks :) | 09:46 |
| iurygregory | dtantsur, \o/ ironic-inspector-non-standalone is working on the ironic-inspector-managed job \o/ https://review.opendev.org/#/c/726121/ | 09:49 |
| patchbot | patch 726121 - ironic-inspector - Merge jobs - 2 patch sets | 09:49 |
| iurygregory | at least is what it seems =) | 09:49 |
| *** Qianbiao has quit IRC | 09:56 | |
| dtantsur | heh | 09:58 |
| dtantsur | nice! | 09:58 |
| iurygregory | we will have to probably adjust some timeouts I think | 09:59 |
| * iurygregory is thinking in try more rechecks to see how it goes... | 09:59 | |
| rpittau | sorry, had a meeting, but yeah I guess we need to wait for another release :) | 10:08 |
| iurygregory | dtantsur, I'm testing removing the openstackdocstheme from the requirements and from the conf.py | 10:08 |
| rpittau | alright time to go hunting, ttyl | 10:09 |
| *** rpittau is now known as rpittau|bbl | 10:09 | |
| iurygregory | done | 10:13 |
| *** Qianbiao has joined #openstack-ironic | 10:14 | |
| gudrutis2 | Hi folks. Why we dont allow python3 install on Centos7? It is available in default yum repos. Just asking if there is any good reason. | 10:16 |
| dtantsur | gudrutis2: IIRC it's not complete python 3, e.g. it misses many libraries that we need | 10:16 |
| dtantsur | there is also this aspect that nobody volunteered to do the work (it's not trivial, believe me) | 10:17 |
| iurygregory | correct ^ | 10:17 |
| * dtantsur exercising, brb | 10:18 | |
| openstackgerrit | wangwei1 proposed openstack/ironic-prometheus-exporter master: Add coverage configuration for tox https://review.opendev.org/728392 | 10:19 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 10:24 |
| openstackgerrit | Merged openstack/sushy master: Remove translation sections from setup.cfg https://review.opendev.org/728030 | 10:26 |
| *** derekh has joined #openstack-ironic | 10:43 | |
| *** Qianbiao has quit IRC | 11:00 | |
| *** ociuhandu has quit IRC | 11:01 | |
| *** Qianbiao has joined #openstack-ironic | 11:01 | |
| *** ociuhandu has joined #openstack-ironic | 11:02 | |
| *** ociuhandu has quit IRC | 11:02 | |
| *** ociuhandu has joined #openstack-ironic | 11:03 | |
| * iurygregory is wondering why docs and unit-with-driver-libs are still falling (the fix is merged O.o) | 11:04 | |
| Qianbiao | iurygregor is it still python-ibmcclient blocks CI? | 11:06 |
| Qianbiao | any review link | 11:06 |
| iurygregory | Qianbiao, we merged the fix, I think probably the opendev mirrors aren't getting the last commit sometimes =) | 11:08 |
| iurygregory | I did recheck in 4 patches, 2 worked 2 didn't | 11:08 |
| Qianbiao | ok | 11:09 |
| *** ociuhandu has quit IRC | 11:13 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/train: Make POST /v1/introspection synchronous when manage_boot==False https://review.opendev.org/728051 | 11:13 |
| openstackgerrit | Merged openstack/ironic-inspector master: extra_hardware: stop storing raw data in swift https://review.opendev.org/728096 | 11:13 |
| openstackgerrit | Merged openstack/ironic-inspector master: Remove translation sections from setup.cfg https://review.opendev.org/727961 | 11:13 |
| *** ociuhandu has joined #openstack-ironic | 11:14 | |
| *** ociuhandu has quit IRC | 11:14 | |
| *** ociuhandu has joined #openstack-ironic | 11:15 | |
| *** ociuhandu has quit IRC | 11:25 | |
| *** dougsz has quit IRC | 11:30 | |
| openstackgerrit | Merged openstack/sushy master: Fix OEM extension loading for different servers https://review.opendev.org/727738 | 11:35 |
| *** Qianbiao has quit IRC | 11:45 | |
| *** threestrands has quit IRC | 11:46 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 11:49 |
| openstackgerrit | Merged openstack/ironic stable/train: Pin the python-ibmcclient upper bounds due to six usage https://review.opendev.org/728389 | 11:50 |
| dtantsur | we cannot merge https://review.opendev.org/#/c/728388/ because of pep8 problem and the pep8 fix because of it :( ideas? | 11:53 |
| patchbot | patch 728388 - ironic (stable/ussuri) - Pin the python-ibmcclient upper bounds due to six ... - 1 patch set | 11:53 |
| dtantsur | although.. maybe with the new ibmcclient we don't need it? | 11:54 |
| iurygregory | dtantsur, squash ? | 11:54 |
| dtantsur | let's try rechecking the pep8 fix, maybe it'll pass with ibmcclient 0.2.2 | 11:55 |
| iurygregory | cool | 11:55 |
| iurygregory | soudns good to me =) | 11:55 |
| iurygregory | sounds* | 11:57 |
| * iurygregory is starting to type things wrong because he is hungry =( | 11:57 | |
| dtantsur | same here | 11:59 |
| dtantsur | folks, could I get some reviews on https://review.opendev.org/#/c/725770/ and https://review.opendev.org/#/c/727154/ please? | 12:03 |
| patchbot | patch 725770 - ironic-inspector - Stop running a periodic task for the noop PXE filter - 2 patch sets | 12:03 |
| patchbot | patch 727154 - ironic-inspector - Stop creating a green thread in the introspection ... - 2 patch sets | 12:03 |
| dtantsur | mgoddard: at some point next week we'll probably need to release stable/train with accumulated changes (at least ironic and inspector) | 12:05 |
| *** ociuhandu has joined #openstack-ironic | 12:05 | |
| dtantsur | oh, and could I also get reviews on https://review.opendev.org/#/c/727169/ or IPA-builder will break us | 12:06 |
| patchbot | patch 727169 - ironic - CI: raise DIB jobs memory to 3 GiB - 1 patch set | 12:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/ussuri: CI: raise DIB jobs memory to 3 GiB https://review.opendev.org/728426 | 12:06 |
| *** rpittau|bbl is now known as rpittau | 12:06 | |
| *** sshnaidm|afk is now known as sshnaidm|off | 12:07 | |
| mgoddard | dtantsur: sure | 12:07 |
| dtantsur | thanks! I'm mentioning it now because I'll be on PTO next week. | 12:09 |
| *** Qianbiao has joined #openstack-ironic | 12:11 | |
| *** ociuhandu has quit IRC | 12:13 | |
| rpittau | the ibmcclient 0.2.2 has still issues with unit tests | 12:13 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 12:16 |
| dtantsur | rpittau: what now? | 12:16 |
| dtantsur | iurygregory, rpittau, maybe update the ibmcclient capping patch with a cap for pycodestyle? | 12:18 |
| dtantsur | so that we can move on with ussuri? | 12:19 |
| rpittau | dtantsur: sounds good, I'm running the unit tests again, maybe it's a quick fix | 12:19 |
| dtantsur | ++ | 12:19 |
| rpittau | 2 weeks of CI crazyness, this was wild | 12:20 |
| openstackgerrit | Merged openstack/ironic-inspector master: No need to import print function anymore https://review.opendev.org/721576 | 12:20 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: add test-requirements to irrelevant-files https://review.opendev.org/728428 | 12:22 |
| dtantsur | rpittau: s/was/is/ :) | 12:22 |
| rpittau | heh yeah | 12:22 |
| *** rloo has joined #openstack-ironic | 12:23 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/ussuri: Pin the python-ibmcclient upper bounds due to six usage https://review.opendev.org/728388 | 12:27 |
| rpittau | dtantsur ^ | 12:27 |
| iurygregory | dtantsur, I'm ok with capping pycodestyle =) | 12:27 |
| rpittau | I added the pycodestyle pin | 12:27 |
| rpittau | s/pin/cap | 12:27 |
| iurygregory | rpittau, ++ | 12:27 |
| rpittau | iurygregory: sorry, quick fix, then we can merge your fix and uncap pycodestyle, then hopefully we can fix ibmcclient and uncap that too :D | 12:28 |
| * iurygregory goes for lunch ( 1h10min delay on the delivery -.-') | 12:28 | |
| rpittau | iurygregory: bon app! | 12:28 |
| iurygregory | rpittau, no worries =) tks! | 12:28 |
| Qianbiao | rpittau any link to the problem log? | 12:39 |
| *** ociuhandu has joined #openstack-ironic | 12:39 | |
| rpittau | Qianbiao: I have a patch already, no worries, small change | 12:40 |
| Qianbiao | ok thanks. | 12:40 |
| rpittau | just finishing the tests | 12:40 |
| rpittau | if we want to use a recent release we probably have to increase the minimum working version | 12:43 |
| dtantsur | not necessary, we can add excludes for 0.2.0 and 0.2.1 | 12:50 |
| Qianbiao | dtantsur rpittau are u talking about ibmcclient? | 12:52 |
| dtantsur | I am, hopefully rpittau as well :) | 12:52 |
| rpittau | dtantsur, Qianbiao, we actally need to start from 0.2.2 | 12:52 |
| Qianbiao | it's ok to just increase version to 0.2.2 | 12:52 |
| rpittau | patch incoming | 12:52 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Change ibmcclient compatible version https://review.opendev.org/728436 | 12:52 |
| dtantsur | we can, but we could also do !=0.2.0,!=0.2.1 | 12:52 |
| rpittau | dtantsur: I don't think we can, there's a breaking change in 0.2.2 in the esceptions | 12:53 |
| rpittau | exceptions | 12:53 |
| dtantsur | ouch | 12:53 |
| dtantsur | okay, then we cannot (and we have to cap on stable branches) | 12:53 |
| rpittau | yeah | 12:53 |
| dtantsur | rpittau: do we want to cap to <0.3.0 to avoid the same problem in the future? | 12:53 |
| dtantsur | or is it too hardcore? | 12:53 |
| rpittau | Qianbiao: can you please that patch and see if the changes makes sense from your side ? | 12:53 |
| dtantsur | Qianbiao: same question ^^ | 12:54 |
| Qianbiao | sure .. | 12:54 |
| Qianbiao | looking | 12:54 |
| rpittau | dtantsur: to be honest with the drivers I would just use a specific version, like ==0.2.2, maybe too much ? | 12:55 |
| dtantsur | rpittau: a bit too much, would prevent getting bugfix releases | 12:55 |
| dtantsur | <0.3.0 sounds acceptable. I think dracclient does it. | 12:55 |
| rpittau | mmm right | 12:55 |
| rpittau | ok | 12:55 |
| * dtantsur hopes Qianbiao won't land breaking changes in 0.2.* | 12:55 | |
| Qianbiao | dtantsur I think wont :) | 12:56 |
| rpittau | Qianbiao: please test the new releases with ironic before releasing :) | 12:56 |
| Qianbiao | almost all feature has done. | 12:56 |
| Qianbiao | rpittau sure. my bad. | 12:57 |
| rpittau | Qianbiao: thanks and no worries! :) | 12:57 |
| dtantsur | don't worry, everyone here has broken something in their life :) | 12:57 |
| * dtantsur has broken CI many times during his 6 years here | 12:58 | |
| rpittau | oh dont get me start with that :D | 12:58 |
| dtantsur | haha, I want to get you started ;) | 12:58 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Change ibmcclient compatible version https://review.opendev.org/728436 | 12:58 |
| dtantsur | we can wait for Friday evening though | 12:58 |
| rpittau | oh man, I did so many bad things | 12:58 |
| Qianbiao | :) I should have test again ironic. Just i am using window, it's not easy to run a test locally, need to do it in remote VM. | 12:59 |
| rpittau | Qianbiao: I think you can use a container or a vm, I do the same even if I don't run windows | 12:59 |
| rpittau | locally I mean | 12:59 |
| dtantsur | Qianbiao: you can do something like $ ./.tox/py3/bin/pip install git+https://github.com/IamFive/python-ibmcclient and re-run unit tests | 13:00 |
| * iurygregory is back | 13:00 | |
| dtantsur | (assuming you have a py3 environment ready) | 13:00 |
| dtantsur | wb iurygregory, how's food? | 13:00 |
| iurygregory | dtantsur, the fried rice was good =) | 13:00 |
| Qianbiao | <rpittau> will try. | 13:00 |
| iurygregory | now I just want chocolate XD | 13:00 |
| * TheJulia wipes sleep from eyes | 13:01 | |
| TheJulia | good morning | 13:01 |
| dtantsur | morning TheJulia, happy Friday | 13:01 |
| rpittau | Qianbiao: thanks for the review :) | 13:01 |
| iurygregory | good morning TheJulia | 13:01 |
| rpittau | good morning TheJulia :) | 13:01 |
| dtantsur | we're still fixing all the CIz, you can go back to sleep | 13:01 |
| TheJulia | dtantsur: happy?!? should I hide? | 13:01 |
| dtantsur | s/hide/nap/ | 13:01 |
| * TheJulia sighs | 13:01 | |
| iurygregory | ^ this | 13:01 |
| TheJulia | fixes in the gate? or??? | 13:01 |
| rpittau | no, no, everything's alright, we're cool and cozy | 13:02 |
| dtantsur | TheJulia: ussuri needs review, I think. everything else back to norm(ish) | 13:02 |
| iurygregory | kinda | 13:02 |
| dtantsur | ah, no, ussuri doesn't need review, I've approved the patch | 13:03 |
| * dtantsur has memory span of a butterfly | 13:03 | |
| iurygregory | o.o | 13:03 |
| rpittau | eheh | 13:03 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 13:04 |
| TheJulia | So... we need to send dtantsur to a butterfly festival? | 13:04 |
| rpittau | on a positive note, I'm very close to make the native ovmf pkg work in ubuntu focal, just seeing an interesting error in ipa, but it's a step forward as we finally see the ramdisk loading | 13:04 |
| dtantsur | would totally go, but social distancing.. | 13:04 |
| TheJulia | dtantsur: naturally | 13:04 |
| dtantsur | rpittau: \o/ | 13:04 |
| TheJulia | rpittau: awesome | 13:04 |
| dtantsur | on a bright side, they're slowly reopening the borders | 13:04 |
| iurygregory | rpittau, \o/ | 13:05 |
| dtantsur | starting today I can travel to the Netherlands without getting quarantined | 13:05 |
| TheJulia | dtantsur: If you have a chance, could you take a glance at https://review.opendev.org/#/c/728315/ and see if you agree with the general idea? It is going to need more work and I'm not quite happy with the direct interface check right but don't want to start mucking with tests too quickly | 13:06 |
| patchbot | patch 728315 - ironic - WIP: Provide a path to set explicit ipxe bootloaders - 1 patch set | 13:06 |
| dtantsur | TheJulia: I took a quick glance this morning and it didn't seem outright insane to me. I'll take a deeper look soon. | 13:07 |
| TheJulia | k, thanks | 13:07 |
| TheJulia | outright insanity is what I was wanting to check | 13:07 |
| Qianbiao | <rpittau> you are welcome | 13:07 |
| Qianbiao | rpittau you take my job and thanks me. | 13:07 |
| dtantsur | TheJulia: there are some dragons around backward compatibility, I suspect | 13:07 |
| Qianbiao | ：） interesting | 13:07 |
| TheJulia | dtantsur: yeah, I think we have a flag on the ipxe interface that only goes so far back, but I think that it is in train, so it might not be too horrible | 13:08 |
| * dtantsur opens the patch again | 13:08 | |
| rpittau | Qianbiao: it's a matter of collaboration, quid pro quo :) | 13:08 |
| Qianbiao | nice ball man. | 13:09 |
| iurygregory | dtantsur, have you tested bluetooh on F32 if is working fine? =) | 13:10 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 13:10 |
| *** michchap has joined #openstack-ironic | 13:11 | |
| dtantsur | TheJulia: a bunch of comments, nothing critical but some food for thought | 13:12 |
| dtantsur | iurygregory: I never use bluetooth. Anything specific I could test for you? | 13:12 |
| iurygregory | dtantsur, I started having problems with my headphones disconnecting from the notebook | 13:14 |
| * iurygregory checking logs XD | 13:14 | |
| dtantsur | ah. I don't have any bluetooth headphones, sorry | 13:14 |
| iurygregory | no worries =) | 13:14 |
| rpittau | iurygregory: works fine for me | 13:14 |
| * dtantsur has actual physical headphones connected to him | 13:14 | |
| iurygregory | wtf the GUI shows bluetooh enable but the service is not running -.-' | 13:18 |
| iurygregory | nice | 13:18 |
| TheJulia | dtantsur: that was exactly what I was seeking, thanks! | 13:18 |
| TheJulia | My bluetooth headphones won't stay connected to my corporate laptop :( | 13:19 |
| TheJulia | It apparently is the bluetooth audio codecs supported | 13:19 |
| iurygregory | D: things were working fine till yesterday | 13:19 |
| iurygregory | and the headphone is working fine with my phone >.< | 13:20 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Raise the JSON RPC connection timeout to 2 minutes https://review.opendev.org/728445 | 13:34 |
| dtantsur | speaking of me breaking things ^^^ :( | 13:34 |
| dtantsur | my apologies, I was naive | 13:34 |
| dtantsur | an example: https://zuul.opendev.org/t/openstack/build/54b51a66c28745cd96091b1129ffb5fb/log/controller/logs/screen-ir-api.txt#3226 | 13:34 |
| dtantsur | on a bright side, the retries are working :D | 13:34 |
| dtantsur | on a dark side, I'm no longer sure we should retry by default | 13:35 |
| dtantsur | in this case we retry a "provide" action and it fails because the first action was in fact in progress | 13:35 |
| dtantsur | opinions? | 13:35 |
| dtantsur | TheJulia: ^^ | 13:36 |
| rpittau | can't we catch the action that is executed and don't trigger the retry ? | 13:37 |
| dtantsur | rpittau: nope. requests doesn't have a way to know if the server is processing a request or just lost it. | 13:37 |
| rpittau | ah it's that thing.... | 13:37 |
| dtantsur | it's HTTP, we don't send anything back until we get the results | 13:38 |
| rpittau | yeah | 13:38 |
| rpittau | retrying by default can stay, if we can figure out a good value for the timeout, I guess the 2 minutes should be ok, considering that it's tweakable | 13:39 |
| dtantsur | another option is to fail with HTTP 500 and let the caller retry | 13:40 |
| dtantsur | I mean, in ironic API | 13:40 |
| dtantsur | with 2 retries we're talking about hanging an API thread for 6 minutes | 13:41 |
| *** jdandrea has joined #openstack-ironic | 13:41 | |
| rpittau | it's a bit long | 13:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Raise the JSON RPC connection timeout to 2 minutes and disable retries https://review.opendev.org/728445 | 13:43 |
| * dtantsur already regrets starting this.. | 13:43 | |
| TheJulia | ugh | 13:44 |
| dtantsur | I also do find it problematic that our synchronous RPC calls can take more than 15 seconds | 13:44 |
| TheJulia | Yeah, vendor passthrough | 13:45 |
| TheJulia | espescialy | 13:45 |
| dtantsur | TheJulia: right, but here it is do_provisioning_action | 13:46 |
| dtantsur | anyway, the patch is up. please review, it will probably improve the stability of the CI. | 13:48 |
| *** uzumaki has joined #openstack-ironic | 13:52 | |
| TheJulia | yeah, validate calls() | 13:55 |
| TheJulia | ? | 13:55 |
| TheJulia | power interface validate | 13:55 |
| TheJulia | or is that just parameters | 13:56 |
| * TheJulia doesn't remember off-hand | 13:56 | |
| *** Goneri has joined #openstack-ironic | 13:56 | |
| TheJulia | and meetings start in moments :( | 13:56 |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/train: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/727906 | 13:59 |
| openstackgerrit | Merged openstack/ironic-inspector master: Stop creating a green thread in the introspection unit tests https://review.opendev.org/727154 | 14:05 |
| rpioso | Good morning, ironic | 14:05 |
| rpioso | dtantsur: Please consider reviewing ^^^ b4 your PTO, since stable/train may be released while you're out. | 14:06 |
| dtantsur | sure | 14:13 |
| *** bnemec is now known as beekneemech | 14:27 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Collect tftpd info from journald https://review.opendev.org/717745 | 14:34 |
| rpittau | hey rpioso, not sure who can approve the changes in sushy-oem-idrac except you, but I have 3 changes that should help on compatibility side, one was already approved by you https://review.opendev.org/717016 https://review.opendev.org/725520 https://review.opendev.org/725521 | 14:38 |
| patchbot | patch 717016 - x/sushy-oem-idrac - Switch to the new canonical constraints URL on master - 2 patch sets | 14:38 |
| patchbot | patch 725520 - x/sushy-oem-idrac - Add pep8 import order check - 1 patch set | 14:38 |
| patchbot | patch 725521 - x/sushy-oem-idrac - Python 3 only - 1 patch set | 14:38 |
| iurygregory | rpittau, I would say that only rpioso can approve since Ilya is not around | 14:44 |
| rpioso | dtantsur: Thank you | 14:45 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 14:49 |
| * rpioso sees he owes rpittau a couple of reviews | 14:49 | |
| rpioso | iurygregory, rpittau, TheJulia: I will also ask a couple of colleagues to take a look at those three. If we're all good, seems like it would be okay for me to Workflow+1 them. Wdyt? | 14:52 |
| iurygregory | rpioso, sounds good to me =) | 14:53 |
| rpittau | rpioso: thanks! | 15:01 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Add global_request_id into the base client methods https://review.opendev.org/725661 | 15:01 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in node methods https://review.opendev.org/725919 | 15:01 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in port and volume https://review.opendev.org/725930 | 15:01 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id for remaining objects https://review.opendev.org/725941 | 15:01 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass additional arguments into adapter https://review.opendev.org/728464 | 15:01 |
| rpioso | rpittau: Thank you for your patience. Busy, busy, ... | 15:05 |
| Qianbiao | <rpittau> will ironic team review 3rd lib too? | 15:07 |
| rpittau | Qianbiao: well, not as "ironic team", but speaking for myself and from experience, members of the team review or contribute to anything that might be of any interest for ironic, including 3rd party libs, and not restricted to python libs :) | 15:12 |
| rpittau | Qianbiao: an example is the python-hardware lib, which is used by ironic-python-agent | 15:12 |
| Qianbiao | rpittau got | 15:16 |
| *** Qianbiao has quit IRC | 15:24 | |
| *** ociuhandu has quit IRC | 15:47 | |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy stable/ussuri: Fix OEM extension loading for different servers https://review.opendev.org/728486 | 16:00 |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy stable/train: Fix OEM extension loading for different servers https://review.opendev.org/728487 | 16:01 |
| *** k_mouza has quit IRC | 16:05 | |
| *** k_mouza has joined #openstack-ironic | 16:06 | |
| *** gyee has joined #openstack-ironic | 16:13 | |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/train: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/727906 | 16:13 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Convert job to dib https://review.opendev.org/718698 | 16:16 |
| iurygregory | rpittau, \o/ ^ | 16:16 |
| rpittau | whooo, unbelievable | 16:16 |
| iurygregory | XD | 16:17 |
| * rpittau will celebrate for the entire weekend | 16:17 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Change ibmcclient compatible version https://review.opendev.org/728436 | 16:18 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Change ibmcclient compatible version https://review.opendev.org/728436 | 16:18 |
| *** Qianbiao has joined #openstack-ironic | 16:21 | |
| Qianbiao | rpittau reply post | 16:27 |
| rpittau | Qianbiao: 0.2.1 is ok with python 3 | 16:28 |
| Qianbiao | yes, but not for py27 ：） | 16:28 |
| rpittau | Qianbiao: yeah, but we don't support py27 in ironic master | 16:29 |
| Qianbiao | it Import typing, but not list in requirements.txt | 16:29 |
| Qianbiao | oh, i see. | 16:29 |
| Qianbiao | But i think use 0.2.2 is better | 16:29 |
| Qianbiao | we can just cherry pick to other branch later. it need >=0.2.2. | 16:30 |
| rpittau | Qianbiao: I see what you mean in this case, alright I'll put 0.2.2, although I'd like to keep the 0.3.0 cap, as failsafe, other drivers do the same | 16:31 |
| Qianbiao | ok | 16:31 |
| Qianbiao | thanks rpittau | 16:31 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Change ibmcclient compatible version https://review.opendev.org/728436 | 16:32 |
| rpittau | Qianbiao: sure, no problem :) | 16:32 |
| Qianbiao | need to go. bye. | 16:34 |
| *** Qianbiao is now known as Qianbiao|afk | 16:34 | |
| *** ociuhandu has joined #openstack-ironic | 16:36 | |
| *** k_mouza has quit IRC | 16:39 | |
| *** ociuhandu has quit IRC | 16:41 | |
| *** ociuhandu has joined #openstack-ironic | 16:42 | |
| *** derekh has quit IRC | 16:47 | |
| openstackgerrit | jacky06 proposed openstack/bifrost master: Remove the unused coding style modules https://review.opendev.org/728501 | 16:51 |
| *** k_mouza has joined #openstack-ironic | 16:58 | |
| dtantsur | folks, I'm on PTO next week, see you in a week | 17:08 |
| *** dtantsur is now known as dtantsur|afk | 17:08 | |
| iurygregory | bye dtantsur|afk enjoy the PTO =) | 17:08 |
| rpittau | bye everyone, have a great weekend! o/ | 17:11 |
| *** rpittau is now known as rpittau|afk | 17:11 | |
| openstackgerrit | Riccardo Pittau proposed openstack/networking-baremetal master: Convert networking-baremetal job to dib https://review.opendev.org/718697 | 17:12 |
| *** k_mouza has quit IRC | 17:15 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Pass global-request-id to ironic-python-agent https://review.opendev.org/728528 | 17:19 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-python-agent master: Add possibility to pass global request ID https://review.opendev.org/728529 | 17:19 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Pass global-request-id to ironic-python-agent https://review.opendev.org/728528 | 17:20 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Pass global-request-id to ironic-python-agent https://review.opendev.org/728528 | 17:25 |
| arne_wiebalck | bye everyone, have a good weekend o/ | 17:29 |
| *** ociuhandu has quit IRC | 17:30 | |
| *** ociuhandu has joined #openstack-ironic | 17:32 | |
| *** ociuhandu has quit IRC | 17:42 | |
| iurygregory | bye arne_wiebalck | 17:43 |
| *** ociuhandu has joined #openstack-ironic | 17:44 | |
| iurygregory | \o | 17:44 |
| *** ociuhandu has quit IRC | 17:54 | |
| *** ociuhandu has joined #openstack-ironic | 17:54 | |
| *** ociuhandu has quit IRC | 17:55 | |
| *** ociuhandu has joined #openstack-ironic | 17:55 | |
| *** ociuhandu has quit IRC | 18:05 | |
| *** ociuhandu has joined #openstack-ironic | 18:06 | |
| *** ociuhandu has quit IRC | 18:06 | |
| *** ociuhandu has joined #openstack-ironic | 18:06 | |
| *** ociuhandu has quit IRC | 18:16 | |
| *** ociuhandu has joined #openstack-ironic | 18:17 | |
| *** iurygregory has quit IRC | 18:35 | |
| *** iurygregory has joined #openstack-ironic | 18:38 | |
| iurygregory | yay for restarting the router on Friday night D: | 18:38 |
| *** iurygregory has quit IRC | 18:45 | |
| *** ociuhandu has quit IRC | 18:57 | |
| *** ociuhandu has joined #openstack-ironic | 18:58 | |
| *** ociuhandu has quit IRC | 18:59 | |
| openstackgerrit | Merged x/sushy-oem-idrac master: Switch to the new canonical constraints URL on master https://review.opendev.org/717016 | 18:59 |
| openstackgerrit | Merged x/sushy-oem-idrac master: Add pep8 import order check https://review.opendev.org/725520 | 18:59 |
| openstackgerrit | Merged x/sushy-oem-idrac master: Python 3 only https://review.opendev.org/725521 | 18:59 |
| *** ociuhandu has joined #openstack-ironic | 18:59 | |
| *** ociuhandu has quit IRC | 19:10 | |
| *** ociuhandu has joined #openstack-ironic | 19:12 | |
| *** ociuhandu has quit IRC | 19:17 | |
| gudrutis2 | anyone still around / | 19:31 |
| gudrutis2 | ? | 19:31 |
| *** k_mouza has joined #openstack-ironic | 19:43 | |
| *** k_mouza has quit IRC | 19:48 | |
| TheJulia | gudrutis2: whats up? | 20:12 |
| gudrutis2 | Nothing much,I figured out my source of my issue | 20:13 |
| gudrutis2 | but thanks for replying | 20:14 |
| TheJulia | gudrutis2: good to hear :) | 20:15 |
| *** iurygregory has joined #openstack-ironic | 20:42 | |
| *** ociuhandu has joined #openstack-ironic | 20:44 | |
| *** ociuhandu has quit IRC | 20:49 | |
| *** rloo has quit IRC | 20:58 | |
| *** lbotell has joined #openstack-ironic | 21:02 | |
| lbotell | Hi all, I have this problem: After IPA finishes the deploying work, NOde tries to boot to instance thru PXE, but openstack does not offer and IP, so it gets stuck in there. Does any one has a suggestion to solve this? | 21:02 |
| NobodyCam | is the bios set to pxe boot? | 21:10 |
| NobodyCam | after deploy it should boot from the local disk | 21:10 |
| lbotell | yes, BIOS is set to PXE boot | 21:12 |
| lbotell | AFAIK there are some choices to boot after deploy: local and netboot, am I wrong? | 21:13 |
| lbotell | Thing is, I would like to boot from PXE always, cause my node changes the boot order to use the last working option | 21:13 |
| NobodyCam | The deploy should set pxe boot when it needs it | 21:14 |
| lbotell | so if it boots from local disks, then I will never be able to make a new deploy | 21:14 |
| lbotell | Ohh, I think I should clarify I am using snmp driver | 21:15 |
| NobodyCam | no pxe will get set for deploy | 21:15 |
| NobodyCam | oh | 21:15 |
| NobodyCam | THat's one I have not used :( | 21:15 |
| lbotell | Oh I see, thank you anyways :) | 21:16 |
| NobodyCam | I'm about to jump into a meeting but might be worth taking a look at the driver code and see what it's doing under the covers | 21:18 |
| lbotell | Hmmm, but snmp controls only a PDU, so we can not instruct the node to boot using a specific interface | 21:19 |
| *** jdandrea has quit IRC | 21:21 | |
| *** lbotell97 has joined #openstack-ironic | 21:22 | |
| *** lbotell is now known as lbotello | 21:25 | |
| *** lbotell97 is now known as lbotell | 21:25 | |
| *** jdandrea has joined #openstack-ironic | 21:29 | |
| *** stevebaker has joined #openstack-ironic | 21:31 | |
| *** ociuhandu has joined #openstack-ironic | 21:32 | |
| TheJulia | o/ | 21:33 |
| openstackgerrit | Merged openstack/ironic-specs master: Propose iso booting https://review.opendev.org/725949 | 21:34 |
| TheJulia | hmm | 21:35 |
| TheJulia | lbotello: I'm glad to hear someone is using that driver! | 21:36 |
| TheJulia | lbotello: so, realistically, it sounds like the default boot option is set to local when it should be network. Also, I wonder what network_interface your using | 21:37 |
| openstackgerrit | Merged openstack/ironic stable/ussuri: Pin the python-ibmcclient upper bounds due to six usage https://review.opendev.org/728388 | 21:38 |
| TheJulia | lbotello: the machine not getting a reply is hte real question, in my mind | 21:39 |
| *** ociuhandu has quit IRC | 21:40 | |
| lbotell | TheJulia: network_interface | 21:44 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: WIP - Victoria Cycle Priorit(y|ies) https://review.opendev.org/720100 | 21:44 |
| lbotell | TheJulia: network_interface | neutron | 21:44 |
| lbotell | is that setting correct? | 21:45 |
| openstackgerrit | Merged openstack/ironic master: Collect ramdisk logs also during cleaning https://review.opendev.org/728071 | 21:46 |
| TheJulia | lbotell: so you have a switch being managed by a neutron ml2 plugin? | 21:51 |
| lbotell | no, the swithc is just set as a single flat VLAN | 21:52 |
| TheJulia | so you want the flat network interface most likely in that case | 21:52 |
| TheJulia | iurygregory: don't let things go to your head :) | 21:59 |
| TheJulia | (in other words, refresh gerrit) | 21:59 |
| iurygregory | TheJulia, ok O.o | 22:05 |
| iurygregory | "with great powers come great responsibilities" | 22:06 |
| TheJulia | Indeed! | 22:07 |
| TheJulia | :) | 22:07 |
| * TheJulia goes and lays down because she tweeked her back... again :( | 22:07 | |
| * TheJulia is just falling apart | 22:07 | |
| iurygregory | ouch =( good luck | 22:08 |
| openstackgerrit | Merged openstack/bifrost master: Remove the unused coding style modules https://review.opendev.org/728501 | 22:20 |
| openstackgerrit | Merged openstack/ironic master: Remove deprecated [cinder]url https://review.opendev.org/728120 | 22:56 |
| lbotello | TheJulia: It worked, I am booting instances thru PXE. Now the only bad new is that I am facing this issue: "Warning: /dev/disk/by-uuid/<some_identifier> does not exist" | 22:58 |
| lbotello | any special advise that you could give me in order to build an image? | 22:58 |
| *** lbotello has quit IRC | 23:01 | |
| TheJulia | lbotell: when/where are you seeing that? | 23:17 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: CI: raise DIB jobs memory to 3 GiB https://review.opendev.org/728426 | 23:26 |
| lbotell | TheJulia: I saw that error while the baremetal node tried to boot a CentOS7 instance | 23:31 |
| lbotell | Good news I just built a Ubuntu18 image and it worked fine | 23:32 |
| TheJulia | if I could get some context I might have an idea or two | 23:32 |
| TheJulia | yay | 23:32 |
| TheJulia | but :( | 23:32 |
| TheJulia | err, some context like the screen shot | 23:32 |
| lbotell | how can I share that photo? | 23:37 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Collect tftpd info from journald https://review.opendev.org/717745 | 23:48 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!