| *** diablo_rojo is now known as Guest2181 | 03:19 | |
| *** ykarel|away is now known as ykarel | 04:18 | |
| *** marios is now known as marios|ruck | 05:16 | |
| *** ysandeep|out is now known as ysandeep | 06:09 | |
| *** iurygregory_ is now known as iurygregory | 06:19 | |
| *** rpittau|afk is now known as rpittau | 07:13 | |
| *** ykarel is now known as ykarel|lunch | 07:38 | |
| opendevreview | Merged openstack/releases master: Create OpenStack-Ansible Wallaby Beta release https://review.opendev.org/c/openstack/releases/+/795974 | 09:28 |
|---|---|---|
| opendevreview | Merged openstack/releases master: release oslo's deliverables for xena https://review.opendev.org/c/openstack/releases/+/795780 | 09:52 |
| *** ykarel|lunch is now known as ykarel | 09:57 | |
| elodilles | hmmm, we have a release job failure: https://zuul.opendev.org/t/openstack/build/b5762a328d414489b84809afb3eea12b | 10:17 |
| elodilles | ssh connection time out :S | 10:17 |
| elodilles | the tag seems to be there: https://opendev.org/openstack/oslo.service/src/tag/2.6.0 | 10:19 |
| elodilles | but announce-release and propose-update-constraints jobs were skipped :S | 10:20 |
| elodilles | hberaud ttx ^^^ | 10:21 |
| hberaud | ack | 10:51 |
| hberaud | elodilles: still related to the previous failure, oslo.service 2.6.0 hasn't been uploaded to pypi | 11:06 |
| hberaud | fungi: Hello, we just experienced a new release job failure related to a ssh connection time out, unfortunatelly the deliverable hasn't been uploaded to pypi https://zuul.opendev.org/t/openstack/build/b5762a328d414489b84809afb3eea12b/log/job-output.txt#731 Please can you reenqueue this job? http://lists.openstack.org/pipermail/release-job-failures/2021-June/001540.html | 11:10 |
| *** ysandeep is now known as ysandeep|afk | 11:24 | |
| fungi | yeah, looks like rsync suddenly couldn't connect from the executor to the job node to pull the built sdist/wheel for publication | 11:24 |
| fungi | that node was in citycloud, wonder if there's some intermittent network connectivity issues recently | 11:25 |
| fungi | zuul enqueue-ref --tenant=openstack --trigger=gerrit --pipeline=release --project=openstack/oslo.service --ref=refs/tags/2.6.0 --newrev=f9eb7d369501a96331a88bf8756482b023af8a1f | 11:27 |
| fungi | hberaud: ^ that's been enqueued now | 11:28 |
| hberaud | fungi: thank you | 11:28 |
| hberaud | release-openstack-python passed | 11:42 |
| fungi | awesome | 11:58 |
| opendevreview | Merged openstack/releases master: python-glanceclient 3.4.0 https://review.opendev.org/c/openstack/releases/+/795836 | 11:59 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 12:27 |
| hberaud | smcginnis, elodilles FYI ^ | 12:27 |
| ttx | nice | 12:32 |
| hberaud | However I've some concern... | 12:32 |
| hberaud | See my comment on the patch | 12:33 |
| *** ysandeep|afk is now known as ysandeep | 12:33 | |
| ttx | yeah, in theory "skip_existing_tags" makes sense for that test... but it does not work as advertised | 12:34 |
| hberaud | yes | 12:34 |
| ttx | So I think that's what should be fixed instead | 12:34 |
| hberaud | yes, I move this patch to WIP | 12:34 |
| opendevreview | Hervé Beraud proposed openstack/releases master: [WIP] Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 12:34 |
| hberaud | ^ | 12:34 |
| ttx | yeah so the logic in includes_new_tag() is only exercised in skip_existing_tags() | 12:36 |
| ttx | so if it's buggy it's likely we haven't noticed | 12:36 |
| hberaud | yes | 12:37 |
| ttx | looks like that gitutils.commit_exists() does not return what we expect it to. Oh well | 12:45 |
| hberaud | yes the return of this function seems to trigger the addition of the tag to the `existing_tag_cache` and so the new tag will be seen as an existing tag and so it will be skipped | 12:52 |
| hberaud | Apparently this method (gitutils.commit_exists) return true when the decorator is called by `validate_pypi_readme` | 13:03 |
| hberaud | true mean that the version exist | 13:04 |
| hberaud | https://opendev.org/openstack/releases/blame/branch/master/openstack_releases/cmds/validate.py#L844 | 13:04 |
| hberaud | and so your validation start to "fail" from this point | 13:05 |
| hberaud | Ah | 13:06 |
| hberaud | I think it's because before this point we check if we can build sdist from the tag, and so, we create it temporarly | 13:07 |
| hberaud | I think that we need to delete this temporary tag at the end of the test | 13:08 |
| hberaud | `git tag -m temporary tag 3.3.1 de0e9d9510e997c5d49fc28e7175e326f216b038 ` | 13:08 |
| hberaud | This is the check that I added during Wallaby to ensure us to catch the tags issue similar to those faced by tripleo when their trailing projects weren't not branched in time and tagged within recent series | 13:12 |
| *** diablo_rojo__ is now known as diablo_rojo | 13:13 | |
| hberaud | so... the output of gitutils.commit_exists is correct, we just need to clean generated stuff at the end of the test | 13:13 |
| opendevreview | Ihar Hrachyshka proposed openstack/releases master: ovsdbapp 1.11.0 https://review.opendev.org/c/openstack/releases/+/796473 | 13:38 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 13:41 |
| opendevreview | Hervé Beraud proposed openstack/releases master: [DNM] PoC to ensure that our validation checks are fixed https://review.opendev.org/c/openstack/releases/+/796485 | 14:12 |
| *** rpittau is now known as rpittau|afk | 14:29 | |
| opendevreview | Hervé Beraud proposed openstack/releases master: Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 14:55 |
| opendevreview | Hervé Beraud proposed openstack/releases master: [DNM] PoC to ensure that our validation checks are fixed https://review.opendev.org/c/openstack/releases/+/796485 | 14:55 |
| *** ykarel_ is now known as ykarel | 14:56 | |
| opendevreview | Merged openstack/releases master: [congress] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795649 | 15:01 |
| opendevreview | Merged openstack/releases master: [searchlight] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795673 | 15:07 |
| *** ysandeep is now known as ysandeep|out | 15:41 | |
| *** marios|ruck is now known as marios|out | 15:57 | |
| hberaud | elodilles, ttx: Please can you double check this last job fail => https://review.opendev.org/c/openstack/releases/+/792820 I was thinking that all the needed config was restored, I surely missed something | 16:05 |
| *** ricolin_ is now known as ricolin | 16:26 | |
| *** ricolin_ is now known as ricolin | 17:32 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!