| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-11 | |||
| 15:48:59 | EugenMayer | but since i do not set this via meta-data it does not make sense that the qcow2 variant works and the other does not, right | |
| 15:49:53 | kevko | sean-k-mooney: hi, maybe I found something weird in my last discussed problem if you remember ... | |
| 15:50:10 | EugenMayer | if using q35 fixes it, i must have used it on my local as on the openstack libvirt. For the latter i did not. The properties i used for running the qcow or iso image where the same: | |
| 15:50:10 | EugenMayer | --property hw_disk_bus=sata --property hw_cpu_policy=dedicated --property img_config_drive=mandatory --property hw_cdrom_bus=ide --property hw_vif_model=e1000 --property hw_boot_menu=true --property hw_qemu_guest_agent=no | |
| 15:50:20 | EugenMayer | or for 7.x | |
| 15:50:25 | EugenMayer | --property hw_disk_bus=sata --property hw_cpu_policy=dedicated --property img_config_drive=mandatory --property hw_cdrom_bus=ide --property hw_vif_model=e1000e --property hw_boot_menu=true --property hw_qemu_guest_agent=no | |
| 15:52:57 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Close Glance image if downloading failed. https://review.opendev.org/c/openstack/nova/+/815347 | |
| 16:20:03 | gmann | lyarwood: gibi sean-k-mooney on tempest things to ssh things. we have two things here 1. all API tests are configurable to pause SSHing via run_validation config option 2. all scenario tests run SSH validation and does not care of run_validation config as most of scenario tests need to ssh | |
| 16:21:24 | lyarwood | gmann: I'm not sure what you mean by 1. do you have an example? | |
| 16:22:13 | lyarwood | what we are looking for here is a check during creation before any additional actions or validations actually happen | |
| 16:22:42 | lyarwood | I've got a WIP locally that I can finish up this evening to show what I mean | |
| 16:24:36 | gmann | lyarwood: we do access remote client only if run_validation is true in API tests | |
| 16:24:41 | gmann | lyarwood: but I think this does not check https://github.com/openstack/tempest/blob/7e96c8e854386f43604ad098a6ec7606ee676145/tempest/common/compute.py#L270 | |
| 16:25:08 | gmann | lyarwood: we should have this with condition CONF.validation.run_validation | |
| 16:25:12 | lyarwood | yeah I'm nuking that implementation FWIW | |
| 16:25:27 | lyarwood | parsing the console it's valid IMHO | |
| 16:25:46 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Revert project-specific APIs for servers https://review.opendev.org/c/openstack/nova/+/816206 | |
| 16:26:09 | dansmith | gmann: johnthetubaguy[m] ^ wasn't too bad to add that new scenario, and it helps to be able to see what works and how in that case | |
| 16:26:48 | gmann | lyarwood: +1 thanks, ping me once you are ready with that or even we can add new config option also in case we do not want all tests doing ssh things and do not want to wait for console ? | |
| 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 | 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 | |