| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-11 | |||
| 16:27:06 | gmann | dansmith: thanks, I will check it after lunch today. | |
| 16:27:39 | lyarwood | gmann: I'm overloading wait_until to accept PINGABLE and SSHABLE, both basically fallback to ACTIVE if validations aren't enabled | |
| 16:28:12 | lyarwood | gmann: but when they are we wait first until the server is ACTIVE and then pingable or sshable etc. | |
| 16:32:01 | gmann | lyarwood: yeah, currently this method make server pingable or sshable if validatable=True but not do actual ping/ssh and with the new way you are making wait_until to do actual ssh/ping and after Active state ? | |
| 16:42:34 | lyarwood | gmann: sorry on a call, yeah correct so wait_until=PINGABLE would wait until the server is ACTIVE and will then wait until it is pingable *if* validatable=True etc | |
| 16:42:56 | lyarwood | gmann: if not it just waits until it's ACTIVE and continues | |
| 16:43:12 | gmann | lyarwood: +1 | |
| 17:01:44 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Close Glance image if downloading failed. https://review.opendev.org/c/openstack/nova/+/815347 | |
| 17:13:54 | gibi | lyarwood, sean-k-mooney: the first run with the 30 sec sleep before detach finished and did not generated any kernel panic https://zuul.opendev.org/t/openstack/build/06cb831dd6c94a2fa8ca373662014dd2/logs | |
| 17:14:25 | gibi | but also the guest did not print anything to the console during that extra 30 seconds | |
| 17:14:36 | gibi | so no hard proof yet that we need to wait | |
| 17:14:54 | gibi | I will recheck it | |
| 17:15:26 | sean-k-mooney | ack | |
| 17:15:59 | sean-k-mooney | so you wait for it to be active then sleep 30 right | |
| 17:16:03 | sean-k-mooney | then attach and detach | |
| 17:17:10 | gibi | nope, the failures are happening during test class cleanup | |
| 17:17:13 | sean-k-mooney | oh you put it in the detach | |
| 17:17:18 | gibi | so I just wait before the detach | |
| 17:18:00 | sean-k-mooney | oh its not in the actual detach test | |
| 17:18:14 | sean-k-mooney | its in the cleanup of a test that did attach? | |
| 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 | hw_machine_type=q35 i guess, right? | |
| 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: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 'autoload' parameter https://review.opendev.org/c/openstack/nova/+/805734 | |
| 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: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: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:32 | opendevreview | Stephen Finucane proposed openstack/nova master: test: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/nova/+/817641 | |
| 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: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: Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/nova/+/804709 | |
| 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:09 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of 'autoload' parameter https://review.opendev.org/c/openstack/nova/+/805734 | |
| 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: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: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: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: Assert quota related API behavior when noop https://review.opendev.org/c/openstack/nova/+/712140 | |
| 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: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: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:46 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 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:47 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 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: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: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 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: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: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: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 | |
| 23:47:53 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| #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* with project* in codes https://review.opendev.org/c/openstack/nova/+/769329 | |
| 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 | |
| 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: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315 | |
| 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: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: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:43 | opendevreview | Brin Zhang proposed openstack/nova master: Replace tenant* with project* in codes https://review.opendev.org/c/openstack/nova/+/769329 | |
| 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: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 | |