*** k_mouza has quit IRC | 00:13 | |
*** ociuhandu has joined #openstack-ironic | 00:36 | |
*** Goneri has quit IRC | 00:41 | |
*** ociuhandu has quit IRC | 00:44 | |
*** tzumainn has quit IRC | 00:44 | |
openstackgerrit | huwenhui proposed openstack/ironic master: Fix the wrong description about singular-plural pair the description about the method "__is_image_href_ordinary_file_name" should be "Check if image_href is an ordinary file name" not "Check if image_href is a ordinary file name" https://review.opendev.org/727547 | 00:46 |
---|---|---|
openstackgerrit | huwenhui proposed openstack/ironic master: Fix the wrong description The description about the method "__is_image_href_ordinary_file_name" should be "Check if image_href is an ordinary file name" not "Check if image_href is a ordinary file name", Fix the error. https://review.opendev.org/727547 | 01:12 |
*** k_mouza has joined #openstack-ironic | 01:14 | |
*** livelace has quit IRC | 01:18 | |
*** k_mouza has quit IRC | 01:20 | |
openstackgerrit | huwenhui proposed openstack/ironic master: Fix the wrong description https://review.opendev.org/727547 | 01:23 |
*** ericlei has joined #openstack-ironic | 01:43 | |
*** ericlei has quit IRC | 01:45 | |
*** jdandrea has quit IRC | 02:14 | |
*** gyee has quit IRC | 03:18 | |
*** dustinc has quit IRC | 03:34 | |
*** cdearborn has quit IRC | 03:53 | |
*** ociuhandu has joined #openstack-ironic | 04:42 | |
*** dmellado has quit IRC | 04:45 | |
*** ociuhandu has quit IRC | 04:49 | |
*** k_mouza has joined #openstack-ironic | 05:16 | |
*** k_mouza has quit IRC | 05:20 | |
arne_wiebalck | TheJulia: Not after deletion, mostly after motherboard replacements. We are working on a small tool to detect these and clean up the DB. | 06:20 |
arne_wiebalck | Good morning, ironic! | 06:21 |
*** ociuhandu has joined #openstack-ironic | 06:30 | |
*** ericlei has joined #openstack-ironic | 06:45 | |
*** ericlei has quit IRC | 06:51 | |
*** Lucas_Gray has joined #openstack-ironic | 07:03 | |
*** yolanda has joined #openstack-ironic | 07:21 | |
*** Lucas_Gray has quit IRC | 07:22 | |
iurygregory | good morning arne_wiebalck and Ironic o/ | 07:27 |
*** ociuhandu has quit IRC | 07:27 | |
*** ociuhandu_ has joined #openstack-ironic | 07:27 | |
iurygregory | stevebaker, hey sorry I went to sleep =), I haven't started anything XD | 07:29 |
*** rpittau|afk is now known as rpittau | 07:29 | |
rpittau | good morning ironic! o/ | 07:29 |
*** amoralej|off is now known as amoralej | 07:34 | |
*** aedc has quit IRC | 07:37 | |
*** dmellado has joined #openstack-ironic | 07:37 | |
*** aedc has joined #openstack-ironic | 07:47 | |
rpittau | anyone around to please approve this https://review.opendev.org/727171 | 07:57 |
patchbot | patch 727171 - ironic-python-agent - CI: raise DIB jobs memory to 3 GiB - 1 patch set | 07:57 |
rpittau | pretty please :) | 07:57 |
*** lucasagomes has joined #openstack-ironic | 07:58 | |
*** livelace has joined #openstack-ironic | 08:12 | |
rpittau | we also need this kind of urgently as we probably need to backport it https://review.opendev.org/726753 | 08:15 |
patchbot | patch 726753 - ironic (stable/ussuri) - Add timeout and retries to JSON RPC client - 1 patch set | 08:15 |
rpittau | whoops wrong | 08:15 |
rpittau | this one https://review.opendev.org/727206 | 08:15 |
patchbot | patch 727206 - ironic - Cap pycodestyle explicitly to be < 2.6.0 - 3 patch sets | 08:15 |
iurygregory | good morning rpittau o/ | 08:19 |
rpittau | hey iurygregory :) | 08:19 |
*** Qianbiao has joined #openstack-ironic | 08:19 | |
openstackgerrit | Riccardo Pittau proposed x/sushy-oem-idrac master: Update lower-constraints.txt https://review.opendev.org/727260 | 08:24 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Update lower-constraints.txt https://review.opendev.org/727253 | 08:30 |
*** xiaolin has quit IRC | 08:31 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Test CI https://review.opendev.org/727661 | 08:32 |
iurygregory | rpittau, I'm trying to test the functional test locally =) | 08:36 |
iurygregory | devstack running so I can run the tests | 08:36 |
openstackgerrit | Riccardo Pittau proposed openstack/networking-baremetal master: Bump hacking version to 3.0.0 and fix pep8 test https://review.opendev.org/727152 | 08:41 |
*** xiaolin has joined #openstack-ironic | 08:45 | |
*** livelace has quit IRC | 08:50 | |
*** riuzen has joined #openstack-ironic | 08:56 | |
*** livelace has joined #openstack-ironic | 08:57 | |
*** livelace has quit IRC | 09:02 | |
*** livelace has joined #openstack-ironic | 09:08 | |
*** xiaolin has quit IRC | 09:10 | |
*** livelace has quit IRC | 09:13 | |
*** dtantsur|afk is now known as dtantsur | 09:13 | |
dtantsur | morning ironic | 09:13 |
iurygregory | good morning dtantsur | 09:14 |
iurygregory | dtantsur, we need to backport https://review.opendev.org/#/c/727141/ to ussuri right? | 09:16 |
patchbot | patch 727141 - ironic - Fix pep8 (MERGED) - 6 patch sets | 09:16 |
dtantsur | iurygregory: yep | 09:16 |
iurygregory | dtantsur, ack working here since has some conflicts | 09:16 |
rpittau | I thought we were capping pycodestyle ? | 09:20 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Remove iDRAC Redfish boot device known issue https://review.opendev.org/727676 | 09:21 |
openstackgerrit | Merged openstack/ironic-lib master: image_convert: retry resource unavailable and make RLIMIT configurable https://review.opendev.org/726378 | 09:22 |
openstackgerrit | Merged openstack/ironic-inspector master: Make POST /v1/introspection synchronous when manage_boot==False https://review.opendev.org/727151 | 09:22 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib stable/ussuri: image_convert: retry resource unavailable and make RLIMIT configurable https://review.opendev.org/727677 | 09:22 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib stable/train: image_convert: retry resource unavailable and make RLIMIT configurable https://review.opendev.org/727678 | 09:22 |
openstackgerrit | Verification of a change to openstack/ironic failed: Improve `redfish` set-boot-mode implementation https://review.opendev.org/727362 | 09:26 |
rpittau | dtantsur: if we backport the fixes for flake8/pycodestyle (not only in ironic), then no need to cap the version ? | 09:27 |
dtantsur | rpittau: probably no (or only to the next version) | 09:33 |
*** livelace has joined #openstack-ironic | 09:34 | |
rpittau | dtantsur: then I better start backporting all the pep8 fixes from yesterday | 09:34 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ussuri: Make POST /v1/introspection synchronous when manage_boot==False https://review.opendev.org/727682 | 09:36 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib stable/ussuri: Fix pep8 test https://review.opendev.org/727683 | 09:36 |
rpittau | arne_wiebalck: thanks for the reviews! if you have one more minute please check https://review.opendev.org/727117 | 09:38 |
patchbot | patch 727117 - ironic-python-agent - Fix pep8 errors - 1 patch set | 09:38 |
arne_wiebalck | rpittau: the others were a few lines ... this is 600 lines | 09:41 |
rpittau | arne_wiebalck: it's just copy paste of stuff, no code change | 09:41 |
arne_wiebalck | I know :) | 09:42 |
iurygregory | arne_wiebalck, is a bit afraid | 09:42 |
iurygregory | don't be =) | 09:42 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic stable/ussuri: Fix pep8 https://review.opendev.org/727689 | 09:42 |
dtantsur | arne_wiebalck: hey! how long does raid.delete_configuration usually take for you? | 09:42 |
arne_wiebalck | have no fear, and press W+1! | 09:42 |
dtantsur | someone tells me that they run delete_configuration+erase_device_metadata+create_configuration and they take 13 mins, 11 mins and 5 mins accordingly | 09:42 |
rpittau | arne_wiebalck: thanks! :) | 09:43 |
arne_wiebalck | dtantsur: this is for software RAID? | 09:43 |
dtantsur | yeah | 09:43 |
arne_wiebalck | these commands should be all really fast | 09:44 |
arne_wiebalck | (and they are for us) | 09:44 |
arne_wiebalck | this is definitely worth a closer look as it seems way too loong | 09:45 |
arne_wiebalck | (even my long is too long) | 09:45 |
openstackgerrit | Riccardo Pittau proposed openstack/sushy stable/ussuri: Fix pep8 test https://review.opendev.org/727693 | 09:50 |
*** hjensas|afk is now known as hjensas | 09:51 | |
iurygregory | rpittau, ERROR: ironic-inspector 10.2.0.dev2 has requirement construct<2.9,>=2.8.10, but you'll have construct 2.10.56 which is incompatible.I think I've found the problem in the functional tests =( | 09:55 |
iurygregory | did we bump construct in requirements? | 09:55 |
rpittau | iurygregory: yeah, I'm seeing the same, we did remove the limitation on construct in requirements | 10:00 |
*** ociuhandu_ has quit IRC | 10:01 | |
openstackgerrit | Merged openstack/ironic master: Fix the wrong description https://review.opendev.org/727547 | 10:01 |
rpittau | I guess we can work around it adding the cap in ironic-inspector-client for the functional test | 10:02 |
*** ociuhandu has joined #openstack-ironic | 10:02 | |
iurygregory | Yeah I'm trying this | 10:03 |
rpittau | I'm working on the construct patch anyway, that should remove the need for the cap | 10:09 |
openstackgerrit | Merged openstack/metalsmith master: Update lower-constraints.txt https://review.opendev.org/727251 | 10:18 |
openstackgerrit | Merged openstack/ironic-python-agent master: CI: raise DIB jobs memory to 3 GiB https://review.opendev.org/727171 | 10:19 |
*** derekh has joined #openstack-ironic | 10:20 | |
iurygregory | rpittau, yeah if we get working we won't need | 10:24 |
rpittau | I'm missing a couple of things, but it's pretty much done | 10:24 |
openstackgerrit | Merged openstack/ironic master: Cap pycodestyle explicitly to be < 2.6.0 https://review.opendev.org/727206 | 10:37 |
*** k_mouza has joined #openstack-ironic | 10:47 | |
*** Qianbiao has quit IRC | 10:50 | |
*** Qianbiao has joined #openstack-ironic | 10:50 | |
*** rpittau is now known as rpittau|bbl | 11:04 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Use latest version of python construct https://review.opendev.org/724822 | 11:04 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Consistently handle errors in the deploy process https://review.opendev.org/727714 | 11:17 |
*** ericlei has joined #openstack-ironic | 11:21 | |
*** ociuhandu has quit IRC | 11:23 | |
openstackgerrit | Merged openstack/sushy-cli master: Update lower-constraints.txt https://review.opendev.org/727242 | 11:26 |
openstackgerrit | Merged openstack/ironic-python-agent master: Fix pep8 errors https://review.opendev.org/727117 | 11:27 |
openstackgerrit | Merged openstack/ironic-inspector stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/723568 | 11:27 |
openstackgerrit | Merged openstack/ironic-lib stable/ussuri: Fix pep8 test https://review.opendev.org/727683 | 11:27 |
*** uzumaki has joined #openstack-ironic | 11:36 | |
*** ericlei has quit IRC | 11:37 | |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Avoid using introspection start delays with non-managed boot https://review.opendev.org/727209 | 11:44 |
openstackgerrit | Merged openstack/networking-baremetal master: Add unit tests for _get_notification_transport_url() https://review.opendev.org/635844 | 11:54 |
openstackgerrit | Verification of a change to openstack/sushy failed: Update lower-constraints.txt https://review.opendev.org/727240 | 11:55 |
*** amoralej is now known as amoralej|lunch | 11:58 | |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: [WIP] Fix OEM extension loading for different servers https://review.opendev.org/727738 | 11:59 |
*** ociuhandu has joined #openstack-ironic | 12:04 | |
*** ociuhandu has quit IRC | 12:10 | |
*** ociuhandu has joined #openstack-ironic | 12:10 | |
*** sshnaidm|afk is now known as sshnaidm | 12:16 | |
*** rpittau|bbl is now known as rpittau | 12:19 | |
*** livelace has quit IRC | 12:23 | |
openstackgerrit | Merged openstack/sushy-tools master: Update lower-constraints.txt https://review.opendev.org/727241 | 12:23 |
*** tkajinam has quit IRC | 12:24 | |
*** Qianbiao has quit IRC | 12:25 | |
*** ociuhandu has quit IRC | 12:27 | |
*** ociuhandu has joined #openstack-ironic | 12:28 | |
*** livelace has joined #openstack-ironic | 12:29 | |
openstackgerrit | Merged openstack/ironic-lib stable/train: image_convert: retry resource unavailable and make RLIMIT configurable https://review.opendev.org/727678 | 12:36 |
iurygregory | rpittau, I think you missed the release note for https://review.opendev.org/#/c/724822/ | 12:39 |
patchbot | patch 724822 - ironic-inspector - Use latest version of python construct - 3 patch sets | 12:39 |
iurygregory | awesome to see working at least \o/ | 12:40 |
rpittau | iurygregory: I did, I know, I'm going to add one | 12:40 |
iurygregory | rpittau, I was about to +2 XD and "oh release note" | 12:40 |
rpittau | iurygregory: yeah, thanks for reviewing :) | 12:40 |
rpittau | feel free to -1 me :D | 12:40 |
iurygregory | nah | 12:41 |
iurygregory | you have 5min =) | 12:41 |
rpittau | gosh, no pressure :) | 12:41 |
iurygregory | hehehe | 12:42 |
iurygregory | or a -1 XD | 12:42 |
rpittau | :( | 12:42 |
iurygregory | rpittau, but if you offer me some lasagna you have a day | 12:43 |
rpittau | iurygregory: I see, we're actually finally going to make tiramisu again this week after 2 months, up to you :) | 12:44 |
iurygregory | rpittau, wow =O sounds like a deal =P | 12:45 |
rpittau | iurygregory: if you're ever in Versailles ;) | 12:49 |
iurygregory | maybe one day hehehe | 12:50 |
dtantsur | I've never had a chance to get to the palace there (only to the gardens), so count me in :) | 12:51 |
*** rloo has joined #openstack-ironic | 12:53 | |
*** ianychoi has quit IRC | 12:55 | |
*** amoralej|lunch is now known as amoralej | 13:00 | |
rpittau | dtantsur: I'd be glad to take you on the palace tour :D | 13:03 |
dtantsur | ++ | 13:03 |
*** Qianbiao has joined #openstack-ironic | 13:03 | |
*** jdandrea has joined #openstack-ironic | 13:07 | |
*** riuzen has quit IRC | 13:11 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-prometheus-exporter master: Stop calling LOG.info on every function calls https://review.opendev.org/727755 | 13:16 |
dtantsur | iurygregory: ^^ | 13:16 |
iurygregory | dtantsur, maybe change for debug? | 13:17 |
dtantsur | iurygregory: most people have debug on, so it won't help | 13:17 |
dtantsur | I don't think why we need to log every input we process | 13:17 |
dtantsur | * see | 13:17 |
iurygregory | for me it was more like a debug XD | 13:18 |
iurygregory | but we don't really need all | 13:18 |
dtantsur | iurygregory: to support my claim with some data: I'm looking at an ironic-conductor log file, where ironic_prometheus_exporter logs occupy 3.7 MiB out of 9.3 MiB total | 13:18 |
iurygregory | dtantsur, gotcha | 13:19 |
* iurygregory wondering who is running the exporter O.o | 13:19 | |
dtantsur | iurygregory: metal3 apparently? | 13:19 |
iurygregory | oh | 13:19 |
*** ociuhandu has quit IRC | 13:24 | |
*** ericlei has joined #openstack-ironic | 13:24 | |
TheJulia | good morning | 13:25 |
rpittau | good morning TheJulia :) | 13:25 |
TheJulia | iurygregory: seems like you should have +2 rights on IPE? | 13:26 |
iurygregory | good morning TheJulia | 13:26 |
iurygregory | maybe | 13:27 |
iurygregory | not sure if it fits in inspector/sushy/bifrost | 13:28 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Use latest version of python construct https://review.opendev.org/724822 | 13:29 |
TheJulia | any objections folks if I give iurygregory rights to the project he put lots of energy into? | 13:30 |
TheJulia | only seems right | 13:30 |
rpittau | I'm actually surprised he was not there already, please do :) | 13:30 |
TheJulia | looks like it requires another group to be created | 13:34 |
iurygregory | that's what I was thinking ^ | 13:34 |
dtantsur | morning TheJulia | 13:35 |
dtantsur | TheJulia: wait, he's not there? Oo | 13:35 |
* iurygregory not sure if it's ok to create a new group | 13:35 | |
dtantsur | it's okay to create a new group, although we can take a trusted delegation approach | 13:35 |
iurygregory | dtantsur, nope =) you can get my +1 and imagine that is a +2 | 13:35 |
dtantsur | i.e. give iurygregory ironic-core and ask to only approve things he's responsible for | 13:35 |
dtantsur | this is how tripleo does it (I have +2 this way there) | 13:36 |
iurygregory | bifrost has a group for itself, maybe rename and add there? | 13:37 |
TheJulia | I think the trusted delegation approach is the right path to take | 13:38 |
TheJulia | iurygregory: what do you think? | 13:38 |
iurygregory | TheJulia, I'm ok with that if the community is ok =) | 13:38 |
* TheJulia expects iury is opening a celebration beer at the moment | 13:38 | |
* iurygregory will have beer at the dinner | 13:39 | |
dtantsur | the benefit of the trusted delegation path is that iurygregory can approve trivial patches on other projects as well | 13:41 |
TheJulia | ++ | 13:41 |
* TheJulia makes a calendar item to send an email | 13:42 | |
iurygregory | ack o/ | 13:42 |
*** ericlei has quit IRC | 13:42 | |
*** tzumainn has joined #openstack-ironic | 13:45 | |
dtantsur | the 10th recheck has failed for https://review.opendev.org/#/c/725239/ time to open a bottle? >_< | 13:54 |
patchbot | patch 725239 - ironic - redfish: handle hardware that is unable to set per... - 11 patch sets | 13:54 |
dtantsur | and people say that unlucky places do not exist, it's just coincidences... | 13:55 |
rpittau | on a positive note, my swear jar is full | 13:55 |
dtantsur | :D | 13:55 |
dtantsur | I need to get one, maybe I can finally take mortgage after a year? | 13:55 |
rpittau | eheh | 13:56 |
iurygregory | dtantsur, they exist in the CI world | 13:56 |
dtantsur | indeed | 13:56 |
* TheJulia reflects and realizes she doesn't actually swear much normally | 13:56 | |
iurygregory | dtantsur, according to https://zuul.openstack.org/builds?job_name=ironic-tempest-ipa-partition-uefi-pxe-grub2 https://zuul.openstack.org/builds?job_name=ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-indirect you are unlucky | 13:58 |
iurygregory | sorry to say that =( | 13:58 |
dtantsur | don't worry, I've known it for a while | 13:58 |
TheJulia | don't tell him he is unlucky! Tell him it will be okay and try to fix it! | 13:58 |
iurygregory | I know arne_wiebalck has some contract with zuul so maybe you can ask for some tips =) | 13:59 |
TheJulia | Without digging into logs, could it just be we have too many jobs? | 13:59 |
iurygregory | yup we do | 13:59 |
TheJulia | the random package mirror can send things down an awful bad path | 13:59 |
TheJulia | seems like we can remove ironic-tempest-partition-bios-redfish-pxe | 14:02 |
dtantsur | in favour of standalone-redfish? | 14:02 |
dtantsur | TheJulia, bfournie, re the ipmitool cipher fun: should we maybe move retries from ipmitool to the ironic level? | 14:03 |
*** ociuhandu has joined #openstack-ironic | 14:05 | |
TheJulia | dtantsur: yeah | 14:05 |
TheJulia | also, ironic-tempest-ipa-partition-pxe_ipmitool seems redundant compared to ironic-standalone | 14:05 |
dtantsur | it is, if we have other partition jobs | 14:05 |
TheJulia | dtthe standalone job has partition + rescue | 14:06 |
TheJulia | with iscsi | 14:06 |
TheJulia | so really the matrix covers it from the standalone job | 14:07 |
TheJulia | Just ideas, two jobs are two jobs | 14:07 |
dtantsur | I'm (mildly) worried about the nova integratin bits | 14:07 |
TheJulia | dtantsur: and we have other jobs that can serve that purpose | 14:07 |
dtantsur | likely so | 14:07 |
* iurygregory likes the idea | 14:08 | |
TheJulia | we have ?10? jobs that cover those same bits | 14:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-prometheus-exporter stable/ussuri: Stop calling LOG.info on every function calls https://review.opendev.org/727772 | 14:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-prometheus-exporter stable/train: Stop calling LOG.info on every function calls https://review.opendev.org/727773 | 14:08 |
*** ociuhandu has quit IRC | 14:11 | |
*** ociuhandu has joined #openstack-ironic | 14:12 | |
*** ociuhandu has quit IRC | 14:18 | |
bfournie | dtantsur: so we would set -R 1 to ipmitool and just have ironic do retries? in the cipher tool case it would get success back and not even need a retry | 14:18 |
dtantsur | right | 14:19 |
* TheJulia tries to remember the reasoning | 14:20 | |
*** Goneri has joined #openstack-ironic | 14:20 | |
TheJulia | I feel like the internal retries were always to handle the protocol level necessary retries, but maybe it is not htat common. Our retry logic, is also built largely upon failure is a hard failure, and success we're done. We would likely need to understand additional retriable cases | 14:24 |
bfournie | dtantsur: it would help in this case - since ipmitool goes through all the retries before it returns success when the cipher query fails, but I wonder if there is anything gained by doing retry in ironic besides working around this ipmitool bug | 14:24 |
*** Qianbiao has quit IRC | 14:25 | |
TheJulia | I guess we're shifting the pain point to something we can actually control | 14:26 |
*** Qianbiao has joined #openstack-ironic | 14:26 | |
dtantsur | right | 14:27 |
TheJulia | I believe the term "necessary evil" is here | 14:28 |
TheJulia | bfournie: do you have any lab machines that are grumpy with ipmi? | 14:28 |
* TheJulia thinks HP ilo2s.... | 14:28 | |
TheJulia | also, ilo2s realistically would only be in test labs... I think | 14:29 |
*** kbaegis has joined #openstack-ironic | 14:29 | |
TheJulia | wow, the last ilo2 firmware was released in 2018 | 14:29 |
bfournie | TheJulia: Yes I can duplicate with the HP machine I have | 14:33 |
*** ociuhandu has joined #openstack-ironic | 14:33 | |
TheJulia | looks like the G5s were ~2009? | 14:33 |
TheJulia | G6 in 2010 | 14:34 |
TheJulia | so basically 10 year old gear, which means only the craziest people would still be running that stuff in production | 14:34 |
iurygregory | hehe | 14:35 |
iurygregory | but we all know how crazy people is | 14:35 |
uzumaki | TheJulia, are we assuming that production people can't be crazy? | 14:35 |
iurygregory | they think is ok to put 999999999999999 for timeout =P | 14:35 |
uzumaki | iurygregory, ^ exactly | 14:36 |
iurygregory | uzumaki, not crazy to have old gear... but everybody is a bit crazy =) | 14:36 |
bfournie | TheJulia: this is a g9 | 14:37 |
uzumaki | oh, not crazy to have it, just don't put critical business application on it is all | 14:37 |
uzumaki | use it as a decoration piece by all means, or to play super mario on it | 14:37 |
iurygregory | XD | 14:37 |
TheJulia | bfournie: not valid then | 14:37 |
TheJulia | the gen5-6 ilos were notoriously problematic | 14:37 |
TheJulia | which is what drove a lot of the retry stuff in the first place | 14:37 |
TheJulia | uzumaki: very true | 14:38 |
*** ociuhandu has quit IRC | 14:39 | |
uzumaki | older hardware reminds me I've sort of 'restored' an old Sega gaming console I had, ordered an AV2HDMI converter unit to use it with an LCD monitor.. let's see how it goes | 14:39 |
TheJulia | I can't seem to find MTBF, but something tells me by now most of the fans would have failed unless the machine was in a clean room and PSUs would be next | 14:39 |
TheJulia | in otherwords, lets try -R 1 against current gear and see how horrible it may be | 14:39 |
TheJulia | psu integrated fans... | 14:40 |
TheJulia | Those generations were also where I think they introduced the always on fans | 14:40 |
* TheJulia had a few racks of these servers a lifetime ago | 14:40 | |
* uzumaki wonders what TheJulia would possibly want with a whole rack of old hardware.. is she one of those mad genuis rocket scientists? (NOT A COMPLEMENT!) | 14:41 | |
TheJulia | uzumaki: too much heat generated... | 14:41 |
uzumaki | you used it to heat you room? =O | 14:42 |
TheJulia | unless someone upgrades my AC power to three phase and maybe somehow some Liebert CRAC units appear | 14:42 |
* TheJulia has an idea, build a data center next to the house. Make it a mid-century themed data center | 14:42 | |
TheJulia | yes that is perfect | 14:42 |
TheJulia | the neighbors won't scream.... much | 14:42 |
uzumaki | mid century as in.. servers in wooden enclosures? with straw as heat sink? | 14:43 |
TheJulia | I was thinking more architecture | 14:43 |
TheJulia | https://www.tripsavvy.com/thmb/YtEHxAyPV5pM1t5UtfvMn1PpoWY=/1602x1068/filters:fill(auto,1)/modernisthouse-7c93fe2acea640358f8e038a8df1fe4d.jpg | 14:43 |
* dtantsur liked CERN's datacenter-in-container | 14:43 | |
* TheJulia thinks that specific house is down the street from the supermarket she frequents | 14:44 | |
* uzumaki wonders what dtantsur is talking about.. | 14:44 | |
dtantsur | uzumaki: they literally have a part of their data center in a shipping container :) or at least used to. arne_wiebalck? | 14:44 |
uzumaki | TheJulia, ^^ that actually looks dope, the aqua green color is so soothing to eyes | 14:44 |
*** livelace has quit IRC | 14:45 | |
bfournie | TheJulia: N 1 takes 10 seconds because of inability to get cipher suites - http://paste.openstack.org/show/793522/ | 14:45 |
uzumaki | dtantsur, Oh! reminds me one of those microsoft underwater data centers.. whatever happened to those.. | 14:45 |
dtantsur | bfournie: cause of -N 10 | 14:45 |
dtantsur | one retry after 10 secs | 14:45 |
uzumaki | I want to visit CERN so bad.. maybe meet arne_wiebalck other there | 14:48 |
uzumaki | over* | 14:48 |
TheJulia | oh, apparently I can stream desert flippers... | 14:49 |
TheJulia | It is funny when I see people I know on that show | 14:49 |
iurygregory | OpenStack Ussuri is officially released! | 14:51 |
iurygregory | \o/ | 14:51 |
iurygregory | wait there is an openstack-announce list? O.o | 14:51 |
TheJulia | there are many lists | 14:52 |
dtantsur | lol | 14:52 |
TheJulia | even lists people can't subscribe to! | 14:52 |
dtantsur | wait wut? | 14:52 |
iurygregory | =O | 14:52 |
bfournie | dtantsur: yeah, even worse as ipmitool increases the time for each retry - R * (N + (R - 1)/2) | 14:52 |
TheJulia | dtantsur: board related | 14:52 |
gmann | TheJulia: dtantsur iurygregory rpittau i saw you backported the pep8 new check code fix in ussuri and stable/train is passing even with latest pycodestyle. I have not checked other older stable but if those are failing the this can be backported (I am not backporting as things are working now) https://review.opendev.org/#/c/727206/ | 14:52 |
patchbot | patch 727206 - ironic - Cap pycodestyle explicitly to be < 2.6.0 (MERGED) - 3 patch sets | 14:52 |
rpittau | thanks gmann | 14:53 |
iurygregory | gmann, tks! | 14:54 |
gmann | I will add hacking min version on master as 3.0.1 so that it new flake8 3.9.0 would break like this. | 14:54 |
*** kbaegis has quit IRC | 14:58 | |
*** ociuhandu has joined #openstack-ironic | 15:01 | |
*** priteau has joined #openstack-ironic | 15:01 | |
uzumaki | iurygregory, Ussuri is out for reals? =O | 15:03 |
iurygregory | uzumaki, yup! | 15:03 |
iurygregory | uzumaki, http://lists.openstack.org/pipermail/openstack-announce/2020-May/002035.html | 15:03 |
uzumaki | Nooicceee | 15:03 |
priteau | On https://www.openstack.org/software/ussuri/ it says: "Ironic (bare metal provisioning) added automated bare metal hardware provisioning" | 15:05 |
TheJulia | dafuq | 15:05 |
uzumaki | priteau, isn't that what ironic kinda... does? | 15:05 |
priteau | What is this referring to? Is there a mistake and it was meant to say "automated hardware decommissioning" | 15:05 |
TheJulia | I don't know... normally they are really good about running the press release past the ptl before sending it out | 15:06 |
uzumaki | it's too weird even when you read it... | 15:06 |
rpittau | hey they finally found out! | 15:07 |
iurygregory | hehehe | 15:07 |
priteau | Worth asking on #openstack-release? Not sure the people doing the web summary are there | 15:07 |
uzumaki | it's like saying.. windows is now an operating system xD | 15:07 |
TheJulia | I'm actually emailing robert and allison now | 15:08 |
dtantsur | heh | 15:08 |
TheJulia | email sent | 15:09 |
uzumaki | so my name isn't in the contributor's list? :'( | 15:09 |
TheJulia | uzumaki: if you have a link to the patch we can check into it | 15:10 |
uzumaki | https://review.opendev.org/#/c/713450/ | 15:10 |
patchbot | patch 713450 - ironic - Implement the bios-interface for idrac-wsman driver (MERGED) - 24 patch sets | 15:10 |
TheJulia | yeah, I just searched my email, I never got a preview into the press release | 15:10 |
TheJulia | sigh | 15:11 |
uzumaki | hopefully, the can't isn't out of the bag... yet? | 15:12 |
uzumaki | cat* | 15:12 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf https://review.opendev.org/716889 | 15:13 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf https://review.opendev.org/716889 | 15:14 |
openstackgerrit | Verification of a change to openstack/sushy failed: Update lower-constraints.txt https://review.opendev.org/727240 | 15:15 |
TheJulia | uzumaki: so that patch was after it looks like they froze the membership and likely generated the list. The deadline for adding a new contributor in the election was march 23rd-march 27th, which is likely where that got frozen at | 15:15 |
TheJulia | uzumaki: on an awesome side, you should definitely be in the victoria list | 15:16 |
uzumaki | oh I see.. :( | 15:16 |
TheJulia | Allison confirmed, they typod what they posted in the public press release | 15:16 |
TheJulia | they are fixing this morning | 15:16 |
* uzumaki *sighs* | 15:16 | |
TheJulia | Apparently they caught it in the briefing deck for the community call earlier today | 15:17 |
TheJulia | uzumaki: would you like me to ask when the list of names was generated/pulled? | 15:17 |
uzumaki | sure, since the patch did make it into the U release, I checked | 15:18 |
openstackgerrit | Merged openstack/ironic master: Update lower-constraints.txt https://review.opendev.org/727220 | 15:21 |
openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Stop calling LOG.info on every function calls https://review.opendev.org/727755 | 15:21 |
TheJulia | uzumaki: Once i finish my current email I'll ask | 15:21 |
uzumaki | thank you! | 15:23 |
iurygregory | rpittau, when https://review.opendev.org/#/c/724822 merges I just need to update the commit in https://github.com/openstack/python-ironic-inspector-client/blob/master/functest-requirements.txt and update https://review.opendev.org/#/c/727161/ right? | 15:25 |
patchbot | patch 724822 - ironic-inspector - Use latest version of python construct - 4 patch sets | 15:25 |
patchbot | patch 727161 - python-ironic-inspector-client - Fix pep8 - 3 patch sets | 15:25 |
dtantsur | folks, a simple patch that has been around for a while: https://review.opendev.org/720405 | 15:25 |
patchbot | patch 720405 - networking-baremetal - Remove the unused oslo.i18n bits - 1 patch set | 15:25 |
rpittau | iurygregory: correct | 15:26 |
iurygregory | rpittau, cool | 15:26 |
rpittau | isn't it nice when you update the lower-constraints with the pkgs of the lower-constraints job and then the lower-constraints job fails? | 15:27 |
*** sshnaidm is now known as sshnaidm|afk | 15:28 | |
iurygregory | rpittau, do you have a link for the patch? | 15:28 |
TheJulia | uzumaki: asked | 15:30 |
TheJulia | iurygregory: email sent to ml | 15:30 |
TheJulia | may I close my email now? | 15:30 |
iurygregory | TheJulia, awesome =) | 15:30 |
iurygregory | you should =) | 15:30 |
iurygregory | rpittau, is the inspector patch ? | 15:30 |
iurygregory | maybe we should test after we merge https://review.opendev.org/720791 =) | 15:31 |
patchbot | patch 720791 - ironic-inspector - Stop configuring install_command in tox. - 4 patch sets | 15:31 |
uzumaki | TheJulia, great, let's see what they say | 15:33 |
TheJulia | uzumaki: they are investigating, they generated it april 29th, so your name should be in the lsit | 15:33 |
TheJulia | I sent them the data, so hopefully they will figure it out and fix it | 15:34 |
* TheJulia may also owe a foundation staffer another margharita | 15:34 | |
*** dtantsur is now known as dtantsur|afk | 15:36 | |
dtantsur|afk | o/ | 15:36 |
uzumaki | TheJulia, thank you so much <3 | 15:36 |
uzumaki | \o dtantsur|afk | 15:36 |
*** ftarasenko has joined #openstack-ironic | 15:40 | |
*** Qianbiao has quit IRC | 15:43 | |
*** gyee has joined #openstack-ironic | 15:43 | |
*** Qianbiao has joined #openstack-ironic | 15:44 | |
openstackgerrit | Merged openstack/ironic-prometheus-exporter stable/train: Stop calling LOG.info on every function calls https://review.opendev.org/727773 | 15:44 |
openstackgerrit | Merged openstack/ironic-prometheus-exporter stable/ussuri: Stop calling LOG.info on every function calls https://review.opendev.org/727772 | 15:46 |
Qianbiao | hello, anyone still online? | 15:54 |
Qianbiao | ironic-dbsync seems has problem. | 15:54 |
Qianbiao | see http://paste.openstack.org/show/793532/ | 15:55 |
Qianbiao | on master branch | 15:55 |
*** Qianbiao has quit IRC | 15:56 | |
*** Qianbiao has joined #openstack-ironic | 15:57 | |
iurygregory | O.o | 15:57 |
Qianbiao | hello, am i online? | 15:57 |
iurygregory | strange | 15:57 |
iurygregory | you are getting 1 but the command doesn't fail? | 15:58 |
Qianbiao | hi iurygregory | 15:58 |
Qianbiao | i thought my network has problems | 15:59 |
Qianbiao | yes, it's wierd. | 15:59 |
Qianbiao | it should return 0 when success. | 15:59 |
Qianbiao | but it says nothing | 15:59 |
Qianbiao | on unsuri, even nothing to upgrade, it will print some message. | 15:59 |
*** lucasagomes has quit IRC | 15:59 | |
Qianbiao | but on master it prints nothing, and return 1 | 16:00 |
iurygregory | maybe because we don't have anything it would upgrade I would say | 16:00 |
iurygregory | so we don't have a message and the workflow assumes it's a problem | 16:00 |
Qianbiao | on unsuri, even nothing to upgrade, it will print messages too. | 16:00 |
Qianbiao | i forget the message content. but it did. | 16:01 |
iurygregory | got it | 16:01 |
Qianbiao | no matter whether there are things to upgrade, it should not return 1 i think. | 16:02 |
Qianbiao | Need to go, it's midnight here. | 16:03 |
Qianbiao | see u. bye. | 16:03 |
*** Qianbiao is now known as Qianbiao|afk | 16:03 | |
*** Qianbiao|afk has quit IRC | 16:05 | |
TheJulia | that seems super weird | 16:07 |
iurygregory | I don't have any problems in my devstack using master | 16:16 |
iurygregory | http://paste.openstack.org/show/793534/ | 16:16 |
iurygregory | O.o | 16:17 |
rpittau | good night! o/ | 16:21 |
*** rpittau is now known as rpittau|afk | 16:21 | |
*** ociuhandu has quit IRC | 16:42 | |
*** ociuhandu has joined #openstack-ironic | 16:43 | |
*** ociuhandu has quit IRC | 16:43 | |
*** ociuhandu has joined #openstack-ironic | 16:43 | |
*** derekh has quit IRC | 17:03 | |
openstackgerrit | Merged openstack/ironic stable/stein: Move ipmi logging to a separate option https://review.opendev.org/717323 | 17:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add ussuri release notes version https://review.opendev.org/726978 | 17:16 |
TheJulia | Any objection if I workflow+1 https://review.opendev.org/#/c/725949/ | 17:17 |
patchbot | patch 725949 - ironic-specs - Propose iso booting - 2 patch sets | 17:17 |
TheJulia | it has 2x +2, and I can do a follow-up for clarity | 17:17 |
arne_wiebalck | dtantsur|afk: uzumaki : yes, we have a few thousand servers in containers | 17:18 |
uzumaki | TheJulia, they've fixed the page now, the typo as well as my name is on there =) THANK YOU | 17:18 |
arne_wiebalck | dtantsur|afk: uzumaki : all nodes we have in there are in Ironic :) | 17:18 |
TheJulia | uzumaki: awesome | 17:18 |
uzumaki | https://www.openstack.org/software/ussuri/ | 17:18 |
TheJulia | \o/ | 17:18 |
uzumaki | arne_wiebalck, that is SO AWESOME | 17:18 |
*** amoralej is now known as amoralej|off | 17:18 | |
arne_wiebalck | the containers are quite impressive :) | 17:19 |
arne_wiebalck | very space efficient | 17:19 |
uzumaki | is it too weird that every time you say container I keep thinking about docker/k8s? | 17:19 |
arne_wiebalck | yep | 17:19 |
uzumaki | xD | 17:19 |
TheJulia | arne_wiebalck: I think... was it... Tim that tweeted a photo? | 17:19 |
arne_wiebalck | me too :) | 17:19 |
arne_wiebalck | Thomas I think ... let me find it ... | 17:20 |
uzumaki | did you say a few thousaaaaa.... =O | 17:20 |
TheJulia | I've been thinking the same thing since docker was only like 6 months old and was the new hot thing.... and I actually priced building a shipping container datacenter because our amazon bill was ABSURD | 17:20 |
*** ociuhandu has quit IRC | 17:21 | |
*** ociuhandu has joined #openstack-ironic | 17:23 | |
arne_wiebalck | https://twitter.com/thomasnomas/status/1126775320000909314 | 17:23 |
uzumaki | I think they wanted to make a wordplay/joke on Docker xD as it was gaining popularity.. arne_wiebalck walks in to an cloud infra bar and he's like "Hey, I've got a containerized data center.. no, i mean literally man" xD | 17:23 |
arne_wiebalck | lol | 17:23 |
uzumaki | =D | 17:23 |
arne_wiebalck | bye everyone, have a nice evening o/ | 17:24 |
TheJulia | o/ | 17:24 |
uzumaki | au revoir arne_wiebalck and thank you for the insight on the containerized matter | 17:25 |
uzumaki | \o | 17:25 |
openstackgerrit | Merged openstack/sushy master: Update lower-constraints.txt https://review.opendev.org/727240 | 17:27 |
uzumaki | ^^^ that comment on the tweet about jobs in containers in containers in containers is just hilarious xD | 17:27 |
*** ociuhandu has quit IRC | 17:33 | |
*** ociuhandu has joined #openstack-ironic | 17:34 | |
*** priteau has quit IRC | 17:38 | |
*** ociuhandu has quit IRC | 17:40 | |
*** k_mouza has quit IRC | 17:40 | |
*** kozhukalov has joined #openstack-ironic | 18:04 | |
*** rloo has quit IRC | 18:11 | |
*** rloo has joined #openstack-ironic | 18:11 | |
*** ociuhandu has joined #openstack-ironic | 18:32 | |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Fix OEM extension loading for different servers https://review.opendev.org/727738 | 18:38 |
*** ociuhandu has quit IRC | 18:40 | |
*** ociuhandu has joined #openstack-ironic | 18:40 | |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Fix OEM extension loading for different servers https://review.opendev.org/727738 | 18:41 |
iurygregory | I love when people put a -1 in a patch that the CI failed and they don't even put a comment XD | 18:46 |
TheJulia | iurygregory: perhaps time for rage against the machine? | 18:54 |
iurygregory | TheJulia, yup | 18:55 |
TheJulia | https://www.youtube.com/watch?v=bWXazVhlyxQ | 18:55 |
TheJulia | just enough time before next meeting | 18:55 |
iurygregory | ++ | 18:56 |
*** rloo has quit IRC | 18:58 | |
*** rloo has joined #openstack-ironic | 18:59 | |
*** kozhukalov has quit IRC | 19:02 | |
*** livelace has joined #openstack-ironic | 19:09 | |
uzumaki | iurygregory, they do that? who is the -1 for then? | 19:21 |
iurygregory | uzumaki, see for yourself https://review.opendev.org/726121 =) | 19:22 |
patchbot | patch 726121 - ironic-inspector - Merge jobs - 2 patch sets | 19:22 |
uzumaki | iurygregory, lol that's hilarious | 19:23 |
iurygregory | yup =) | 19:23 |
uzumaki | TheJulia, the first comment on the video says " This song makes me wanna read the terms and conditions then DECLINE " LMAO | 19:23 |
uzumaki | " I fired my boss after listening to this " OMG these comments are hilarious.. =)) | 19:24 |
*** ociuhandu has quit IRC | 19:27 | |
*** ociuhandu has joined #openstack-ironic | 19:28 | |
*** ociuhandu has quit IRC | 19:39 | |
*** ociuhandu has joined #openstack-ironic | 19:40 | |
*** ociuhandu has quit IRC | 19:40 | |
*** ociuhandu has joined #openstack-ironic | 19:40 | |
iurygregory | uzumaki, that's the poing of Rage Against The Machine \o/ | 19:46 |
iurygregory | bye everyone o/ | 19:46 |
uzumaki | adios iurygregory ! Yasss that's the whole point! \o/ | 19:47 |
*** ociuhandu has quit IRC | 19:51 | |
*** ociuhandu has joined #openstack-ironic | 19:52 | |
*** ociuhandu has quit IRC | 19:52 | |
*** ociuhandu has joined #openstack-ironic | 19:52 | |
TheJulia | stepping away, need to take my wife to the doctor | 20:00 |
*** ociuhandu has quit IRC | 20:02 | |
*** ociuhandu has joined #openstack-ironic | 20:03 | |
*** ociuhandu has quit IRC | 20:03 | |
*** ociuhandu has joined #openstack-ironic | 20:03 | |
uzumaki | hope nothing too serious... | 20:05 |
*** ociuhandu has quit IRC | 20:14 | |
*** ociuhandu has joined #openstack-ironic | 20:15 | |
*** ociuhandu has quit IRC | 20:26 | |
*** ociuhandu has joined #openstack-ironic | 20:27 | |
*** ociuhandu has quit IRC | 20:27 | |
*** ociuhandu has joined #openstack-ironic | 20:27 | |
*** ociuhandu has quit IRC | 20:37 | |
*** ociuhandu has joined #openstack-ironic | 20:38 | |
*** ociuhandu has quit IRC | 20:38 | |
*** ociuhandu has joined #openstack-ironic | 20:39 | |
*** ociuhandu has quit IRC | 20:49 | |
*** ociuhandu has joined #openstack-ironic | 20:50 | |
*** ociuhandu has quit IRC | 20:50 | |
*** ociuhandu has joined #openstack-ironic | 20:50 | |
*** ociuhandu has quit IRC | 21:01 | |
*** ociuhandu has joined #openstack-ironic | 21:02 | |
*** ociuhandu has quit IRC | 21:07 | |
*** livelace has quit IRC | 21:11 | |
*** livelace has joined #openstack-ironic | 21:17 | |
*** livelace has quit IRC | 21:18 | |
*** Goneri has quit IRC | 22:07 | |
*** Goneri has joined #openstack-ironic | 22:10 | |
*** alexmcleod has quit IRC | 22:24 | |
TheJulia | eh, life | 22:35 |
TheJulia | and my terminal windows all crashed while I was away | 22:35 |
uzumaki | that just sucks! | 22:35 |
openstackgerrit | Merged openstack/ironic master: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/725239 | 22:39 |
*** tkajinam has joined #openstack-ironic | 22:46 | |
rpioso | ^^^ \o/ | 22:49 |
rpioso | TheJulia: When time permits, please take a look at https://review.opendev.org/#/c/727738 | 22:50 |
patchbot | patch 727738 - sushy - Fix OEM extension loading for different servers - 3 patch sets | 22:50 |
rpioso | Without it, only one server can be managed. | 22:52 |
rpioso | TheJulia: Are there any other core reviewers you could recommend look at sushy stuffs while etingof is away? | 22:54 |
TheJulia | off the top of my head, I don't konw but we can figure something out | 22:56 |
TheJulia | worst comes to worst we'll single core review it if we have to | 22:56 |
TheJulia | I'll have to look in the morning, I just got home and feel like someone is drilling into my skull | 22:57 |
rpioso | TheJulia: Thank you! It's a subtle, somewhat disorienting bug -- stevedore, entry points, factories galore, and caching. | 22:58 |
TheJulia | eww | 22:59 |
TheJulia | sounds like it | 22:59 |
* rpioso hopes TheJulia and her wife soon feel better | 22:59 | |
TheJulia | So... a morning espresso based beverage and code review | 22:59 |
rpioso | A most excellent plan :-) | 23:00 |
* rpioso goes for a walk. We're burning daylight in New England. | 23:01 | |
TheJulia | ++ | 23:02 |
*** threestrands has joined #openstack-ironic | 23:29 | |
*** rcernin has quit IRC | 23:31 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: WIP - Victoria Cycle Priorit(y|ies) https://review.opendev.org/720100 | 23:32 |
TheJulia | gah, it became a longish list | 23:32 |
*** rcernin has joined #openstack-ironic | 23:32 | |
*** ociuhandu has joined #openstack-ironic | 23:36 | |
*** early has quit IRC | 23:38 | |
*** ociuhandu has quit IRC | 23:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!