| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-11 | |||
| 19:19:31 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of legacy select() calling style https://review.opendev.org/c/openstack/nova/+/805735 | |
| 19:19:31 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace 'insert.inline' parameter with 'Insert.inline()' method https://review.opendev.org/c/openstack/nova/+/805736 | |
| 19:19:32 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Don't pass strings to 'Connection.execute' https://review.opendev.org/c/openstack/nova/+/805737 | |
| 19:19:32 | opendevreview | Stephen Finucane proposed openstack/nova master: test: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/nova/+/817641 | |
| 19:26:14 | stephenfin | Had a fun time https://review.opendev.org/c/openstack/nova/+/817641 I was about to throw my tea at zzzeek before I finally realized it wasn't SQLA doing something weird with warning filter but rather us | |
| 19:26:39 | stephenfin | *Had a fun time figuring out | |
| 19:31:08 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/nova/+/817641 | |
| 19:31:08 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/nova/+/804709 | |
| 19:31:09 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of Engine.scalar(), Engine.execute() https://review.opendev.org/c/openstack/nova/+/804878 | |
| 19:31:09 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of 'autoload' parameter https://review.opendev.org/c/openstack/nova/+/805734 | |
| 19:31:10 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of legacy select() calling style https://review.opendev.org/c/openstack/nova/+/805735 | |
| 19:31:10 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace 'insert.inline' parameter with 'Insert.inline()' method https://review.opendev.org/c/openstack/nova/+/805736 | |
| 19:31:11 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Don't pass strings to 'Connection.execute' https://review.opendev.org/c/openstack/nova/+/805737 | |
| 21:10:00 | opendevreview | Julia Kreger proposed openstack/nova master: WIP Ironic - Reconcile compute host records on rebalance https://review.opendev.org/c/openstack/nova/+/817467 | |
| 23:47:44 | opendevreview | melanie witt proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/c/openstack/nova/+/712137 | |
| 23:47:44 | opendevreview | melanie witt proposed openstack/nova master: Assert quota related API behavior when noop https://review.opendev.org/c/openstack/nova/+/712140 | |
| 23:47:45 | opendevreview | melanie witt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/c/openstack/nova/+/712141 | |
| 23:47:45 | opendevreview | melanie witt proposed openstack/nova master: DNM Run against unmerged oslo.limit changes https://review.opendev.org/c/openstack/nova/+/812236 | |
| 23:47:46 | opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 23:47:46 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 23:47:47 | opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 23:47:47 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 23:47:48 | opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 23:47:49 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 23:47:49 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 23:47:51 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 23:47:51 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 23:47:53 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 23:47:53 | opendevreview | melanie witt proposed openstack/nova master: WIP Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| #openstack-nova - 2021-11-12 | |||
| 01:26:28 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726 | |
| 01:47:12 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704 | |
| 03:02:57 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Show usage APIs https://review.opendev.org/c/openstack/nova/+/768509 | |
| 03:14:59 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315 | |
| 06:35:23 | opendevreview | Brin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage https://review.opendev.org/c/openstack/nova/+/768852 | |
| 06:47:26 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in os-quota-sets path https://review.opendev.org/c/openstack/nova/+/768851 | |
| 06:47:27 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in Limits API https://review.opendev.org/c/openstack/nova/+/768862 | |
| 06:47:27 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant* with project* in codes https://review.opendev.org/c/openstack/nova/+/769329 | |
| 07:09:40 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704 | |
| 07:09:41 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Show usage APIs https://review.opendev.org/c/openstack/nova/+/768509 | |
| 07:09:41 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315 | |
| 07:09:42 | opendevreview | Brin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage https://review.opendev.org/c/openstack/nova/+/768852 | |
| 07:09:42 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in os-quota-sets path https://review.opendev.org/c/openstack/nova/+/768851 | |
| 07:09:43 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in Limits API https://review.opendev.org/c/openstack/nova/+/768862 | |
| 07:09:43 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant* with project* in codes https://review.opendev.org/c/openstack/nova/+/769329 | |
| 07:11:06 | brinzhang | hi team, the series of remove tenant_id patches have been updated, please review them while you are free, thanks | |
| 07:19:56 | gibi | stephenfin: nice catch | |
| 08:14:49 | bauzas | good morning fokls | |
| 08:21:31 | lyarwood | Morning | |
| 08:22:09 | gibi | bauzas, lyarwood: happy Friday! | |
| 08:22:37 | bauzas | oh, the Friday dress code is back ? | |
| 08:23:21 | giblet | my neighbor decided to renovate his flat and the workers demolishing his bathroom at the moment which has a common wall with my flat. there is a constant noise so I feel like giblet already | |
| 08:23:39 | giblet | hence the name | |
| 08:24:54 | bauwser | hah, the likes of remote working | |
| 08:26:58 | lyarwood | urgh that's awful giblet | |
| 08:29:17 | giblet | I will survive | |
| 08:30:45 | bauwser | every last Thurday of the month, I appreciate my neighbor | |
| 08:31:41 | bauwser | just because then he gets 5 people for mowing his lawn | |
| 08:32:01 | giblet | a lot of lawn I assume | |
| 08:32:16 | bauwser | yeah, around ~2500sqm | |
| 08:32:31 | bauwser | but they are not having mowers | |
| 08:32:42 | bauwser | rather lawn tractors... | |
| 08:33:04 | bauwser | you can imagine the noise | |
| 08:39:20 | giblet | :/ | |
| 09:06:13 | opendevreview | Balazs Gibizer proposed openstack/nova stable/pike: Add a WA flag waiting for vif-plugged event during reboot https://review.opendev.org/c/openstack/nova/+/813437 | |
| 09:46:45 | opendevreview | Brin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage https://review.opendev.org/c/openstack/nova/+/768852 | |
| 09:46:46 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in os-quota-sets path https://review.opendev.org/c/openstack/nova/+/768851 | |
| 09:46:46 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in Limits API https://review.opendev.org/c/openstack/nova/+/768862 | |
| 09:52:35 | opendevreview | Merged openstack/nova master: tests: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/nova/+/817641 | |
| 09:56:53 | bauwser | gibi: thanks for reviewing https://review.opendev.org/c/openstack/nova/+/816861 | |
| 09:57:06 | bauwser | ah, giblet ^ | |
| 09:57:26 | bauwser | giblet: your comment looks good to me, I'll provide a new revision | |
| 10:02:38 | giblet | bauwser: how do you feel about my top level comment there that I'm supportive to allows non-cores to state priority but not for patch owners to self +1? | |
| 10:03:09 | bauwser | giblet: honestly, I don't have any opinion | |
| 10:03:29 | bauwser | giblet: maybe we should not use this label for asking folks to look at our change | |
| 10:03:49 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant* with project* in codes https://review.opendev.org/c/openstack/nova/+/769329 | |
| 10:04:02 | bauwser | giblet: but we should try to find some way to help contributors to ask for reviews without needing to go in IRC | |
| 10:04:30 | bauwser | so I'll provide 2 different points | |
| 10:04:41 | bauwser | and we'll discuss this in the change | |
| 10:17:34 | giblet | hm | |
| 10:17:50 | giblet | why people cannot come to IRC? | |
| 10:18:47 | giblet | is the problem that it is real time and therefore time zone dependent? | |
| 10:18:59 | giblet | then they can use the ML | |
| 10:19:34 | giblet | (but also I keep my client up and read scrollback so I can be reached from other timezones via IRC too) | |
| 10:34:05 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: [doc] propose Review-Priority label for contribs https://review.opendev.org/c/openstack/nova/+/816861 | |
| 10:36:29 | bauwser | giblet: we provided a way to async ask for reviews previously with the etherpad | |
| 10:36:41 | bauwser | now, you need to ping people, ideally synchronously | |
| 10:37:05 | bauwser | and I don't want the ML to be used for begging reviews or other non-nova contributors will yell at us :) | |
| 10:38:42 | bauwser | but I need to go to gym | |
| 10:38:56 | bauwser | I'm starting to look at https://gerrit-review.googlesource.com/Documentation/user-attention-set.html | |
| 10:38:59 | bauwser | we could use it | |
| 11:18:48 | giblet | bauwser: attentions set is built on people added in CC and in Review field of a patch | |
| 11:19:05 | giblet | btw, why don't we say, add me in the review if you need me to look at the patch | |
| 11:19:17 | giblet | that is async and gerrit based | |
| 11:27:25 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/nova/+/804709 | |
| 11:27:25 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of Executable.scalar(), Executable.execute() https://review.opendev.org/c/openstack/nova/+/804878 | |
| 11:27:26 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of 'autoload' parameter https://review.opendev.org/c/openstack/nova/+/805734 | |
| 11:27:26 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of legacy select() calling style https://review.opendev.org/c/openstack/nova/+/805735 | |
| 11:27:27 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace 'insert.inline' parameter with 'Insert.inline()' method https://review.opendev.org/c/openstack/nova/+/805736 | |
| 11:27:27 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Don't pass strings to 'Connection.execute' https://review.opendev.org/c/openstack/nova/+/805737 | |
| 11:27:28 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Don't use legacy 'Row()' methods https://review.opendev.org/c/openstack/nova/+/817746 | |