Wednesday, 2020-05-13

« Tuesday, 2020-05-12 Index Thursday, 2020-05-14 »
*** k_mouza has quit IRC00:13
*** ociuhandu has joined #openstack-ironic00:36
*** Goneri has quit IRC00:41
*** ociuhandu has quit IRC00:44
*** tzumainn has quit IRC00:44
openstackgerrithuwenhui 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/72754700:46
openstackgerrithuwenhui 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/72754701:12
*** k_mouza has joined #openstack-ironic01:14
*** livelace has quit IRC01:18
*** k_mouza has quit IRC01:20
openstackgerrithuwenhui proposed openstack/ironic master: Fix the wrong description  https://review.opendev.org/72754701:23
*** ericlei has joined #openstack-ironic01:43
*** ericlei has quit IRC01:45
*** jdandrea has quit IRC02:14
*** gyee has quit IRC03:18
*** dustinc has quit IRC03:34
*** cdearborn has quit IRC03:53
*** ociuhandu has joined #openstack-ironic04:42
*** dmellado has quit IRC04:45
*** ociuhandu has quit IRC04:49
*** k_mouza has joined #openstack-ironic05:16
*** k_mouza has quit IRC05:20
arne_wiebalckTheJulia: 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_wiebalckGood morning, ironic!06:21
*** ociuhandu has joined #openstack-ironic06:30
*** ericlei has joined #openstack-ironic06:45
*** ericlei has quit IRC06:51
*** Lucas_Gray has joined #openstack-ironic07:03
*** yolanda has joined #openstack-ironic07:21
*** Lucas_Gray has quit IRC07:22
iurygregorygood morning arne_wiebalck and Ironic o/07:27
*** ociuhandu has quit IRC07:27
*** ociuhandu_ has joined #openstack-ironic07:27
iurygregorystevebaker, hey sorry I went to sleep =), I haven't started anything XD07:29
*** rpittau|afk is now known as rpittau07:29
rpittaugood morning ironic! o/07:29
*** amoralej|off is now known as amoralej07:34
*** aedc has quit IRC07:37
*** dmellado has joined #openstack-ironic07:37
*** aedc has joined #openstack-ironic07:47
rpittauanyone around to please approve this https://review.opendev.org/72717107:57
patchbotpatch 727171 - ironic-python-agent - CI: raise DIB jobs memory to 3 GiB - 1 patch set07:57
rpittaupretty please :)07:57
*** lucasagomes has joined #openstack-ironic07:58
*** livelace has joined #openstack-ironic08:12
rpittauwe also need this kind of urgently as we probably need to backport it https://review.opendev.org/72675308:15
patchbotpatch 726753 - ironic (stable/ussuri) - Add timeout and retries to JSON RPC client - 1 patch set08:15
rpittauwhoops wrong08:15
rpittauthis one https://review.opendev.org/72720608:15
patchbotpatch 727206 - ironic - Cap pycodestyle explicitly to be < 2.6.0 - 3 patch sets08:15
iurygregorygood morning rpittau o/08:19
rpittauhey iurygregory :)08:19
*** Qianbiao has joined #openstack-ironic08:19
openstackgerritRiccardo Pittau proposed x/sushy-oem-idrac master: Update lower-constraints.txt  https://review.opendev.org/72726008:24
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Update lower-constraints.txt  https://review.opendev.org/72725308:30
*** xiaolin has quit IRC08:31
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Test CI  https://review.opendev.org/72766108:32
iurygregoryrpittau, I'm trying to test the functional test locally =)08:36
iurygregorydevstack running so I can run the tests08:36
openstackgerritRiccardo Pittau proposed openstack/networking-baremetal master: Bump hacking version to 3.0.0 and fix pep8 test  https://review.opendev.org/72715208:41
*** xiaolin has joined #openstack-ironic08:45
*** livelace has quit IRC08:50
*** riuzen has joined #openstack-ironic08:56
*** livelace has joined #openstack-ironic08:57
*** livelace has quit IRC09:02
*** livelace has joined #openstack-ironic09:08
*** xiaolin has quit IRC09:10
*** livelace has quit IRC09:13
*** dtantsur|afk is now known as dtantsur09:13
dtantsurmorning ironic09:13
iurygregorygood morning dtantsur09:14
iurygregorydtantsur, we need to backport https://review.opendev.org/#/c/727141/ to ussuri right?09:16
patchbotpatch 727141 - ironic - Fix pep8 (MERGED) - 6 patch sets09:16
dtantsuriurygregory: yep09:16
iurygregorydtantsur, ack working here since has some conflicts09:16
rpittauI thought we were capping pycodestyle ?09:20
openstackgerritAija Jaunteva proposed openstack/ironic master: Remove iDRAC Redfish boot device known issue  https://review.opendev.org/72767609:21
openstackgerritMerged openstack/ironic-lib master: image_convert: retry resource unavailable and make RLIMIT configurable  https://review.opendev.org/72637809:22
openstackgerritMerged openstack/ironic-inspector master: Make POST /v1/introspection synchronous when manage_boot==False  https://review.opendev.org/72715109:22
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/ussuri: image_convert: retry resource unavailable and make RLIMIT configurable  https://review.opendev.org/72767709:22
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/train: image_convert: retry resource unavailable and make RLIMIT configurable  https://review.opendev.org/72767809:22
openstackgerritVerification of a change to openstack/ironic failed: Improve `redfish` set-boot-mode implementation  https://review.opendev.org/72736209:26
rpittaudtantsur: if we backport the fixes for flake8/pycodestyle (not only in ironic), then no need to cap the version ?09:27
dtantsurrpittau: probably no (or only to the next version)09:33
*** livelace has joined #openstack-ironic09:34
rpittaudtantsur: then I better start backporting all the pep8 fixes from yesterday09:34
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector stable/ussuri: Make POST /v1/introspection synchronous when manage_boot==False  https://review.opendev.org/72768209:36
openstackgerritRiccardo Pittau proposed openstack/ironic-lib stable/ussuri: Fix pep8 test  https://review.opendev.org/72768309:36
rpittauarne_wiebalck: thanks for the reviews! if you have one more minute please check https://review.opendev.org/72711709:38
patchbotpatch 727117 - ironic-python-agent - Fix pep8 errors - 1 patch set09:38
arne_wiebalckrpittau: the others were a few lines ... this is 600 lines09:41
rpittauarne_wiebalck: it's just copy paste of stuff, no code change09:41
arne_wiebalckI know :)09:42
iurygregoryarne_wiebalck, is a bit afraid09:42
iurygregorydon't be =)09:42
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic stable/ussuri: Fix pep8  https://review.opendev.org/72768909:42
dtantsurarne_wiebalck: hey! how long does raid.delete_configuration usually take for you?09:42
arne_wiebalckhave no fear, and press W+1!09:42
dtantsursomeone tells me that they run delete_configuration+erase_device_metadata+create_configuration and they take 13 mins, 11 mins and 5 mins accordingly09:42
rpittauarne_wiebalck: thanks! :)09:43
arne_wiebalckdtantsur: this is for software RAID?09:43
dtantsuryeah09:43
arne_wiebalckthese commands should be all really fast09:44
arne_wiebalck(and they are for us)09:44
arne_wiebalckthis is definitely worth a closer look as it seems way too loong09:45
arne_wiebalck(even my long is too long)09:45
openstackgerritRiccardo Pittau proposed openstack/sushy stable/ussuri: Fix pep8 test  https://review.opendev.org/72769309:50
*** hjensas|afk is now known as hjensas09:51
iurygregoryrpittau, 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
iurygregorydid we bump construct in requirements?09:55
rpittauiurygregory: yeah, I'm seeing the same, we did remove the limitation on construct in requirements10:00
*** ociuhandu_ has quit IRC10:01
openstackgerritMerged openstack/ironic master: Fix the wrong description  https://review.opendev.org/72754710:01
rpittauI guess we can work around it adding the cap in ironic-inspector-client for the functional test10:02
*** ociuhandu has joined #openstack-ironic10:02
iurygregoryYeah I'm trying this10:03
rpittauI'm working on the construct patch anyway, that should remove the need for the cap10:09
openstackgerritMerged openstack/metalsmith master: Update lower-constraints.txt  https://review.opendev.org/72725110:18
openstackgerritMerged openstack/ironic-python-agent master: CI: raise DIB jobs memory to 3 GiB  https://review.opendev.org/72717110:19
*** derekh has joined #openstack-ironic10:20
iurygregoryrpittau, yeah if we get working we won't need10:24
rpittauI'm missing a couple of things, but it's pretty much done10:24
openstackgerritMerged openstack/ironic master: Cap pycodestyle explicitly to be < 2.6.0  https://review.opendev.org/72720610:37
*** k_mouza has joined #openstack-ironic10:47
*** Qianbiao has quit IRC10:50
*** Qianbiao has joined #openstack-ironic10:50
*** rpittau is now known as rpittau|bbl11:04
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Use latest version of python construct  https://review.opendev.org/72482211:04
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Consistently handle errors in the deploy process  https://review.opendev.org/72771411:17
*** ericlei has joined #openstack-ironic11:21
*** ociuhandu has quit IRC11:23
openstackgerritMerged openstack/sushy-cli master: Update lower-constraints.txt  https://review.opendev.org/72724211:26
openstackgerritMerged openstack/ironic-python-agent master: Fix pep8 errors  https://review.opendev.org/72711711:27
openstackgerritMerged openstack/ironic-inspector stable/ussuri: Update .gitreview for stable/ussuri  https://review.opendev.org/72356811:27
openstackgerritMerged openstack/ironic-lib stable/ussuri: Fix pep8 test  https://review.opendev.org/72768311:27
*** uzumaki has joined #openstack-ironic11:36
*** ericlei has quit IRC11:37
openstackgerritVerification of a change to openstack/ironic-inspector failed: Avoid using introspection start delays with non-managed boot  https://review.opendev.org/72720911:44
openstackgerritMerged openstack/networking-baremetal master: Add unit tests for _get_notification_transport_url()  https://review.opendev.org/63584411:54
openstackgerritVerification of a change to openstack/sushy failed: Update lower-constraints.txt  https://review.opendev.org/72724011:55
*** amoralej is now known as amoralej|lunch11:58
openstackgerritAija Jaunteva proposed openstack/sushy master: [WIP] Fix OEM extension loading for different servers  https://review.opendev.org/72773811:59
*** ociuhandu has joined #openstack-ironic12:04
*** ociuhandu has quit IRC12:10
*** ociuhandu has joined #openstack-ironic12:10
*** sshnaidm|afk is now known as sshnaidm12:16
*** rpittau|bbl is now known as rpittau12:19
*** livelace has quit IRC12:23
openstackgerritMerged openstack/sushy-tools master: Update lower-constraints.txt  https://review.opendev.org/72724112:23
*** tkajinam has quit IRC12:24
*** Qianbiao has quit IRC12:25
*** ociuhandu has quit IRC12:27
*** ociuhandu has joined #openstack-ironic12:28
*** livelace has joined #openstack-ironic12:29
openstackgerritMerged openstack/ironic-lib stable/train: image_convert: retry resource unavailable and make RLIMIT configurable  https://review.opendev.org/72767812:36
iurygregoryrpittau, I think you missed the release note for https://review.opendev.org/#/c/724822/12:39
patchbotpatch 724822 - ironic-inspector - Use latest version of python construct - 3 patch sets12:39
iurygregoryawesome to see working at least \o/12:40
rpittauiurygregory: I did, I know, I'm going to add one12:40
iurygregoryrpittau, I was about to +2 XD and "oh release note"12:40
rpittauiurygregory: yeah, thanks for reviewing :)12:40
rpittaufeel free to -1 me :D12:40
iurygregorynah12:41
iurygregoryyou have 5min =)12:41
rpittaugosh, no pressure :)12:41
iurygregoryhehehe12:42
iurygregoryor a -1 XD12:42
rpittau:(12:42
iurygregoryrpittau, but if you offer me some lasagna you have a day12:43
rpittauiurygregory: I see, we're actually finally going to make tiramisu again this week after 2 months, up to you :)12:44
iurygregoryrpittau, wow =O sounds like a deal =P12:45
rpittauiurygregory: if you're ever in Versailles ;)12:49
iurygregorymaybe one day hehehe12:50
dtantsurI'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-ironic12:53
*** ianychoi has quit IRC12:55
*** amoralej|lunch is now known as amoralej13:00
rpittaudtantsur: I'd be glad to take you on the palace tour :D13:03
dtantsur++13:03
*** Qianbiao has joined #openstack-ironic13:03
*** jdandrea has joined #openstack-ironic13:07
*** riuzen has quit IRC13:11
openstackgerritDmitry Tantsur proposed openstack/ironic-prometheus-exporter master: Stop calling LOG.info on every function calls  https://review.opendev.org/72775513:16
dtantsuriurygregory: ^^13:16
iurygregorydtantsur, maybe change for debug?13:17
dtantsuriurygregory: most people have debug on, so it won't help13:17
dtantsurI don't think why we need to log every input we process13:17
dtantsur* see13:17
iurygregoryfor me it was more like a debug XD13:18
iurygregorybut we don't really need all13:18
dtantsuriurygregory: 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 total13:18
iurygregorydtantsur, gotcha13:19
* iurygregory wondering who is running the exporter O.o13:19
dtantsuriurygregory: metal3 apparently?13:19
iurygregoryoh13:19
*** ociuhandu has quit IRC13:24
*** ericlei has joined #openstack-ironic13:24
TheJuliagood morning13:25
rpittaugood morning TheJulia :)13:25
TheJuliaiurygregory: seems like you should have +2 rights on IPE?13:26
iurygregorygood morning TheJulia13:26
iurygregorymaybe13:27
iurygregorynot sure if it fits in inspector/sushy/bifrost13:28
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Use latest version of python construct  https://review.opendev.org/72482213:29
TheJuliaany objections folks if I give iurygregory rights to the project he put lots of energy into?13:30
TheJuliaonly seems right13:30
rpittauI'm actually surprised he was not there already, please do :)13:30
TheJulialooks like it requires another group to be created13:34
iurygregorythat's what I was thinking ^13:34
dtantsurmorning TheJulia13:35
dtantsurTheJulia: wait, he's not there? Oo13:35
* iurygregory not sure if it's ok to create a new group13:35
dtantsurit's okay to create a new group, although we can take a trusted delegation approach13:35
iurygregorydtantsur, nope =) you can get my +1 and imagine that is a +213:35
dtantsuri.e. give iurygregory ironic-core and ask to only approve things he's responsible for13:35
dtantsurthis is how tripleo does it (I have +2 this way there)13:36
iurygregorybifrost has a group for itself, maybe rename and add there?13:37
TheJuliaI think the trusted delegation approach is the right path to take13:38
TheJuliaiurygregory: what do you think?13:38
iurygregoryTheJulia, I'm ok with that if the community is ok =)13:38
* TheJulia expects iury is opening a celebration beer at the moment13:38
* iurygregory will have beer at the dinner13:39
dtantsurthe benefit of the trusted delegation path is that iurygregory can approve trivial patches on other projects as well13:41
TheJulia++13:41
* TheJulia makes a calendar item to send an email13:42
iurygregoryack o/13:42
*** ericlei has quit IRC13:42
*** tzumainn has joined #openstack-ironic13:45
dtantsurthe 10th recheck has failed for https://review.opendev.org/#/c/725239/ time to open a bottle? >_<13:54
patchbotpatch 725239 - ironic - redfish: handle hardware that is unable to set per... - 11 patch sets13:54
dtantsurand people say that unlucky places do not exist, it's just coincidences...13:55
rpittauon a positive note, my swear jar is full13:55
dtantsur:D13:55
dtantsurI need to get one, maybe I can finally take mortgage after a year?13:55
rpittaueheh13:56
iurygregorydtantsur, they exist in the CI world13:56
dtantsurindeed13:56
* TheJulia reflects and realizes she doesn't actually swear much normally13:56
iurygregorydtantsur, 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 unlucky13:58
iurygregorysorry to say that =(13:58
dtantsurdon't worry, I've known it for a while13:58
TheJuliadon't tell him he is unlucky! Tell him it will be okay and try to fix it!13:58
iurygregoryI know arne_wiebalck has some contract with zuul so maybe you can ask for some tips =)13:59
TheJuliaWithout digging into logs, could it just be we have too many jobs?13:59
iurygregoryyup we do13:59
TheJuliathe random package mirror can send things down an awful bad path13:59
TheJuliaseems like we can remove ironic-tempest-partition-bios-redfish-pxe14:02
dtantsurin favour of standalone-redfish?14:02
dtantsurTheJulia, bfournie, re the ipmitool cipher fun: should we maybe move retries from ipmitool to the ironic level?14:03
*** ociuhandu has joined #openstack-ironic14:05
TheJuliadtantsur: yeah14:05
TheJuliaalso, ironic-tempest-ipa-partition-pxe_ipmitool seems redundant compared to ironic-standalone14:05
dtantsurit is, if we have other partition jobs14:05
TheJuliadtthe standalone job has partition + rescue14:06
TheJuliawith iscsi14:06
TheJuliaso really the matrix covers it from the standalone job14:07
TheJuliaJust ideas, two jobs are two jobs14:07
dtantsurI'm (mildly) worried about the nova integratin bits14:07
TheJuliadtantsur: and we have other jobs that can serve that purpose14:07
dtantsurlikely so14:07
* iurygregory likes the idea14:08
TheJuliawe have ?10? jobs that cover those same bits14:08
openstackgerritDmitry Tantsur proposed openstack/ironic-prometheus-exporter stable/ussuri: Stop calling LOG.info on every function calls  https://review.opendev.org/72777214:08
openstackgerritDmitry Tantsur proposed openstack/ironic-prometheus-exporter stable/train: Stop calling LOG.info on every function calls  https://review.opendev.org/72777314:08
*** ociuhandu has quit IRC14:11
*** ociuhandu has joined #openstack-ironic14:12
*** ociuhandu has quit IRC14:18
bfourniedtantsur: 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 retry14:18
dtantsurright14:19
* TheJulia tries to remember the reasoning14:20
*** Goneri has joined #openstack-ironic14:20
TheJuliaI 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 cases14:24
bfourniedtantsur: 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 bug14:24
*** Qianbiao has quit IRC14:25
TheJuliaI guess we're shifting the pain point to something we can actually control14:26
*** Qianbiao has joined #openstack-ironic14:26
dtantsurright14:27
TheJuliaI believe the term "necessary evil" is here14:28
TheJuliabfournie: do you have any lab machines that are grumpy with ipmi?14:28
* TheJulia thinks HP ilo2s....14:28
TheJuliaalso, ilo2s realistically would only be in test labs... I think14:29
*** kbaegis has joined #openstack-ironic14:29
TheJuliawow, the last ilo2 firmware was released in 201814:29
bfournieTheJulia: Yes I can duplicate with the HP machine I have14:33
*** ociuhandu has joined #openstack-ironic14:33
TheJulialooks like the G5s were ~2009?14:33
TheJuliaG6 in 201014:34
TheJuliaso basically 10 year old gear, which means only the craziest people would still be running that stuff in production14:34
iurygregoryhehe14:35
iurygregorybut we all know how crazy people is14:35
uzumakiTheJulia, are we assuming that production people can't be crazy?14:35
iurygregorythey think is ok to put 999999999999999 for timeout =P14:35
uzumakiiurygregory, ^ exactly14:36
iurygregoryuzumaki, not crazy to have old gear... but everybody is a bit crazy =)14:36
bfournieTheJulia: this is a g914:37
uzumakioh, not crazy to have it, just don't put critical business application on it is all14:37
uzumakiuse it as a decoration piece by all means, or to play super mario on it14:37
iurygregoryXD14:37
TheJuliabfournie: not valid then14:37
TheJuliathe gen5-6 ilos were notoriously problematic14:37
TheJuliawhich is what drove a lot of the retry stuff in the first place14:37
TheJuliauzumaki: very true14:38
*** ociuhandu has quit IRC14:39
uzumakiolder 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 goes14:39
TheJuliaI 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 next14:39
TheJuliain otherwords, lets try -R 1 against current gear and see how horrible it may be14:39
TheJuliapsu integrated fans...14:40
TheJuliaThose generations were also where I think they introduced the always on fans14:40
* TheJulia had a few racks of these servers a lifetime ago14: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
TheJuliauzumaki: too much heat generated...14:41
uzumakiyou used it to heat you room? =O14:42
TheJuliaunless someone upgrades my AC power to three phase and maybe somehow some Liebert CRAC units appear14:42
* TheJulia has an idea, build a data center next to the house. Make it a mid-century themed data center14:42
TheJuliayes that is perfect14:42
TheJuliathe neighbors won't scream.... much14:42
uzumakimid century as in.. servers in wooden enclosures? with straw as heat sink?14:43
TheJuliaI was thinking more architecture14:43
TheJuliahttps://www.tripsavvy.com/thmb/YtEHxAyPV5pM1t5UtfvMn1PpoWY=/1602x1068/filters:fill(auto,1)/modernisthouse-7c93fe2acea640358f8e038a8df1fe4d.jpg14:43
* dtantsur liked CERN's datacenter-in-container14:43
* TheJulia thinks that specific house is down the street from the supermarket she frequents14:44
* uzumaki wonders what dtantsur is talking about..14:44
dtantsuruzumaki: they literally have a part of their data center in a shipping container :) or at least used to. arne_wiebalck?14:44
uzumakiTheJulia, ^^ that actually looks dope, the aqua green color is so soothing to eyes14:44
*** livelace has quit IRC14:45
bfournieTheJulia: N 1 takes 10 seconds because of inability to get cipher suites - http://paste.openstack.org/show/793522/14:45
uzumakidtantsur, Oh! reminds me one of those microsoft underwater data centers.. whatever happened to those..14:45
dtantsurbfournie: cause of -N 1014:45
dtantsurone retry after 10 secs14:45
uzumakiI want to visit CERN so bad.. maybe meet arne_wiebalck other there14:48
uzumakiover*14:48
TheJuliaoh, apparently I can stream desert flippers...14:49
TheJuliaIt is funny when I see people I know on that show14:49
iurygregoryOpenStack Ussuri is officially released!14:51
iurygregory\o/14:51
iurygregorywait there is an openstack-announce list? O.o14:51
TheJuliathere are many lists14:52
dtantsurlol14:52
TheJuliaeven lists people can't subscribe to!14:52
dtantsurwait wut?14:52
iurygregory=O14:52
bfourniedtantsur: yeah, even worse as ipmitool increases the time for each retry - R * (N + (R - 1)/2)14:52
TheJuliadtantsur: board related14:52
gmannTheJulia: 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
patchbotpatch 727206 - ironic - Cap pycodestyle explicitly to be < 2.6.0 (MERGED) - 3 patch sets14:52
rpittauthanks gmann14:53
iurygregorygmann, tks!14:54
gmannI 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 IRC14:58
*** ociuhandu has joined #openstack-ironic15:01
*** priteau has joined #openstack-ironic15:01
uzumakiiurygregory, Ussuri is out for reals? =O15:03
iurygregoryuzumaki, yup!15:03
iurygregoryuzumaki, http://lists.openstack.org/pipermail/openstack-announce/2020-May/002035.html15:03
uzumakiNooicceee15:03
priteauOn https://www.openstack.org/software/ussuri/ it says: "Ironic (bare metal provisioning) added automated bare metal hardware provisioning"15:05
TheJuliadafuq15:05
uzumakipriteau, isn't that what ironic kinda... does?15:05
priteauWhat is this referring to? Is there a mistake and it was meant to say "automated hardware decommissioning"15:05
TheJuliaI don't know... normally they are really good about running the press release past the ptl before sending it out15:06
uzumakiit's too weird even when you read it...15:06
rpittauhey they finally found out!15:07
iurygregoryhehehe15:07
priteauWorth asking on #openstack-release? Not sure the people doing the web summary are there15:07
uzumakiit's like saying.. windows is now an operating system xD15:07
TheJuliaI'm actually emailing robert and allison now15:08
dtantsurheh15:08
TheJuliaemail sent15:09
uzumakiso my name isn't in the contributor's list? :'(15:09
TheJuliauzumaki: if you have a link to the patch we can check into it15:10
uzumakihttps://review.opendev.org/#/c/713450/15:10
patchbotpatch 713450 - ironic - Implement the bios-interface for idrac-wsman driver (MERGED) - 24 patch sets15:10
TheJuliayeah, I just searched my email, I never got a preview into the press release15:10
TheJuliasigh15:11
uzumakihopefully, the can't isn't out of the bag... yet?15:12
uzumakicat*15:12
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf  https://review.opendev.org/71688915:13
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf  https://review.opendev.org/71688915:14
openstackgerritVerification of a change to openstack/sushy failed: Update lower-constraints.txt  https://review.opendev.org/72724015:15
TheJuliauzumaki: 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 at15:15
TheJuliauzumaki: on an awesome side, you should definitely be in the victoria list15:16
uzumakioh I see.. :(15:16
TheJuliaAllison confirmed, they typod what they posted in the public press release15:16
TheJuliathey are fixing this morning15:16
* uzumaki *sighs*15:16
TheJuliaApparently they caught it in the briefing deck for the community call earlier today15:17
TheJuliauzumaki: 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 checked15:18
openstackgerritMerged openstack/ironic master: Update lower-constraints.txt  https://review.opendev.org/72722015:21
openstackgerritMerged openstack/ironic-prometheus-exporter master: Stop calling LOG.info on every function calls  https://review.opendev.org/72775515:21
TheJuliauzumaki: Once i finish my current email I'll ask15:21
uzumakithank you!15:23
iurygregoryrpittau, 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
patchbotpatch 724822 - ironic-inspector - Use latest version of python construct - 4 patch sets15:25
patchbotpatch 727161 - python-ironic-inspector-client - Fix pep8 - 3 patch sets15:25
dtantsurfolks, a simple patch that has been around for a while: https://review.opendev.org/72040515:25
patchbotpatch 720405 - networking-baremetal - Remove the unused oslo.i18n bits - 1 patch set15:25
rpittauiurygregory: correct15:26
iurygregoryrpittau, cool15:26
rpittauisn'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|afk15:28
iurygregoryrpittau, do you have a link for the patch?15:28
TheJuliauzumaki: asked15:30
TheJuliaiurygregory: email sent to ml15:30
TheJuliamay I close my email now?15:30
iurygregoryTheJulia, awesome =)15:30
iurygregoryyou should =)15:30
iurygregoryrpittau, is the inspector patch ?15:30
iurygregorymaybe we should test after we merge https://review.opendev.org/720791 =)15:31
patchbotpatch 720791 - ironic-inspector - Stop configuring install_command in tox. - 4 patch sets15:31
uzumakiTheJulia, great, let's see what they say15:33
TheJuliauzumaki: they are investigating, they generated it april 29th, so your name should be in the lsit15:33
TheJuliaI sent them the data, so hopefully they will figure it out and fix it15:34
* TheJulia may also owe a foundation staffer another margharita15:34
*** dtantsur is now known as dtantsur|afk15:36
dtantsur|afko/15:36
uzumakiTheJulia, thank you so much <315:36
uzumaki\o dtantsur|afk15:36
*** ftarasenko has joined #openstack-ironic15:40
*** Qianbiao has quit IRC15:43
*** gyee has joined #openstack-ironic15:43
*** Qianbiao has joined #openstack-ironic15:44
openstackgerritMerged openstack/ironic-prometheus-exporter stable/train: Stop calling LOG.info on every function calls  https://review.opendev.org/72777315:44
openstackgerritMerged openstack/ironic-prometheus-exporter stable/ussuri: Stop calling LOG.info on every function calls  https://review.opendev.org/72777215:46
Qianbiaohello, anyone still online?15:54
Qianbiaoironic-dbsync seems has problem.15:54
Qianbiaosee http://paste.openstack.org/show/793532/15:55
Qianbiaoon master branch15:55
*** Qianbiao has quit IRC15:56
*** Qianbiao has joined #openstack-ironic15:57
iurygregoryO.o15:57
Qianbiaohello, am i online?15:57
iurygregorystrange15:57
iurygregoryyou are getting 1 but the command doesn't fail?15:58
Qianbiaohi iurygregory15:58
Qianbiaoi thought my network has problems15:59
Qianbiaoyes, it's wierd.15:59
Qianbiaoit should return 0 when success.15:59
Qianbiaobut it says nothing15:59
Qianbiaoon unsuri, even nothing to upgrade, it will print some message.15:59
*** lucasagomes has quit IRC15:59
Qianbiaobut on master it prints nothing, and return 116:00
iurygregorymaybe because we don't have anything it would upgrade I would say16:00
iurygregoryso we don't have a message and the workflow assumes it's a problem16:00
Qianbiao on unsuri, even nothing to upgrade, it will print messages too.16:00
Qianbiaoi forget the message content. but it did.16:01
iurygregorygot it16:01
Qianbiaono matter whether there are things to upgrade, it should not return 1 i think.16:02
QianbiaoNeed to go, it's midnight here.16:03
Qianbiaosee u. bye.16:03
*** Qianbiao is now known as Qianbiao|afk16:03
*** Qianbiao|afk has quit IRC16:05
TheJuliathat seems super weird16:07
iurygregoryI don't have any problems in my devstack using master16:16
iurygregoryhttp://paste.openstack.org/show/793534/16:16
iurygregoryO.o16:17
rpittaugood night! o/16:21
*** rpittau is now known as rpittau|afk16:21
*** ociuhandu has quit IRC16:42
*** ociuhandu has joined #openstack-ironic16:43
*** ociuhandu has quit IRC16:43
*** ociuhandu has joined #openstack-ironic16:43
*** derekh has quit IRC17:03
openstackgerritMerged openstack/ironic stable/stein: Move ipmi logging to a separate option  https://review.opendev.org/71732317:04
openstackgerritJulia Kreger proposed openstack/ironic master: Add ussuri release notes version  https://review.opendev.org/72697817:16
TheJuliaAny objection if I workflow+1 https://review.opendev.org/#/c/725949/17:17
patchbotpatch 725949 - ironic-specs - Propose iso booting - 2 patch sets17:17
TheJuliait has 2x +2, and I can do a follow-up for clarity17:17
arne_wiebalckdtantsur|afk: uzumaki : yes, we have a few thousand servers in containers17:18
uzumakiTheJulia, they've fixed the page now, the typo as well as my name is on there =) THANK YOU17:18
arne_wiebalckdtantsur|afk: uzumaki : all nodes we have in there are in Ironic :)17:18
TheJuliauzumaki: awesome17:18
uzumakihttps://www.openstack.org/software/ussuri/17:18
TheJulia\o/17:18
uzumakiarne_wiebalck, that is SO AWESOME17:18
*** amoralej is now known as amoralej|off17:18
arne_wiebalckthe containers are quite impressive :)17:19
arne_wiebalckvery space efficient17:19
uzumakiis it too weird that every time you say container I keep thinking about docker/k8s?17:19
arne_wiebalckyep17:19
uzumakixD17:19
TheJuliaarne_wiebalck: I think... was it... Tim that tweeted a photo?17:19
arne_wiebalckme too :)17:19
arne_wiebalckThomas I think ... let me find it ...17:20
uzumakidid you say a few thousaaaaa.... =O17:20
TheJuliaI'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 ABSURD17:20
*** ociuhandu has quit IRC17:21
*** ociuhandu has joined #openstack-ironic17:23
arne_wiebalckhttps://twitter.com/thomasnomas/status/112677532000090931417:23
uzumakiI 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" xD17:23
arne_wiebalcklol17:23
uzumaki=D17:23
arne_wiebalckbye everyone, have a nice evening o/17:24
TheJuliao/17:24
uzumakiau revoir arne_wiebalck and thank you for the insight on the containerized matter17:25
uzumaki\o17:25
openstackgerritMerged openstack/sushy master: Update lower-constraints.txt  https://review.opendev.org/72724017:27
uzumaki^^^ that comment on the tweet about jobs in containers in containers in containers is just hilarious xD17:27
*** ociuhandu has quit IRC17:33
*** ociuhandu has joined #openstack-ironic17:34
*** priteau has quit IRC17:38
*** ociuhandu has quit IRC17:40
*** k_mouza has quit IRC17:40
*** kozhukalov has joined #openstack-ironic18:04
*** rloo has quit IRC18:11
*** rloo has joined #openstack-ironic18:11
*** ociuhandu has joined #openstack-ironic18:32
openstackgerritAija Jaunteva proposed openstack/sushy master: Fix OEM extension loading for different servers  https://review.opendev.org/72773818:38
*** ociuhandu has quit IRC18:40
*** ociuhandu has joined #openstack-ironic18:40
openstackgerritAija Jaunteva proposed openstack/sushy master: Fix OEM extension loading for different servers  https://review.opendev.org/72773818:41
iurygregoryI love when people put a -1 in a patch that the CI failed and they don't even put a comment XD18:46
TheJuliaiurygregory: perhaps time for rage against the machine?18:54
iurygregoryTheJulia, yup18:55
TheJuliahttps://www.youtube.com/watch?v=bWXazVhlyxQ18:55
TheJuliajust enough time before next meeting18:55
iurygregory++18:56
*** rloo has quit IRC18:58
*** rloo has joined #openstack-ironic18:59
*** kozhukalov has quit IRC19:02
*** livelace has joined #openstack-ironic19:09
uzumakiiurygregory, they do that? who is the -1 for then?19:21
iurygregoryuzumaki, see for yourself https://review.opendev.org/726121 =)19:22
patchbotpatch 726121 - ironic-inspector - Merge jobs - 2 patch sets19:22
uzumakiiurygregory, lol that's hilarious19:23
iurygregoryyup =)19:23
uzumakiTheJulia, the first comment on the video says " This song makes me wanna read the terms and conditions  then DECLINE " LMAO19:23
uzumaki" I fired my boss after listening to this " OMG these comments are hilarious.. =))19:24
*** ociuhandu has quit IRC19:27
*** ociuhandu has joined #openstack-ironic19:28
*** ociuhandu has quit IRC19:39
*** ociuhandu has joined #openstack-ironic19:40
*** ociuhandu has quit IRC19:40
*** ociuhandu has joined #openstack-ironic19:40
iurygregoryuzumaki, that's the poing of Rage Against The Machine \o/19:46
iurygregorybye everyone o/19:46
uzumakiadios iurygregory ! Yasss that's the whole point! \o/19:47
*** ociuhandu has quit IRC19:51
*** ociuhandu has joined #openstack-ironic19:52
*** ociuhandu has quit IRC19:52
*** ociuhandu has joined #openstack-ironic19:52
TheJuliastepping away, need to take my wife to the doctor20:00
*** ociuhandu has quit IRC20:02
*** ociuhandu has joined #openstack-ironic20:03
*** ociuhandu has quit IRC20:03
*** ociuhandu has joined #openstack-ironic20:03
uzumakihope nothing too serious...20:05
*** ociuhandu has quit IRC20:14
*** ociuhandu has joined #openstack-ironic20:15
*** ociuhandu has quit IRC20:26
*** ociuhandu has joined #openstack-ironic20:27
*** ociuhandu has quit IRC20:27
*** ociuhandu has joined #openstack-ironic20:27
*** ociuhandu has quit IRC20:37
*** ociuhandu has joined #openstack-ironic20:38
*** ociuhandu has quit IRC20:38
*** ociuhandu has joined #openstack-ironic20:39
*** ociuhandu has quit IRC20:49
*** ociuhandu has joined #openstack-ironic20:50
*** ociuhandu has quit IRC20:50
*** ociuhandu has joined #openstack-ironic20:50
*** ociuhandu has quit IRC21:01
*** ociuhandu has joined #openstack-ironic21:02
*** ociuhandu has quit IRC21:07
*** livelace has quit IRC21:11
*** livelace has joined #openstack-ironic21:17
*** livelace has quit IRC21:18
*** Goneri has quit IRC22:07
*** Goneri has joined #openstack-ironic22:10
*** alexmcleod has quit IRC22:24
TheJuliaeh, life22:35
TheJuliaand my terminal windows all crashed while I was away22:35
uzumakithat just sucks!22:35
openstackgerritMerged openstack/ironic master: redfish: handle hardware that is unable to set persistent boot  https://review.opendev.org/72523922:39
*** tkajinam has joined #openstack-ironic22:46
rpioso^^^ \o/22:49
rpiosoTheJulia: When time permits, please take a look at https://review.opendev.org/#/c/72773822:50
patchbotpatch 727738 - sushy - Fix OEM extension loading for different servers - 3 patch sets22:50
rpiosoWithout it, only one server can be managed.22:52
rpiosoTheJulia: Are there any other core reviewers you could recommend look at sushy stuffs while etingof is away?22:54
TheJuliaoff the top of my head, I don't konw but we can figure something out22:56
TheJuliaworst comes to worst we'll single core review it if we have to22:56
TheJuliaI'll have to look in the morning, I just got home and feel like someone is drilling into my skull22:57
rpiosoTheJulia: Thank you! It's a subtle, somewhat disorienting bug -- stevedore, entry points, factories galore, and caching.22:58
TheJuliaeww22:59
TheJuliasounds like it22:59
* rpioso hopes TheJulia and her wife soon feel better22:59
TheJuliaSo... a morning espresso based beverage and code review22:59
rpiosoA 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-ironic23:29
*** rcernin has quit IRC23:31
openstackgerritJulia Kreger proposed openstack/ironic-specs master: WIP - Victoria Cycle Priorit(y|ies)  https://review.opendev.org/72010023:32
TheJuliagah, it became a longish list23:32
*** rcernin has joined #openstack-ironic23:32
*** ociuhandu has joined #openstack-ironic23:36
*** early has quit IRC23:38
*** ociuhandu has quit IRC23:52
« Tuesday, 2020-05-12 Index Thursday, 2020-05-14 »

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