| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-11 | |||
| 17:18:45 | gibi | yes | |
| 17:18:56 | sean-k-mooney | i see why not just remove https://review.opendev.org/c/openstack/tempest/+/817560/1/tempest/api/compute/base.py#619 | |
| 17:18:58 | gibi | created server, attached volume, did a live migration and then cleanup | |
| 17:20:01 | sean-k-mooney | we want to delete the voluem after the test sure but we dont need to detach it just delete the vm | |
| 17:22:00 | gibi | I'm not that familiar with tempest that I can say it is always OK not to detach as volume create and volume attach are two seperate action from the test case | |
| 17:26:32 | lyarwood | yeah volume attach always adds in the detach cleanup in tempest | |
| 17:26:53 | lyarwood | I've talked about removing that in the past but people didn't want to drop the coverage of detach etc | |
| 17:28:25 | sean-k-mooney | which given we have detach test i dont really get | |
| 17:28:41 | sean-k-mooney | we did talk about it for the compute test before definetly | |
| 18:12:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add debug log for scheduler weight calculation https://review.opendev.org/c/openstack/nova/+/817632 | |
| 18:14:59 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: DNM - Testing volume detach failures https://review.opendev.org/c/openstack/nova/+/817636 | |
| 18:15:32 | lyarwood | gibi ^ sorry been stuck on more calls, I'll try to clean this up later or early tomorrow but that's the general outline I had in mind | |
| 18:16:35 | gibi | ok, lets see how it goes with the testing | |
| 18:21:29 | gmann | gibi: can you check this, it help to remove this broken opensuse job from tempest https://review.opendev.org/c/openstack/nova/+/816574 | |
| 18:36:40 | EugenMayer | sean-k-mooney interesting, suddenly i cannot get the 7.x working again. The nic is detected but DHCP does not work (nic type is e1000e), with 6.5 and e1000 DHCP does actually work. I'am really not sure if i kind of messed it up beforehand and deployed 6.5 instead of 7.0 and did not notice or it is something else. Your hint with q35 would be | |
| 18:36:40 | EugenMayer | hw_machine_type=q35 i guess, right? | |
| 18:38:19 | sean-k-mooney | yes if you want to use e100e | |
| 18:38:25 | sean-k-mooney | *e1000e | |
| 18:38:29 | sean-k-mooney | with pcie | |
| 18:38:54 | sean-k-mooney | but i have never tied to run esxi on openstack so your millage will vary | |
| 18:56:03 | opendevreview | Merged openstack/placement stable/xena: Use 'functional-without-sample-db-tests' tox env for placement nova job https://review.opendev.org/c/openstack/placement/+/817255 | |
| 18:59:43 | EugenMayer | sean-k-mooney did not work either. Odd. I must have gotten confused and picked the wrong image to verify 7.x - i just cannot get it reprodroduced. | |
| 19:19:29 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/nova/+/804709 | |
| 19:19:30 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of Engine.scalar(), Engine.execute() https://review.opendev.org/c/openstack/nova/+/804878 | |
| 19:19:30 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of 'autoload' parameter https://review.opendev.org/c/openstack/nova/+/805734 | |
| 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 | |