| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-15 | |||
| 17:11:02 | artom | cfriesen, not a bad idea :) | |
| 17:11:17 | jaypipes | honestly, we really shouldn't be using the ENUM type anyway... but meh | |
| 17:11:38 | artom | So, I have to bounce of a lunch thing, and ideally I'd have liked more discussion about this before rewriting the spec, but I think I'll just bite the bullet and rewrite the spec :) | |
| 17:12:29 | dansmith | jaypipes: yeah I'm not a fan myself | |
| 17:13:48 | claudiub|2 | dansmith: ack, done that, but it seems like postgresql is dropping the column on alter, for some reason, and this exception is raised: oslo_db.exception.DBError: (psycopg2.InternalError) cannot drop type migration_type because other objects depend on it | |
| 17:13:48 | claudiub|2 | DETAIL: table migrations column migration_type depends on type migration_type | |
| 17:13:52 | cfriesen | artom: I'm just updating the review with some comments right now. gimme a couple minutes | |
| 17:14:05 | dansmith | claudiub|2: I don't think I can help you with that one :) | |
| 17:14:11 | dansmith | claudiub|2: but I don't think it's any of our doing | |
| 17:15:24 | claudiub|2 | I sea. | |
| 17:16:18 | openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553496 | |
| 17:16:38 | claudiub|2 | jaypipes: do you have any ideas? ^ | |
| 17:18:15 | openstackgerrit | Merged openstack/nova master: Update deprecated log-config option in docs https://review.openstack.org/551825 | |
| 17:20:43 | mriedem | efried: how do i get an endpoint url from a ksa adapter object? | |
| 17:21:06 | efried | mriedem: stand by. | |
| 17:21:30 | cfriesen | artom: okay, updated the review with some extra info | |
| 17:21:58 | efried | mriedem: .get_endpoint() | |
| 17:25:21 | sean-k-mooney | mriedem: exposing the vlans for trunk ports in metadata, i think the kuryr team did but honest answer is i dont know | |
| 17:25:46 | tssurya | dansmith, mriedem: since I had to rebase this regression (https://review.openstack.org/#/c/550967/) bug on top of the new purge command got merged before this, does this mean we will have to backport that new command into queens too ? | |
| 17:26:38 | dansmith | tssurya: generally just fix up the backport and note the reason for the conflicts in the commit message | |
| 17:26:54 | dansmith | I wouldn't be too opposed to backporting the purge stuff personally, but it's technically not a candidate | |
| 17:26:55 | tssurya | dansmith: okay thanks | |
| 17:27:07 | tssurya | yea I can fix up the backport | |
| 17:27:20 | tssurya | I didn't know if it was an allowed practice | |
| 17:27:20 | dansmith | tssurya: if you survey some other backports you'll see some "Conflicts:" examples | |
| 17:27:38 | tssurya | dansmith: thanks will look them up | |
| 17:27:49 | dansmith | tssurya: example: https://review.openstack.org/#/c/540145/ | |
| 17:28:16 | jaypipes | claudiub|2: added review note. | |
| 17:28:25 | tssurya | dansmith: perfect thank you! | |
| 17:28:31 | claudiub|2 | \o/ thanks. :D | |
| 17:56:23 | cfriesen | are there any known issues with nic tagging in Newton? | |
| 17:57:07 | stephenfin | artom: ^ ? | |
| 17:57:09 | cfriesen | I'm failing schema validation, wondering if it's something we screwed up but I don't see us changing anything in that area. | |
| 17:57:19 | cfriesen | (by "we" I mean my organization) | |
| 17:58:12 | cdent | jaypipes: you might enjoy this buglet: https://bugs.launchpad.net/nova/+bug/1756151 | |
| 17:58:13 | openstack | Launchpad bug 1756151 in OpenStack Compute (nova) "placement os-traits sync checked every request" [Low,Triaged] | |
| 18:03:02 | cfriesen | stephenfin: artom: ah yes, microversion 2.37 broke tagging, and 2.42 added it back in. | |
| 18:13:15 | mriedem | melwitt: have you heard anything new about a project update session at the summit? | |
| 18:15:09 | melwitt | mriedem: no, I'm gonna ask anne about it | |
| 18:15:15 | mriedem | nova project update doesn't fall under the category of CI/CD, HPC or EDGE so probably not | |
| 18:15:19 | mriedem | i replied to the ML thread | |
| 18:15:25 | mriedem | quite dickishly | |
| 18:16:13 | melwitt | heh. I tried to find them in the summit schedule and found none | |
| 18:16:19 | mriedem | same | |
| 18:17:29 | mriedem | nova project update: feature k8s! | |
| 18:17:34 | mriedem | *featuring | |
| 18:17:35 | mriedem | damn | |
| 18:20:48 | jaypipes | efried, edleafe: where are we on your battling microversion changes? | |
| 18:21:20 | efried | jaypipes: It's tied up on the home stretch while zuul unwinds its panties. | |
| 18:21:27 | efried | See topic | |
| 18:21:34 | jaypipes | k | |
| 18:24:51 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345 | |
| 18:24:54 | mriedem | efried: i've tried to rebase this but there are some known broken things in it ^ | |
| 18:25:44 | efried | mriedem: Yeah, I had to put it aside for other "more urgent" things. | |
| 18:26:01 | efried | I swear I had it working at some point in the cycle. | |
| 18:26:34 | efried | but by the end, it was definitely busted and I couldn't figure out how to fix it without spending a big chunk of time. | |
| 18:28:31 | openstackgerrit | Claudiu Belu proposed openstack/nova master: compute: Adds instance live-resize https://review.openstack.org/248581 | |
| 18:28:31 | openstackgerrit | Claudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type https://review.openstack.org/185961 | |
| 18:34:20 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add disabled field to CellMapping object https://review.openstack.org/550090 | |
| 18:34:20 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Add CellMappingList.get_all_enabled() query method https://review.openstack.org/550188 | |
| 18:40:24 | cdent | nice message efried | |
| 18:40:40 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add CellMappingList.get_all_enabled() query method https://review.openstack.org/550188 | |
| 18:40:53 | efried | Thanks cdent. I guess if anyone was gonna bother to read the whole thing, it'd be you :) | |
| 18:41:21 | cdent | yeah, spose so | |
| 18:41:34 | cdent | I think you'll find there's a vast army of silent readers out there | |
| 18:41:48 | cdent | their silence can sometimes be rather disturbing | |
| 18:46:06 | efried | cdent: I'm quite a slow reader, and I often feel that pain when something's important enough that I know I gotta read it, but really long (like dhellman's missive on requirements). I guess for people who read at normal speeds, it's not such an onerous task. | |
| 18:47:25 | cdent | efried: yeah, I've been reminded many times that my attitude towards reading comes from something of a position of privilege. Apparently I read _very_ fast when it comes to email and similar forms. | |
| 18:47:42 | efried | That explains a lot. | |
| 18:48:49 | efried | I envy those who can read fast and not miss stuff. That's why I read slow - because I'm being real thorough (terrified of missing some detail or - gods forbid - failing to catch a typo!) | |
| 18:49:23 | edleafe | heh, just started reading efried's email | |
| 18:50:12 | edleafe | An hourglass would be good enough :) | |
| 18:52:35 | cdent | efried: I'm certain that I miss stuff, but I'm usually grazing for meaning, not details | |
| 18:52:58 | efried | I should develop that skill. FOMO. | |
| 18:53:18 | cdent | maybe not, probably useful to have both styles around | |
| 18:53:23 | mriedem | mgoddard_: stephenfin: dansmith: so maybe the numa topology filter is ok with ironic http://logs.openstack.org/12/553412/1/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/2b781af/logs/screen-n-sch.txt.gz#_Mar_15_15_24_11_093172 | |
| 18:53:27 | mriedem | that devstack change didn't blow up | |
| 18:53:33 | dansmith | sweet | |
| 18:53:41 | cdent | for me it's in part a learned skill to intentionally miss out on some stuff | |
| 18:55:03 | mriedem | could have sworn something about the IronicNodeState object was different such that the filter would fail with it | |
| 18:55:12 | mriedem | also, it's not like we actually have pci requests in these CI jobs | |
| 18:55:30 | edleafe | I tried to learn to skim back in college. I always felt like I never got anything out of it, so I went back to reading details | |
| 18:55:58 | edleafe | efried: excellent recap | |
| 18:55:59 | dansmith | mriedem: that's true, I guess a flavor or request with pci or numa could break if there are ironic hosts in there somehow, | |
| 18:56:09 | dansmith | but I dunno what that would be really | |
| 18:56:29 | efried | edleafe: Thanks. | |
| 18:56:36 | mriedem | dansmith: the other thing might have been something to do with allocation ratios, but grasping at straws | |
| 18:57:29 | dansmith | we don't have ratios for those types though | |
| 18:58:04 | mriedem | goes into the numa topology limits | |
| 18:58:08 | mriedem | the cpu and ram allocation ratois | |
| 18:58:55 | cfriesen | mriedem: we have a private patch to enable ironic and regular nodes...had to make some of the scheduler filters check the hypervisor type. | |
| 18:59:12 | mriedem | cfriesen: is there anything you guys don't have a private patch for? | |
| 18:59:33 | cfriesen | mriedem: we try to upstream stuff, but it takes forever | |
| 18:59:40 | mgoddard_ | mriedem: that's good news! | |
| 19:00:01 | cfriesen | mriedem: plus, we only need to worry about one hypervisor | |
| 19:00:16 | mriedem | mgoddard_: well, it's kind of a false positive i think | |
| 19:00:34 | mriedem | cfriesen: it takes even longer when you don't even propose them | |
| 19:00:45 | dansmith | mriedem: it's not a false positive, it's just not comprehensive.. it means something, it just doesn't mean it all works fine :) | |
| 19:01:39 | mriedem | i think host_topology_and_format_from_host might be the thing | |
| 19:02:20 | mriedem | that's just always None for ironic i think | |