| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Apache: Increase timeout https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890409 | 00:43 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Apache: Increase timeout https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890409 | 00:44 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest master: Remove support for Glance v1 API parameter https://review.opendev.org/c/openstack/puppet-tempest/+/890604 | 00:52 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-cookiecutter master: replace validate_legacy with proper data types https://review.opendev.org/c/openstack/puppet-openstack-cookiecutter/+/890605 | 01:13 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Create only required ceph pools https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890607 | 01:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceph master: Add recent operating system versions to supported versions https://review.opendev.org/c/openstack/puppet-ceph/+/890608 | 01:34 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceph master: Deprecate support for mod_fastcgi https://review.opendev.org/c/openstack/puppet-ceph/+/890609 | 02:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceph master: Do not use params to define parameter defaults of defined types https://review.opendev.org/c/openstack/puppet-ceph/+/890610 | 02:43 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Remove workaround for old puppetlabs-apache https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890424 | 03:09 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Enable caching in some services https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890422 | 03:34 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Apache: Disable default mods and vhosts https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890612 | 03:41 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest master: Add support for [object-storage] reseller_admin_role https://review.opendev.org/c/openstack/puppet-tempest/+/890623 | 07:05 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: DNM: Run swift tests with RADOS Gateway https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890042 | 07:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: DNM: Run swift tests with RADOS Gateway https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890042 | 07:20 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceph master: Add support for rgw_keystone_accepted_admin_roles https://review.opendev.org/c/openstack/puppet-ceph/+/890627 | 07:24 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Apache: Disable default mods https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890612 | 07:58 |
| opendevreview | Yadnesh Kulkarni proposed openstack/puppet-ceilometer stable/xena: Add support for oslo.cache parameters https://review.opendev.org/c/openstack/puppet-ceilometer/+/889356 | 11:18 |
| opendevreview | Yadnesh Kulkarni proposed openstack/puppet-ceilometer stable/xena: Deprecate cache parameters in the base class https://review.opendev.org/c/openstack/puppet-ceilometer/+/889687 | 11:18 |
| opendevreview | Yadnesh Kulkarni proposed openstack/puppet-ceilometer stable/xena: Deprecate cache parameters in the base class https://review.opendev.org/c/openstack/puppet-ceilometer/+/889687 | 11:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ironic master: Bump upper version of puppetlabs-xinetd https://review.opendev.org/c/openstack/puppet-ironic/+/890646 | 12:24 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Bump upper version of puppetlabs-xinetd https://review.opendev.org/c/openstack/puppet-swift/+/890647 | 12:25 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-designate master: Bump upper version of puppet-dns https://review.opendev.org/c/openstack/puppet-designate/+/890648 | 12:26 |
| amoralej | hi! release-openstack-puppet job is failing with puppet-nova 20.4.1 with what looks like an infra error from pupetforge | 12:56 |
| amoralej | https://zuul.openstack.org/builds?project=openstack%252Fpuppet-nova&pipeline=release&skip=0 | 12:56 |
| amoralej | you know if there is any way to report this error to the puppetforge team? | 12:56 |
| amoralej | apparently it's specific to this particular package and release, maybe related to size or content... dunno | 12:57 |
| amoralej | otherwise, there may be a way to build and push the release manually? | 12:58 |
| tkajinam | let me bring it to the infra team | 13:01 |
| amoralej | tkajinam, fungi is already working on it | 13:02 |
| amoralej | he suggesting pinging here too :) | 13:02 |
| fungi | yeah, keep in mind that "the infra team" really has no connection or responsibility here other than we happen to maintain the servers where your release job is running. we don't really use puppet or have contacts at puppetforge, so i'm recommending someone in the puppet-openstack team see if locally building puppet-nova 20.4.1 and trying to push a copy of that to a test account at | 13:04 |
| fungi | puppetforge may be more useful in order to provide the puppetforge admins with something actionable to look into | 13:04 |
| fungi | the script that's uploading your puppet modules seems to be able to successfully upload all the others, but consistently fails to upload puppet-nova 20.4.1, so it seems likely it's a problem specific to that module (and maybe even that precise version) | 13:06 |
| tkajinam | fungi, ok. sorry I should have looked into the error more carefully. my initial intention of bringing this was just to retrigger job to see whether it succeeds now but it might be a consistent problem | 13:09 |
| tkajinam | in a meeting now... will have a deeper look | 13:09 |
| fungi | tkajinam: yes, we've now run it four times since thursday and every time it fails with the exact same error | 13:10 |
| tkajinam | :-( | 13:11 |
| amoralej | the tarball contains the whole git history, that's expected ? | 13:20 |
| tkajinam | https://github.com/puppetlabs/puppetlabs-package/blob/main/.pdkignore | 13:26 |
| tkajinam | we should probably add .pdkignore to exclude .git/ | 13:26 |
| amoralej | https://www.puppet.com/docs/puppet/7/modules_publishing.html#modules_publishing_prep-publishing-exclude-files | 13:29 |
| amoralej | looks like a good idea | 13:29 |
| amoralej | .pkdignore or .pmtignore depending on what we use | 13:30 |
| *** amoralej is now known as amoralej|lunch | 13:30 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Add .pdkignore to exclude some files from tar ball https://review.opendev.org/c/openstack/puppet-nova/+/890654 | 13:30 |
| jcapitao[m] | amoralej: ouch, 33M the .git folder in 20.4.0 extracted tarball | 13:33 |
| jcapitao[m] | in GH the whole tarball is only ~3MB | 13:34 |
| jcapitao[m] | so yeah it worth removing it | 13:34 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Add .pdkignore to exclude some files from tar ball https://review.opendev.org/c/openstack/puppet-nova/+/890654 | 13:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Add .pdkignore to exclude some files from tar ball https://review.opendev.org/c/openstack/puppet-nova/+/890654 | 13:37 |
| tkajinam | we can probably merge and backport these back to yoga and create a new release. | 13:38 |
| tkajinam | these = the above 890654 I proposed | 13:38 |
| jcapitao[m] | yep | 13:39 |
| fungi | that sounds like a win all around. users have been downloading entire copies of the git repos for every project with every updated version of the modules | 13:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Add .pdkignore to exclude some files from tar ball https://review.opendev.org/c/openstack/puppet-nova/+/890654 | 13:39 |
| fungi | so even if it doesn't fix the upload error, it's probably going to save a ton of bandwidth | 13:39 |
| tkajinam | yeah | 13:39 |
| *** amoralej|lunch is now known as amoralej | 13:54 | |
| amoralej | tkajinam, lgtm | 13:55 |
| tkajinam | fungi, amoralej, jcapitao[m] : thank you all ! let me take care of the change, backport and the new release. | 13:57 |
| fungi | sounds good. here's hoping it solves the problem | 13:57 |
| amoralej | thanks tkajinam and fungi | 13:57 |
| amoralej | i hope it will | 13:57 |
| jcapitao[m] | thank you both for taking care | 13:58 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Skio most of jobs for update in .pdkignore https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890660 | 14:27 |
| opendevreview | Merged openstack/puppet-ceph master: Add recent operating system versions to supported versions https://review.opendev.org/c/openstack/puppet-ceph/+/890608 | 15:11 |
| opendevreview | Merged openstack/puppet-octavia master: Bump upper-version of puppet-ssh_keygen https://review.opendev.org/c/openstack/puppet-octavia/+/890402 | 15:17 |
| opendevreview | Merged openstack/puppet-openstack-cookiecutter master: replace validate_legacy with proper data types https://review.opendev.org/c/openstack/puppet-openstack-cookiecutter/+/890605 | 15:25 |
| opendevreview | Merged openstack/puppet-ironic master: Disable Index in http server for iPXE boot https://review.opendev.org/c/openstack/puppet-ironic/+/890390 | 16:19 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Increase nova-api worker https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890474 | 16:20 |
| opendevreview | Merged openstack/puppet-ceph master: Deprecate support for mod_fastcgi https://review.opendev.org/c/openstack/puppet-ceph/+/890609 | 16:26 |
| opendevreview | Merged openstack/puppet-ceph master: Do not use params to define parameter defaults of defined types https://review.opendev.org/c/openstack/puppet-ceph/+/890610 | 16:26 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Revert "bind: Disable config check in Ubuntu" https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890585 | 16:35 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Create only required ceph pools https://review.opendev.org/c/openstack/puppet-openstack-integration/+/890607 | 16:35 |
| opendevreview | Merged openstack/puppet-neutron master: CentOS: Add default package/service name for rpc-server https://review.opendev.org/c/openstack/puppet-neutron/+/890586 | 17:02 |
| opendevreview | Merged openstack/puppet-nova master: Add support for [libvirt] tb_cache_size https://review.opendev.org/c/openstack/puppet-nova/+/890502 | 18:51 |
| opendevreview | Merged openstack/puppet-swift master: Bump upper version of puppetlabs-xinetd https://review.opendev.org/c/openstack/puppet-swift/+/890647 | 19:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!