| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-15 | |||
| 11:59:34 | stephenfin | Do we still support that? There was talk of dropping it some time back, though cfriesen had objections | |
| 11:59:35 | mnaser | right but you'd have to install that yourself (i.e. it shouldn't be inside test-requirements.txt in openstack/nova ?) | |
| 11:59:52 | mnaser | oh i found this https://review.opendev.org/#/c/296604/ | |
| 12:00:31 | stephenfin | What's the opportunistic testing he refers to? | |
| 12:00:51 | mnaser | this is 3 years ago, but i think there's a tc patch saying we dont support postgres. | |
| 12:01:03 | sean-k-mooney | we have a db mixing that will use mysql or postgress instead of sqlitght if it avioable i think | |
| 12:01:33 | cdent | mnaser: that patch never landed in full | |
| 12:01:34 | sean-k-mooney | mnaser: there was discussion or it yes | |
| 12:01:42 | mnaser | ah, ok. | |
| 12:01:48 | cdent | instead we decided that we wouldn't explicitly disallow it | |
| 12:02:02 | mnaser | i'm just annoyed by it because i just had tox fail on me cause i didnt have postgres development headers.. | |
| 12:02:03 | cdent | instead projects could try to use sqlachemy in the way it was intended if they like | |
| 12:02:15 | sean-k-mooney | mnaser: are you just annoyed by the fact that requires use to install the postgres client and complie it | |
| 12:02:21 | sean-k-mooney | and that breaks on stable | |
| 12:02:28 | sean-k-mooney | ya | |
| 12:02:38 | mnaser | i have to get the postgres client to run tox locally (its inside test-requirements, not requirements) | |
| 12:02:39 | sean-k-mooney | we could replacy it with psycopg2-binary | |
| 12:03:06 | mnaser | oh thats a reasonable suggestion | |
| 12:03:32 | sean-k-mooney | "The binary package is a practical choice for development and testing but in production it is advised to use the package built from sources.£ | |
| 12:03:44 | cdent | mnaser: yeah, in placement we did this: https://review.opendev.org/#/c/671249/ which may be a method for possible hacking around things | |
| 12:03:47 | sean-k-mooney | we are using it for testing so i think it makes sense | |
| 12:04:08 | mnaser | oh thats a nice workaround too | |
| 12:04:10 | mnaser | TIL about sys_platform | |
| 12:04:25 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion 2.76: Add 'power-update' external event https://review.opendev.org/645611 | |
| 12:04:48 | sean-k-mooney | cdent: so when you develop on macos you dont run all the db test and leave the gate check it | |
| 12:05:02 | sean-k-mooney | unfortunetly i dont think we will be allowed do the same on linux | |
| 12:05:20 | sean-k-mooney | maybe i should go back ot using windows | |
| 12:05:22 | cdent | sean-k-mooney: the db tests are opportunistic so the migrations are testing on sqlite but not msyql/postgresql | |
| 12:06:16 | cdent | i'm not suggesting that we do exactly what's done in that patch, rather that there might be _some_ strategy for making those requirements optional | |
| 12:06:32 | sean-k-mooney | ya but back to the binary package is there any reason not to jsut use that instead | |
| 12:06:53 | sean-k-mooney | cdent: well we could make it a tox siblig too | |
| 12:07:03 | sean-k-mooney | rather then have it in test requirements | |
| 12:12:28 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682 | |
| 12:18:24 | openstackgerrit | Mohammed Naser proposed openstack/nova master: config: remove deprecated checksum options https://review.opendev.org/676539 | |
| 12:18:25 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Remove nova.image.download.modules extension point https://review.opendev.org/676540 | |
| 12:18:25 | openstackgerrit | Mohammed Naser proposed openstack/nova master: requirements: move to using psycopg2-binary https://review.opendev.org/676684 | |
| 12:25:31 | stephenfin | cdent: That's easy - 'extras' | |
| 12:25:50 | stephenfin | e.g. https://github.com/openstack/oslo.messaging/blob/master/setup.cfg#L22-L29 | |
| 12:26:32 | sean-k-mooney | stephenfin ah ya that is what i was tihnking of when i said tox siblings | |
| 12:27:27 | sean-k-mooney | but we would have to be able to use tox -e py37[mysql,postgres] or whatever in the upstream gate | |
| 12:27:59 | sean-k-mooney | if we could not run the tox env with the extras installed then it wont help | |
| 12:28:38 | stephenfin | sean-k-mooney: Not necessarily | |
| 12:29:19 | stephenfin | sean-k-mooney: https://tox.readthedocs.io/en/latest/config.html#conf-extras | |
| 12:30:13 | sean-k-mooney | the goal is to not need to install postgres header locally to be able to run tox | |
| 12:30:34 | sean-k-mooney | but to still enable all the testing with the optional deps in the gate | |
| 12:31:47 | sean-k-mooney | so im not sure that actully helps | |
| 12:36:48 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the target in os-services APIs policy https://review.opendev.org/676688 | |
| 12:38:44 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in Admin Actions https://review.opendev.org/657823 | |
| 12:39:10 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682 | |
| 12:39:28 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095 | |
| 13:20:42 | gmann | melwitt: can you review this. This is followup fix for policy default refresh spec - https://review.opendev.org/#/c/669196/ | |
| 13:21:44 | gmann | melwitt: johnthetubaguy also I have prepared the first set of changes and looking for feedback before I proceed on other policy changes. | |
| 13:22:28 | gmann | melwitt: johnthetubaguy this and its base patches are the complete set of os-services policy change - https://review.opendev.org/#/c/676688/ and | |
| 13:22:46 | gmann | sent the review guide on ML also http://lists.openstack.org/pipermail/openstack-discuss/2019-August/008504.html | |
| 13:40:31 | mriedem | gibi: sorry i didn't address these before rebasing yesterday, but replied now https://review.opendev.org/#/c/635080/38 | |
| 13:42:28 | sean-k-mooney | mriedem: o/ | |
| 13:42:39 | sean-k-mooney | mriedem: i played around with lxc a bit last night | |
| 13:42:58 | sean-k-mooney | so i managed to boot a contianer but there are issues | |
| 13:43:48 | sean-k-mooney | i was able to boot with the .img file in https://cloud-images.ubuntu.com/bionic/current/bionic-server-cloudimg-amd64.tar.gz | |
| 13:44:09 | sean-k-mooney | that is a img file create from a raw ext4 partion with the full file system | |
| 13:44:36 | sean-k-mooney | i confimed the cirros image seam to use /init instead of /sbin/init and we hardcode the path | |
| 13:45:13 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5055-L5057 | |
| 13:45:52 | sean-k-mooney | but i also found we do not resize the image and as a result cloud init does not run | |
| 13:46:33 | sean-k-mooney | * we might resize the image but we dont resize the filesystem in the image | |
| 13:47:17 | gmann | alex_xu: cc for ^^ policy default refresh changes review and early feedback- http://lists.openstack.org/pipermail/openstack-discuss/2019-August/008504.html | |
| 13:47:50 | gmann | mriedem: can you check this now, added functional tests - https://review.opendev.org/#/c/676275/ | |
| 13:48:01 | sean-k-mooney | mriedem: so tl;dr if you have a image preprepared with enough space for cloud-init to run we might be oke but i dont know if cloud inti can resize the file system when dealing with lxc | |
| 13:55:28 | openstackgerrit | Chris Dent proposed openstack/nova master: DNM: add profiling to scheduler https://review.opendev.org/674810 | |
| 14:02:10 | yonglihe | mriedem: I got Alex's +2, you recently reviewed this patch, Hope you had bandwidth to review it again: https://review.opendev.org/#/c/621476/ | |
| 14:02:30 | efried | melwitt: Thank you for sending that. The u-c bump merged as well. That opens up several improvements in nova. | |
| 14:12:34 | mordred | efried: did your mock.patch thing wind up working? | |
| 14:12:42 | efried | yes! | |
| 14:12:43 | gibi | mriedem: ack, I try to look at it today | |
| 14:13:12 | efried | mordred: which excites me not because it's a lovely solution, but because it lets me use all those shiny new features in nova now. | |
| 14:13:44 | sean-k-mooney | efried: you are not useing mock.patch outside of tests are you? | |
| 14:13:45 | efried | which I will start doing just as soooon as I wake up. | |
| 14:14:00 | efried | sean-k-mooney: Why, would that be a bad thing? | |
| 14:14:07 | efried | it's exactly the same as temporary_mutation | |
| 14:14:24 | openstackgerrit | Mohammed Naser proposed openstack/nova master: config: remove deprecated checksum options https://review.opendev.org/676539 | |
| 14:14:24 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Remove nova.image.download.modules extension point https://review.opendev.org/676540 | |
| 14:14:25 | openstackgerrit | Mohammed Naser proposed openstack/nova master: requirements: move to using psycopg2-binary https://review.opendev.org/676684 | |
| 14:14:35 | efried | sean-k-mooney: I'm yanking your chain. We're talking about this https://review.opendev.org/676495 | |
| 14:14:52 | sean-k-mooney | because you are monkeypatchin other code at runtime which can fail in really hard to debug ways | |
| 14:15:03 | sean-k-mooney | :) | |
| 14:15:47 | sean-k-mooney | ok that is less scary | |
| 14:16:46 | cdent | sigh: the nova functional tests are _still_ reading /etc/nova/nova.conf (if it exists). I thought we fixed that? | |
| 14:17:07 | sean-k-mooney | what test? | |
| 14:17:19 | sean-k-mooney | its proably missing a config fixture | |
| 14:18:03 | aspiers | AFAICS it just doesn't do what it claims to | |
| 14:18:03 | tssurya | mriedem, dansmith: hmm I was thinking for the UnexpectedTaskStateError exception for the power-update event, should we also handle the faults/error notifications in case its actually another action that won the race ? | |
| 14:18:26 | tssurya | as of now if we simply return the event is considered as a successful attempt | |
| 14:19:25 | dansmith | tssurya: I'm not sure what you're asking | |
| 14:20:30 | openstackgerrit | Chris Dent proposed openstack/nova master: single pass instance info fetch in host manager https://review.opendev.org/623558 | |
| 14:20:40 | mriedem | yonglihe: i can review again based on my previous comments but i likely won't be approving that change, because (1) we're trying to land tssurya's change for 2.76 since there is an approved ironic change that depends on the nova change (see the ML) and (2) i can't vote on the low-level contents of what's going into the numa topology response, i think dansmith or others that were more involved in the spec approval should revie | |
| 14:20:40 | mriedem | ose details. | |
| 14:21:08 | mriedem | sean-k-mooney: ack on the lxc thing; we can define an image to use in the job, but if that hard-coding in the libvirt driver is a problem we'll need to do something about that. | |
| 14:21:42 | mriedem | sean-k-mooney: i'm not sure how much we care about cloud-init in the lxc job, but someone does have some patches up for making cloud-init work with lxc (so saying that, it might not work today anyway) | |
| 14:21:56 | tssurya | dansmith: ah nvm, I got the answer | |
| 14:21:58 | mriedem | sean-k-mooney: yeah this https://review.opendev.org/#/c/667976/ | |
| 14:22:05 | tssurya | :D sorry for the noise | |