| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Refactor validation of ssh_key parameters by Struct data type https://review.opendev.org/c/openstack/puppet-nova/+/930838 | 01:12 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-trove master: Debian/Ubuntu: Fix wsgi script source https://review.opendev.org/c/openstack/puppet-trove/+/930933 | 01:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cloudkitty master: Expose parameter to customize [oslo_messaging_notifications] retry https://review.opendev.org/c/openstack/puppet-cloudkitty/+/930901 | 01:41 |
| opendevreview | Merged openstack/puppet-cloudkitty master: Expose parameter to customize [oslo_messaging_notifications] retry https://review.opendev.org/c/openstack/puppet-cloudkitty/+/930901 | 02:15 |
| opendevreview | Merged openstack/puppet-watcher master: Expose parameter to customize [oslo_messaging_notifications] retry https://review.opendev.org/c/openstack/puppet-watcher/+/930917 | 02:34 |
| opendevreview | Merged openstack/puppet-trove master: Expose parameter to customize [oslo_messaging_notifications] retry https://review.opendev.org/c/openstack/puppet-trove/+/930902 | 02:38 |
| opendevreview | Merged openstack/puppet-magnum master: Expose parameter to customize [oslo_messaging_notifications] retry https://review.opendev.org/c/openstack/puppet-magnum/+/930906 | 02:43 |
| opendevreview | Merged openstack/puppet-vitrage master: Expose parameter to customize [oslo_messaging_notifications] retry https://review.opendev.org/c/openstack/puppet-vitrage/+/930907 | 02:51 |
| opendevreview | Merged openstack/puppet-nova master: Refactor validation of ssh_key parameters by Struct data type https://review.opendev.org/c/openstack/puppet-nova/+/930838 | 04:02 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-magnum master: Use Stdlib::Filemode to validate file mode https://review.opendev.org/c/openstack/puppet-magnum/+/930973 | 04:38 |
| opendevreview | Merged openstack/puppet-horizon master: Validate policy definitions using the new data type https://review.opendev.org/c/openstack/puppet-horizon/+/930855 | 04:38 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceph master: Use Stdlib::Filemode to validate file mode parameter https://review.opendev.org/c/openstack/puppet-ceph/+/930974 | 04:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-vitrage master: Add support for [oslo_messaging_rabbit] enable_cancel_on_failover https://review.opendev.org/c/openstack/puppet-vitrage/+/930976 | 05:40 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher master: Add support for [oslo_messaging_rabbit] enable_cancel_on_failover https://review.opendev.org/c/openstack/puppet-watcher/+/930977 | 05:43 |
| zigo | tkajinam: Hi there! On one of our deployment, puppet tries to do "/usr/bin/openstack project list --quiet --format csv --long" though it timesout too fast, as we have really a huge amount of project. Do you know where the timeout is set? | 10:27 |
| tkajinam | zigo, maybe https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L17 ? | 10:29 |
| zigo | Thanks, trying. | 10:29 |
| zigo | Yeah, that was it, thanks ! :) | 10:33 |
| zigo | tkajinam: Would you accept a patch to double this value? | 10:34 |
| * zigo counts how many project in that deployment | 10:41 | |
| zigo | tkajinam: In our case, we have 164959 projects in Keystone ! :) | 10:42 |
| zigo | That's a swift only cluster that we use for our drive service. | 10:43 |
| opendevreview | Merged openstack/puppet-trove master: Drop notification options from guest agent https://review.opendev.org/c/openstack/puppet-trove/+/930924 | 10:44 |
| opendevreview | Merged openstack/puppet-keystone master: Handle new message format for openstackclient >= 7.0.0 https://review.opendev.org/c/openstack/puppet-keystone/+/930900 | 11:24 |
| opendevreview | Merged openstack/puppet-trove master: Deprecate ineffective rabbit_notification_topic https://review.opendev.org/c/openstack/puppet-trove/+/930903 | 11:40 |
| tkajinam | zigo, does 60 seconds work or you need 80s ? | 12:01 |
| zigo | I can run "time" on my project list, hang on. | 12:01 |
| zigo | tkajinam: Took 1m7s ... so 60 isn't enough. | 12:04 |
| opendevreview | Thomas Goirand proposed openstack/puppet-openstacklib master: Increase @@command_timeout to 120s. https://review.opendev.org/c/openstack/puppet-openstacklib/+/931000 | 12:06 |
| tkajinam | zigo, you have to tune the other timeouts, too. increasing only command_timeout reduces number of retris for timeout | 12:20 |
| zigo | Oh... | 12:21 |
| zigo | tkajinam: Like request_timeout ? | 12:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: Extend command timeout https://review.opendev.org/c/openstack/puppet-openstacklib/+/931003 | 12:22 |
| tkajinam | ^^^ zigo this is what I'd suggest | 12:22 |
| zigo | Works for me. | 12:22 |
| tkajinam | we can probably replace the whole timeout by retry counts to avoid being stuck for too long period for server problems. | 12:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: Extend command timeout https://review.opendev.org/c/openstack/puppet-openstacklib/+/931003 | 13:43 |
| opendevreview | Alfredo Moralejo proposed openstack/puppet-keystone master: Fix message format for openstackclient >= 7.0.0 https://review.opendev.org/c/openstack/puppet-keystone/+/931019 | 13:59 |
| tkajinam | :facepalm: | 14:00 |
| amoralej | tkajinam, i've tested that locally and works mostly, still one more error: | 14:06 |
| amoralej | 2024-10-01 09:57:40 -0400 /Stage[main]/Heat::Keystone::Domain/Keystone_user_role[heat_admin::heat@::heat] (err): Could not evaluate: Execution of '/bin/openstack role assignment list --quiet --format csv --names --domain heat --user heat_admin --user-domain heat' returned 1: Could not load 'metric_clean-tombstones': 'HOME' | 14:06 |
| amoralej | # /bin/openstack --os-cloud system role assignment list --quiet --format csv --names --domain heat --user heat_admin --user-domain heat | 14:06 |
| amoralej | No User found for heat_admin | 14:06 |
| amoralej | maybe it's related to previous failure, i'm not sure, it was not clean deployment | 14:09 |
| tkajinam | amoralej, I've seen a similar error in https://review.opendev.org/c/openstack/aodh/+/922698 ... it could be a different regression | 14:33 |
| tkajinam | amoralej, I wonder if you can give me access to the env if that's not a private one ? | 14:33 |
| tkajinam | or can you pick up the id of the "heat" domain and use its id in --user-domain ? | 14:35 |
| amoralej | i'd need to spawn a new env | 14:50 |
| tkajinam | ok | 14:51 |
| tkajinam | let me check if I can reproduce it in my local | 14:51 |
| opendevreview | Merged openstack/puppet-watcher master: Add support for [oslo_messaging_rabbit] enable_cancel_on_failover https://review.opendev.org/c/openstack/puppet-watcher/+/930977 | 14:56 |
| tkajinam | ok I could reproduce the failure | 15:02 |
| opendevreview | Merged openstack/puppet-vitrage master: Add support for [oslo_messaging_rabbit] enable_cancel_on_failover https://review.opendev.org/c/openstack/puppet-vitrage/+/930976 | 15:02 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: Extend command timeout https://review.opendev.org/c/openstack/puppet-openstacklib/+/931003 | 15:41 |
| opendevreview | Merged openstack/puppet-keystone master: Fix message format for openstackclient >= 7.0.0 https://review.opendev.org/c/openstack/puppet-keystone/+/931019 | 16:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!