Wednesday, 2020-09-23

*** zzzeek has quit IRC00:04
*** zzzeek has joined #openstack-ironic00:05
*** zzzeek has quit IRC00:47
*** zzzeek has joined #openstack-ironic00:49
*** zigo has quit IRC00:53
*** zzzeek has quit IRC01:01
openstackgerritMerged openstack/ironic master: Make the standalone-redfish job voting  https://review.opendev.org/75337401:01
*** zzzeek has joined #openstack-ironic01:02
*** Goneri has quit IRC01:10
openstackgerritGhanshyam Mann proposed openstack/ironic master: migrate testing to ubuntu focal  https://review.opendev.org/73944801:18
*** gyee has quit IRC01:33
*** cdearborn has quit IRC01:45
*** zzzeek has quit IRC02:13
*** zzzeek has joined #openstack-ironic02:15
*** tkajinam_ has joined #openstack-ironic02:17
*** zigo has joined #openstack-ironic02:26
*** tkajinam has quit IRC02:27
*** johnsom has quit IRC02:27
*** ajya has quit IRC02:27
*** Qianbiao has joined #openstack-ironic02:33
*** johnsom has joined #openstack-ironic02:33
*** ajya has joined #openstack-ironic02:33
openstackgerritVerification of a change to openstack/ironic failed: Limit inspector jobs to 1 testing VM  https://review.opendev.org/75309402:41
*** stevebaker has quit IRC03:16
*** stevebaker has joined #openstack-ironic03:45
*** ociuhandu has joined #openstack-ironic03:47
*** ociuhandu has quit IRC03:52
*** tzumainn has quit IRC03:58
openstackgerritSteve Baker proposed openstack/ironic master: Convert nodes endpoint to plain JSON  https://review.opendev.org/74782004:01
openstackgerritSteve Baker proposed openstack/ironic master: Convert ports endpoint to plain JSON  https://review.opendev.org/75012004:01
openstackgerritSteve Baker proposed openstack/ironic master: Convert portgroups endpoint to plain JSON  https://review.opendev.org/75086404:01
openstackgerritSteve Baker proposed openstack/ironic master: Convert ramdisk endpoint to plain JSON  https://review.opendev.org/75085704:02
openstackgerritSteve Baker proposed openstack/ironic master: Convert volume endpoint to plain JSON  https://review.opendev.org/75085804:02
openstackgerritSteve Baker proposed openstack/ironic master: Convert volume/connectors endpoint to plain JSON  https://review.opendev.org/75085904:02
openstackgerritSteve Baker proposed openstack/ironic master: Convert volume/targets endpoint to plain JSON  https://review.opendev.org/75086004:02
openstackgerritSteve Baker proposed openstack/ironic master: Expunge the internal version of WSME  https://review.opendev.org/75086104:02
*** tbonds has quit IRC04:16
*** tbonds has joined #openstack-ironic04:18
*** uzumaki has joined #openstack-ironic04: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 upload04:45
*** stevebaker has quit IRC04:54
uzumakimorning ironic! o/05:06
*** cp- has quit IRC05:37
*** cp- has joined #openstack-ironic05:42
*** abdysn has joined #openstack-ironic05:50
arne_wiebalckGood morning, ironic!05:54
iurygregorygood morning uzumaki arne_wiebalck and Ironic o/06:08
arne_wiebalckhey iurygregory o/06:09
uzumakiheyyyy everybody! iurygregory arne_wiebalck how's the morning? (assuming it's morning there)06:09
iurygregoryit's morning06:11
iurygregoryI need coffee06:11
*** ianychoi_ is now known as ianychoi06:16
*** cp- has quit IRC06:23
QianbiaoMorning Ironic。06:27
Qianbiaoarne_wiebalck send you a feedback.06:28
iurygregorymorning Qianbiao06:28
*** cp- has joined #openstack-ironic06:29
QianbiaoHey boy o/ iurygregory06:29
iurygregoryo/06:29
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/6.3: CI: Lower memory usage of VMs/Increase swap  https://review.opendev.org/75346006:37
arne_wiebalckQianbiao: yes, you're right ... I replied (and still would like a reno :-)06:41
Qianbiaoarne_wiebalck sure06:43
Qianbiaomay i add it at fellow commit? let this one pass first.06:43
arne_wiebalckthere are plans to backport this, no?06:45
Qianbiaoyes06:46
Qianbiaoneed reno backport too?06:46
iurygregoryyes06:46
arne_wiebalckthat is what I am thinking06:46
QianbiaoI thought stable branch does not need that06:46
Qianbiaook06:46
arne_wiebalckI think it'd be easier to add it now06:47
Qianbiaook06:47
iurygregorywell depends if you want to backport, but I would say a reno is necessary06:47
arne_wiebalckotherwise I am +2 on the change :)06:47
iurygregorythe change impacts the operator no?06:47
Qianbiaoiurygregory no, no behavior changes.06:48
iurygregoryoh ok =)06:48
QianbiaoBut fix a bug06:48
iurygregoryhummm06:48
QianbiaoI think should no one have use the infiniband hardware with Ironic before.06:49
Qianbiaoarne_wiebalck we should break the patch into two at first.06:59
Qianbiaoone for inherit issue, one for CNA card disable lldp agent.06:59
*** jtomasek has joined #openstack-ironic07:00
arne_wiebalckand you would only backport the disable part?07:01
Qianbiaoyes07:02
Qianbiaono, i will backport the inherit07:02
Qianbiaopart not disable part.07:02
Qianbiaoinherit part fixes the dispatch mechanism issue. the disable part is another topic.07:03
arne_wiebalckStrictly 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_wiebalckDid anyone suggest to split it?07:07
Qianbiaonope... i do not want to split it now ..07:09
QianbiaoI mean if at first.07:09
Qianbiao:)07:09
arne_wiebalckIn that case I would suggest to add a reno and see how it goes :-D07:10
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75202407:10
Qianbiao<arne_wiebalck> ^^^07:10
* arne_wiebalck sees that Qianbiao can add a reno in 7 seconds!07:10
Qianbiao:)07:11
Qianbiao6 seconds.07:11
QianbiaoIt's a english test for me to write so long a paragraph07:13
* arne_wiebalck has a meeting now, will check after07:14
Qianbiaosure thanks.07:14
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/6.2: CI: Lower memory usage of VMs/Increase swap  https://review.opendev.org/75346107:17
*** tosky has joined #openstack-ironic07:19
*** dougsz has joined #openstack-ironic07:30
iurygregorymgoddard, hey if you have some time this week https://review.opendev.org/#/c/752042/ o/07:33
patchbotpatch 752042 - bifrost - Add ipe installation to bifrost - 4 patch sets07:33
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75202407:37
*** yolanda has joined #openstack-ironic07:42
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic bugfix/15.1: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75346907:44
*** rcernin has quit IRC07:45
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic bugfix/15.2: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75346807:45
*** ociuhandu has joined #openstack-ironic07:47
*** priteau has joined #openstack-ironic07:50
*** patchbot has quit IRC07:57
*** patchbot has joined #openstack-ironic07:59
*** dtantsur|afk is now known as dtantsur08:03
dtantsurmorning ironic, says a man trying to open his eyes08:04
iurygregorymorning dtantsur o/08:04
mgoddardiurygregory: argh ipe ipa ipxe08:06
mgoddardiurygregory: I'll put it on my list, but busy this morning08:06
iurygregorymgoddard, sorry for the acronym =)08:08
*** belmoreira has joined #openstack-ironic08:12
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector bugfix/10.3: Limit inspector jobs to 1 testing VM  https://review.opendev.org/75327308:12
*** lucasagomes has joined #openstack-ironic08:12
*** ociuhandu has quit IRC08:13
*** ociuhandu has joined #openstack-ironic08:13
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector bugfix/10.2: Limit inspector jobs to 1 testing VM  https://review.opendev.org/75346308:13
iurygregorydtantsur, great news docs job should be fine08:18
*** dtantsur has quit IRC08:18
iurygregorywe didn't have to merge the change to disable pdf08:19
*** ociuhandu has quit IRC08:20
*** dtantsur has joined #openstack-ironic08:20
dtantsuriurygregory: some ipe comments inline08:22
iurygregorydtantsur, ack08:23
*** ociuhandu has joined #openstack-ironic08:25
iurygregorydtantsur, instead of ipe_* ironicprometheusexporter_* | ironic_prometheus_exporter_* | prometheus_exporter_* ?08:29
dtantsuriurygregory: I prefer the last one, the 2nd one works as well08:29
dtantsurI'd rather not do toomanywordswithoutasinglebreak :)08:29
iurygregoryi've put the first only because of ironicinspectorclient_git_folder....08:30
*** Lucas_Gray has joined #openstack-ironic08:31
dtantsuryeah, I don't like it too, but changing it is a pain08:31
iurygregoryI can imagine...08:32
*** tkajinam_ is now known as tkajinam08:34
*** derekh has joined #openstack-ironic08:41
iurygregorydtantsur, I've update some of TheJulia patches on the bugfix branches since we had the problem with the l-c jobs08:42
dtantsurcool. I'll try to get to them when I wake up08:42
iurygregoryit's the time for CI to finish running XD08:42
*** ociuhandu has quit IRC08:45
*** Qianbiao has quit IRC08:46
*** Qianbiao has joined #openstack-ironic08:47
*** k_mouza has joined #openstack-ironic08:50
iurygregorydtantsur, 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
jandersgood morning Ironic o/08:54
dtantsuriurygregory: no objections as long as they don't become too heavy (which I doubt)08:54
dtantsurgood evening janders :)08:54
iurygregorymorning janders o/08:55
iurygregorydtantsur, 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
dtantsuriurygregory: we can think about it later, I guess? I'm only worried about testing the ability to install it08:56
*** priteau has quit IRC08:56
dtantsuralso half-joking: do we need to install prometheus as well? :)08:56
iurygregorydtantsur, I see08:57
iurygregoryno we don't need to install XD08:57
iurygregoryin the IPE CI we test that we can retrive the metrics from the endpoint08:57
dtantsuryeah, I know, but I wonder if this feature is complete without bifrost installing prometheus (probably yes)08:58
iurygregoryfor me it's08:58
iurygregorysince I run prometheus in a different server =)08:58
iurygregoryI only need to add the endpoint to the config08:58
dtantsuriurygregory: in your patch IPE listens on localhost, will it work this way?09:01
dtantsuror do we need to use internal_ip?09:01
iurygregorywell when deploying I set the ip in the group_vars .-.09:02
dtantsuriurygregory: maybe default to internal_ip then?09:02
dtantsurI don't quite like using group_vars.. for anything :)09:02
iurygregorycan be09:02
iurygregoryI will look at how to do that =)09:02
dtantsuriurygregory: oh, do we have http_basic auth for IPE?09:03
dtantsuror is it not prometheus-compatible?09:03
* dtantsur feels he should stop bombarding iurygregory with ideas09:03
iurygregoryis not necessary afaik09:03
iurygregoryin my tests I'm running ironic with http_basic09:03
dtantsurwell, ideally we should not allow anyone on the network to read metrics09:04
iurygregorymaybe we need to add something for the flask application to require the auth09:04
iurygregorymaybe something for the future of IPE it's important09:05
iurygregorydtantsur, you can continue bombarding me with ideas I don't mind =)09:05
dtantsurI think it's useful if prometheus can use basic auth09:05
dtantsurnot blocking this patch of course :)09:05
*** strigazi has joined #openstack-ironic09:06
iurygregoryack09:06
*** Abdallahyas has joined #openstack-ironic09:12
*** abdysn has quit IRC09:15
openstackgerritMerged openstack/ironic-inspector master: Correct the docstring for _should_enable_unknown_hosts  https://review.opendev.org/75343009:21
dtantsurTheJulia: 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
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Switch integration testing to Ubuntu Focal  https://review.opendev.org/75355309:37
dtantsurI guess we're supposed to do ^^^ before releasing victoria09:37
*** yolanda has quit IRC09:38
*** yolanda has joined #openstack-ironic09:39
iurygregoryyeah09:40
Qianbiaomay core give a workflow for this patch https://review.opendev.org/#/c/75202409:54
patchbotpatch 752024 - ironic-python-agent - Fix: make Intel CNA hardware manager none generic - 10 patch sets09:54
iurygregorydtantsur, internal_ip would be ansible_default_ipv4.address I assume right?10:03
*** Abdallahyas has quit IRC10:05
*** abdysn has joined #openstack-ironic10:05
openstackgerritAnkit Kumar proposed openstack/ironic master: Adding changes for iso less vmedia support  https://review.opendev.org/75200110:06
*** Lucas_Gray has quit IRC10:07
*** fgofurov has joined #openstack-ironic10:08
fgofurovHello ironic team10:09
fgofurovwe are facing an error during introspection10:11
fgofurov2020-09-23 06:16:54.272 1 ERROR oslo_messaging.rpc.server     'Could not find a node for attributes %s') % attributes)10:11
fgofurov2020-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
fgofurovit keeps throwing: AttributeError: 'Node' object has no attribute 'uuid'10:11
fgofurovany ideas on this?10:12
dtantsuriurygregory: I think internal_ip is already defined, it's a bit more complex than that10:13
dtantsurfgofurov: a complete traceback (actually, two tracebacks) could be useful10:14
dtantsurQianbiao: the release notes failure seems real - see inline10:15
fgofurov2020-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
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server Traceback (most recent call last):10:17
fgofurov2020-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_info10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     mac=macs)10:17
fgofurov2020-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_node10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     'Could not find a node for attributes %s') % attributes)10:17
fgofurov2020-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
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server During handling of the above exception, another exception occurred:10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server Traceback (most recent call last):10:17
fgofurov2020-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_incoming10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     res = self.dispatcher.dispatch(message)10:17
fgofurov2020-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 dispatch10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     return self._do_dispatch(endpoint, method, ctxt, args)10:17
fgofurov2020-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_dispatch10:17
Qianbiao<dtantsur> checking10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     result = func(ctxt, **new_args)10:17
fgofurov2020-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 inner10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     return func(*args, **kwargs)10:17
fgofurov2020-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_continue10:17
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     return process.process(data)10:18
fgofurov2020-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 process10:18
Qianbiaofgofurov do not paste log here.10:18
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     node_info = _find_node_info(introspection_data, failures)10:18
fgofurov2020-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_info10:18
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     macs=macs)10:18
fgofurov2020-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_node10:18
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     {'node': node.uuid,10:18
fgofurov2020-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
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     raise e10:18
fgofurov2020-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
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server     return object.__getattribute__(self, name)10:18
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server AttributeError: 'Node' object has no attribute 'uuid'10:18
fgofurov2020-09-22 07:46:53.780 1 ERROR oslo_messaging.rpc.server [00m10:18
fgofurov2020-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
fgofurov2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main Traceback (most recent call last):10:18
fgofurov2020-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_info10:18
fgofurov2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main     mac=macs)10:18
fgofurov2020-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_node10:18
fgofurov2020-09-22 07:46:53.782 1 ERROR ironic_inspector.main     'Could not find a node for attributes %s') % attributes)10:18
fgofurovsorry for flooding a chat10:18
iurygregoryfgofurov, try to use a paste tool please =)10:20
iurygregorydtantsur, /me checks10:20
fgofurovOkay, newbie to this, my apologies10:21
iurygregoryfgofurov, http://paste.openstack.org/10:21
iurygregoryfgofurov, no worries10:21
fgofurovthanks a lot!10:21
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75202410:24
Qianbiaodtantsur fixed10:25
*** ociuhandu has joined #openstack-ironic10:29
*** ociuhandu has quit IRC10:38
dtantsurfgofurov: 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
dtantsurthe fix was https://review.opendev.org/#/c/739730/10:41
patchbotpatch 739730 - ironic-inspector - Use node.id instead of node.uuid in record_node (MERGED) - 1 patch set10:41
dtantsuriurygregory: if you have a second, could you request an ironic-inspector ussuri release so that we have ^^ released?10:44
fgofurovWe are using ironic-inspector 10.1.0 btw10:45
iurygregorydtantsur, can I do after our meeting (I'm finishing lunch atm)10:45
dtantsuryeah, the fix will be in 10.1.110:45
dtantsuriurygregory: enjoy your lunch then, I'll request myself10:45
iurygregoryack10:45
dtantsurfgofurov: 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 often10:46
*** kashifkhan has joined #openstack-ironic10:48
kashifkhanHi, 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-ironic10:53
*** zzzeek has quit IRC10:55
dtantsurkashifkhan: 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 it10:56
dtantsurah, got it10:56
kashifkhanits kind of problematic for us, to uplift the image at this moment, so we were thinking of thereis any other workaround10:57
*** zzzeek has joined #openstack-ironic10:58
dtantsurI don't think so, it's a bug that has to be fixed in code10:59
kashifkhan@<dtantsur>ok, thanks10:59
fgofurovthanks a lot dtansur11:03
*** ociuhandu has joined #openstack-ironic11:05
*** zzzeek has quit IRC11:09
*** zzzeek has joined #openstack-ironic11:11
openstackgerritMerged openstack/ironic master: Limit inspector jobs to 1 testing VM  https://review.opendev.org/75309411:12
*** JamesBenson has joined #openstack-ironic11:21
openstackgerritDmitry Tantsur proposed openstack/ironic stable/ussuri: Limit inspector jobs to 1 testing VM  https://review.opendev.org/75358011:22
iurygregorydtantsur, anything else we need releases?11:40
iurygregorynow I can push some11:41
iurygregoryI know we can cut IPE stable/victoria11:41
iurygregorywondering if it's ok to wait for the backports for ussuri/train to get in so I push everything in one release patch11:41
dtantsuriurygregory: yep. also waiting with ironic because of the unmerged patches to ussuri (sigh). maybe you could check train?11:59
iurygregorydtantsur, sure11:59
*** tosky_ has joined #openstack-ironic12:02
*** tosky is now known as Guest7321912:03
*** tosky_ is now known as tosky12:03
*** fgofurov has quit IRC12:05
*** Guest73219 has quit IRC12:05
*** Goneri has joined #openstack-ironic12:06
janderssee you tomorrow Ironic o/12:09
jandershave a great day everyone12:09
iurygregorybye janders o/12:10
openstackgerritMahendra Kamble proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port  https://review.opendev.org/69332812:13
openstackgerritMerged openstack/ironic-python-agent master: Generate a TLS certificate and send it to ironic  https://review.opendev.org/74993012:14
iurygregory\o/12:15
iurygregorymerged!12:15
janders\o/12:15
janderswell done!12:15
iurygregorynow dtantsur is happy12:16
dtantsurw000t \o/12:27
openstackgerritShivanand Tendulker proposed openstack/ironic master: Use correct error handler instead of calling process_event('fail')  https://review.opendev.org/75359512:27
*** rloo has joined #openstack-ironic12:28
dtantsurnow I guess I should write some docs12:29
*** priteau has joined #openstack-ironic12:35
*** tkajinam has quit IRC12:51
*** tkajinam has joined #openstack-ironic12:52
openstackgerritDmitry Tantsur proposed openstack/ironic master: Expose agent_verify_ca in configuration and correctly handle booleans  https://review.opendev.org/75364113:04
dtantsura missing bit in the TLS story ^^^13:04
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal  https://review.opendev.org/75364213:05
*** Abdallahyas has joined #openstack-ironic13:06
* iurygregory fingers crossed for this one ^13:06
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Document ramdisk TLS and update existing TLS docs  https://review.opendev.org/75364513:08
dtantsurJayF: ^^13:08
dtantsurkashifkhan, ironic-inspector 10.1.1 has been released13:09
*** abdysn has quit IRC13:10
Qianbiaothanks you dtantsur i see the cna card hardware manager patch has started the gateway job.13:12
dtantsuryep, hopefully will land soon13:12
QianbiaoShould i wait it merged and then start cherry-pick?13:12
Qianbiaoor i can start cherry pick now.13:12
*** abdysn has joined #openstack-ironic13:12
dtantsurQianbiao: you can cherry-pick now, but note that gerrit UI will not add the necessary "cherry-picked from" line until it merges13:12
dtantsurso better cherry-pick it locally with `git cherry-pick -x`13:13
Qianbiaook13:13
Qianbiaowill wait :)13:13
*** Abdallahyas has quit IRC13:15
Qianbiaodtantsur no hurry, i can wait :)13:19
*** Goneri has quit IRC13:21
dtantsur:)13:21
openstackgerritAija Jaunteva proposed openstack/ironic master: Update tests for Redfish BIOS apply_configuration  https://review.opendev.org/75369413:25
*** tosky_ has joined #openstack-ironic13:26
*** tosky is now known as Guest1333613:26
*** tosky_ is now known as tosky13:26
*** Guest13336 has quit IRC13:28
*** tosky_ has joined #openstack-ironic13:29
*** ociuhandu_ has joined #openstack-ironic13:31
*** tosky is now known as Guest6102013:31
*** tosky_ is now known as tosky13:31
*** Guest61020 has quit IRC13:32
*** Goneri has joined #openstack-ironic13:34
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add Redfish BIOS interface to idrac HW type  https://review.opendev.org/74924013:34
*** ociuhandu has quit IRC13:35
openstackgerritAija Jaunteva proposed openstack/ironic master: Update tests for Redfish BIOS apply_configuration  https://review.opendev.org/75369413:38
ajyasorry dtantsur had to make a minor clean up ^ :)13:40
*** uzumaki has quit IRC13:40
*** ociuhandu_ has quit IRC13:41
*** ociuhandu has joined #openstack-ironic13:42
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal  https://review.opendev.org/75364213:45
*** tzumainn has joined #openstack-ironic13:46
*** cdearborn has joined #openstack-ironic13:54
dtantsurokay, I'll check again once the CI runas13:59
TheJuliagood morning14:03
dtantsurmorning TheJulia14:03
* TheJulia wipes sleep from her eyes14:04
TheJuliadtantsur: I hadn't started a full etherpad yet, only tried to collect items at the midcycle14:04
dtantsurwe can probably repurpose https://meetpad.opendev.org/etherpad/p/ironic-ptg14:05
TheJulia++14:05
*** tkajinam has quit IRC14:05
iurygregorygood morning TheJulia =)14:05
TheJuliaiurygregory: by chance did you look at the stable/ussuri job to see why it failed?14:15
TheJuliatalking about the patches to fix ci14:16
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/ussuri: CI: Lower memory usage of VMs/Increase swap  https://review.opendev.org/75346214:16
iurygregoryTheJulia, 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 see14:16
TheJuliawheeee14:17
iurygregorythe bugfix branches I've updated because we need to set l-c to bionic14:17
iurygregoryor update requirements...14:17
openstackgerritMerged openstack/ironic master: Fix redfish BIOS to use @Redfish.SettingsApplyTime  https://review.opendev.org/75261414:17
iurygregoryfor inspector bugfix .2 I've let a question in the patch14:18
TheJuliak14:18
TheJuliaugh, yeah, l-c changes were landed to bugfix branches yet14:18
iurygregorydo 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
iurygregoryhttps://review.opendev.org/75327714:20
patchbotpatch 753277 - ironic-inspector (bugfix/10.3) - Fix l-c and mysql GRANT syntex for Ubuntu Focal (ABANDONED) - 1 patch set14:20
TheJuliaok, that might be fine then14:20
openstackgerritMudit proposed openstack/ironic master: Implement reporting GPU devices in iDRAC driver  https://review.opendev.org/75100614:20
iurygregoryI'm ok with both approaches =)'14:20
TheJuliatechnically victoria artifiats are supposed to be good against focal14:22
iurygregoryhumm so we would need to bump14:22
TheJulianeed to bump the minor?14:23
iurygregorywell we also bump req14:24
iurygregorynot only l-c14:24
TheJuliaso in that case we only propose backports to lock the nodes to bionic? That is viable to me14:25
iurygregoryyeah in the bugfix branches only14:25
iurygregorythe master/victoria we have should be fine14:26
TheJuliayeah14:26
openstackgerritJulia Kreger proposed openstack/ironic stable/stein: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75370714:30
dtantsuryeah, for victoria we can make another minor release14:30
dtantsurfor bugfix branches it contradicts the very idea of bugfix branches :)14:30
TheJuliayeah14:31
openstackgerritJulia Kreger proposed openstack/ironic stable/train: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75347114:31
openstackgerritLeo McGann proposed openstack/ironic-specs master: Add security interface spec  https://review.opendev.org/57671814:34
TheJuliadtantsur: do you agree with bacporting https://review.opendev.org/#/c/752710/ ?14:34
patchbotpatch 752710 - ironic (stable/train) - Allow configuring IPMI cipher suite - 1 patch set14:34
iurygregoryI have the feeling it fits on our idea that we are making the operators life easier ...14:40
*** abdysn has quit IRC14:46
TheJuliathen +1 it (I really need to figure out what it will take to get the stable groups changed)14:47
iurygregorydone =)14:48
dtantsurTheJulia: I do, just never got to approving it14:54
openstackgerritIury Gregory Melo Ferreira proposed openstack/bifrost master: Add ipe installation to bifrost  https://review.opendev.org/75204214:54
iurygregorylet's see what CI will say14:54
iurygregorylocally it worked14:54
dtantsurTheJulia: I talked to ttx and we agreed that the situation with ipmitool falls under "regression" category14:55
dtantsur(which is also how consumers tend to perceive it)14:55
iurygregoryagree14:55
TheJuliaYeah, I remember remarking that you guys basically re-discussed what the TC already said was okay and worthwhile14:55
openstackgerritChristopher Dearborn proposed openstack/ironic master: Redfish driver firmware update  https://review.opendev.org/74961914:57
dtantsurTheJulia: yeah, I wanted to clarify the situation before it comes up on a release review14:57
openstackgerritMerged openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75202414:57
TheJulia++14:58
TheJuliayay an ipa patch merged14:58
iurygregorysecond14:59
iurygregoryTLS patch merged14:59
TheJuliasweet14:59
iurygregoryalmost a merge party from zuul today14:59
dtantsurapproved14:59
dtantsura missing bit from the TLS story: https://review.opendev.org/75364114:59
patchbotpatch 753641 - ironic - Expose agent_verify_ca in configuration and correc... - 1 patch set14:59
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent stable/ussuri: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75372015:00
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent stable/train: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75206515:03
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent stable/stein: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75373015:03
Qianbiaothose three branches i should backport, right?15:04
*** k_mouza has quit IRC15:04
arne_wiebalckdtantsur: you have a moment? there is a RAID cleaning issue and I'd like to discuss how to address it15:05
TheJuliaQianbiao:  depends on the patch :)15:05
dtantsuron a meeting, will ping you15:06
TheJuliaQianbiao: if you've got a link, I can give you an idea15:06
arne_wiebalcktks15:06
QianbiaoTheJulia https://review.opendev.org/75373015:06
patchbotpatch 753730 - ironic-python-agent (stable/stein) - Fix: make Intel CNA hardware manager none generic - 1 patch set15:06
openstackgerritLeo McGann proposed openstack/ironic-specs master: Add security interface spec  https://review.opendev.org/57671815:07
*** k_mouza has joined #openstack-ironic15:10
*** jtriley has joined #openstack-ironic15:22
jtrileyhjensas: 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 testing15:23
patchbotpatch 753435 - ironic-inspector - dnsmasq pxe filter - option to always block unknow... - 1 patch set15:23
jtrileyseems like just ironic_inspector - sorry for the noise15:25
hjensasjtriley: ironic_inspector only, I think ironic_inspector_dnsmasq is just the actual dnsmasq process?15:25
jtrileyhjensas: thanks makes sense15:28
jtrileyhjensas: I got the patch applied - should I podmount umount before bouncing the systemd service?15:30
hjensasjtriley: Should work without unmounting, but it does'nt hurt. :)15:31
*** VisBits has quit IRC15:32
jtrileyhjensas: and obviously I'll need to add that setting to the inspector.conf under /var/lib/config-data?15:33
*** k_mouza has quit IRC15:38
jtrileyhjensas: ok looks like the patch and config inside the container look as they should after restart - time to test :D15:38
hjensasjtriley: yes, setting in /var/lib/config-data is bind mounted in the container.15:40
*** stbenjam has joined #openstack-ironic15:43
openstackgerritVerification of a change to openstack/ironic failed: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75347015:44
*** gyee has joined #openstack-ironic15:45
*** ociuhandu has quit IRC15:49
arne_wiebalckdtantsur: I will have to leave shortly, sorry. I have summarised the issue and the options I see here: https://storyboard.openstack.org/#!/story/200818615:50
arne_wiebalckTheJulia: ^^15:50
dtantsurthx!15:50
openstackgerritMerged openstack/ironic bugfix/15.2: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75346815:51
openstackgerritMerged openstack/ironic bugfix/15.1: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75346915:51
*** uzumaki has joined #openstack-ironic15:55
jtrileyhjensas: looks like the unknown_hosts_filter is still getting the ";ignore" stripped so both nics are still getting an IP15:56
jtrileyhjensas: taking a look at the dnsmasq container in case it needs the patch as well15:56
*** belmoreira has quit IRC15:57
*** belmoreira has joined #openstack-ironic15:59
*** tosky has quit IRC15:59
*** belmoreira has quit IRC16:00
*** ociuhandu has joined #openstack-ironic16:01
*** lucasagomes has quit IRC16:04
TheJuliaQianbiao: re the patch, sorry I thought I had hit enter16:05
TheJuliaYes, stable/ussuri, stable/train <-- should be good enough for minimally, you can also backport to the two bugfix branches if you want16:06
Qianbiaoyou mean rocky and queens?16:07
*** ociuhandu has quit IRC16:07
hjensasjtriley: strange, you have deny_unknown_macs = true in the dnsmasq_pxe_filter section of ironic inspector conf?16:07
jtrileyhjensas: blah, I put the config setting in the wrong spot16:07
jtrileyhjensas: haha yup that's it16:07
jtrileyretrying16:07
QianbiaoTheJulia np, i thought you got caught by something16:07
Qianbiao you mean rocky and queens?16:07
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Switch integration testing to Ubuntu Focal  https://review.opendev.org/75355316:08
dtantsurmgoddard: oops, updated ^^16:09
openstackgerritIury Gregory Melo Ferreira proposed openstack/bifrost master: Add ipe installation to bifrost  https://review.opendev.org/75204216:09
TheJuliaQianbiao: you can to rocky and queens as well16:12
mgoddarddtantsur: thanks16:12
dtantsurTheJulia: assuming rocky and queens have any chances of working..16:12
TheJuliaoh jeeze yeah16:12
TheJuliaWell, I got both working ~3-4 weeks ago16:13
*** Lucas_Gray has quit IRC16:13
* dtantsur would like to reduce the number of supported branches16:13
jtrileyhjensas: looks like it's working - letting it go for a bit but we already see it denying the other mac :)16:13
Qianbiaook TheJulia and for branch train, is it wrong? there are no topic info.16:14
TheJuliaAs 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 too16:14
Qianbiaobecause i mis-submit it from my local train branch before master merges.16:14
TheJuliaQianbiao: stable/<release_name>16:14
Qianbiaoyes that is what i mean https://review.opendev.org/#/c/752065/16:15
patchbotpatch 752065 - ironic-python-agent (stable/train) - Fix: make Intel CNA hardware manager none generic - 4 patch sets16:15
hjensasjtriley: 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
Qianbiaohow i can delete it , and re cherry pick16:15
TheJuliayou 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 up16:15
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent stable/rocky: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75378516:15
TheJuliaoh, `git reset HEAD~1` and `git pull origin stable/<releasename>` is your friend16:16
TheJuliaerr16:16
TheJuliathat should be git pull --ff-only origin stable/<releasename>16:16
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent stable/queens: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75378616:17
TheJuliaQianbiao: 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
Qianbiaoi think it should be ok16:19
jtrileyhjensas: our please, thanks for the fix!16:21
jtrileyhjensas: let us know if you want us to test again after the re-factor16:22
QianbiaoI did not understand which version dib fix this issue: https://bugs.launchpad.net/tripleo/+bug/186442716:22
openstackLaunchpad 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
QianbiaoI have update dib to latest version, still come across same issue16:23
TheJuliaQianbiao: looks like because it was fixed only for RDO16:25
TheJuliahmm16:25
TheJuliathe ironic-agent element in diskimage-builder is also deprecated if memory serves16:25
TheJuliain favor of ironic-python-agent-builder16: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
QianbiaoI thought the same issue should have be fixed in dib.16:26
Qianbiaobecause ipa-builder copy code from dib if i am right.16:26
TheJuliayeah, they only fixed it for rdo becaues it was filed against tripleo16:27
Qianbiao:(16:27
TheJuliaI agree, it should have been fixed in dib16:27
*** dougsz has quit IRC16:30
TheJuliathe google image search built into slides returns some _weird_ results16:33
jtrileyhjensas: 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
QianbiaoTheJulia  can i just update locate 'tgt': 'scsi-target-utils' to tgt': 'targetcli' to fix it?16:42
hjensasjtriley: not check of the source mack, that would'nt work in a l3 network.16:43
TheJuliaQianbiao: I think so yes16:43
Qianbiaowill try ty16:43
TheJuliajust upload the change to diskimage-builder :)16:43
Qianbiao:) ok will try first16:43
*** dtantsur is now known as dtantsur|afk16:43
dtantsur|afko/16:43
*** gouthamr has joined #openstack-ironic16:44
hjensasjtriley: I don't think there is any security measures like that in place, maby TheJulia can correct me?16:44
TheJuliano, 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
TheJulianow, 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 it16:45
TheJuliasince it looks at node cache and mappings to figure out what to do16:45
arne_wiebalckbye everyone o/16:46
Qianbiaobye arne_wiebalck16:48
QianbiaoTheJulia new issue "Error: Unable to find a match: busybox16:49
Qianbiao" ..16:49
Qianbiaoit seems dib has bugs with centos 816:49
QianbiaoNot sure how many.16:49
TheJuliaQianbiao: are you running this command on a centos machine?16:54
Qianbiaono16:54
Qianbiaoa ubuntu 18. must I?16:54
*** martalais has joined #openstack-ironic16:55
openstackgerritMerged openstack/ironic stable/train: Allow configuring IPMI cipher suite  https://review.opendev.org/75271016:59
*** derekh has quit IRC17:01
*** priteau has quit IRC17:02
openstackgerritQianBiao Ng proposed openstack/ironic-python-agent stable/train: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75206517:04
TheJuliaQianbiao: yeah, you need to be on centos. Centos really only builds on centos machines, ubuntu/debian is kind of similar17:05
Qianbiaook thanks.17:05
TheJulia\o/ something merged to stable/train!17:05
QianbiaoTheJulia the topic is still as it is.17:05
TheJuliatrandles: janders: I think my slides are good, please sanity check :)17:05
TheJuliatrandles: also, trinity's water cooling install is in the deck now17:06
TheJuliaQianbiao: looking17:17
TheJuliaQianbiao: what do you mean about topic, I'm slightly confused17:17
QianbiaoTheJulia ty, the patch's code look good. all the same17:17
TheJuliaQianbiao: excellent!17:20
Qianbiao:) ty17:21
TheJuliaQianbiao: also you backporting that patch helps provide us visibility into which branches we may need to fix CI jobs on17:28
trandlesTheJulia, 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
TheJuliaQianbiao: It may. I've already approved one of them. :) One step at a time.17:31
Qianbiaonice17:32
*** bdodd has joined #openstack-ironic17:36
*** zaneb has quit IRC17:37
*** zaneb has joined #openstack-ironic17:38
*** jtomasek has quit IRC17:41
QianbiaoTheJulia  on centos, dib fails also.17:43
Qianbiao"No match for argument: busybox", I googled, busybox does not package for centos17:43
Qianbiaoi will try dracut-ramdisk instead of ramdisk17:45
*** uzumaki has quit IRC17:48
*** Qianbiao has quit IRC17:53
openstackgerritVerification of a change to openstack/ironic failed: Add Redfish BIOS interface to idrac HW type  https://review.opendev.org/74924017:55
*** Qianbiao has joined #openstack-ironic17:56
openstackgerritAija Jaunteva proposed openstack/ironic master: Update tests for Redfish BIOS apply_configuration  https://review.opendev.org/75369418:00
*** Qianbiao has quit IRC18:00
*** martalais has quit IRC18:03
openstackgerritJulia Kreger proposed openstack/ironic stable/ussuri: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75347018:04
*** zzzeek has quit IRC18:07
*** zzzeek has joined #openstack-ironic18:10
TheJuliaadditional core reviews on https://review.opendev.org/#/c/750204/10 would be much appreciated18:18
patchbotpatch 750204 - ironic - Deprecate the iscsi deploy interface - 10 patch sets18:18
*** chenhaw has joined #openstack-ironic18:19
dkingWhen running the clean step "erase_devices", at what point are the partitions erased?18:25
openstackgerritMerged openstack/bifrost master: Ensure that repositories are owned by the running users  https://review.opendev.org/75247318:26
openstackgerritMerged openstack/bifrost master: bifrost-cli: do not provide any release if non can be detected  https://review.opendev.org/75265818:26
openstackgerritMerged openstack/bifrost master: Switch integration testing to Ubuntu Focal  https://review.opendev.org/75355318:26
dkingI 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
rpiosoTheJulia: 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
patchbotpatch 748927 - sushy - Make message parsing more resilient - 4 patch sets18:29
openstackgerritMerged openstack/bifrost master: Move CLI validation to bifrost-keystone-client-config  https://review.opendev.org/75246618:30
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal stable/victoria: Update .gitreview for stable/victoria  https://review.opendev.org/75381818:34
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria  https://review.opendev.org/75381918:34
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/victoria  https://review.opendev.org/75382018:34
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal master: Add Python3 wallaby unit tests  https://review.opendev.org/75382118:34
TheJuliadking: erasing the base devices wipes the partition tables out18:37
TheJuliarpioso: awesome18:38
dkingTheJulia: I've run hardware.GenericHardwareManager()._ata_erase() multiple times, and the partitions are still there.18:40
TheJuliadking: have you rebooted?18:40
dkingNo, 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-ironic18:49
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch stable/victoria: Update .gitreview for stable/victoria  https://review.opendev.org/75382518:55
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria  https://review.opendev.org/75382618:55
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/victoria  https://review.opendev.org/75382718:55
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch master: Add Python3 wallaby unit tests  https://review.opendev.org/75382818:55
TheJuliadking: exactly18:57
TheJuliayou can use kpartx to update the cache18:58
dkingThanks! 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 IRC19:02
TheJuliaI think that just lists the kernel recognized data19:13
* TheJulia goes and lays down for a little bit19:13
*** tosky has joined #openstack-ironic19:15
*** jtriley has quit IRC19:20
openstackgerritMerged openstack/ironic-inspector master: Power off before inspection  https://review.opendev.org/74984519:34
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Fix: make Intel CNA hardware manager none generic  https://review.opendev.org/75372019:34
*** adrianc_ has quit IRC19:35
*** adrianc has joined #openstack-ironic19:35
openstackgerritMerged openstack/sushy master: Make message parsing more resilient  https://review.opendev.org/74892719:46
openstackgerritVerification of a change to openstack/ironic failed: Use correct error handler instead of calling process_event('fail')  https://review.opendev.org/75359519:50
openstackgerritJulia Kreger proposed openstack/sushy stable/victoria: Make message parsing more resilient  https://review.opendev.org/75383719:52
openstackgerritJulia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job  https://review.opendev.org/74831519:55
*** stevebaker has joined #openstack-ironic20:01
stevebakermorning20:03
openstackgerritMerged openstack/bifrost master: Add some interfaces for development hardware types  https://review.opendev.org/75321320:29
iurygregorymorning stevebaker o/20:30
stevebakeriurygregory: hey!20:30
iurygregorytoo early for you no?20:30
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Don't update the grubenv  https://review.opendev.org/75230820:31
stevebakeriurygregory: its 8:30am not too early20:33
iurygregoryI thought it was 7:3020:34
openstackgerritJulia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job  https://review.opendev.org/74831520:34
TheJuliagood morning stevebaker20:37
TheJuliaAny luck with the first patch of the api change series in CI?20:37
*** yolanda has quit IRC20:37
stevebakerTheJulia: good afternoon, its 1337 oclock20:37
TheJuliahow leet20:38
* TheJulia suspects stevebaker is in a bunker with a neat world clock :)20:38
stevebakerTheJulia: https://review.opendev.org/#/c/742795 is passing zuul ci, just not all in the same run. only ironic-standalone-redfish failed the last recheck20:40
patchbotpatch 742795 - ironic - New argument validate decorator - 13 patch sets20:40
TheJuliastevebaker: did you find any reason why?20:40
openstackgerritMerged openstack/sushy stable/victoria: Make message parsing more resilient  https://review.opendev.org/75383720:44
stevebakerTheJulia: 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 related20:45
TheJuliastevebaker: did you check the console logs to at least see if the instance booted?20:49
*** phrobb has quit IRC20:49
*** vkmc has quit IRC20:49
*** rajinir has quit IRC20:49
stevebakerTheJulia: I'll dig into that failure today20:50
*** sri_ has quit IRC20:50
*** buhman has quit IRC20:50
*** rpittau|afk has quit IRC20:50
*** gmann has quit IRC20:50
*** mnasiadka has quit IRC20:50
TheJuliak, either the vm didn't boot up or neutron networking blew up20:50
*** ildikov has quit IRC20:51
*** NobodyCam has quit IRC20:51
*** rpioso has quit IRC20:51
*** vkmc has joined #openstack-ironic20:51
*** phrobb has joined #openstack-ironic20:51
*** arne_wiebalck has quit IRC20:51
*** PrinzElvis has quit IRC20:51
*** erbarr has quit IRC20:51
*** sri_ has joined #openstack-ironic20:52
*** vdrok has quit IRC20:52
*** NobodyCam has joined #openstack-ironic20:52
*** zzzeek has quit IRC20:52
*** mnasiadka has joined #openstack-ironic20:52
*** vdrok has joined #openstack-ironic20:52
*** rpioso has joined #openstack-ironic20:52
*** PrinzElvis has joined #openstack-ironic20:52
*** ildikov has joined #openstack-ironic20:52
*** buhman has joined #openstack-ironic20:52
*** rajinir has joined #openstack-ironic20:53
*** arne_wiebalck has joined #openstack-ironic20:53
*** erbarr has joined #openstack-ironic20:53
*** gmann has joined #openstack-ironic20:53
*** rpittau|afk has joined #openstack-ironic20:53
*** zzzeek has joined #openstack-ironic20:56
dkingDoes 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
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents  https://review.opendev.org/74804921:26
openstackgerritVerification of a change to openstack/ironic failed: Adding changes for iso less vmedia support  https://review.opendev.org/75200121:32
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: WIP Add fstab pointer to EFI partition  https://review.opendev.org/74826021:33
openstackgerritMerged openstack/ironic stable/train: CI: Patch to fix CI jobs in bugfix/stable branches  https://review.opendev.org/75347121:37
openstackgerritMerged openstack/ironic-python-agent master: Document ramdisk TLS and update existing TLS docs  https://review.opendev.org/75364521:37
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents  https://review.opendev.org/74804921:38
rpiosoTheJulia: When you have a chance, we would greatly appreciate your further review of https://review.opendev.org/749619.21:55
patchbotpatch 749619 - ironic - Redfish driver firmware update - 21 patch sets21:55
trandlesTheJulia: janders: your slides look good to me21:56
trandlesI'm working on mine now...21:56
TheJuliaumm the follow-up?21:58
TheJuliatrandles: awesome21:58
*** JamesBenson has quit IRC21:59
TheJuliarpioso: okay, thats a different one22:00
openstackgerritMerged openstack/ironic-python-agent master: When reporting that agent is busy, report the executed command  https://review.opendev.org/75271922:00
openstackgerritMerged openstack/ironic master: Update tests for Redfish BIOS apply_configuration  https://review.opendev.org/75369422:00
*** JamesBenson has joined #openstack-ironic22:00
TheJuliarpioso: I'll look at it a little later today. I've got a pile of stuff to take care of today22:00
*** phrobb has quit IRC22:01
rpiosoTheJulia: Thank you so much! I can only imagine.22:02
*** phrobb has joined #openstack-ironic22:03
rpiosoTheJulia:  I believe you looked at that one last Friday, but your caffeine level was waning. Hopefully, it'll look familiar.22:04
TheJuliaI should actually make caffeine22:04
TheJuliaand pizza douh22:04
TheJuliadough22:04
*** JamesBenson has quit IRC22:05
*** k_mouza has joined #openstack-ironic22:20
*** k_mouza has quit IRC22:24
openstackgerritMerged openstack/ironic-python-agent master: Remove unused jobs  https://review.opendev.org/74972522:32
*** zzzeek has quit IRC22:35
*** JamesBenson has joined #openstack-ironic22:37
*** zzzeek has joined #openstack-ironic22:37
*** k_mouza has joined #openstack-ironic22:39
*** JamesBenson has quit IRC22:41
*** k_mouza has quit IRC22:44
*** zzzeek has quit IRC22:45
*** rcernin has joined #openstack-ironic22:46
*** zzzeek has joined #openstack-ironic22:46
openstackgerritMerged openstack/ironic-inspector master: Add NVIDIA V100 to known accelerator devices  https://review.opendev.org/75166022:49
*** rpioso has quit IRC22:50
*** PrinzElvis has quit IRC22:50
*** sri_ has quit IRC22:50
*** ildikov has quit IRC22:50
*** v1k0d3n has quit IRC22:50
*** phrobb has quit IRC22:50
*** erbarr has quit IRC22:50
*** bbezak has quit IRC22:50
*** arne_wiebalck has quit IRC22:50
*** johnsom has quit IRC22:50
*** andrein has quit IRC22:50
*** m1p has quit IRC22:50
*** mnasiadka has quit IRC22:50
*** vdrok has quit IRC22:50
*** portdirect has quit IRC22:50
*** mnaser has quit IRC22:50
*** NobodyCam has quit IRC22:51
*** vkmc has quit IRC22:51
*** erbarr has joined #openstack-ironic22:51
*** bbezak has joined #openstack-ironic22:51
*** v1k0d3n has joined #openstack-ironic22:51
*** sri_ has joined #openstack-ironic22:51
*** arne_wiebalck has joined #openstack-ironic22:51
*** PrinzElvis has joined #openstack-ironic22:51
*** ildikov has joined #openstack-ironic22:51
*** rpioso has joined #openstack-ironic22:51
*** m1p has joined #openstack-ironic22:52
*** andrein has joined #openstack-ironic22:52
*** seongsoocho has quit IRC22:52
*** seongsoocho has joined #openstack-ironic22:52
*** phrobb has joined #openstack-ironic22:53
*** mnaser has joined #openstack-ironic22:53
*** rpittau|afk has quit IRC22:54
*** vdrok has joined #openstack-ironic22:54
*** portdirect has joined #openstack-ironic22:54
*** mnasiadka has joined #openstack-ironic22:54
*** johnsom has joined #openstack-ironic22:55
*** rpittau|afk has joined #openstack-ironic22:56
*** vkmc has joined #openstack-ironic22:56
*** andrein has quit IRC22:58
*** NobodyCam has joined #openstack-ironic22:59
*** andrein has joined #openstack-ironic23:00
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition  https://review.opendev.org/74826023:02
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Change integration test for Metalsmith to use Centos8/UEFI  https://review.opendev.org/74977223:02
*** tkajinam has joined #openstack-ironic23:03
*** zzzeek has quit IRC23:04
TheJulialooks like we need to drop the test concurrency on the multinode job :\23:05
*** zzzeek has joined #openstack-ironic23:06
stevebakerTheJulia: 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#414223:07
stevebakerTheJulia: '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 argument23:07
trandlesTheJulia: janders my slides are ready for review I think. I'm not thrilled, feedback very welcome.23:10
* TheJulia blinks23:10
TheJuliatrandles: why not thrilled?!?23:11
trandleseh...I'm rarely thrilled with my slides23:12
TheJuliastevebaker: what the...23:12
TheJuliastevebaker: maybe make some noise in the neutron channel?!?23:12
* TheJulia hopes this is not the curse of the end of cycle release23:12
*** k_mouza has joined #openstack-ironic23:13
* trandles is closing this laptop for the night...23:13
TheJuliagoodnight!23:14
trandlesnighty night :)23:14
stevebakerTheJulia: I'll keep digging for a bit before I ask them23:14
TheJuliastevebaker: that is just bizzar....23:15
openstackgerritJulia Kreger proposed openstack/ironic master: CI: Tune down multinode concurrency  https://review.opendev.org/75386323:16
stevebakerTheJulia: its like client_id is assumed to be a list, and is joined into a string with a space23:16
*** k_mouza has quit IRC23:18
TheJuliarpioso: that firmware update patch is likely going to need another revision, working on comments now23:22
rpiosoTheJulia: Thank you so much. We look forward to your feedback.23:23
rpiosocdearborn: ^^^23:24
stevebakerTheJulia: boom http://paste.openstack.org/show/798297/23:32
TheJuliarpioso: basically looking like the periodic. mainly in 20 seconds is super aggressive23:33
jandersgood morning Ironic o/23:34
*** rcernin has quit IRC23:34
*** rcernin has joined #openstack-ironic23:34
rpiosoTheJulia: If memory serves, cdearborn has reported it's quickly completed.23:37
rpiosoTheJulia: But, I'll defer to him.23:37
rpiosoTheJulia: 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 IRC23:41
*** zzzeek has joined #openstack-ironic23:45
TheJuliarpioso: conundrum is we're getting clusters with hundreds and thousands of nodes23:45
TheJuliaso... yeah. :\23:45
TheJuliaI'm still typing notes out, hopefully it will be fully clear and cdearborn if you want to chat tomorrow I can make some time23:45
rpiosoTheJulia: Love the conundrum! ironic Redfish firmware update automation might be helpful :-)23:47
rpiosoTheJulia: That would be awesome if you could make time. Wanna get something on our calendars?23:48
*** rloo has quit IRC23:48
TheJuliarpioso: after 9:30 am pacific I'm free until noon pacific.23:49
TheJuliarpioso: cdearborn comments posted23:49
*** paulbrowne has joined #openstack-ironic23:49
*** tosky has quit IRC23:49
rpiosoTheJulia: My calendar is clear during that period. BlueJeans?23:50
TheJuliasure, just send me an meeting invite to my rh address23:50
openstackgerritVerification of a change to openstack/ironic failed: Use correct error handler instead of calling process_event('fail')  https://review.opendev.org/75359523:52
rpiosoTheJulia: 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
rpiosoTheJulia: Sent23:56
TheJuliarpioso: ack23:56
*** paulbrowne has quit IRC23:56
rpiosoTheJulia: Please feel free to invite others.23:57
TheJuliarpioso: I don't see it yet but I've blocked the time out on my calendar if it doesn't appear before I forget23:58

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!