*** zzzeek has quit IRC | 00:04 | |
*** zzzeek has joined #openstack-ironic | 00:05 | |
*** zzzeek has quit IRC | 00:47 | |
*** zzzeek has joined #openstack-ironic | 00:49 | |
*** zigo has quit IRC | 00:53 | |
*** zzzeek has quit IRC | 01:01 | |
openstackgerrit | Merged openstack/ironic master: Make the standalone-redfish job voting https://review.opendev.org/753374 | 01:01 |
---|---|---|
*** zzzeek has joined #openstack-ironic | 01:02 | |
*** Goneri has quit IRC | 01:10 | |
openstackgerrit | Ghanshyam Mann proposed openstack/ironic master: migrate testing to ubuntu focal https://review.opendev.org/739448 | 01:18 |
*** gyee has quit IRC | 01:33 | |
*** cdearborn has quit IRC | 01:45 | |
*** zzzeek has quit IRC | 02:13 | |
*** zzzeek has joined #openstack-ironic | 02:15 | |
*** tkajinam_ has joined #openstack-ironic | 02:17 | |
*** zigo has joined #openstack-ironic | 02:26 | |
*** tkajinam has quit IRC | 02:27 | |
*** johnsom has quit IRC | 02:27 | |
*** ajya has quit IRC | 02:27 | |
*** Qianbiao has joined #openstack-ironic | 02:33 | |
*** johnsom has joined #openstack-ironic | 02:33 | |
*** ajya has joined #openstack-ironic | 02:33 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Limit inspector jobs to 1 testing VM https://review.opendev.org/753094 | 02:41 |
*** stevebaker has quit IRC | 03:16 | |
*** stevebaker has joined #openstack-ironic | 03:45 | |
*** ociuhandu has joined #openstack-ironic | 03:47 | |
*** ociuhandu has quit IRC | 03:52 | |
*** tzumainn has quit IRC | 03:58 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert nodes endpoint to plain JSON https://review.opendev.org/747820 | 04:01 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ports endpoint to plain JSON https://review.opendev.org/750120 | 04:01 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert portgroups endpoint to plain JSON https://review.opendev.org/750864 | 04:01 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ramdisk endpoint to plain JSON https://review.opendev.org/750857 | 04:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume endpoint to plain JSON https://review.opendev.org/750858 | 04:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/connectors endpoint to plain JSON https://review.opendev.org/750859 | 04:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/targets endpoint to plain JSON https://review.opendev.org/750860 | 04:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Expunge the internal version of WSME https://review.opendev.org/750861 | 04:02 |
*** tbonds has quit IRC | 04:16 | |
*** tbonds has joined #openstack-ironic | 04:18 | |
*** uzumaki has joined #openstack-ironic | 04:24 | |
-openstackstatus- NOTICE: A failing log storage endpoint has been removed, you can recheck any recent jobs with POST_FAILURE where logs have failed to upload | 04:45 | |
*** stevebaker has quit IRC | 04:54 | |
uzumaki | morning ironic! o/ | 05:06 |
*** cp- has quit IRC | 05:37 | |
*** cp- has joined #openstack-ironic | 05:42 | |
*** abdysn has joined #openstack-ironic | 05:50 | |
arne_wiebalck | Good morning, ironic! | 05:54 |
iurygregory | good morning uzumaki arne_wiebalck and Ironic o/ | 06:08 |
arne_wiebalck | hey iurygregory o/ | 06:09 |
uzumaki | heyyyy everybody! iurygregory arne_wiebalck how's the morning? (assuming it's morning there) | 06:09 |
iurygregory | it's morning | 06:11 |
iurygregory | I need coffee | 06:11 |
*** ianychoi_ is now known as ianychoi | 06:16 | |
*** cp- has quit IRC | 06:23 | |
Qianbiao | Morning Ironic。 | 06:27 |
Qianbiao | arne_wiebalck send you a feedback. | 06:28 |
iurygregory | morning Qianbiao | 06:28 |
*** cp- has joined #openstack-ironic | 06:29 | |
Qianbiao | Hey boy o/ iurygregory | 06:29 |
iurygregory | o/ | 06:29 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/6.3: CI: Lower memory usage of VMs/Increase swap https://review.opendev.org/753460 | 06:37 |
arne_wiebalck | Qianbiao: yes, you're right ... I replied (and still would like a reno :-) | 06:41 |
Qianbiao | arne_wiebalck sure | 06:43 |
Qianbiao | may i add it at fellow commit? let this one pass first. | 06:43 |
arne_wiebalck | there are plans to backport this, no? | 06:45 |
Qianbiao | yes | 06:46 |
Qianbiao | need reno backport too? | 06:46 |
iurygregory | yes | 06:46 |
arne_wiebalck | that is what I am thinking | 06:46 |
Qianbiao | I thought stable branch does not need that | 06:46 |
Qianbiao | ok | 06:46 |
arne_wiebalck | I think it'd be easier to add it now | 06:47 |
Qianbiao | ok | 06:47 |
iurygregory | well depends if you want to backport, but I would say a reno is necessary | 06:47 |
arne_wiebalck | otherwise I am +2 on the change :) | 06:47 |
iurygregory | the change impacts the operator no? | 06:47 |
Qianbiao | iurygregory no, no behavior changes. | 06:48 |
iurygregory | oh ok =) | 06:48 |
Qianbiao | But fix a bug | 06:48 |
iurygregory | hummm | 06:48 |
Qianbiao | I think should no one have use the infiniband hardware with Ironic before. | 06:49 |
Qianbiao | arne_wiebalck we should break the patch into two at first. | 06:59 |
Qianbiao | one for inherit issue, one for CNA card disable lldp agent. | 06:59 |
*** jtomasek has joined #openstack-ironic | 07:00 | |
arne_wiebalck | and you would only backport the disable part? | 07:01 |
Qianbiao | yes | 07:02 |
Qianbiao | no, i will backport the inherit | 07:02 |
Qianbiao | part not disable part. | 07:02 |
Qianbiao | inherit part fixes the dispatch mechanism issue. the disable part is another topic. | 07:03 |
arne_wiebalck | Strictly speaking you are correct that these are independent issues. IMO, all this is entangled (and small) enough to keep it as one patch. | 07:06 |
arne_wiebalck | Did anyone suggest to split it? | 07:07 |
Qianbiao | nope... i do not want to split it now .. | 07:09 |
Qianbiao | I mean if at first. | 07:09 |
Qianbiao | :) | 07:09 |
arne_wiebalck | In that case I would suggest to add a reno and see how it goes :-D | 07:10 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/752024 | 07:10 |
Qianbiao | <arne_wiebalck> ^^^ | 07:10 |
* arne_wiebalck sees that Qianbiao can add a reno in 7 seconds! | 07:10 | |
Qianbiao | :) | 07:11 |
Qianbiao | 6 seconds. | 07:11 |
Qianbiao | It's a english test for me to write so long a paragraph | 07:13 |
* arne_wiebalck has a meeting now, will check after | 07:14 | |
Qianbiao | sure thanks. | 07:14 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/6.2: CI: Lower memory usage of VMs/Increase swap https://review.opendev.org/753461 | 07:17 |
*** tosky has joined #openstack-ironic | 07:19 | |
*** dougsz has joined #openstack-ironic | 07:30 | |
iurygregory | mgoddard, hey if you have some time this week https://review.opendev.org/#/c/752042/ o/ | 07:33 |
patchbot | patch 752042 - bifrost - Add ipe installation to bifrost - 4 patch sets | 07:33 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/752024 | 07:37 |
*** yolanda has joined #openstack-ironic | 07:42 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic bugfix/15.1: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753469 | 07:44 |
*** rcernin has quit IRC | 07:45 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic bugfix/15.2: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753468 | 07:45 |
*** ociuhandu has joined #openstack-ironic | 07:47 | |
*** priteau has joined #openstack-ironic | 07:50 | |
*** patchbot has quit IRC | 07:57 | |
*** patchbot has joined #openstack-ironic | 07:59 | |
*** dtantsur|afk is now known as dtantsur | 08:03 | |
dtantsur | morning ironic, says a man trying to open his eyes | 08:04 |
iurygregory | morning dtantsur o/ | 08:04 |
mgoddard | iurygregory: argh ipe ipa ipxe | 08:06 |
mgoddard | iurygregory: I'll put it on my list, but busy this morning | 08:06 |
iurygregory | mgoddard, sorry for the acronym =) | 08:08 |
*** belmoreira has joined #openstack-ironic | 08:12 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector bugfix/10.3: Limit inspector jobs to 1 testing VM https://review.opendev.org/753273 | 08:12 |
*** lucasagomes has joined #openstack-ironic | 08:12 | |
*** ociuhandu has quit IRC | 08:13 | |
*** ociuhandu has joined #openstack-ironic | 08:13 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector bugfix/10.2: Limit inspector jobs to 1 testing VM https://review.opendev.org/753463 | 08:13 |
iurygregory | dtantsur, great news docs job should be fine | 08:18 |
*** dtantsur has quit IRC | 08:18 | |
iurygregory | we didn't have to merge the change to disable pdf | 08:19 |
*** ociuhandu has quit IRC | 08:20 | |
*** dtantsur has joined #openstack-ironic | 08:20 | |
dtantsur | iurygregory: some ipe comments inline | 08:22 |
iurygregory | dtantsur, ack | 08:23 |
*** ociuhandu has joined #openstack-ironic | 08:25 | |
iurygregory | dtantsur, instead of ipe_* ironicprometheusexporter_* | ironic_prometheus_exporter_* | prometheus_exporter_* ? | 08:29 |
dtantsur | iurygregory: I prefer the last one, the 2nd one works as well | 08:29 |
dtantsur | I'd rather not do toomanywordswithoutasinglebreak :) | 08:29 |
iurygregory | i've put the first only because of ironicinspectorclient_git_folder.... | 08:30 |
*** Lucas_Gray has joined #openstack-ironic | 08:31 | |
dtantsur | yeah, I don't like it too, but changing it is a pain | 08:31 |
iurygregory | I can imagine... | 08:32 |
*** tkajinam_ is now known as tkajinam | 08:34 | |
*** derekh has joined #openstack-ironic | 08:41 | |
iurygregory | dtantsur, I've update some of TheJulia patches on the bugfix branches since we had the problem with the l-c jobs | 08:42 |
dtantsur | cool. I'll try to get to them when I wake up | 08:42 |
iurygregory | it's the time for CI to finish running XD | 08:42 |
*** ociuhandu has quit IRC | 08:45 | |
*** Qianbiao has quit IRC | 08:46 | |
*** Qianbiao has joined #openstack-ironic | 08:47 | |
*** k_mouza has joined #openstack-ironic | 08:50 | |
iurygregory | dtantsur, about enabling the exporter in the bifrost CI I was thinking in bifrost-integration-tinyipa-keystone-centos-8 or bifrost-integration-tinyipa-keystone-ubuntu-bionic since they are using redfish wdyt? | 08:54 |
janders | good morning Ironic o/ | 08:54 |
dtantsur | iurygregory: no objections as long as they don't become too heavy (which I doubt) | 08:54 |
dtantsur | good evening janders :) | 08:54 |
iurygregory | morning janders o/ | 08:55 |
iurygregory | dtantsur, the only thing if we want to *see* the metrics on logs or something we would probably need to add some wait (since only from time to time the sensor data is collected) | 08:55 |
dtantsur | iurygregory: we can think about it later, I guess? I'm only worried about testing the ability to install it | 08:56 |
*** priteau has quit IRC | 08:56 | |
dtantsur | also half-joking: do we need to install prometheus as well? :) | 08:56 |
iurygregory | dtantsur, I see | 08:57 |
iurygregory | no we don't need to install XD | 08:57 |
iurygregory | in the IPE CI we test that we can retrive the metrics from the endpoint | 08:57 |
dtantsur | yeah, I know, but I wonder if this feature is complete without bifrost installing prometheus (probably yes) | 08:58 |
iurygregory | for me it's | 08:58 |
iurygregory | since I run prometheus in a different server =) | 08:58 |
iurygregory | I only need to add the endpoint to the config | 08:58 |
dtantsur | iurygregory: in your patch IPE listens on localhost, will it work this way? | 09:01 |
dtantsur | or do we need to use internal_ip? | 09:01 |
iurygregory | well when deploying I set the ip in the group_vars .-. | 09:02 |
dtantsur | iurygregory: maybe default to internal_ip then? | 09:02 |
dtantsur | I don't quite like using group_vars.. for anything :) | 09:02 |
iurygregory | can be | 09:02 |
iurygregory | I will look at how to do that =) | 09:02 |
dtantsur | iurygregory: oh, do we have http_basic auth for IPE? | 09:03 |
dtantsur | or is it not prometheus-compatible? | 09:03 |
* dtantsur feels he should stop bombarding iurygregory with ideas | 09:03 | |
iurygregory | is not necessary afaik | 09:03 |
iurygregory | in my tests I'm running ironic with http_basic | 09:03 |
dtantsur | well, ideally we should not allow anyone on the network to read metrics | 09:04 |
iurygregory | maybe we need to add something for the flask application to require the auth | 09:04 |
iurygregory | maybe something for the future of IPE it's important | 09:05 |
iurygregory | dtantsur, you can continue bombarding me with ideas I don't mind =) | 09:05 |
dtantsur | I think it's useful if prometheus can use basic auth | 09:05 |
dtantsur | not blocking this patch of course :) | 09:05 |
*** strigazi has joined #openstack-ironic | 09:06 | |
iurygregory | ack | 09:06 |
*** Abdallahyas has joined #openstack-ironic | 09:12 | |
*** abdysn has quit IRC | 09:15 | |
openstackgerrit | Merged openstack/ironic-inspector master: Correct the docstring for _should_enable_unknown_hosts https://review.opendev.org/753430 | 09:21 |
dtantsur | TheJulia: do we have a PTG etherpad at this point? I'd like to raise the topic of stable branch lifetime (again) | 09:29 |
janders | +1 (I wanted to add NVMe/SSD erasure improvements) | 09:31 |
janders | (gotta be careful not to discard the Etherpad :) | 09:31 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Switch integration testing to Ubuntu Focal https://review.opendev.org/753553 | 09:37 |
dtantsur | I guess we're supposed to do ^^^ before releasing victoria | 09:37 |
*** yolanda has quit IRC | 09:38 | |
*** yolanda has joined #openstack-ironic | 09:39 | |
iurygregory | yeah | 09:40 |
Qianbiao | may core give a workflow for this patch https://review.opendev.org/#/c/752024 | 09:54 |
patchbot | patch 752024 - ironic-python-agent - Fix: make Intel CNA hardware manager none generic - 10 patch sets | 09:54 |
iurygregory | dtantsur, internal_ip would be ansible_default_ipv4.address I assume right? | 10:03 |
*** Abdallahyas has quit IRC | 10:05 | |
*** abdysn has joined #openstack-ironic | 10:05 | |
openstackgerrit | Ankit Kumar proposed openstack/ironic master: Adding changes for iso less vmedia support https://review.opendev.org/752001 | 10:06 |
*** Lucas_Gray has quit IRC | 10:07 | |
*** fgofurov has joined #openstack-ironic | 10:08 | |
fgofurov | Hello ironic team | 10:09 |
fgofurov | we are facing an error during introspection | 10:11 |
fgofurov | 2020-09-23 06:16:54.272 1 ERROR oslo_messaging.rpc.server 'Could not find a node for attributes %s') % attributes) | 10:11 |
fgofurov | 2020-09-23 06:16:54.272 1 ERROR oslo_messaging.rpc.server ironic_inspector.utils.NotFoundInCacheError: Could not find a node for attributes {'bmc_address': | 10:11 |
fgofurov | it keeps throwing: AttributeError: 'Node' object has no attribute 'uuid' | 10:11 |
fgofurov | any ideas on this? | 10:12 |
dtantsur | iurygregory: I think internal_ip is already defined, it's a bit more complex than that | 10:13 |
dtantsur | fgofurov: a complete traceback (actually, two tracebacks) could be useful | 10:14 |
dtantsur | Qianbiao: the release notes failure seems real - see inline | 10:15 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server [-] Exception during message handling: AttributeError: 'Node' object has no attribute 'uuid' | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server Traceback (most recent call last): | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/ironic_inspector/process.py", line 87, in _find_node_info | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server mac=macs) | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/ironic_inspector/node_cache.py", line 819, in find_node | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server 'Could not find a node for attributes %s') % attributes) | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server ironic_inspector.utils.NotFoundInCacheError: Could not find a node for attributes {'bmc_address': | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server During handling of the above exception, another exception occurred: | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server Traceback (most recent call last): | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/oslo_messaging/rpc/server.py", line 165, in _process_incoming | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server res = self.dispatcher.dispatch(message) | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/oslo_messaging/rpc/dispatcher.py", line 276, in dispatch | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server return self._do_dispatch(endpoint, method, ctxt, args) | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/oslo_messaging/rpc/dispatcher.py", line 196, in _do_dispatch | 10:17 |
Qianbiao | <dtantsur> checking | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server result = func(ctxt, **new_args) | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/oslo_messaging/rpc/server.py", line 241, in inner | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server return func(*args, **kwargs) | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/ironic_inspector/conductor/manager.py", line 188, in do_continue | 10:17 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server return process.process(data) | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/ironic_inspector/process.py", line 206, in process | 10:18 |
Qianbiao | fgofurov do not paste log here. | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server node_info = _find_node_info(introspection_data, failures) | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/ironic_inspector/process.py", line 92, in _find_node_info | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server macs=macs) | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/ironic_inspector/node_cache.py", line 961, in record_node | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server {'node': node.uuid, | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/openstack/resource.py", line 588, in __getattribute__ | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server raise e | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/openstack/resource.py", line 582, in __getattribute__ | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server return object.__getattribute__(self, name) | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server AttributeError: 'Node' object has no attribute 'uuid' | 10:18 |
fgofurov | 2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server [00m | 10:18 |
fgofurov | 2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main [req-44447b7f-aa5c-4262-a28f-053d98355b80 - - - - -] Internal server error: AttributeError: 'Node' object has no attribute 'uuid' | 10:18 |
fgofurov | 2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main Traceback (most recent call last): | 10:18 |
fgofurov | 2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main File "/usr/lib/python3.6/site-packages/ironic_inspector/process.py", line 87, in _find_node_info | 10:18 |
fgofurov | 2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main mac=macs) | 10:18 |
fgofurov | 2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main File "/usr/lib/python3.6/site-packages/ironic_inspector/node_cache.py", line 819, in find_node | 10:18 |
fgofurov | 2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main 'Could not find a node for attributes %s') % attributes) | 10:18 |
fgofurov | sorry for flooding a chat | 10:18 |
iurygregory | fgofurov, try to use a paste tool please =) | 10:20 |
iurygregory | dtantsur, /me checks | 10:20 |
fgofurov | Okay, newbie to this, my apologies | 10:21 |
iurygregory | fgofurov, http://paste.openstack.org/ | 10:21 |
iurygregory | fgofurov, no worries | 10:21 |
fgofurov | thanks a lot! | 10:21 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/752024 | 10:24 |
Qianbiao | dtantsur fixed | 10:25 |
*** ociuhandu has joined #openstack-ironic | 10:29 | |
*** ociuhandu has quit IRC | 10:38 | |
dtantsur | fgofurov: okay, so the 2nd problem is a red herring, and it has been fixed recently (I wonder why you don't have a fix) | 10:39 |
dtantsur | the fix was https://review.opendev.org/#/c/739730/ | 10:41 |
patchbot | patch 739730 - ironic-inspector - Use node.id instead of node.uuid in record_node (MERGED) - 1 patch set | 10:41 |
dtantsur | iurygregory: if you have a second, could you request an ironic-inspector ussuri release so that we have ^^ released? | 10:44 |
fgofurov | We are using ironic-inspector 10.1.0 btw | 10:45 |
iurygregory | dtantsur, can I do after our meeting (I'm finishing lunch atm) | 10:45 |
dtantsur | yeah, the fix will be in 10.1.1 | 10:45 |
dtantsur | iurygregory: enjoy your lunch then, I'll request myself | 10:45 |
iurygregory | ack | 10:45 |
dtantsur | fgofurov: back to the 1st (real) problem. the IPMI address and MAC addresses after "Could not find a node for attributes", do they match the node you're trying to introspect? | 10:45 |
* dtantsur considers creating a troubleshooting FAQ since this problem happens quite often | 10:46 | |
*** kashifkhan has joined #openstack-ironic | 10:48 | |
kashifkhan | Hi, I a deployment we have ironic-inspector version 10.1.0 and we see the error "'Node' object has no attribute 'uuid'", we also noticed that there was a bug fix for this on 10.1.0-19, the question is , is there any other workaround to this issue without uplifting the ironic-inspector image? | 10:51 |
*** Lucas_Gray has joined #openstack-ironic | 10:53 | |
*** zzzeek has quit IRC | 10:55 | |
dtantsur | kashifkhan: hi, funny, another person reported the same issue just minutes ago :) is there any problem with updating the image? | 10:55 |
kashifkhan | @<dtantsur>, yes we are talking about the same issue, but since I logged in later, I didnt see it | 10:56 |
dtantsur | ah, got it | 10:56 |
kashifkhan | its kind of problematic for us, to uplift the image at this moment, so we were thinking of thereis any other workaround | 10:57 |
*** zzzeek has joined #openstack-ironic | 10:58 | |
dtantsur | I don't think so, it's a bug that has to be fixed in code | 10:59 |
kashifkhan | @<dtantsur>ok, thanks | 10:59 |
fgofurov | thanks a lot dtansur | 11:03 |
*** ociuhandu has joined #openstack-ironic | 11:05 | |
*** zzzeek has quit IRC | 11:09 | |
*** zzzeek has joined #openstack-ironic | 11:11 | |
openstackgerrit | Merged openstack/ironic master: Limit inspector jobs to 1 testing VM https://review.opendev.org/753094 | 11:12 |
*** JamesBenson has joined #openstack-ironic | 11:21 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Limit inspector jobs to 1 testing VM https://review.opendev.org/753580 | 11:22 |
iurygregory | dtantsur, anything else we need releases? | 11:40 |
iurygregory | now I can push some | 11:41 |
iurygregory | I know we can cut IPE stable/victoria | 11:41 |
iurygregory | wondering if it's ok to wait for the backports for ussuri/train to get in so I push everything in one release patch | 11:41 |
dtantsur | iurygregory: yep. also waiting with ironic because of the unmerged patches to ussuri (sigh). maybe you could check train? | 11:59 |
iurygregory | dtantsur, sure | 11:59 |
*** tosky_ has joined #openstack-ironic | 12:02 | |
*** tosky is now known as Guest73219 | 12:03 | |
*** tosky_ is now known as tosky | 12:03 | |
*** fgofurov has quit IRC | 12:05 | |
*** Guest73219 has quit IRC | 12:05 | |
*** Goneri has joined #openstack-ironic | 12:06 | |
janders | see you tomorrow Ironic o/ | 12:09 |
janders | have a great day everyone | 12:09 |
iurygregory | bye janders o/ | 12:10 |
openstackgerrit | Mahendra Kamble proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port https://review.opendev.org/693328 | 12:13 |
openstackgerrit | Merged openstack/ironic-python-agent master: Generate a TLS certificate and send it to ironic https://review.opendev.org/749930 | 12:14 |
iurygregory | \o/ | 12:15 |
iurygregory | merged! | 12:15 |
janders | \o/ | 12:15 |
janders | well done! | 12:15 |
iurygregory | now dtantsur is happy | 12:16 |
dtantsur | w000t \o/ | 12:27 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Use correct error handler instead of calling process_event('fail') https://review.opendev.org/753595 | 12:27 |
*** rloo has joined #openstack-ironic | 12:28 | |
dtantsur | now I guess I should write some docs | 12:29 |
*** priteau has joined #openstack-ironic | 12:35 | |
*** tkajinam has quit IRC | 12:51 | |
*** tkajinam has joined #openstack-ironic | 12:52 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Expose agent_verify_ca in configuration and correctly handle booleans https://review.opendev.org/753641 | 13:04 |
dtantsur | a missing bit in the TLS story ^^^ | 13:04 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal https://review.opendev.org/753642 | 13:05 |
*** Abdallahyas has joined #openstack-ironic | 13:06 | |
* iurygregory fingers crossed for this one ^ | 13:06 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Document ramdisk TLS and update existing TLS docs https://review.opendev.org/753645 | 13:08 |
dtantsur | JayF: ^^ | 13:08 |
dtantsur | kashifkhan, ironic-inspector 10.1.1 has been released | 13:09 |
*** abdysn has quit IRC | 13:10 | |
Qianbiao | thanks you dtantsur i see the cna card hardware manager patch has started the gateway job. | 13:12 |
dtantsur | yep, hopefully will land soon | 13:12 |
Qianbiao | Should i wait it merged and then start cherry-pick? | 13:12 |
Qianbiao | or i can start cherry pick now. | 13:12 |
*** abdysn has joined #openstack-ironic | 13:12 | |
dtantsur | Qianbiao: you can cherry-pick now, but note that gerrit UI will not add the necessary "cherry-picked from" line until it merges | 13:12 |
dtantsur | so better cherry-pick it locally with `git cherry-pick -x` | 13:13 |
Qianbiao | ok | 13:13 |
Qianbiao | will wait :) | 13:13 |
*** Abdallahyas has quit IRC | 13:15 | |
Qianbiao | dtantsur no hurry, i can wait :) | 13:19 |
*** Goneri has quit IRC | 13:21 | |
dtantsur | :) | 13:21 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Update tests for Redfish BIOS apply_configuration https://review.opendev.org/753694 | 13:25 |
*** tosky_ has joined #openstack-ironic | 13:26 | |
*** tosky is now known as Guest13336 | 13:26 | |
*** tosky_ is now known as tosky | 13:26 | |
*** Guest13336 has quit IRC | 13:28 | |
*** tosky_ has joined #openstack-ironic | 13:29 | |
*** ociuhandu_ has joined #openstack-ironic | 13:31 | |
*** tosky is now known as Guest61020 | 13:31 | |
*** tosky_ is now known as tosky | 13:31 | |
*** Guest61020 has quit IRC | 13:32 | |
*** Goneri has joined #openstack-ironic | 13:34 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add Redfish BIOS interface to idrac HW type https://review.opendev.org/749240 | 13:34 |
*** ociuhandu has quit IRC | 13:35 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Update tests for Redfish BIOS apply_configuration https://review.opendev.org/753694 | 13:38 |
ajya | sorry dtantsur had to make a minor clean up ^ :) | 13:40 |
*** uzumaki has quit IRC | 13:40 | |
*** ociuhandu_ has quit IRC | 13:41 | |
*** ociuhandu has joined #openstack-ironic | 13:42 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal https://review.opendev.org/753642 | 13:45 |
*** tzumainn has joined #openstack-ironic | 13:46 | |
*** cdearborn has joined #openstack-ironic | 13:54 | |
dtantsur | okay, I'll check again once the CI runas | 13:59 |
TheJulia | good morning | 14:03 |
dtantsur | morning TheJulia | 14:03 |
* TheJulia wipes sleep from her eyes | 14:04 | |
TheJulia | dtantsur: I hadn't started a full etherpad yet, only tried to collect items at the midcycle | 14:04 |
dtantsur | we can probably repurpose https://meetpad.opendev.org/etherpad/p/ironic-ptg | 14:05 |
TheJulia | ++ | 14:05 |
*** tkajinam has quit IRC | 14:05 | |
iurygregory | good morning TheJulia =) | 14:05 |
TheJulia | iurygregory: by chance did you look at the stable/ussuri job to see why it failed? | 14:15 |
TheJulia | talking about the patches to fix ci | 14:16 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: CI: Lower memory usage of VMs/Increase swap https://review.opendev.org/753462 | 14:16 |
iurygregory | TheJulia, I know there was some problem on CI "NOTICE: A failing log storage endpoint has been removed, you can recheck any recent jobs with POST_FAILURE where logs have failed to upload" so I just went with some rechecks to see | 14:16 |
TheJulia | wheeee | 14:17 |
iurygregory | the bugfix branches I've updated because we need to set l-c to bionic | 14:17 |
iurygregory | or update requirements... | 14:17 |
openstackgerrit | Merged openstack/ironic master: Fix redfish BIOS to use @Redfish.SettingsApplyTime https://review.opendev.org/752614 | 14:17 |
iurygregory | for inspector bugfix .2 I've let a question in the patch | 14:18 |
TheJulia | k | 14:18 |
TheJulia | ugh, yeah, l-c changes were landed to bugfix branches yet | 14:18 |
iurygregory | do we need to land? I was trying to backport but dtantsur raised a good point on my backport so I just went to switch the jobs to bionic =) | 14:19 |
iurygregory | https://review.opendev.org/753277 | 14:20 |
patchbot | patch 753277 - ironic-inspector (bugfix/10.3) - Fix l-c and mysql GRANT syntex for Ubuntu Focal (ABANDONED) - 1 patch set | 14:20 |
TheJulia | ok, that might be fine then | 14:20 |
openstackgerrit | Mudit proposed openstack/ironic master: Implement reporting GPU devices in iDRAC driver https://review.opendev.org/751006 | 14:20 |
iurygregory | I'm ok with both approaches =)' | 14:20 |
TheJulia | technically victoria artifiats are supposed to be good against focal | 14:22 |
iurygregory | humm so we would need to bump | 14:22 |
TheJulia | need to bump the minor? | 14:23 |
iurygregory | well we also bump req | 14:24 |
iurygregory | not only l-c | 14:24 |
TheJulia | so in that case we only propose backports to lock the nodes to bionic? That is viable to me | 14:25 |
iurygregory | yeah in the bugfix branches only | 14:25 |
iurygregory | the master/victoria we have should be fine | 14:26 |
TheJulia | yeah | 14:26 |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/stein: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753707 | 14:30 |
dtantsur | yeah, for victoria we can make another minor release | 14:30 |
dtantsur | for bugfix branches it contradicts the very idea of bugfix branches :) | 14:30 |
TheJulia | yeah | 14:31 |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/train: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753471 | 14:31 |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/576718 | 14:34 |
TheJulia | dtantsur: do you agree with bacporting https://review.opendev.org/#/c/752710/ ? | 14:34 |
patchbot | patch 752710 - ironic (stable/train) - Allow configuring IPMI cipher suite - 1 patch set | 14:34 |
iurygregory | I have the feeling it fits on our idea that we are making the operators life easier ... | 14:40 |
*** abdysn has quit IRC | 14:46 | |
TheJulia | then +1 it (I really need to figure out what it will take to get the stable groups changed) | 14:47 |
iurygregory | done =) | 14:48 |
dtantsur | TheJulia: I do, just never got to approving it | 14:54 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Add ipe installation to bifrost https://review.opendev.org/752042 | 14:54 |
iurygregory | let's see what CI will say | 14:54 |
iurygregory | locally it worked | 14:54 |
dtantsur | TheJulia: I talked to ttx and we agreed that the situation with ipmitool falls under "regression" category | 14:55 |
dtantsur | (which is also how consumers tend to perceive it) | 14:55 |
iurygregory | agree | 14:55 |
TheJulia | Yeah, I remember remarking that you guys basically re-discussed what the TC already said was okay and worthwhile | 14:55 |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Redfish driver firmware update https://review.opendev.org/749619 | 14:57 |
dtantsur | TheJulia: yeah, I wanted to clarify the situation before it comes up on a release review | 14:57 |
openstackgerrit | Merged openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/752024 | 14:57 |
TheJulia | ++ | 14:58 |
TheJulia | yay an ipa patch merged | 14:58 |
iurygregory | second | 14:59 |
iurygregory | TLS patch merged | 14:59 |
TheJulia | sweet | 14:59 |
iurygregory | almost a merge party from zuul today | 14:59 |
dtantsur | approved | 14:59 |
dtantsur | a missing bit from the TLS story: https://review.opendev.org/753641 | 14:59 |
patchbot | patch 753641 - ironic - Expose agent_verify_ca in configuration and correc... - 1 patch set | 14:59 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent stable/ussuri: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/753720 | 15:00 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent stable/train: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/752065 | 15:03 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent stable/stein: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/753730 | 15:03 |
Qianbiao | those three branches i should backport, right? | 15:04 |
*** k_mouza has quit IRC | 15:04 | |
arne_wiebalck | dtantsur: you have a moment? there is a RAID cleaning issue and I'd like to discuss how to address it | 15:05 |
TheJulia | Qianbiao: depends on the patch :) | 15:05 |
dtantsur | on a meeting, will ping you | 15:06 |
TheJulia | Qianbiao: if you've got a link, I can give you an idea | 15:06 |
arne_wiebalck | tks | 15:06 |
Qianbiao | TheJulia https://review.opendev.org/753730 | 15:06 |
patchbot | patch 753730 - ironic-python-agent (stable/stein) - Fix: make Intel CNA hardware manager none generic - 1 patch set | 15:06 |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/576718 | 15:07 |
*** k_mouza has joined #openstack-ironic | 15:10 | |
*** jtriley has joined #openstack-ironic | 15:22 | |
jtriley | hjensas: For https://review.opendev.org/#/c/753435/ would this patch go in the ironic_inspector container only or ironic_inspect and ironic_inspector_dnsmasq? Trying to apply this now for testing | 15:23 |
patchbot | patch 753435 - ironic-inspector - dnsmasq pxe filter - option to always block unknow... - 1 patch set | 15:23 |
jtriley | seems like just ironic_inspector - sorry for the noise | 15:25 |
hjensas | jtriley: ironic_inspector only, I think ironic_inspector_dnsmasq is just the actual dnsmasq process? | 15:25 |
jtriley | hjensas: thanks makes sense | 15:28 |
jtriley | hjensas: I got the patch applied - should I podmount umount before bouncing the systemd service? | 15:30 |
hjensas | jtriley: Should work without unmounting, but it does'nt hurt. :) | 15:31 |
*** VisBits has quit IRC | 15:32 | |
jtriley | hjensas: and obviously I'll need to add that setting to the inspector.conf under /var/lib/config-data? | 15:33 |
*** k_mouza has quit IRC | 15:38 | |
jtriley | hjensas: ok looks like the patch and config inside the container look as they should after restart - time to test :D | 15:38 |
hjensas | jtriley: yes, setting in /var/lib/config-data is bind mounted in the container. | 15:40 |
*** stbenjam has joined #openstack-ironic | 15:43 | |
openstackgerrit | Verification of a change to openstack/ironic failed: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753470 | 15:44 |
*** gyee has joined #openstack-ironic | 15:45 | |
*** ociuhandu has quit IRC | 15:49 | |
arne_wiebalck | dtantsur: I will have to leave shortly, sorry. I have summarised the issue and the options I see here: https://storyboard.openstack.org/#!/story/2008186 | 15:50 |
arne_wiebalck | TheJulia: ^^ | 15:50 |
dtantsur | thx! | 15:50 |
openstackgerrit | Merged openstack/ironic bugfix/15.2: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753468 | 15:51 |
openstackgerrit | Merged openstack/ironic bugfix/15.1: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753469 | 15:51 |
*** uzumaki has joined #openstack-ironic | 15:55 | |
jtriley | hjensas: looks like the unknown_hosts_filter is still getting the ";ignore" stripped so both nics are still getting an IP | 15:56 |
jtriley | hjensas: taking a look at the dnsmasq container in case it needs the patch as well | 15:56 |
*** belmoreira has quit IRC | 15:57 | |
*** belmoreira has joined #openstack-ironic | 15:59 | |
*** tosky has quit IRC | 15:59 | |
*** belmoreira has quit IRC | 16:00 | |
*** ociuhandu has joined #openstack-ironic | 16:01 | |
*** lucasagomes has quit IRC | 16:04 | |
TheJulia | Qianbiao: re the patch, sorry I thought I had hit enter | 16:05 |
TheJulia | Yes, stable/ussuri, stable/train <-- should be good enough for minimally, you can also backport to the two bugfix branches if you want | 16:06 |
Qianbiao | you mean rocky and queens? | 16:07 |
*** ociuhandu has quit IRC | 16:07 | |
hjensas | jtriley: strange, you have deny_unknown_macs = true in the dnsmasq_pxe_filter section of ironic inspector conf? | 16:07 |
jtriley | hjensas: blah, I put the config setting in the wrong spot | 16:07 |
jtriley | hjensas: haha yup that's it | 16:07 |
jtriley | retrying | 16:07 |
Qianbiao | TheJulia np, i thought you got caught by something | 16:07 |
Qianbiao | you mean rocky and queens? | 16:07 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Switch integration testing to Ubuntu Focal https://review.opendev.org/753553 | 16:08 |
dtantsur | mgoddard: oops, updated ^^ | 16:09 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Add ipe installation to bifrost https://review.opendev.org/752042 | 16:09 |
TheJulia | Qianbiao: you can to rocky and queens as well | 16:12 |
mgoddard | dtantsur: thanks | 16:12 |
dtantsur | TheJulia: assuming rocky and queens have any chances of working.. | 16:12 |
TheJulia | oh jeeze yeah | 16:12 |
TheJulia | Well, I got both working ~3-4 weeks ago | 16:13 |
*** Lucas_Gray has quit IRC | 16:13 | |
* dtantsur would like to reduce the number of supported branches | 16:13 | |
jtriley | hjensas: looks like it's working - letting it go for a bit but we already see it denying the other mac :) | 16:13 |
Qianbiao | ok TheJulia and for branch train, is it wrong? there are no topic info. | 16:14 |
TheJulia | As long as we're supporting queens downstream for RH, I'd like us to be able to merge/test on that branch. Nova is taking a similar approach. In fact, I proposed something downstream and they went and did it upstream too | 16:14 |
Qianbiao | because i mis-submit it from my local train branch before master merges. | 16:14 |
TheJulia | Qianbiao: stable/<release_name> | 16:14 |
Qianbiao | yes that is what i mean https://review.opendev.org/#/c/752065/ | 16:15 |
patchbot | patch 752065 - ironic-python-agent (stable/train) - Fix: make Intel CNA hardware manager none generic - 4 patch sets | 16:15 |
hjensas | jtriley: ok, based on dtantsur feedback I will re-factor the patch so that it's also possible to do this with the other drivers. Thanks for testing! | 16:15 |
Qianbiao | how i can delete it , and re cherry pick | 16:15 |
TheJulia | you need to checkout that branch specifically and then run `git review -X changenumber` to pull down and apply the cherry pick to your local branch and once you've resolved cherrypicking merge conflicts, you run `git review` to push things back up | 16:15 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent stable/rocky: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/753785 | 16:15 |
TheJulia | oh, `git reset HEAD~1` and `git pull origin stable/<releasename>` is your friend | 16:16 |
TheJulia | err | 16:16 |
TheJulia | that should be git pull --ff-only origin stable/<releasename> | 16:16 |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent stable/queens: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/753786 | 16:17 |
TheJulia | Qianbiao: typically we go in most recent to oldest, but given you've done both of those in rapid succession I suspect they are clean backports :) | 16:17 |
Qianbiao | <TheJulia> the modification is pretty clear. | 16:19 |
Qianbiao | i think it should be ok | 16:19 |
jtriley | hjensas: our please, thanks for the fix! | 16:21 |
jtriley | hjensas: let us know if you want us to test again after the re-factor | 16:22 |
Qianbiao | I did not understand which version dib fix this issue: https://bugs.launchpad.net/tripleo/+bug/1864427 | 16:22 |
openstack | Launchpad bug 1864427 in tripleo "CentOS 8 buildimage-ironic-python-agent job failing with Error: Unable to find a match: scsi-target-utils" [Critical,Fix released] | 16:22 |
Qianbiao | I have update dib to latest version, still come across same issue | 16:23 |
TheJulia | Qianbiao: looks like because it was fixed only for RDO | 16:25 |
TheJulia | hmm | 16:25 |
TheJulia | the ironic-agent element in diskimage-builder is also deprecated if memory serves | 16:25 |
TheJulia | in favor of ironic-python-agent-builder | 16:25 |
Qianbiao | <TheJulia> I did not use ironic-agent, i build iamge like "disk-image-create centos baremetal ramdisk mellanox devuser openssh-server -o centos-8-ramdisk" | 16:26 |
Qianbiao | I thought the same issue should have be fixed in dib. | 16:26 |
Qianbiao | because ipa-builder copy code from dib if i am right. | 16:26 |
TheJulia | yeah, they only fixed it for rdo becaues it was filed against tripleo | 16:27 |
Qianbiao | :( | 16:27 |
TheJulia | I agree, it should have been fixed in dib | 16:27 |
*** dougsz has quit IRC | 16:30 | |
TheJulia | the google image search built into slides returns some _weird_ results | 16:33 |
jtriley | hjensas: the introspection succeeded with that in place which is great. question: when it goes to post the introspection report does the inspector require the request to come from the MAC being introspected? | 16:40 |
Qianbiao | TheJulia can i just update locate 'tgt': 'scsi-target-utils' to tgt': 'targetcli' to fix it? | 16:42 |
hjensas | jtriley: not check of the source mack, that would'nt work in a l3 network. | 16:43 |
TheJulia | Qianbiao: I think so yes | 16:43 |
Qianbiao | will try ty | 16:43 |
TheJulia | just upload the change to diskimage-builder :) | 16:43 |
Qianbiao | :) ok will try first | 16:43 |
*** dtantsur is now known as dtantsur|afk | 16:43 | |
dtantsur|afk | o/ | 16:43 |
*** gouthamr has joined #openstack-ironic | 16:44 | |
hjensas | jtriley: I don't think there is any security measures like that in place, maby TheJulia can correct me? | 16:44 |
TheJulia | no, it does not require it to come from anywhere specific since it is basically stateless data collection on what is considered a trusted network. | 16:44 |
TheJulia | now, now that I think about it, I _think_ it actually looks at the data and doesn't post to a specific endpoint to determine what to do with it | 16:45 |
TheJulia | since it looks at node cache and mappings to figure out what to do | 16:45 |
arne_wiebalck | bye everyone o/ | 16:46 |
Qianbiao | bye arne_wiebalck | 16:48 |
Qianbiao | TheJulia new issue "Error: Unable to find a match: busybox | 16:49 |
Qianbiao | " .. | 16:49 |
Qianbiao | it seems dib has bugs with centos 8 | 16:49 |
Qianbiao | Not sure how many. | 16:49 |
TheJulia | Qianbiao: are you running this command on a centos machine? | 16:54 |
Qianbiao | no | 16:54 |
Qianbiao | a ubuntu 18. must I? | 16:54 |
*** martalais has joined #openstack-ironic | 16:55 | |
openstackgerrit | Merged openstack/ironic stable/train: Allow configuring IPMI cipher suite https://review.opendev.org/752710 | 16:59 |
*** derekh has quit IRC | 17:01 | |
*** priteau has quit IRC | 17:02 | |
openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent stable/train: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/752065 | 17:04 |
TheJulia | Qianbiao: yeah, you need to be on centos. Centos really only builds on centos machines, ubuntu/debian is kind of similar | 17:05 |
Qianbiao | ok thanks. | 17:05 |
TheJulia | \o/ something merged to stable/train! | 17:05 |
Qianbiao | TheJulia the topic is still as it is. | 17:05 |
TheJulia | trandles: janders: I think my slides are good, please sanity check :) | 17:05 |
TheJulia | trandles: also, trinity's water cooling install is in the deck now | 17:06 |
TheJulia | Qianbiao: looking | 17:17 |
TheJulia | Qianbiao: what do you mean about topic, I'm slightly confused | 17:17 |
Qianbiao | TheJulia ty, the patch's code look good. all the same | 17:17 |
TheJulia | Qianbiao: excellent! | 17:20 |
Qianbiao | :) ty | 17:21 |
TheJulia | Qianbiao: also you backporting that patch helps provide us visibility into which branches we may need to fix CI jobs on | 17:28 |
trandles | TheJulia, janders I'll take a look in a little bit, and also wrap up my own. Been in meetings all morning...sigh... | 17:29 |
Qianbiao | <TheJulia> XD .. seems it will be a long story before merge. | 17:29 |
TheJulia | Qianbiao: It may. I've already approved one of them. :) One step at a time. | 17:31 |
Qianbiao | nice | 17:32 |
*** bdodd has joined #openstack-ironic | 17:36 | |
*** zaneb has quit IRC | 17:37 | |
*** zaneb has joined #openstack-ironic | 17:38 | |
*** jtomasek has quit IRC | 17:41 | |
Qianbiao | TheJulia on centos, dib fails also. | 17:43 |
Qianbiao | "No match for argument: busybox", I googled, busybox does not package for centos | 17:43 |
Qianbiao | i will try dracut-ramdisk instead of ramdisk | 17:45 |
*** uzumaki has quit IRC | 17:48 | |
*** Qianbiao has quit IRC | 17:53 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Add Redfish BIOS interface to idrac HW type https://review.opendev.org/749240 | 17:55 |
*** Qianbiao has joined #openstack-ironic | 17:56 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Update tests for Redfish BIOS apply_configuration https://review.opendev.org/753694 | 18:00 |
*** Qianbiao has quit IRC | 18:00 | |
*** martalais has quit IRC | 18:03 | |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753470 | 18:04 |
*** zzzeek has quit IRC | 18:07 | |
*** zzzeek has joined #openstack-ironic | 18:10 | |
TheJulia | additional core reviews on https://review.opendev.org/#/c/750204/10 would be much appreciated | 18:18 |
patchbot | patch 750204 - ironic - Deprecate the iscsi deploy interface - 10 patch sets | 18:18 |
*** chenhaw has joined #openstack-ironic | 18:19 | |
dking | When running the clean step "erase_devices", at what point are the partitions erased? | 18:25 |
openstackgerrit | Merged openstack/bifrost master: Ensure that repositories are owned by the running users https://review.opendev.org/752473 | 18:26 |
openstackgerrit | Merged openstack/bifrost master: bifrost-cli: do not provide any release if non can be detected https://review.opendev.org/752658 | 18:26 |
openstackgerrit | Merged openstack/bifrost master: Switch integration testing to Ubuntu Focal https://review.opendev.org/753553 | 18:26 |
dking | I know that when I run the step, I end up without partitions, but when I'm looking at the code, or if I try to run the method directly, I still have partitions. | 18:26 |
rpioso | TheJulia: Here's an easy one: https://review.opendev.org/#/c/748927/. You previously Workflow+1 it. It did not get through the gate, because it needed to be rebased. It has been rebased. | 18:29 |
patchbot | patch 748927 - sushy - Make message parsing more resilient - 4 patch sets | 18:29 |
openstackgerrit | Merged openstack/bifrost master: Move CLI validation to bifrost-keystone-client-config https://review.opendev.org/752466 | 18:30 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/753818 | 18:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/753819 | 18:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/victoria https://review.opendev.org/753820 | 18:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal master: Add Python3 wallaby unit tests https://review.opendev.org/753821 | 18:34 |
TheJulia | dking: erasing the base devices wipes the partition tables out | 18:37 |
TheJulia | rpioso: awesome | 18:38 |
dking | TheJulia: I've run hardware.GenericHardwareManager()._ata_erase() multiple times, and the partitions are still there. | 18:40 |
TheJulia | dking: have you rebooted? | 18:40 |
dking | No, I have not. I suppose that could make sense. The partition is gone, but the OS doesn't know about it yet? | 18:41 |
*** uzumaki has joined #openstack-ironic | 18:49 | |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/753825 | 18:55 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/753826 | 18:55 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/victoria https://review.opendev.org/753827 | 18:55 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch master: Add Python3 wallaby unit tests https://review.opendev.org/753828 | 18:55 |
TheJulia | dking: exactly | 18:57 |
TheJulia | you can use kpartx to update the cache | 18:58 |
dking | Thanks! I just thought I was missing something. I think was getting confused because I thought hm.list_block_devices was being called with include_partitions. | 18:59 |
*** uzumaki has quit IRC | 19:02 | |
TheJulia | I think that just lists the kernel recognized data | 19:13 |
* TheJulia goes and lays down for a little bit | 19:13 | |
*** tosky has joined #openstack-ironic | 19:15 | |
*** jtriley has quit IRC | 19:20 | |
openstackgerrit | Merged openstack/ironic-inspector master: Power off before inspection https://review.opendev.org/749845 | 19:34 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/753720 | 19:34 |
*** adrianc_ has quit IRC | 19:35 | |
*** adrianc has joined #openstack-ironic | 19:35 | |
openstackgerrit | Merged openstack/sushy master: Make message parsing more resilient https://review.opendev.org/748927 | 19:46 |
openstackgerrit | Verification of a change to openstack/ironic failed: Use correct error handler instead of calling process_event('fail') https://review.opendev.org/753595 | 19:50 |
openstackgerrit | Julia Kreger proposed openstack/sushy stable/victoria: Make message parsing more resilient https://review.opendev.org/753837 | 19:52 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 19:55 |
*** stevebaker has joined #openstack-ironic | 20:01 | |
stevebaker | morning | 20:03 |
openstackgerrit | Merged openstack/bifrost master: Add some interfaces for development hardware types https://review.opendev.org/753213 | 20:29 |
iurygregory | morning stevebaker o/ | 20:30 |
stevebaker | iurygregory: hey! | 20:30 |
iurygregory | too early for you no? | 20:30 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't update the grubenv https://review.opendev.org/752308 | 20:31 |
stevebaker | iurygregory: its 8:30am not too early | 20:33 |
iurygregory | I thought it was 7:30 | 20:34 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 20:34 |
TheJulia | good morning stevebaker | 20:37 |
TheJulia | Any luck with the first patch of the api change series in CI? | 20:37 |
*** yolanda has quit IRC | 20:37 | |
stevebaker | TheJulia: good afternoon, its 1337 oclock | 20:37 |
TheJulia | how leet | 20:38 |
* TheJulia suspects stevebaker is in a bunker with a neat world clock :) | 20:38 | |
stevebaker | TheJulia: https://review.opendev.org/#/c/742795 is passing zuul ci, just not all in the same run. only ironic-standalone-redfish failed the last recheck | 20:40 |
patchbot | patch 742795 - ironic - New argument validate decorator - 13 patch sets | 20:40 |
TheJulia | stevebaker: did you find any reason why? | 20:40 |
openstackgerrit | Merged openstack/sushy stable/victoria: Make message parsing more resilient https://review.opendev.org/753837 | 20:44 |
stevebaker | TheJulia: test_ip_access_to_server failed, but that job passed in the previous run. That first 5 changes in that series are all new code which isn't used until change 6, so functional test failures are unlikely to be related | 20:45 |
TheJulia | stevebaker: did you check the console logs to at least see if the instance booted? | 20:49 |
*** phrobb has quit IRC | 20:49 | |
*** vkmc has quit IRC | 20:49 | |
*** rajinir has quit IRC | 20:49 | |
stevebaker | TheJulia: I'll dig into that failure today | 20:50 |
*** sri_ has quit IRC | 20:50 | |
*** buhman has quit IRC | 20:50 | |
*** rpittau|afk has quit IRC | 20:50 | |
*** gmann has quit IRC | 20:50 | |
*** mnasiadka has quit IRC | 20:50 | |
TheJulia | k, either the vm didn't boot up or neutron networking blew up | 20:50 |
*** ildikov has quit IRC | 20:51 | |
*** NobodyCam has quit IRC | 20:51 | |
*** rpioso has quit IRC | 20:51 | |
*** vkmc has joined #openstack-ironic | 20:51 | |
*** phrobb has joined #openstack-ironic | 20:51 | |
*** arne_wiebalck has quit IRC | 20:51 | |
*** PrinzElvis has quit IRC | 20:51 | |
*** erbarr has quit IRC | 20:51 | |
*** sri_ has joined #openstack-ironic | 20:52 | |
*** vdrok has quit IRC | 20:52 | |
*** NobodyCam has joined #openstack-ironic | 20:52 | |
*** zzzeek has quit IRC | 20:52 | |
*** mnasiadka has joined #openstack-ironic | 20:52 | |
*** vdrok has joined #openstack-ironic | 20:52 | |
*** rpioso has joined #openstack-ironic | 20:52 | |
*** PrinzElvis has joined #openstack-ironic | 20:52 | |
*** ildikov has joined #openstack-ironic | 20:52 | |
*** buhman has joined #openstack-ironic | 20:52 | |
*** rajinir has joined #openstack-ironic | 20:53 | |
*** arne_wiebalck has joined #openstack-ironic | 20:53 | |
*** erbarr has joined #openstack-ironic | 20:53 | |
*** gmann has joined #openstack-ironic | 20:53 | |
*** rpittau|afk has joined #openstack-ironic | 20:53 | |
*** zzzeek has joined #openstack-ironic | 20:56 | |
dking | Does GenericHardwareManager happen to support cleaning SCSI, other than shred? I have some SCSI spinners which should have Instant Secure Erase, but it looks like all that's attempted is hdparm and then shred? | 21:09 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 21:26 |
openstackgerrit | Verification of a change to openstack/ironic failed: Adding changes for iso less vmedia support https://review.opendev.org/752001 | 21:32 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: WIP Add fstab pointer to EFI partition https://review.opendev.org/748260 | 21:33 |
openstackgerrit | Merged openstack/ironic stable/train: CI: Patch to fix CI jobs in bugfix/stable branches https://review.opendev.org/753471 | 21:37 |
openstackgerrit | Merged openstack/ironic-python-agent master: Document ramdisk TLS and update existing TLS docs https://review.opendev.org/753645 | 21:37 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 21:38 |
rpioso | TheJulia: When you have a chance, we would greatly appreciate your further review of https://review.opendev.org/749619. | 21:55 |
patchbot | patch 749619 - ironic - Redfish driver firmware update - 21 patch sets | 21:55 |
trandles | TheJulia: janders: your slides look good to me | 21:56 |
trandles | I'm working on mine now... | 21:56 |
TheJulia | umm the follow-up? | 21:58 |
TheJulia | trandles: awesome | 21:58 |
*** JamesBenson has quit IRC | 21:59 | |
TheJulia | rpioso: okay, thats a different one | 22:00 |
openstackgerrit | Merged openstack/ironic-python-agent master: When reporting that agent is busy, report the executed command https://review.opendev.org/752719 | 22:00 |
openstackgerrit | Merged openstack/ironic master: Update tests for Redfish BIOS apply_configuration https://review.opendev.org/753694 | 22:00 |
*** JamesBenson has joined #openstack-ironic | 22:00 | |
TheJulia | rpioso: I'll look at it a little later today. I've got a pile of stuff to take care of today | 22:00 |
*** phrobb has quit IRC | 22:01 | |
rpioso | TheJulia: Thank you so much! I can only imagine. | 22:02 |
*** phrobb has joined #openstack-ironic | 22:03 | |
rpioso | TheJulia: I believe you looked at that one last Friday, but your caffeine level was waning. Hopefully, it'll look familiar. | 22:04 |
TheJulia | I should actually make caffeine | 22:04 |
TheJulia | and pizza douh | 22:04 |
TheJulia | dough | 22:04 |
*** JamesBenson has quit IRC | 22:05 | |
*** k_mouza has joined #openstack-ironic | 22:20 | |
*** k_mouza has quit IRC | 22:24 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Remove unused jobs https://review.opendev.org/749725 | 22:32 |
*** zzzeek has quit IRC | 22:35 | |
*** JamesBenson has joined #openstack-ironic | 22:37 | |
*** zzzeek has joined #openstack-ironic | 22:37 | |
*** k_mouza has joined #openstack-ironic | 22:39 | |
*** JamesBenson has quit IRC | 22:41 | |
*** k_mouza has quit IRC | 22:44 | |
*** zzzeek has quit IRC | 22:45 | |
*** rcernin has joined #openstack-ironic | 22:46 | |
*** zzzeek has joined #openstack-ironic | 22:46 | |
openstackgerrit | Merged openstack/ironic-inspector master: Add NVIDIA V100 to known accelerator devices https://review.opendev.org/751660 | 22:49 |
*** rpioso has quit IRC | 22:50 | |
*** PrinzElvis has quit IRC | 22:50 | |
*** sri_ has quit IRC | 22:50 | |
*** ildikov has quit IRC | 22:50 | |
*** v1k0d3n has quit IRC | 22:50 | |
*** phrobb has quit IRC | 22:50 | |
*** erbarr has quit IRC | 22:50 | |
*** bbezak has quit IRC | 22:50 | |
*** arne_wiebalck has quit IRC | 22:50 | |
*** johnsom has quit IRC | 22:50 | |
*** andrein has quit IRC | 22:50 | |
*** m1p has quit IRC | 22:50 | |
*** mnasiadka has quit IRC | 22:50 | |
*** vdrok has quit IRC | 22:50 | |
*** portdirect has quit IRC | 22:50 | |
*** mnaser has quit IRC | 22:50 | |
*** NobodyCam has quit IRC | 22:51 | |
*** vkmc has quit IRC | 22:51 | |
*** erbarr has joined #openstack-ironic | 22:51 | |
*** bbezak has joined #openstack-ironic | 22:51 | |
*** v1k0d3n has joined #openstack-ironic | 22:51 | |
*** sri_ has joined #openstack-ironic | 22:51 | |
*** arne_wiebalck has joined #openstack-ironic | 22:51 | |
*** PrinzElvis has joined #openstack-ironic | 22:51 | |
*** ildikov has joined #openstack-ironic | 22:51 | |
*** rpioso has joined #openstack-ironic | 22:51 | |
*** m1p has joined #openstack-ironic | 22:52 | |
*** andrein has joined #openstack-ironic | 22:52 | |
*** seongsoocho has quit IRC | 22:52 | |
*** seongsoocho has joined #openstack-ironic | 22:52 | |
*** phrobb has joined #openstack-ironic | 22:53 | |
*** mnaser has joined #openstack-ironic | 22:53 | |
*** rpittau|afk has quit IRC | 22:54 | |
*** vdrok has joined #openstack-ironic | 22:54 | |
*** portdirect has joined #openstack-ironic | 22:54 | |
*** mnasiadka has joined #openstack-ironic | 22:54 | |
*** johnsom has joined #openstack-ironic | 22:55 | |
*** rpittau|afk has joined #openstack-ironic | 22:56 | |
*** vkmc has joined #openstack-ironic | 22:56 | |
*** andrein has quit IRC | 22:58 | |
*** NobodyCam has joined #openstack-ironic | 22:59 | |
*** andrein has joined #openstack-ironic | 23:00 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 23:02 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Change integration test for Metalsmith to use Centos8/UEFI https://review.opendev.org/749772 | 23:02 |
*** tkajinam has joined #openstack-ironic | 23:03 | |
*** zzzeek has quit IRC | 23:04 | |
TheJulia | looks like we need to drop the test concurrency on the multinode job :\ | 23:05 |
*** zzzeek has joined #openstack-ironic | 23:06 | |
stevebaker | TheJulia: regarding the ironic-standalone-redfish failure, the node deploys, I found this wrong client_id https://zuul.opendev.org/t/openstack/build/ee0efe9616b04dc384aa3f4d0ca62168/log/controller/logs/screen-q-dhcp.txt#4142 | 23:07 |
stevebaker | TheJulia: 'client_id': '01:52:54:00:f5:77:63' becomes 0 1 : 5 2 : 5 4 : 0 0 : f 5 : 7 7 : 6 3 in the cli argument | 23:07 |
trandles | TheJulia: janders my slides are ready for review I think. I'm not thrilled, feedback very welcome. | 23:10 |
* TheJulia blinks | 23:10 | |
TheJulia | trandles: why not thrilled?!? | 23:11 |
trandles | eh...I'm rarely thrilled with my slides | 23:12 |
TheJulia | stevebaker: what the... | 23:12 |
TheJulia | stevebaker: maybe make some noise in the neutron channel?!? | 23:12 |
* TheJulia hopes this is not the curse of the end of cycle release | 23:12 | |
*** k_mouza has joined #openstack-ironic | 23:13 | |
* trandles is closing this laptop for the night... | 23:13 | |
TheJulia | goodnight! | 23:14 |
trandles | nighty night :) | 23:14 |
stevebaker | TheJulia: I'll keep digging for a bit before I ask them | 23:14 |
TheJulia | stevebaker: that is just bizzar.... | 23:15 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: CI: Tune down multinode concurrency https://review.opendev.org/753863 | 23:16 |
stevebaker | TheJulia: its like client_id is assumed to be a list, and is joined into a string with a space | 23:16 |
*** k_mouza has quit IRC | 23:18 | |
TheJulia | rpioso: that firmware update patch is likely going to need another revision, working on comments now | 23:22 |
rpioso | TheJulia: Thank you so much. We look forward to your feedback. | 23:23 |
rpioso | cdearborn: ^^^ | 23:24 |
stevebaker | TheJulia: boom http://paste.openstack.org/show/798297/ | 23:32 |
TheJulia | rpioso: basically looking like the periodic. mainly in 20 seconds is super aggressive | 23:33 |
janders | good morning Ironic o/ | 23:34 |
*** rcernin has quit IRC | 23:34 | |
*** rcernin has joined #openstack-ironic | 23:34 | |
rpioso | TheJulia: If memory serves, cdearborn has reported it's quickly completed. | 23:37 |
rpioso | TheJulia: But, I'll defer to him. | 23:37 |
rpioso | TheJulia: Oh! And I think each type of firmware -- iDRAC, BIOS, RAID, NIC, etc. -- has to be applied one-by-one. | 23:39 |
*** zzzeek has quit IRC | 23:41 | |
*** zzzeek has joined #openstack-ironic | 23:45 | |
TheJulia | rpioso: conundrum is we're getting clusters with hundreds and thousands of nodes | 23:45 |
TheJulia | so... yeah. :\ | 23:45 |
TheJulia | I'm still typing notes out, hopefully it will be fully clear and cdearborn if you want to chat tomorrow I can make some time | 23:45 |
rpioso | TheJulia: Love the conundrum! ironic Redfish firmware update automation might be helpful :-) | 23:47 |
rpioso | TheJulia: That would be awesome if you could make time. Wanna get something on our calendars? | 23:48 |
*** rloo has quit IRC | 23:48 | |
TheJulia | rpioso: after 9:30 am pacific I'm free until noon pacific. | 23:49 |
TheJulia | rpioso: cdearborn comments posted | 23:49 |
*** paulbrowne has joined #openstack-ironic | 23:49 | |
*** tosky has quit IRC | 23:49 | |
rpioso | TheJulia: My calendar is clear during that period. BlueJeans? | 23:50 |
TheJulia | sure, just send me an meeting invite to my rh address | 23:50 |
openstackgerrit | Verification of a change to openstack/ironic failed: Use correct error handler instead of calling process_event('fail') https://review.opendev.org/753595 | 23:52 |
rpioso | TheJulia: Will do! cdearborn shows as being on line, but he's not responding to IMs. I'll propose 11 am-noon your time. We can take it from there. | 23:53 |
rpioso | TheJulia: Sent | 23:56 |
TheJulia | rpioso: ack | 23:56 |
*** paulbrowne has quit IRC | 23:56 | |
rpioso | TheJulia: Please feel free to invite others. | 23:57 |
TheJulia | rpioso: I don't see it yet but I've blocked the time out on my calendar if it doesn't appear before I forget | 23:58 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!