*** ociuhandu has quit IRC | 00:00 | |
*** rloo has quit IRC | 00:22 | |
*** ociuhandu has joined #openstack-ironic | 00:28 | |
*** k_mouza has joined #openstack-ironic | 00:34 | |
*** k_mouza has quit IRC | 00:39 | |
*** cdearborn has quit IRC | 00:47 | |
*** ociuhandu has quit IRC | 00:54 | |
*** ociuhandu has joined #openstack-ironic | 01:25 | |
*** ociuhandu has quit IRC | 01:31 | |
*** Goneri has quit IRC | 01:43 | |
*** threestrands has joined #openstack-ironic | 02:33 | |
*** hwoarang has quit IRC | 02:43 | |
*** hwoarang has joined #openstack-ironic | 02:43 | |
*** ricolin_ has joined #openstack-ironic | 02:44 | |
*** mkrai has joined #openstack-ironic | 02:51 | |
*** rh-jelabarre has quit IRC | 03:07 | |
*** ociuhandu has joined #openstack-ironic | 03:12 | |
*** ociuhandu has quit IRC | 03:17 | |
*** jdandrea has quit IRC | 03:19 | |
*** gyee has quit IRC | 05:29 | |
*** ociuhandu has joined #openstack-ironic | 05:54 | |
*** ociuhandu has quit IRC | 05:59 | |
*** ociuhandu has joined #openstack-ironic | 06:01 | |
*** ociuhandu has quit IRC | 06:06 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721656 | 06:20 |
---|---|---|
*** mkrai has quit IRC | 06:37 | |
arne_wiebalck | Good morning, ironic! | 06:41 |
openstackgerrit | Merged openstack/bifrost master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721659 | 06:43 |
iurygregory | good morning arne_wiebalck and Ironic o/ | 06:47 |
arne_wiebalck | hey iurygregory o/ | 06:47 |
*** mkrai has joined #openstack-ironic | 06:50 | |
*** ociuhandu has joined #openstack-ironic | 06:50 | |
*** mkrai has quit IRC | 06:56 | |
*** belmoreira has joined #openstack-ironic | 06:58 | |
*** adrianc has quit IRC | 06:59 | |
*** mkrai has joined #openstack-ironic | 07:00 | |
*** adrianc has joined #openstack-ironic | 07:12 | |
*** mkrai has quit IRC | 07:21 | |
*** ociuhandu has quit IRC | 07:23 | |
*** yolanda has joined #openstack-ironic | 07:24 | |
*** ociuhandu has joined #openstack-ironic | 07:24 | |
openstackgerrit | Merged openstack/ironic-lib master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721647 | 07:28 |
*** ociuhandu has quit IRC | 07:30 | |
*** rpittau|afk is now known as rpittau | 07:39 | |
rpittau | good morning ironic! o/ | 07:39 |
iurygregory | good morning rpittau o/ | 07:40 |
rpittau | hey iurygregory :) | 07:40 |
*** yaawang has quit IRC | 07:43 | |
*** yaawang has joined #openstack-ironic | 07:45 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721655 | 07:57 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic stable/train: Fix RAID configuration with idrac-wsman interface https://review.opendev.org/722222 | 08:00 |
*** ociuhandu has joined #openstack-ironic | 08:01 | |
*** mkrai has joined #openstack-ironic | 08:04 | |
*** ociuhandu has quit IRC | 08:11 | |
*** ociuhandu has joined #openstack-ironic | 08:12 | |
*** igordc has quit IRC | 08:13 | |
*** ociuhandu has quit IRC | 08:17 | |
*** lucasagomes has joined #openstack-ironic | 08:19 | |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: No need to import print from future anymore https://review.opendev.org/722227 | 08:25 |
*** mkrai has quit IRC | 08:27 | |
*** mkrai_ has joined #openstack-ironic | 08:27 | |
* iurygregory is holding the patch from tenks for future ^ | 08:30 | |
*** ociuhandu has joined #openstack-ironic | 08:32 | |
*** dtantsur|afk is now known as dtantsur | 08:32 | |
dtantsur | morning ironic | 08:32 |
iurygregory | morning dtantsur o/ | 08:33 |
*** threestrands has quit IRC | 08:33 | |
*** derekh has joined #openstack-ironic | 08:46 | |
*** sshnaidm|afk has quit IRC | 08:55 | |
*** k_mouza has joined #openstack-ironic | 08:58 | |
*** tkajinam has quit IRC | 09:03 | |
*** ociuhandu has quit IRC | 09:07 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Native zuulv3 grenade job for inspector https://review.opendev.org/703995 | 09:10 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade job for ironic https://review.opendev.org/703098 | 09:11 |
*** ociuhandu has joined #openstack-ironic | 09:12 | |
rpittau | the issue with the missing mount is hit hard :/ | 09:13 |
iurygregory | rpittau, yup =( | 09:14 |
rpittau | and I don't know how to write anymore | 09:14 |
iurygregory | I'm testing locally to see if something helps =( | 09:14 |
*** ociuhandu has quit IRC | 09:17 | |
rpittau | TheJulia: when you have a minute, I think we can move on with this https://review.opendev.org/708827 | 09:23 |
patchbot | patch 708827 - ironic-python-agent (stable/train) - Rescan after restarting the md device - 2 patch sets | 09:23 |
*** ociuhandu has joined #openstack-ironic | 09:26 | |
openstackgerrit | Merged openstack/ironic master: Add link to other Redfish parms to iDRAC doc https://review.opendev.org/721836 | 09:28 |
*** k_mouza has quit IRC | 09:30 | |
*** k_mouza has joined #openstack-ironic | 09:34 | |
*** k_mouza has quit IRC | 09:35 | |
*** k_mouza has joined #openstack-ironic | 09:35 | |
openstackgerrit | Merged openstack/bifrost master: No need to import print from future anymore https://review.opendev.org/722227 | 09:51 |
*** k_mouza has quit IRC | 09:58 | |
*** rcernin has quit IRC | 10:03 | |
openstackgerrit | Merged openstack/ironic-inspector master: Use autospec=True in mocks https://review.opendev.org/721819 | 10:06 |
*** k_mouza has joined #openstack-ironic | 10:07 | |
*** sshnaidm has joined #openstack-ironic | 10:14 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Restore missing node.save() in agent_base.py https://review.opendev.org/722268 | 10:15 |
dtantsur | mgoddard: ^^ | 10:15 |
mgoddard | dtantsur: ack | 10:18 |
*** rpittau is now known as rpittau|bbl | 10:21 | |
*** riuzen has joined #openstack-ironic | 10:21 | |
*** riuzen has quit IRC | 10:22 | |
iurygregory | I'm having the feeling that there is too many timeout variables in grenade oh god | 10:48 |
iurygregory | REMAINING_TIME is a result from DEVSTACK_GATE_TIMEOUT | DEVSTACK_GATE_TIMEOUT is BUILD_TIMEOUT - DEVSTACK_GATE_TIMEOUT_BUFFER | 10:51 |
iurygregory | we renamed BUILD_TIMEOUT to IRONIC_TEMPEST_BUILD_TIMEOUT and in grenade playbook it's set to 1200 sec not sure if this is always enough O.o since the smoke tests can take 1hr (seems like smoke uses some other values) | 10:52 |
iurygregory | or maybe I saw to many variables and Im going cray \o/ | 10:53 |
* iurygregory brb | 11:00 | |
*** jerry1 has quit IRC | 11:09 | |
*** ociuhandu has quit IRC | 11:29 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721655 | 11:30 |
*** ociuhandu has joined #openstack-ironic | 11:31 | |
*** ociuhandu has quit IRC | 11:35 | |
openstackgerrit | Merged openstack/ironic-tempest-plugin master: Actually log state transition timeouts in debug log https://review.opendev.org/720620 | 11:38 |
openstackgerrit | Mahendra Kamble proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port https://review.opendev.org/693328 | 11:41 |
openstackgerrit | Verification of a change to openstack/ironic-tempest-plugin failed: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721664 | 11:46 |
*** rh-jelabarre has joined #openstack-ironic | 11:52 | |
*** mkrai_ has quit IRC | 11:53 | |
*** mkrai__ has joined #openstack-ironic | 11:53 | |
*** rpittau|bbl is now known as rpittau | 12:02 | |
iurygregory | CI why you don't help and merge things pls <3 | 12:03 |
dtantsur | ++ | 12:03 |
iurygregory | after 3 months I decided to cut my hair \o/ | 12:10 |
*** dougsz has joined #openstack-ironic | 12:15 | |
openstackgerrit | Merged openstack/networking-baremetal master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721661 | 12:16 |
ajya | What's the status with CI on older branches? I have backports failing in stein and queens? Few of the errors are: No valid host was found. Reason: No conductor service registered which supports driver fake-hardware, tempest requires Python '>=3.6' but the running Python is 3.5.2 | 12:22 |
ajya | s/quuens?/queens. | 12:22 |
iurygregory | you did pure backport? | 12:23 |
iurygregory | maybe you need to do some changes since there is quite a lot of changes between queens and ussuri | 12:23 |
ajya | they were bug fixes, one very small, other a bit bigger | 12:25 |
ajya | the backports to train and rocky were fine | 12:25 |
openstackgerrit | Merged openstack/networking-generic-switch master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721662 | 12:27 |
iurygregory | ajya, do you have the link? I can try to take a look | 12:27 |
iurygregory | OMG ironic-inspecor-grenade green with zuulv3 <3 | 12:29 |
rpittau | ajya: re: the python version, that sounds like an old distribution there, is that Queens ? | 12:29 |
ajya | rpittau: no, this I see in stein | 12:31 |
ajya | iurygregory: these are queens patches: https://review.opendev.org/#/c/721712/ https://review.opendev.org/#/c/721811/ | 12:31 |
patchbot | patch 721712 - ironic (stable/queens) - Fix SpanLength calculation for DRAC RAID configura... - 1 patch set | 12:31 |
patchbot | patch 721811 - ironic (stable/queens) - Fix issue where server fails to reboot - 1 patch set | 12:31 |
ajya | and stein: https://review.opendev.org/#/c/721708/ https://review.opendev.org/#/c/721797/ | 12:32 |
patchbot | patch 721708 - ironic (stable/stein) - Fix SpanLength calculation for DRAC RAID configura... - 1 patch set | 12:32 |
patchbot | patch 721797 - ironic (stable/stein) - Fix issue where server fails to reboot - 1 patch set | 12:32 |
ajya | the same jobs are failing for both bugs in each branch | 12:32 |
ajya | also it appears consistent - same jobs failing after recheck | 12:32 |
*** Mahendra22 has joined #openstack-ironic | 12:32 | |
rpittau | mmm interesting, tempest is requiring python 3.6 but using xenial in stein that ships 3.5 :) | 12:34 |
rpittau | oh we're using constraints from master | 12:37 |
rpittau | well that probably won't work in stein | 12:37 |
dtantsur | rpittau: I thought gmann fixed that.. | 12:38 |
rpittau | it doesn't look like it | 12:38 |
rpittau | 2020-04-23 08:49:46.481860 | controller | full installdeps: -chttps://releases.openstack.org/constraints/upper/master, -r/opt/stack/tempest/requirements.txt | 12:38 |
iurygregory | ouch | 12:39 |
iurygregory | this is bad | 12:39 |
rpittau | this is stein btw | 12:39 |
iurygregory | WUT?! | 12:39 |
rpittau | https://zuul.opendev.org/t/openstack/build/d0b2a199c3f54a808c7445984d763931/log/job-output.txt#13038 | 12:39 |
iurygregory | it's late to start drinking? | 12:40 |
dtantsur | it's never too late! | 12:40 |
rpittau | I never stop | 12:41 |
rpittau | just to be sure | 12:41 |
dtantsur | very sensible | 12:42 |
iurygregory | hehehehe | 12:42 |
iurygregory | "you can only be drunk if you stop drinking" | 12:42 |
dtantsur | this is science, kids | 12:43 |
rpittau | I just take Dr. Banner as model | 12:43 |
rpittau | I'm asking in #openstack-qa btw | 12:45 |
*** mkrai__ has quit IRC | 12:48 | |
*** mkrai has joined #openstack-ironic | 12:48 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade job for ironic https://review.opendev.org/703098 | 12:50 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Native zuulv3 grenade job for inspector https://review.opendev.org/703995 | 12:50 |
*** mkrai has quit IRC | 12:52 | |
*** mkrai_ has joined #openstack-ironic | 12:52 | |
iurygregory | Seems we weren't copying the ironic-bm-logs, let's see now | 12:53 |
*** ociuhandu has joined #openstack-ironic | 12:53 | |
rpittau | ok uhmmm interesting, that is the only job that still uses xenial and not bionic | 12:53 |
iurygregory | in stein? | 12:54 |
rpittau | yeah | 12:54 |
iurygregory | maybe we should try switching to see how it goes | 12:55 |
rpittau | if we switch it will succeed | 12:55 |
rpittau | the only problem is the pythone version | 12:55 |
rpittau | but the main issue is the problem with uppper constraints | 12:55 |
rpittau | it should not require python 3.6 in stein | 12:56 |
iurygregory | yeah | 12:56 |
iurygregory | I saw some thread on openstack-discuss about problems in stable/branches | 12:56 |
iurygregory | they said the problem was fixed... | 12:57 |
iurygregory | rpittau, you want to push the change or should I? | 12:57 |
rpittau | iurygregory: I'm going to push the patch and see how it goes | 12:58 |
iurygregory | rpittau, cool =) | 12:58 |
*** rloo has joined #openstack-ironic | 13:03 | |
*** igordc has joined #openstack-ironic | 13:07 | |
*** mkrai has joined #openstack-ironic | 13:08 | |
*** mkrai_ has quit IRC | 13:11 | |
*** Goneri has joined #openstack-ironic | 13:15 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/stein: Workaround for uefi job with ubuntu bionic https://review.opendev.org/722317 | 13:29 |
rpittau | dtantsur: ^^ | 13:29 |
dtantsur | yeah, thanks! | 13:30 |
rpittau | fingers crossed | 13:30 |
openstackgerrit | Verification of a change to openstack/ironic failed: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721646 | 13:33 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf https://review.opendev.org/716889 | 13:37 |
rpittau | docs job timing out, yeah, why not? | 13:37 |
*** mkrai has quit IRC | 13:38 | |
rpioso | Good morning, ironic | 13:39 |
rpittau | hey rpioso :) | 13:42 |
rpioso | rpittau: Hi :-) | 13:43 |
*** mkrai has joined #openstack-ironic | 13:44 | |
*** priteau has joined #openstack-ironic | 13:44 | |
*** tkajinam has joined #openstack-ironic | 13:44 | |
iurygregory | morning rpioso | 13:45 |
*** Mahendra22 has quit IRC | 13:46 | |
*** tzumainn has joined #openstack-ironic | 13:46 | |
rpioso | iurygregory: Afternoon :-) | 13:47 |
iurygregory | yeah =) | 13:47 |
*** uzumaki has joined #openstack-ironic | 13:50 | |
iurygregory | rpittau, it's the magic of our CI | 13:52 |
iurygregory | docs failing \o/ | 13:52 |
iurygregory | well | 13:57 |
iurygregory | CI seems RED | 13:57 |
rpioso | rpittau: I wonder if https://review.opendev.org/#/c/647687 fixes the kernel panic I was seeing on my bionic devstack while integration testing Redfish virtual media. It panicked before any PXE boot console messages were displayed. I believe only workflow was affected: boot_option:local and partition disk image. | 13:57 |
patchbot | patch 647687 - ironic - Workaround for uefi job with ubuntu bionic (MERGED) - 5 patch sets | 13:57 |
iurygregory | yay Github down \o/ | 13:58 |
rpioso | s/only/only one/ | 13:58 |
iurygregory | Connecting to github.com (github.com)|140.82.114.4|:443... connected. HTTP request sent, awaiting response... 503 Service Unavailable | 13:58 |
rpittau | rpioso: that patch is present since train, not sure | 13:58 |
rpioso | rpittau: I was testing it on master in ~February. | 14:00 |
*** jdandrea has joined #openstack-ironic | 14:01 | |
*** sshnaidm has quit IRC | 14:01 | |
rpittau | rpioso: the patch is there since more than a year :) | 14:01 |
*** sshnaidm has joined #openstack-ironic | 14:02 | |
* iurygregory wondering if there is a way to not request to download cirros | 14:03 | |
rpioso | rpittau: Ugh! I thought it was merged last month. | 14:03 |
* rpioso needs more caffeine | 14:03 | |
iurygregory | maybe a mirror that we could fallback if we can't get in github | 14:03 |
*** mkrai has quit IRC | 14:08 | |
*** mkrai_ has joined #openstack-ironic | 14:08 | |
iurygregory | well just hold rechecks https://www.githubstatus.com/ | 14:09 |
*** jdandrea has quit IRC | 14:10 | |
openstackgerrit | Merged openstack/ironic master: Improve the command status checks in the agent's process_next_step https://review.opendev.org/718358 | 14:17 |
openstackgerrit | Merged openstack/ironic stable/train: Fix RAID configuration with idrac-wsman interface https://review.opendev.org/722222 | 14:17 |
uzumaki | what's the status on the dell 3rd party CI? | 14:21 |
erbarr | uzumaki: still broken, there are patches that didn't get really tested cause they hit the tempest ussuri tag transition thing, I updated one job to use overcloud-full image and it looks like that one passed master earlier today but then the next job failed at devstack, i need to check on that | 14:25 |
erbarr | uzumaki: also looks like some older branches are failing, I need to look into that too | 14:26 |
uzumaki | oh gosh | 14:28 |
uzumaki | it's a house of cards! | 14:28 |
erbarr | lol | 14:29 |
iurygregory | erbarr,I hope you have zuulv3 in the backlog for dell CI | 14:30 |
iurygregory | would make things a lot easier I would say | 14:30 |
dtantsur | iurygregory: what are we even using github for? I'd assume infra mirrors everything.. | 14:31 |
dtantsur | I mean, except that building tinyipa | 14:31 |
iurygregory | dtantsur, we are downloading cirros... | 14:32 |
iurygregory | and it fails | 14:32 |
dtantsur | why on earth are we downloading it? Oo it's definitely cached by infra.. | 14:32 |
iurygregory | functions:upload_image:99 : wget --progress=dot:giga -c http://download.cirros-cloud.net/0.4.0/cirros-0.4.0-x86_64-uec.tar.gz -O /opt/stack/devstack/files/cirros-0.4.0-x86_64-uec.tar.gz | 14:32 |
iurygregory | dtantsur, good to know | 14:32 |
dtantsur | hmmmm, maybe I misunderstand something.. | 14:32 |
dtantsur | usually it's the point when clarkb appears and explains everything :) | 14:32 |
iurygregory | wait for the magic | 14:33 |
iurygregory | it will happen | 14:33 |
iurygregory | XD | 14:33 |
dtantsur | mmm, okay, downloading cirros. but it's not on github, is it? | 14:33 |
dtantsur | I see a redirect to aws when accessing that link | 14:33 |
iurygregory | it redirects to github | 14:34 |
iurygregory | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_de5/703098/12/check/ironic-standalone/de59b73/job-output.txt | 14:34 |
dtantsur | orly? | 14:34 |
iurygregory | and boom | 14:34 |
uzumaki | erbarr, anything I can do to help? | 14:34 |
*** jdandrea has joined #openstack-ironic | 14:34 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Restore missing node.save() in agent_base.py https://review.opendev.org/722268 | 14:34 |
dtantsur | omg wut | 14:35 |
erbarr | iurygregory: yea, is on the backlog | 14:35 |
iurygregory | dtantsur, infra doesn't seem to have cirros http://mirror.mtl01.inap.openstack.org/ | 14:36 |
iurygregory | or maybe I'm looking at the wrong place \o/ | 14:36 |
dtantsur | discussing on #openstack-infra | 14:37 |
*** tkajinam has quit IRC | 14:37 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Add function to calculate memory https://review.opendev.org/722334 | 14:37 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Fix TypeError with newer version of lshw https://review.opendev.org/722335 | 14:37 |
TheJulia | o/ | 14:39 |
dtantsur | TheJulia: welcome to the world of broken CI \o/ | 14:39 |
TheJulia | that is awesome | 14:41 |
TheJulia | I'm going back to bed after a meeting because food poisoning results in awful sleep | 14:41 |
rpittau | oh :/ | 14:41 |
*** mkrai_ has quit IRC | 14:42 | |
iurygregory | good morning TheJulia o/ | 14:43 |
iurygregory | wellcome to github breaking our CI | 14:43 |
TheJulia | srsly? | 14:43 |
iurygregory | this would fit better for know... | 14:43 |
iurygregory | https://bbb78882b87a10b2abd5-579228d175b3a8315f0bb2a7cc508a16.ssl.cf5.rackcdn.com/703098/12/check/ironic-grenade/71110bd/job-output.txt yup | 14:43 |
iurygregory | ERROR 503: Service Unavailable \o/ | 14:44 |
dtantsur | TheJulia: we download cirros images from there.. | 14:44 |
dtantsur | I'm working with infra to cache them | 14:45 |
rpittau | iurygregory: btw https://download.cirros-cloud.net/ | 14:45 |
rpittau | maybe this link can help ? | 14:45 |
iurygregory | rpittau, yep but Location: https://github.com/cirros-dev/cirros/releases/download/0.4.0/cirros-0.4.0-x86_64-uec.tar.gz [following] | 14:45 |
dtantsur | rpittau: it redirects, at least sometimes to github | 14:45 |
iurygregory | yup | 14:45 |
rpittau | mmmm ok | 14:45 |
iurygregory | TheJulia, but at least before the failures I got ironic-inspector-grenade working with zuulv3 \o/ | 14:46 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: migrate UEFI jobs to Cirros 0.5.1 https://review.opendev.org/722340 | 14:48 |
dtantsur | another part of the picture ^^ | 14:48 |
dtantsur | I hope it passes. if it doesn't, we need to fix it ASAP. | 14:48 |
dtantsur | (will fail because of github and incomplete caching) | 14:48 |
*** cdearborn has joined #openstack-ironic | 14:49 | |
rpittau | I just tested the download from my laptop and works fine | 14:49 |
rpittau | maybe it's getting better | 14:50 |
dtantsur | rpittau: the redirect may be 1) random, 2) based on your location | 14:50 |
rpittau | dtantsur: yeah, sorry, I meant from github | 14:50 |
dtantsur | well, github may be up and down, as we see :) | 14:51 |
dtantsur | in any case, we should stop downloading stuff from the internet in our main jobs | 14:51 |
dtantsur | I've proposed a patch to infra images, let's see how it goes | 14:51 |
dtantsur | https://review.opendev.org/722338 | 14:51 |
patchbot | patch 722338 - project-config - Cache cirros UEC images - 1 patch set | 14:51 |
rpittau | hope that doesn't include python packages :) | 14:51 |
iurygregory | dtantsur, nice | 14:52 |
dtantsur | meanwhile, https://review.opendev.org/#/c/698768/ feels cold and lonely without your reviews! | 14:52 |
patchbot | patch 698768 - ironic - Support executing in-band deploy steps - 36 patch sets | 14:52 |
dtantsur | rpittau: our infra definitely has a pip cache | 14:52 |
iurygregory | I will look after the meeting | 14:52 |
*** k_mouza has quit IRC | 14:53 | |
*** cdearborn has quit IRC | 14:53 | |
*** k_mouza has joined #openstack-ironic | 14:53 | |
*** cdearborn has joined #openstack-ironic | 14:53 | |
*** ricolin_ has quit IRC | 15:06 | |
*** sshnaidm has quit IRC | 15:22 | |
*** jtomasek_ has quit IRC | 15:28 | |
*** belmoreira has quit IRC | 15:31 | |
*** gyee has joined #openstack-ironic | 15:53 | |
*** sshnaidm has joined #openstack-ironic | 15:55 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Convert jobs to dib https://review.opendev.org/718627 | 16:05 |
openstackgerrit | Merged openstack/ironic-inspector master: Give more time for grenade job https://review.opendev.org/721638 | 16:22 |
dtantsur | is there a reason we don't use timeout and retries in https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/extensions/standby.py#L73 | 16:28 |
* iurygregory is not aware of any reason | 16:29 | |
* dtantsur wonders if jroll or JayF are around | 16:30 | |
jroll | O_o | 16:30 |
jroll | morning dtantsur :) | 16:30 |
*** Lucas_Gray has joined #openstack-ironic | 16:30 | |
JayF | dtantsur: MVP, agile methodologies, and gotta-go-fast? | 16:30 |
dtantsur | ah :) so no dont-do-it-please opinion? | 16:30 |
jroll | yeah, I'm going to go with nobody thought to do it | 16:31 |
JayF | not in my memory. My memory is lossy though :) | 16:31 |
jroll | timeouts are great, and I don't mind retries | 16:31 |
dtantsur | okay, thanks guys. I hope life is treating you well nowadays :) | 16:31 |
jroll | thanks, I'm well, hope y'all are too | 16:32 |
jroll | one day I'll find the time to be here again :) | 16:32 |
dtantsur | I hope :) | 16:32 |
iurygregory | dtantsur, used his power to summon people in the irc =) | 16:32 |
dtantsur | it's not my exclusive power :D | 16:32 |
iurygregory | hehehe | 16:32 |
*** yaawang_ has joined #openstack-ironic | 16:32 | |
dtantsur | can someone please double-check my understanding: the 'timeout' argument in requests applies only to connecting, not to the whole process? | 16:33 |
*** yaawang has quit IRC | 16:33 | |
iurygregory | dtantsur, "You can also pass a tuple to timeout with the first element being a connect timeout (the time it allows for the client to establish a connection to the server), and the second being a read timeout (the time it will wait on a response once your client has established a connection)" | 16:35 |
dtantsur | yeah, but the read timeout also isn't a full timeout of the whole operation, right? | 16:35 |
dtantsur | so if I set timeout=60 it will cover all possible cases (even if the download itself takes minutes)? | 16:35 |
rpittau | dtantsur: it will wait for a response for that time and then fail | 16:36 |
iurygregory | it's the time to wait for the response only | 16:36 |
dtantsur | okay, lemme try a patch | 16:36 |
*** uzumaki has quit IRC | 16:38 | |
*** ociuhandu has quit IRC | 16:39 | |
*** ociuhandu has joined #openstack-ironic | 16:40 | |
*** ociuhandu has quit IRC | 16:45 | |
* iurygregory wondering if we should only do the SPUC next week or after next week so people will be able to vote... | 16:47 | |
iurygregory | thoughts? | 16:47 |
rpittau | good night! o/ | 16:49 |
*** rpittau is now known as rpittau|afk | 16:49 | |
*** derekh has quit IRC | 16:49 | |
*** lucasagomes has quit IRC | 16:51 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Native zuulv3 grenade job for inspector https://review.opendev.org/703995 | 16:52 |
*** jdandrea_ has joined #openstack-ironic | 16:54 | |
*** jdandrea has quit IRC | 16:55 | |
*** k_mouza has quit IRC | 16:57 | |
openstackgerrit | Merged openstack/ironic-tempest-plugin master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721664 | 16:59 |
*** cdearborn has quit IRC | 17:02 | |
*** belmoreira has joined #openstack-ironic | 17:09 | |
*** dougsz has quit IRC | 17:10 | |
*** belmoreira has quit IRC | 17:16 | |
*** priteau has quit IRC | 17:16 | |
*** ociuhandu has joined #openstack-ironic | 17:22 | |
*** Lucas_Gray has quit IRC | 17:24 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add timeout and retries when connection to an image server https://review.opendev.org/722409 | 17:25 |
dtantsur | the patch ^^ | 17:28 |
*** dtantsur is now known as dtantsur|afk | 17:28 | |
dtantsur|afk | o/ | 17:28 |
*** ociuhandu has quit IRC | 17:29 | |
dtantsur|afk | rpittau|afk and iurygregory: when you're back tomorrow, https://review.opendev.org/#/c/722340/ failing UEFI jobs is a bad sign. help appreciated. | 17:30 |
patchbot | patch 722340 - ironic - CI: migrate UEFI jobs to Cirros 0.5.1 - 1 patch set | 17:30 |
iurygregory | dtantsur|afk, I will look at that | 17:32 |
arne_wiebalck | bye everyone o/ | 17:34 |
JayF | dtantsur|afk: fwiw, I idle here during most workdays, I never mind a ping for a quick question or review. It's literally the least I can do :) | 17:37 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add timeout and retries when connection to an image server https://review.opendev.org/722409 | 17:44 |
*** sshnaidm is now known as sshnaidm|afk | 17:50 | |
*** yolanda has quit IRC | 17:55 | |
*** ociuhandu has joined #openstack-ironic | 18:12 | |
*** bdodd has quit IRC | 18:30 | |
openstackgerrit | Merged openstack/ironic master: Restore missing node.save() in agent_base.py https://review.opendev.org/722268 | 18:35 |
*** Lucas_Gray has joined #openstack-ironic | 18:42 | |
*** Lucas_Gray has quit IRC | 18:55 | |
*** bdodd has joined #openstack-ironic | 18:56 | |
*** k_mouza has joined #openstack-ironic | 18:58 | |
*** Lucas_Gray has joined #openstack-ironic | 18:58 | |
*** dougsz has joined #openstack-ironic | 19:01 | |
*** k_mouza has quit IRC | 19:02 | |
openstackgerrit | Doug Szumski proposed openstack/ironic stable/train: Add note about UEFI support in Train https://review.opendev.org/722438 | 19:12 |
*** ociuhandu has quit IRC | 19:19 | |
*** vesper has joined #openstack-ironic | 19:37 | |
*** vesper11 has quit IRC | 19:38 | |
*** igordc has quit IRC | 19:39 | |
*** Lucas_Gray has quit IRC | 19:41 | |
openstackgerrit | Merged openstack/ironic stable/stein: Workaround for uefi job with ubuntu bionic https://review.opendev.org/722317 | 19:46 |
*** Lucas_Gray has joined #openstack-ironic | 19:48 | |
*** priteau has joined #openstack-ironic | 19:56 | |
*** igordc has joined #openstack-ironic | 20:32 | |
*** aedc has quit IRC | 20:49 | |
*** dougsz has quit IRC | 20:50 | |
*** m1p has joined #openstack-ironic | 21:09 | |
m1p | hello, any chance these two will get backported to stable/train soon? | 21:10 |
m1p | https://review.opendev.org/#/c/713209/ | 21:10 |
m1p | https://review.opendev.org/#/c/716542/ | 21:10 |
patchbot | patch 713209 - ironic-python-agent - Allow specifying target devices for software RAID (MERGED) - 3 patch sets | 21:10 |
patchbot | patch 716542 - ironic-python-agent - Only check for partitions on devices that are part... (MERGED) - 3 patch sets | 21:10 |
*** jdandrea has joined #openstack-ironic | 21:15 | |
*** jdandrea_ has quit IRC | 21:19 | |
*** priteau has quit IRC | 21:33 | |
*** jdandrea has quit IRC | 21:33 | |
*** jdandrea has joined #openstack-ironic | 21:34 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Native zuulv3 grenade job for inspector https://review.opendev.org/703995 | 21:38 |
*** ociuhandu has joined #openstack-ironic | 21:39 | |
*** ociuhandu has quit IRC | 21:45 | |
*** Lucas_Gray has quit IRC | 21:45 | |
*** igordc has quit IRC | 21:51 | |
*** rcernin has joined #openstack-ironic | 22:10 | |
*** rcernin has quit IRC | 22:15 | |
*** rcernin has joined #openstack-ironic | 22:15 | |
*** Goneri has quit IRC | 22:21 | |
*** igordc has joined #openstack-ironic | 22:26 | |
*** tkajinam has joined #openstack-ironic | 22:58 | |
*** igordc has quit IRC | 23:05 | |
*** rcernin has quit IRC | 23:09 | |
*** rcernin has joined #openstack-ironic | 23:16 | |
*** uzumaki has joined #openstack-ironic | 23:36 | |
*** k_mouza has joined #openstack-ironic | 23:44 | |
*** k_mouza has quit IRC | 23:49 | |
*** rloo has quit IRC | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!