| *** tbachman has quit IRC | 00:09 | |
| *** tbachman_ has joined #openstack-nova | 00:09 | |
| *** jawad_axd has joined #openstack-nova | 00:15 | |
| *** ttsiouts has quit IRC | 00:18 | |
| *** hamalq_ has quit IRC | 00:20 | |
| *** tetsuro has joined #openstack-nova | 00:25 | |
| *** jamesden_ has joined #openstack-nova | 00:42 | |
| *** jamesden_ has quit IRC | 00:42 | |
| *** jamesden_ has joined #openstack-nova | 00:43 | |
| *** jamesdenton has quit IRC | 00:43 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: testing nova gate on ubuntu focal https://review.opendev.org/738126 | 00:49 |
|---|---|---|
| *** jawad_axd has quit IRC | 00:50 | |
| *** ttsiouts has joined #openstack-nova | 00:50 | |
| *** markvoelker has joined #openstack-nova | 00:54 | |
| *** ttsiouts has quit IRC | 00:55 | |
| *** ociuhandu has joined #openstack-nova | 00:56 | |
| *** ociuhandu has quit IRC | 00:59 | |
| *** markvoelker has quit IRC | 00:59 | |
| *** ociuhandu has joined #openstack-nova | 00:59 | |
| *** ociuhandu has quit IRC | 01:03 | |
| *** markvoelker has joined #openstack-nova | 01:09 | |
| *** markvoelker has quit IRC | 01:14 | |
| *** avolkov has quit IRC | 01:25 | |
| *** ttsiouts has joined #openstack-nova | 01:30 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: DNM: testing os-vif gate on ubuntu focal https://review.opendev.org/738130 | 01:45 |
| mnaser | dansmith, mriedem: in my case, i'm getting non-json content (because there's a load balancer infront of placement) | 01:48 |
| mnaser | content-type seems sane though | 01:48 |
| *** ttsiouts has quit IRC | 02:03 | |
| *** jawad_axd has joined #openstack-nova | 02:06 | |
| *** vishalmanchanda has joined #openstack-nova | 02:29 | |
| openstackgerrit | Jinsheng Zhang proposed openstack/nova master: Add API to support multiple boot-volumes with boot order selection https://review.opendev.org/737913 | 02:32 |
| *** chenhaw has quit IRC | 02:37 | |
| gmann | dansmith: melwitt sean-k-mooney elod gibi pep8 job cherry pick script failure was just rebase issue (somehow zuul way of merging the code at runtime did not work for this script). after rebase, script can find all hashes correctly and job pass - https://review.opendev.org/#/c/728057/ | 02:40 |
| *** jawad_axd has quit IRC | 02:41 | |
| sean-k-mooney | gmann: it might be down to the branch name | 02:43 |
| sean-k-mooney | e.g. how zuul create the reiview branch and merges things | 02:44 |
| sean-k-mooney | but glad it worked. | 02:44 |
| gmann | yeah and commit msg is somethings not being picked correctly that is why hashes were not picked by script | 02:45 |
| sean-k-mooney | anyway im going to go to sleep now o/ i was just getting my phone which i left on my desk :) | 02:48 |
| *** hongbin has joined #openstack-nova | 02:59 | |
| *** ttsiouts has joined #openstack-nova | 03:05 | |
| *** psachin has joined #openstack-nova | 03:29 | |
| openstackgerrit | Merged openstack/nova master: Raise InstanceMappingNotFound if StaleDataError is encountered https://review.opendev.org/734400 | 03:32 |
| *** hongbin has quit IRC | 03:33 | |
| openstackgerrit | Wenping Song proposed openstack/nova master: WIP: Cyborg shelve/unshelve support https://review.opendev.org/729563 | 03:35 |
| *** ttsiouts has quit IRC | 03:39 | |
| *** jawad_axd has joined #openstack-nova | 03:58 | |
| *** ttsiouts has joined #openstack-nova | 04:14 | |
| *** spatel has joined #openstack-nova | 04:17 | |
| *** ttsiouts has quit IRC | 04:19 | |
| *** spatel has quit IRC | 04:21 | |
| *** jawad_axd has quit IRC | 04:31 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #openstack-nova | 04:33 | |
| *** ttsiouts has joined #openstack-nova | 04:51 | |
| *** ratailor has joined #openstack-nova | 05:12 | |
| *** links has joined #openstack-nova | 05:16 | |
| *** ttsiouts has quit IRC | 05:24 | |
| *** udesale has joined #openstack-nova | 05:40 | |
| *** maciejjozefczyk has joined #openstack-nova | 05:50 | |
| *** jawad_axd has joined #openstack-nova | 06:09 | |
| *** jawad_axd has quit IRC | 06:14 | |
| *** danpawlik is now known as dpawlik|EoD | 06:15 | |
| *** damien_r has joined #openstack-nova | 06:15 | |
| *** ralonsoh has joined #openstack-nova | 06:17 | |
| *** ttsiouts has joined #openstack-nova | 06:18 | |
| *** damien_r has quit IRC | 06:20 | |
| *** dklyle has quit IRC | 06:20 | |
| *** damien_r has joined #openstack-nova | 06:21 | |
| *** jsuchome has joined #openstack-nova | 06:26 | |
| *** rpittau|afk is now known as rpittau | 06:29 | |
| *** ttsiouts has quit IRC | 06:33 | |
| *** ttsiouts has joined #openstack-nova | 06:34 | |
| *** ttsiouts has quit IRC | 06:39 | |
| *** slaweq has joined #openstack-nova | 06:57 | |
| *** jawad_axd has joined #openstack-nova | 07:00 | |
| *** ttsiouts has joined #openstack-nova | 07:05 | |
| *** tesseract has joined #openstack-nova | 07:13 | |
| *** ratailor has quit IRC | 07:19 | |
| *** ratailor has joined #openstack-nova | 07:19 | |
| *** jawad_axd has quit IRC | 07:22 | |
| *** jawad_axd has joined #openstack-nova | 07:22 | |
| *** damien_r has quit IRC | 07:25 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:25 | |
| *** bhagyashris|afk is now known as bhagyashris | 07:27 | |
| *** dtantsur|afk is now known as dtantsur | 07:33 | |
| *** tosky has joined #openstack-nova | 07:35 | |
| *** ociuhandu has joined #openstack-nova | 07:37 | |
| *** ttsiouts has quit IRC | 07:59 | |
| *** ttsiouts has joined #openstack-nova | 07:59 | |
| *** JamesBen_ has joined #openstack-nova | 08:02 | |
| *** ttsiouts has quit IRC | 08:03 | |
| *** JamesBenson has quit IRC | 08:04 | |
| openstackgerrit | Jinsheng Zhang proposed openstack/nova master: Add API to support multiple boot-volumes with boot order selection https://review.opendev.org/737913 | 08:07 |
| *** tbachman_ has quit IRC | 08:13 | |
| *** tbachman has joined #openstack-nova | 08:14 | |
| *** ttsiouts has joined #openstack-nova | 08:16 | |
| openstackgerrit | Wenping Song proposed openstack/nova master: cyborg evacuate support https://review.opendev.org/715326 | 08:38 |
| *** derekh has joined #openstack-nova | 08:43 | |
| *** markvoelker has joined #openstack-nova | 08:49 | |
| openstackgerrit | Merged openstack/nova master: Remove deprecated nova.image.download hook https://review.opendev.org/725338 | 08:53 |
| openstackgerrit | Merged openstack/nova master: libvirt: Remove workaround for unsupported libvirt version https://review.opendev.org/729656 | 08:53 |
| *** markvoelker has quit IRC | 08:54 | |
| gibi | sean-k-mooney: responeded in https://bugs.launchpad.net/nova/+bug/1884068 | 08:54 |
| openstack | Launchpad bug 1884068 in OpenStack Compute (nova) "Instance stuck in build state when some/one compute node is unreachable" [Undecided,New] | 08:54 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512 | 09:05 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565 | 09:05 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898 | 09:05 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | 09:05 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757 | 09:06 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777 | 09:06 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094 | 09:06 |
| *** udesale has quit IRC | 09:07 | |
| *** avolkov has joined #openstack-nova | 09:10 | |
| *** xek has joined #openstack-nova | 09:10 | |
| zigo | Does anyone know what's going on with this bug ? https://bugs.debian.org/963339 | 09:15 |
| openstack | Debian bug 963339 in src:nova "nova: FTBFS: ValueError: Sentinels must not start with _" [Serious,Open] | 09:15 |
| zigo | Is there any solution already commited to master? | 09:15 |
| *** udesale has joined #openstack-nova | 09:15 | |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109 | 09:17 |
| gibi | zigo: I haven't seen this before. let me try to reproduce the test failure locally | 09:17 |
| zigo | gibi: I don't think you will unless you use the same set of dependency as in Debian Sid. | 09:17 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | 09:18 |
| zigo | It's probably one dependency that was upgraded or something similar. | 09:18 |
| *** luksky has joined #openstack-nova | 09:18 | |
| zigo | Maybe this is py3.8 related? | 09:19 |
| gibi | zigo: what version of oslo_utils that build uses? | 09:19 |
| zigo | gibi: Probably 4.1.1 | 09:20 |
| zigo | checking ... | 09:20 |
| zigo | Yeah, that's correct. | 09:21 |
| openstackgerrit | Elod Illes proposed openstack/nova stable/ussuri: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736957 | 09:21 |
| gibi | hm the ValueError is coming from here https://github.com/openstack/oslo.utils/blame/master/oslo_utils/fixture.py#L82 but that piece of code is 2 years old | 09:21 |
| zigo | Oh ... | 09:22 |
| zigo | Indeed, I was missreading the logs. | 09:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736962 | 09:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736962 | 09:24 |
| zigo | gibi: Do you think it'd have any consequence if we were to remove the underscore check? | 09:24 |
| gibi | zigo: as far as I understand the logs this fails in the test code part of the test case so I think it is save to gut that check temporarly | 09:26 |
| gibi | s/save/safe/ | 09:26 |
| gibi | looking more to the stacktrace I suspect this is some kind of mock autospecing issue. So far the same test passes for me locally in py38 | 09:28 |
| zigo | gibi: The full log with what version of each component is here: qa-logs.debian.net/2020/06/20/nova_21.0.0-2_unstable.log | 09:29 |
| gibi | zigo: thanks | 09:29 |
| * zigo tries to rebuilds nova with the patch in oslo.utils, removing that check. | 09:33 | |
| * gibi tries to find the difference between zigo's env and his own env | 09:35 | |
| zigo | gibi: The easiest way is probably to run Debian Sid, and do "apt-get build-dep nova" then pip freeze, then compare the output with your venv. | 09:36 |
| zigo | gibi: I can do the sid part send send it somewhere if you want. | 09:37 |
| gibi | zigo: yeah please provide that info for me | 09:38 |
| sean-k-mooney | gibi: oh your right it was --availabity-zone <az> <server name> | 09:39 |
| zigo | I'm installing a fresh sid machine. | 09:40 |
| gibi | thanks | 09:40 |
| *** masterpe has joined #openstack-nova | 09:48 | |
| masterpe | Hi, I have a question about a upgrade from Rocky to Stein. | 09:50 |
| openstackgerrit | Alexandre Arents proposed openstack/nova master: Limit the number of concurrent snapshots https://review.opendev.org/736169 | 09:50 |
| gibi | zigo: need to join a call for the next two hours, but I continue looking at the issue you see | 09:51 |
| zigo | Thanks. | 09:51 |
| zigo | I'm nearly done with the sid box. | 09:52 |
| zigo | (well, VM actually...) | 09:52 |
| zigo | gibi: output of pip freeze for me: | 09:55 |
| zigo | http://paste.openstack.org/show/795248/ | 09:55 |
| gibi | thanks | 09:55 |
| gibi | I've spotted some differences already but so far no luck reproducing the issue | 09:56 |
| masterpe | I try to upgrade a compute node from Rocky to Stein as a first step I upgraded a empty compute node and also libvirt and qemu to version 5.0.0-1~cloud0 and 1:3.1+dfsg-2ubuntu3.7~cloud0 on a ubuntu bionic system | 09:58 |
| *** priteau has joined #openstack-nova | 10:04 | |
| *** rpittau is now known as rpittau|bbl | 10:04 | |
| zigo | gibi: With the test removed from oslo.utils, now I only got a single failure which seems to be an unrelated broken test (which I'm ok to just disable): http://paste.openstack.org/show/795250/ | 10:06 |
| gibi | zigo: OK. I will let you know if I found something on my side | 10:06 |
| *** tetsuro has quit IRC | 10:08 | |
| *** ttsiouts has quit IRC | 10:14 | |
| *** ttsiouts has joined #openstack-nova | 10:14 | |
| *** ttsiouts has quit IRC | 10:16 | |
| *** ttsiouts has joined #openstack-nova | 10:16 | |
| gibi | zigo: there are different versions in http://paste.openstack.org/show/795248/ and in qa-logs.debian.net/2020/06/20/nova_21.0.0-2_unstable.log | 10:21 |
| gibi | like oslo.utils is 4.1.1 in http://paste.openstack.org/show/795248/ | 10:21 |
| openstackgerrit | Alexandre Arents proposed openstack/nova master: Snapshot: offload glance upload in a native thread https://review.opendev.org/734776 | 10:22 |
| gibi | zigo: never mind a looked at the wrong file | 10:22 |
| gibi | zigo: I found the problem | 10:23 |
| zigo | :) | 10:23 |
| gibi | zigo: you have mock==4.0.2 if I install that then I have the same failure | 10:24 |
| zigo | Ok. | 10:24 |
| gibi | zigo: openstack pins that to 3.0.5 | 10:25 |
| gibi | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L168 | 10:25 |
| *** martinkennelly has joined #openstack-nova | 10:27 | |
| zigo | gibi: Any idea how to solve the problem with this new mock version then? | 10:29 |
| zigo | I obviously can't get the sid package to downgrade ... | 10:29 |
| zigo | I'll get in touch with the package maintainer still though. | 10:30 |
| gibi | zigo: that is a good question, right now I don't | 10:30 |
| *** slaweq has quit IRC | 10:40 | |
| *** ccamacho has quit IRC | 10:42 | |
| *** slaweq has joined #openstack-nova | 10:42 | |
| gibi | zigo: this seems related https://github.com/testing-cabal/mock/issues/487 | 10:46 |
| gibi | reading that it seems it is an issue in the python standard mock lib | 10:49 |
| *** markvoelker has joined #openstack-nova | 10:50 | |
| *** markvoelker has quit IRC | 10:54 | |
| zigo | Thanks for the investigations. | 10:55 |
| *** k_mouza has joined #openstack-nova | 11:10 | |
| *** raildo has joined #openstack-nova | 11:18 | |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: DNM: debug check-cherry-picks.sh on gate https://review.opendev.org/738189 | 11:23 |
| aarents | gibi: I just read meeting minutes, I added bp/max-concurrent-snapshots in runways, Thanks! | 11:24 |
| *** tinwood has quit IRC | 11:37 | |
| *** tinwood has joined #openstack-nova | 11:38 | |
| *** k_mouza has quit IRC | 11:47 | |
| *** k_mouza has joined #openstack-nova | 11:48 | |
| *** martinkennelly has quit IRC | 11:53 | |
| *** martinkennelly has joined #openstack-nova | 11:53 | |
| gibi | zigo: looking at the python reference __getattr__ should raise AttributeError instead of ValueError if the attribute is not found on the object. So https://github.com/openstack/oslo.utils/blob/master/oslo_utils/fixture.py#L82 is wrong. If I change the exception type to AttributeError then the problem is gone | 11:56 |
| gibi | aarents: ack. I will look at it | 11:56 |
| zigo | gibi: I'll do that in the oslo.utils package then ! | 11:57 |
| gibi | zigo: I will file a bug to oslo.utils to fix this | 12:00 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | 12:05 |
| *** rpittau|bbl is now known as rpittau | 12:12 | |
| *** ociuhandu has quit IRC | 12:14 | |
| *** udesale has quit IRC | 12:29 | |
| *** derekh has quit IRC | 12:32 | |
| *** ociuhandu has joined #openstack-nova | 12:34 | |
| jsuchome | gibi: Hi, thanks for the review! However, the part about splitting somerefactoring into another the patch ... the first versions of the patch did not include the refactoring and it is only included as it was requested by someone else... | 12:43 |
| jsuchome | I mean it's hard to please everyone... | 12:45 |
| sean-k-mooney | jsuchome: well you could do the refactoring first then the functional chagne as the second patch right that would keep both happy | 12:50 |
| sean-k-mooney | jsuchome: not actully sure what patch you are refering too but just a suggestion | 12:50 |
| *** markvoelker has joined #openstack-nova | 12:51 | |
| jsuchome | the direct-rbd download one... | 12:51 |
| jsuchome | https://review.opendev.org/#/c/574301/ | 12:51 |
| sean-k-mooney | ah i see. i can kind of see why gibi asked for them to be seperate | 12:54 |
| stephenfin | bauzas: Care to look at https://review.opendev.org/#/c/730865/ and the following two patches? Trivial cleanup | 12:55 |
| bauzas | stephenfin: ack, sure | 12:55 |
| sean-k-mooney | the content of _verify_and_write looks very similar to the previous cod ebut its not clear if the differences are due to the refactor or the patch | 12:55 |
| stephenfin | ignore the failing test - it's not an issue (see the following patch) | 12:55 |
| sean-k-mooney | jsuchome: at least not at a 30 second glance i could figure it out if i reviewed it properly just more mental load | 12:56 |
| *** markvoelker has quit IRC | 12:56 | |
| jsuchome | sean-k-mooney: I can understand that. It's only because I've seen it hundred times it does not seem weird to me | 12:58 |
| openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Remove stale nested backport from InstancePCIRequests https://review.opendev.org/725932 | 12:58 |
| stephenfin | elod: I rebased https://review.opendev.org/#/c/725932/ to drop lyarwood's -W. Care to +W again? | 12:59 |
| *** jamesden_ is now known as jamesdenton | 12:59 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova stable/stein: Remove stale nested backport from InstancePCIRequests https://review.opendev.org/738199 | 12:59 |
| elod | stephenfin: thanks, yes, let's +W it :] | 13:01 |
| elod | there it is | 13:02 |
| stephenfin | ta | 13:02 |
| elod | np | 13:03 |
| *** derekh has joined #openstack-nova | 13:04 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove custom comparison methods https://review.opendev.org/472285 | 13:04 |
| *** nweinber has joined #openstack-nova | 13:06 | |
| openstackgerrit | Merged openstack/nova master: hardware: Don't consider overhead CPUs for unpinned instances https://review.opendev.org/714698 | 13:12 |
| openstackgerrit | Merged openstack/nova master: hardware: Remove handling of pre-Train compute nodes https://review.opendev.org/714699 | 13:12 |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: DNM: debug check-cherry-picks.sh on gate https://review.opendev.org/738189 | 13:12 |
| *** dtantsur is now known as dtantsur|afk | 13:21 | |
| *** mordred has quit IRC | 13:34 | |
| *** dave-mccowan has quit IRC | 13:37 | |
| *** dave-mccowan has joined #openstack-nova | 13:38 | |
| gibi | zigo: bug https://bugs.launchpad.net/oslo.utils/+bug/1885281 bugfix https://review.opendev.org/#/c/738207/ | 13:39 |
| openstack | Launchpad bug 1885281 in oslo.utils "uuidsentinel does not follow __getattr__ protocol" [Undecided,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | 13:39 |
| openstackgerrit | Merged openstack/nova master: Make libvirt able to trigger a backend image copy when needed https://review.opendev.org/656998 | 13:42 |
| *** udesale has joined #openstack-nova | 13:47 | |
| zigo | gibi: Super cool ! :) | 13:50 |
| *** itandops has joined #openstack-nova | 13:51 | |
| itandops | hi all, I'm installing openstack with openstack-ansible and get this error during nova install http://paste.openstack.org/show/795265/ | 13:54 |
| itandops | Is it required to have Placement API or any suggestion to solve this please ? | 13:54 |
| *** mlavalle has joined #openstack-nova | 13:58 | |
| gibi | aarents: thanks for adding max-concurrent-snapshots into the runway etherpad. Just a note, next time please add your patches to the Queue section, then I will make the move to the runway slot | 14:00 |
| aarents | gibi: noted | 14:01 |
| gibi | jsuchome: as sean-k-mooney put it, it would be nice to have the refactoring in the first patch and then the feature implementation in another patch on top of the refactor | 14:02 |
| *** ttsiouts has quit IRC | 14:06 | |
| *** ttsiouts has joined #openstack-nova | 14:07 | |
| *** dklyle has joined #openstack-nova | 14:08 | |
| *** mriedem has joined #openstack-nova | 14:09 | |
| *** ttsiouts has quit IRC | 14:12 | |
| melwitt | elod: I think you're likely right that the cherry pick check only works if the hash being looked for is in the local branch, I'm wondering if we need to add '-r' to the --contains command here https://github.com/openstack/nova/blob/4ae5b61c4f9819e26c13c147b46ab2344bd3cc3a/tools/check-cherry-picks.sh#L11 https://git-scm.com/docs/git-branch#Documentation/git-branch.txt--r cc gmann | 14:19 |
| elod | melwitt: it seems that even the 'git show --format='%b' --quiet $1' gives empty result back | 14:27 |
| melwitt | elod: oh... sorry I didn't realize that's what it's failing on. I'm stumped about that then : | 14:28 |
| melwitt | :/ | 14:28 |
| elod | melwitt: https://zuul.opendev.org/t/openstack/build/490f4df6fc384d48ba895af3a9cf9862/log/job-output.txt#865-866 | 14:29 |
| elod | melwitt: might be that the way how the repo is cloned causes the weirdness: https://opendev.org/openstack/project-config/src/branch/master/roles/use-cached-repos/tasks/main.yaml | 14:31 |
| melwitt | hrm | 14:32 |
| elod | only the .git is there I guess | 14:33 |
| *** ratailor has quit IRC | 14:36 | |
| elod | maybe if the patch is not on the top of the branch then it can just see a merge patch ( like http://paste.openstack.org/show/795267/ ) | 14:37 |
| elod | i've pushed a dummy patch to see what's there exactly: https://review.opendev.org/738189 | 14:38 |
| *** priteau has quit IRC | 14:44 | |
| elod | but correct me if I'm on a wrong track or poking the obvious :] | 14:48 |
| *** markvoelker has joined #openstack-nova | 14:52 | |
| *** markvoelker has quit IRC | 14:57 | |
| gmann | melwitt: elod : it is still issue, rebasing patches on top of script merged one works fine | 15:03 |
| gmann | * it is still issue? | 15:03 |
| elod | gmann: it is still an issue :( see, e.g.: https://review.opendev.org/#/c/736957/ | 15:10 |
| gmann | elod: its is working fine on 736957 right? | 15:11 |
| gmann | as it got rebased on top of ussuri lates code which has script changes, script got all the hashes correctly. | 15:12 |
| elod | gmann: it works fine when it is rebased | 15:13 |
| gmann | yeah | 15:13 |
| gmann | same like stable/train changes we rebased yesterday. | 15:14 |
| elod | gmann: but currently it seems to me that the script works only if the patch is on top of the given branch | 15:15 |
| elod | i might be wrong though | 15:16 |
| *** ttsiouts has joined #openstack-nova | 15:16 | |
| gmann | elod: oh is it? or on top of where script merged. | 15:17 |
| gmann | that I have not tested. do we have any such change on stable where it is on top of script-merged-change but not on latest branch changes | 15:18 |
| elod | gmann: I've created this dummy patch: https://review.opendev.org/#/c/738189/ | 15:18 |
| elod | but it is on stein, where the script is not even merged yet | 15:19 |
| gmann | yeah | 15:19 |
| gmann | we can wait for either of these merge and then recheck on other before rebase - https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/ussuri | 15:20 |
| elod | my thinking is what I wrote here: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2020-06-26.log.html#t2020-06-26T14:33:02 | 15:23 |
| elod | but that's also a good check what you are saying | 15:24 |
| *** priteau has joined #openstack-nova | 15:31 | |
| *** mordred has joined #openstack-nova | 15:35 | |
| *** JamesBen_ has quit IRC | 15:35 | |
| *** jawad_axd has quit IRC | 15:36 | |
| *** k_mouza has quit IRC | 15:49 | |
| *** ttsiouts has quit IRC | 15:49 | |
| *** k_mouza has joined #openstack-nova | 15:52 | |
| *** xek has quit IRC | 15:59 | |
| *** rpittau is now known as rpittau|afk | 15:59 | |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: DNM: debug check-cherry-picks.sh on gate https://review.opendev.org/738189 | 16:07 |
| stephenfin | bauzas: Got another one for you https://review.opendev.org/#/c/706013/ | 16:08 |
| *** k_mouza has quit IRC | 16:14 | |
| *** vishalmanchanda has quit IRC | 16:15 | |
| itandops | hello folks any suggestion about my error ? | 16:17 |
| *** psachin has quit IRC | 16:18 | |
| *** k_mouza has joined #openstack-nova | 16:19 | |
| *** k_mouza has quit IRC | 16:22 | |
| *** k_mouza has joined #openstack-nova | 16:22 | |
| *** jangutter has joined #openstack-nova | 16:23 | |
| *** jangutter has quit IRC | 16:25 | |
| *** tbachman has quit IRC | 16:26 | |
| *** ttsiouts has joined #openstack-nova | 16:28 | |
| *** ttsiouts has quit IRC | 16:32 | |
| *** mordred has quit IRC | 16:33 | |
| *** gyee has joined #openstack-nova | 16:35 | |
| *** ociuhandu_ has joined #openstack-nova | 16:36 | |
| *** tbachman has joined #openstack-nova | 16:36 | |
| stephenfin | gibi: comment on https://review.opendev.org/#/c/736167/ | 16:37 |
| *** mordred has joined #openstack-nova | 16:38 | |
| *** hamalq has joined #openstack-nova | 16:38 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Use imports instead of type aliases https://review.opendev.org/738018 | 16:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove wrappers around ovo mixins https://review.opendev.org/738019 | 16:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add type hints https://review.opendev.org/738020 | 16:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Don't use generic 'Field' container https://review.opendev.org/738239 | 16:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove unnecessary type aliases, exceptions https://review.opendev.org/738240 | 16:38 |
| *** ociuhandu has quit IRC | 16:38 | |
| *** k_mouza has quit IRC | 16:39 | |
| stephenfin | itandops: That looks like placement isn't registered in the service catalog https://docs.openstack.org/keystone/latest/contributor/service-catalog.html | 16:39 |
| stephenfin | Placement is mandatory from OSP 13 or so | 16:40 |
| stephenfin | Sorry, Newton | 16:40 |
| *** ociuhandu_ has quit IRC | 16:40 | |
| *** hamalq_ has joined #openstack-nova | 16:40 | |
| *** hamalq has quit IRC | 16:44 | |
| openstackgerrit | Merged openstack/os-traits master: Fix hacking min version to 3.0.1 https://review.opendev.org/727559 | 16:48 |
| *** priteau has quit IRC | 16:48 | |
| *** markvoelker has joined #openstack-nova | 16:53 | |
| *** markvoelker has quit IRC | 16:58 | |
| *** derekh has quit IRC | 17:00 | |
| *** ttsiouts has joined #openstack-nova | 17:04 | |
| *** ociuhandu has joined #openstack-nova | 17:11 | |
| *** tesseract has quit IRC | 17:13 | |
| *** ociuhandu has quit IRC | 17:18 | |
| *** jsuchome has quit IRC | 17:26 | |
| *** maciejjozefczyk has quit IRC | 17:28 | |
| *** ttsiouts has quit IRC | 17:37 | |
| *** udesale has quit IRC | 17:39 | |
| openstackgerrit | Merged openstack/nova master: Update interop repo link https://review.opendev.org/735427 | 17:40 |
| *** dcapone2004 has joined #openstack-nova | 18:03 | |
| *** mtreinish has quit IRC | 18:04 | |
| *** martinkennelly has quit IRC | 18:05 | |
| *** martinkennelly has joined #openstack-nova | 18:05 | |
| *** mtreinish has joined #openstack-nova | 18:05 | |
| *** ralonsoh has quit IRC | 18:25 | |
| *** ttsiouts has joined #openstack-nova | 18:42 | |
| *** markvoelker has joined #openstack-nova | 18:49 | |
| *** markvoelker has quit IRC | 18:54 | |
| *** lbragstad_ has joined #openstack-nova | 19:04 | |
| *** lbragstad has quit IRC | 19:06 | |
| *** ttsiouts has quit IRC | 19:16 | |
| *** efried has quit IRC | 19:28 | |
| openstackgerrit | Merged openstack/nova master: Delete resource provider in tree by top-down traversable order https://review.opendev.org/719163 | 19:40 |
| openstackgerrit | Elod Illes proposed openstack/nova master: Fix cherry-pick check for merge patch https://review.opendev.org/738271 | 19:52 |
| *** links has quit IRC | 20:00 | |
| *** slaweq has quit IRC | 20:02 | |
| elod | melwitt dansmith gmann : ^^^^ I've tested it locally, but will do some testing on my DNM patch in zuul as well | 20:02 |
| dansmith | elod: ah, makes sense | 20:02 |
| melwitt | elod: awesome, thanks for tackling it | 20:02 |
| dansmith | I dunno why we're on a merge and sometimes not, but probably good to account for that | 20:03 |
| melwitt | it's something about how zuul applies the patch in CI | 20:05 |
| elod | if patch's parent is not the latest patch on stable/xxxxx then zuul rebases it, to see if the patch works with the latest state | 20:06 |
| elod | yes, as melwitt says :) | 20:06 |
| *** slaweq has joined #openstack-nova | 20:06 | |
| *** slaweq has quit IRC | 20:10 | |
| dansmith | ah, zuul not git or gerrit? | 20:13 |
| elod | i think so | 20:20 |
| *** ttsiouts has joined #openstack-nova | 20:26 | |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: DNM: debug check-cherry-picks.sh on gate https://review.opendev.org/738189 | 20:26 |
| *** markvoelker has joined #openstack-nova | 20:35 | |
| *** efried has joined #openstack-nova | 20:36 | |
| *** mriedem has left #openstack-nova | 20:39 | |
| *** markvoelker has quit IRC | 20:39 | |
| *** raildo has quit IRC | 20:42 | |
| openstackgerrit | Merged openstack/nova stable/train: Remove stale nested backport from InstancePCIRequests https://review.opendev.org/725932 | 20:50 |
| *** ttsiouts has quit IRC | 21:00 | |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: DNM: debug check-cherry-picks.sh on gate https://review.opendev.org/738189 | 21:01 |
| *** nweinber has quit IRC | 21:16 | |
| *** lbragstad_ has quit IRC | 21:38 | |
| *** ttsiouts has joined #openstack-nova | 22:08 | |
| *** luksky has quit IRC | 22:22 | |
| *** markvoelker has joined #openstack-nova | 22:35 | |
| *** markvoelker has quit IRC | 22:40 | |
| *** ttsiouts has quit IRC | 22:42 | |
| *** tosky has quit IRC | 23:00 | |
| *** ttsiouts has joined #openstack-nova | 23:14 | |
| *** ttsiouts has quit IRC | 23:19 | |
| openstackgerrit | Merged openstack/nova stable/ussuri: Guard against missing image cache directory https://review.opendev.org/737990 | 23:29 |
| *** ttsiouts has joined #openstack-nova | 23:47 | |
| *** martinkennelly has quit IRC | 23:49 | |
| *** martinkennelly has joined #openstack-nova | 23:49 | |
| *** martinkennelly has quit IRC | 23:58 | |
| *** martinkennelly has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!