| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-23 | |||
| 16:58:56 | sean-k-mooney | i think its becasue we were geting it form an object of type sqlalchemy.engine.url.URL | |
| 16:59:01 | sean-k-mooney | and then casting that to a string | |
| 16:59:16 | lyarwood | ah | |
| 16:59:18 | sean-k-mooney | so it was likely being url encoded by the str call | |
| 17:19:20 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Query the service group API within check_instance_host https://review.opendev.org/c/openstack/nova/+/802317 | |
| 17:19:21 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Increase service_down_time beyond INITIAL_REPORTING_DELAY in test https://review.opendev.org/c/openstack/nova/+/805667 | |
| 17:20:00 | lyarwood | gibi: ^ there was a silly race in my func test causing https://bugs.launchpad.net/nova/+bug/1940741 that should hopefully fix it now | |
| 17:21:07 | gibi | lyarwood: good catch! | |
| 17:21:24 | gibi | I did not know there is an initial delay | |
| 17:21:29 | lyarwood | ah wait, might need to refine that a little | |
| 17:32:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add two new hacking rules https://review.opendev.org/c/openstack/nova/+/805668 | |
| 17:32:50 | gibi | lyarwood: the hacking rule ^^ | |
| 17:34:22 | gibi | and that marks my end of the day o/ see you tomorrow folks | |
| 17:35:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add two new hacking rules https://review.opendev.org/c/openstack/nova/+/805668 | |
| 17:37:12 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Increase service_down_time beyond INITIAL_REPORTING_DELAY in test https://review.opendev.org/c/openstack/nova/+/805667 | |
| 17:37:13 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Query the service group API within check_instance_host https://review.opendev.org/c/openstack/nova/+/802317 | |
| 17:37:14 | lyarwood | Aborting. | |
| 17:37:14 | lyarwood | Type 'yes' to confirm, other to cancel: yes | |
| 17:37:14 | lyarwood | Do you really want to submit the above commits? | |
| 18:18:50 | melwitt | hmm.. why isn't zuul voting on a change when it passes? https://review.opendev.org/c/openstack/nova/+/805654 sorry if I missed the discussion | |
| 18:30:17 | melwitt | maybe a fluke. just rechecked it | |
| 18:34:23 | melwitt | sean-k-mooney: hey, in case you didn't see, I replied to your comments on https://review.opendev.org/c/openstack/placement/+/679441 | |
| 18:34:33 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use cell with more free memory first https://review.opendev.org/c/openstack/nova/+/805649 | |
| 18:58:33 | opendevreview | Lee Yarwood proposed openstack/nova master: db: Handle parameters in DB strings https://review.opendev.org/c/openstack/nova/+/805663 | |
| 19:32:52 | opendevreview | Merged openstack/nova master: VMWare: Use WithRetrieval to get all results https://review.opendev.org/c/openstack/nova/+/804968 | |
| 22:22:02 | opendevreview | Merged openstack/nova master: Avoid modifying the Mock class in test https://review.opendev.org/c/openstack/nova/+/805657 | |
| 23:14:20 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of Engine.scalar(), Engine.execute() https://review.opendev.org/c/openstack/nova/+/804878 | |
| 23:14:20 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/nova/+/804709 | |
| 23:14:21 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of legacy select() calling style https://review.opendev.org/c/openstack/nova/+/805735 | |
| 23:14:21 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace use of 'autoload' parameter https://review.opendev.org/c/openstack/nova/+/805734 | |
| 23:14:22 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Don't pass strings to 'Connection.execute' https://review.opendev.org/c/openstack/nova/+/805737 | |
| 23:14:22 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Replace 'insert.inline' parameter with 'Insert.inline()' method https://review.opendev.org/c/openstack/nova/+/805736 | |
| 23:15:14 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Add migration to resolve shadow table discrepancies https://review.opendev.org/c/openstack/nova/+/805738 | |
| #openstack-nova - 2021-08-24 | |||
| 06:47:02 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Avoid modifying the Mock class in test https://review.opendev.org/c/openstack/nova/+/805759 | |
| 06:57:28 | opendevreview | Johannes Kulik proposed openstack/nova master: api: Log actual number of UUIDs queried https://review.opendev.org/c/openstack/nova/+/805786 | |
| 07:24:37 | gibi | fungi: I re-opened the bug https://bugs.launchpad.net/nova/+bug/1927677 should I reopne the OSA part of it too? | |
| 08:19:50 | admin1 | when i do a rescue, in horizon , it asks for pass .but that pas does not work for root or debian/ubuntu .. | |
| 08:19:54 | admin1 | so how exactly is a rescue done | |
| 08:25:33 | lyarwood | admin1: so we attempt to inject the password during the rescue into the rescue image, either directly or via a configdrive and cloud-init | |
| 08:26:24 | lyarwood | admin1: direct injection into the rescue image depends on the config in your env | |
| 08:26:39 | lyarwood | admin1: and injection by configdrive depends on cloud-init being used by the rescue image | |
| 08:27:52 | lyarwood | https://github.com/openstack/nova/blob/2d2fb2debab9e1705231ebee1bd9305c5b22fa15/nova/virt/libvirt/driver.py#L4635-L4638 controls the direct injection | |
| 08:29:27 | admin1 | is there a good recommended resuce image that works ? | |
| 08:39:32 | opendevreview | Lee Yarwood proposed openstack/nova master: api: Introduce microversion 2.89 adjusting os-volume_attachments https://review.opendev.org/c/openstack/nova/+/804275 | |
| 08:50:46 | opendevreview | Merged openstack/nova master: address open redirect with 3 forward slashes https://review.opendev.org/c/openstack/nova/+/805654 | |
| 08:59:42 | lyarwood | admin1: You can use pretty much anything, I tend to virt-customize the image with a password etc before use | |
| 08:59:59 | lyarwood | admin1: but there are plenty of small rescue images made for this purpose with physical hosts like http://tinycorelinux.net/ | |
| 09:00:25 | lyarwood | https://www.techradar.com/uk/best/best-linux-repair-and-rescue-distros has loads | |
| 09:11:40 | admin1 | lyarwood, thanks | |
| 09:32:53 | opendevreview | Lee Yarwood proposed openstack/python-novaclient master: Microversion 2.89 - os-volume_attachments https://review.opendev.org/c/openstack/python-novaclient/+/805022 | |
| 09:51:23 | lyarwood | melwitt: https://review.opendev.org/c/openstack/nova/+/805667 - would you mind hitting this when you're online to resolve a gate-failure | |
| 10:44:12 | opendevreview | Lee Yarwood proposed openstack/nova master: api: Introduce microversion 2.89 adjusting os-volume_attachments https://review.opendev.org/c/openstack/nova/+/804275 | |
| 11:00:17 | gibi | lyarwood: I have doc questions in https://review.opendev.org/c/openstack/nova/+/804275 otherwise it looks good | |
| 11:06:28 | opendevreview | Balazs Gibizer proposed openstack/placement master: Bump os-traits to latest 2.6.0 https://review.opendev.org/c/openstack/placement/+/805830 | |
| 11:09:07 | opendevreview | Lee Yarwood proposed openstack/nova master: api: Introduce microversion 2.89 adjusting os-volume_attachments https://review.opendev.org/c/openstack/nova/+/804275 | |
| 11:11:24 | fungi | gibi: don't worry about the ossa task, i'll reopen it in a bit | |
| 11:11:27 | fungi | and thanks! | |
| 11:11:33 | gibi | fungi: thanks you | |
| 11:12:26 | fungi | unfortunately i have a number of them to catch up on after breakfast | |
| 11:42:36 | gibi | is it a known fact that the metadata service will only have generic metadata while the config drive will have virt driver specific metadata? as per https://bugs.launchpad.net/nova/+bug/1940894 | |
| 11:48:11 | opendevreview | Merged openstack/nova master: docs: Drop references to non-filter scheduler drivers https://review.opendev.org/c/openstack/nova/+/773645 | |
| 11:55:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix documentation about cpu topologies https://review.opendev.org/c/openstack/nova/+/805833 | |
| 11:55:27 | gibi | stephenfin: an simple doc fix ^^ | |
| 12:53:40 | sean-k-mooney | gibi: actully that is only true if you are suing the new cpu_share_set and cpu dedicated set options | |
| 12:54:11 | sean-k-mooney | although you are not really chanign the content | |
| 12:54:18 | sean-k-mooney | so i gues your change is fine | |
| 12:54:49 | sean-k-mooney | your main correct is with to without which indeed was a bug | |
| 12:55:48 | sean-k-mooney | gibi: in case you were not aware if you use vcpu_pin_set and dont use the new options we still support the old cpu pinning behavior eventough that is deprecated | |
| 12:56:16 | sean-k-mooney | stephenfin: was going to submit patchs to remove that but im not sure he had time to do that yet | |
| 12:56:38 | gibi | ack | |
| 13:37:06 | opendevreview | Merged openstack/nova master: api: Log actual number of UUIDs queried https://review.opendev.org/c/openstack/nova/+/805786 | |
| 13:40:46 | opendevreview | Merged openstack/nova master: Retry on vmware create_vm when it fails https://review.opendev.org/c/openstack/nova/+/764586 | |
| 13:41:19 | opendevreview | Merged openstack/nova master: Add missing [oslo_reports] options https://review.opendev.org/c/openstack/nova/+/804568 | |
| 13:45:59 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: address open redirect with 3 forward slashes https://review.opendev.org/c/openstack/nova/+/805818 | |
| 15:25:18 | opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: Avoid modifying the Mock class in test https://review.opendev.org/c/openstack/nova/+/805759 | |
| 15:26:02 | opendevreview | Stephen Finucane proposed openstack/nova stable/victoria: Avoid modifying the Mock class in test https://review.opendev.org/c/openstack/nova/+/805823 | |
| 15:40:52 | gmann | lyarwood: takashi comments are good to fix to correct api-ref. how you will like to do in follow up or in same patch ? https://review.opendev.org/c/openstack/nova/+/804275/10 | |
| 15:44:55 | gibi | fyi, nova meeting starts in 15 minutes here in the channel | |
| 16:00:24 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:24 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:24 | opendevmeet | Meeting started Tue Aug 24 16:00:24 2021 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:24 | gibi | #startmeeting nova | |
| 16:00:29 | gibi | o/ | |
| 16:03:10 | elodilles | o/ | |
| 16:03:30 | gibi | thanks elodilles, I felt alone | |
| 16:03:57 | elodilles | :] | |
| 16:04:28 | gibi | is it only the two of us or other still listening quietly? | |
| 16:05:41 | gibi | I won't do a full meeting just for the two of us | |
| 16:06:04 | gibi | lets see what is important from the agenda | |
| 16:06:35 | gibi | we have one critical bug https://bugs.launchpad.net/nova/+bug/1940555 around SQLAlchemy and the fix is being discussed in: https://review.opendev.org/c/openstack/nova/+/805663 | |
| 16:07:21 | gibi | I guess it is a blocker for tripleoo | |
| 16:08:07 | gibi | overall the nova gate is is an acceptable state. we are landing patches | |
| 16:08:55 | gibi | about the PTG: every info is in the PTG etherpad #link https://etherpad.opendev.org/p/nova-yoga-ptg | |
| 16:09:17 | gibi | we will have FF next week | |
| 16:09:31 | gibi | release todos are being tracked in #link https://etherpad.opendev.org/p/nova-xena-rc-potential | |
| 16:09:50 | gibi | if you see release critical bugs then please tag them with xena-rc-potential tag | |
| 16:10:08 | gibi | as per elodilles the stable gate also look healthy | |
| 16:10:16 | gibi | is there any question before I close this? | |
| 16:10:25 | elodilles | nope :X | |
| 16:12:08 | gibi | then lets close this | |