| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-13 | |||
| 09:37:29 | takashin | stephenfin: I could not reproduce failure in my local environment. | |
| 09:37:30 | sapd_ | gmann: I think this is a important feature when want to create instance use volume based instead of create volume after launch instance | |
| 09:38:51 | stephenfin | takashin: I'm guessing there's some kind of race going on. Let me see if I can reproduce quickly | |
| 09:39:13 | takashin | stephenfin: Thank you. | |
| 09:41:13 | sahid | stephenfin: can you re-ACK 2 first patches here: https://review.openstack.org/#/c/560455/? | |
| 09:41:45 | sahid | they lost the bit because of a rebase | |
| 09:43:51 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 7 https://review.openstack.org/560748 | |
| 10:39:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Fix how context managers are mocked https://review.openstack.org/561190 | |
| 10:39:25 | stephenfin | takashin: That should do the trick ^ | |
| 10:42:17 | openstackgerrit | Merged openstack/nova master: only increment disk address unit for scsi devices https://review.openstack.org/538310 | |
| 10:47:54 | openstackgerrit | Merged openstack/nova master: trivial: fix a comment typo https://review.openstack.org/560349 | |
| 10:49:13 | kashyap | I wonder if Gerrit can be tweaked to retain the +2 post-rebase | |
| 10:49:24 | kashyap | (Post-successful rebase) | |
| 10:50:42 | odyssey4me | kashyap it already does, assuming the only thing changing in the rebase is the parent commit and date/time | |
| 10:51:05 | kashyap | lyarwood: Seems like 'pep8' failures on that Ocata backport; will look post meetings | |
| 10:51:16 | kashyap | odyssey4me: Ah, I meant +W | |
| 10:51:21 | lyarwood | kashyap: yeah sorting it out now | |
| 10:51:27 | odyssey4me | ah no, workflow is lost | |
| 10:57:22 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/561168 | |
| 10:57:43 | lyarwood | kashyap: ^ just a missing _LW | |
| 10:58:13 | kashyap | lyarwood: Ah-ha; /me quickly takes a peek | |
| 11:08:44 | openstackgerrit | sahid proposed openstack/nova stable/queens: only increment disk address unit for scsi devices https://review.openstack.org/561196 | |
| 11:47:07 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Record the host info in EventReporter https://review.openstack.org/556746 | |
| 11:47:08 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Extract generate_hostid method into utils.py https://review.openstack.org/555282 | |
| 11:47:09 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145 | |
| 12:03:24 | sahid | dansmith: in case you have a moment to look at this today https://review.openstack.org/#/c/484997/ | |
| 12:14:06 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Address nits in Idf57fb5fbc611abb83943bd7e36d3cebf03b3977 https://review.openstack.org/561205 | |
| 12:15:29 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Address nits in Idf57fb5fbc611abb83943bd7e36d3cebf03b3977 https://review.openstack.org/561205 | |
| 12:23:34 | bhujay | Hello everybody , would be grateful if someone can give me a hint why my openstack environment timestamp is not showing localtime that is set in operating system /etc/timezone | |
| 12:24:56 | bhujay | nova vm create , key create or any such creation date is showing utc time and i cant figure out how to change this to localtime | |
| 12:55:40 | fried_rice | ö/ | |
| 12:59:16 | fried_rice | stephenfin: is https://review.openstack.org/561190 intended to fix test_non_ami? | |
| 12:59:24 | fried_rice | (Which is French for "No, friend") | |
| 12:59:43 | stephenfin | fried_rice: That's the intention, yup | |
| 13:04:31 | fried_rice | finucannot: +A (which is French for "Oui, ami") | |
| 13:22:35 | Spaz-Home | Happy friday folks.. time for me to call it an ight | |
| 13:34:03 | openstackgerrit | Merged openstack/nova stable/queens: libvirt: Block swapping to an encrypted volume when using QEMU to decrypt https://review.openstack.org/559987 | |
| 13:42:14 | mriedem | finucannot: thanks for https://review.openstack.org/#/c/561190/ | |
| 13:44:44 | finucannot | mriedem: np | |
| 13:52:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: cleanup API guide about instance faults https://review.openstack.org/560178 | |
| 13:54:23 | mriedem | that would have been merged by now except with the zuulv3 reswizzle on irrelevant_files we're running tempest jobs on unit test only changes still | |
| 13:55:41 | mriedem | finucannot: can't we define our own nova-tox-py27 and then define our own irrelevant-files list? | |
| 13:55:43 | mriedem | gmann: ^? | |
| 13:55:50 | mriedem | i remember gibi asking about this in the ML | |
| 13:56:07 | finucannot | mriedem: IIRC, we can't override the global state | |
| 13:56:12 | finucannot | for that one anyway | |
| 13:57:29 | mriedem | we can't just define our own job that doesn't extend openstack-tox-py27? | |
| 13:57:40 | mriedem | like, copy openstack-tox-py27 and tweak but not extend | |
| 13:57:51 | finucannot | Ohh, I guess we could do that, yeah | |
| 13:58:58 | finucannot | I think openstack-tox-py27 is run by default though. I'm not sure how/if we can disable that | |
| 13:59:11 | mriedem | it's run because of the "openstack-python-jobs" usage in project-config | |
| 13:59:17 | finucannot | i.e. it's defined in openstack/project-config or the likes | |
| 13:59:18 | finucannot | yup | |
| 13:59:47 | mriedem | which runs: | |
| 13:59:47 | finucannot | *openstack-infra/project-config | |
| 13:59:48 | mriedem | - openstack-tox-pep8 | |
| 13:59:48 | mriedem | - openstack-tox-py27 | |
| 14:00:17 | mriedem | yeah so if we defined our own job, we'd have to list openstack-tox-pep8 in our own .zuul.yaml along with nova-tox-py27, but that seems ok, | |
| 14:00:30 | mriedem | and we'd have to backport it to all stable branches before removing the template usage from project-config | |
| 14:00:54 | mriedem | trivial but time consuming | |
| 14:01:10 | finucannot | Hmm. Can zuulv3 not support this use case? | |
| 14:01:32 | mriedem | i don't know why you can't extend an existing job and override attributes of it, like irrelevant-files | |
| 14:01:36 | mriedem | if we could do that, we'd be set | |
| 14:02:14 | finucannot | That sounds like a more effective use of time, if we can do it. I wonder if mordred is around? | |
| 14:02:25 | mriedem | i'm asking again in -infra, | |
| 14:02:32 | mriedem | but last time this came up there were lots of words in the ML | |
| 14:02:45 | mriedem | my head exploded, and i moved on | |
| 14:03:58 | mriedem | just sucks that we have to wait 2+ hours for a 15 min job | |
| 14:04:22 | mriedem | heh, maybe i should be asking, why is this taking nearly 2 hours | |
| 14:04:23 | mriedem | tempest-full-py3SUCCESS in 1h 50m 17s | |
| 14:04:49 | fried_rice | mriedem: How long does it take when you run it locally? Did you say you can run the whole thing in 15m? | |
| 14:05:07 | mriedem | fried_rice: i just mean the unit test job takes 15 minutes | |
| 14:05:17 | mriedem | and that's all we care about in this unit test only patch | |
| 14:05:25 | fried_rice | ah, I see. | |
| 14:05:46 | fried_rice | Well, I know I've seen special cases where what you've changed influences which jobs are run. | |
| 14:05:59 | fried_rice | Can't remember specifically for sure, but I think e.g. if you only change docs, only docs jobs are run. | |
| 14:06:05 | fried_rice | So there's a precedent for it. | |
| 14:06:32 | mriedem | yes that's how things used to all work when the jobs were centralized in project-config, pre-zuulv3 | |
| 14:06:46 | mriedem | looking at http://logs.openstack.org/90/561190/1/check/tempest-full-py3/41adeca/controller/logs/stackviz/#/stdin/timeline some of the tempest tests are taking over 7 minutes | |
| 14:07:18 | mriedem | maybe that's all related to meltdown/spectre fallout... | |
| 14:07:19 | mriedem | slow nodes | |
| 14:07:37 | mriedem | alright, i'll take my cup o coffee and go back into my old man hole now | |
| 14:25:49 | mordred | finucannot: uhoh | |
| 14:25:53 | mordred | finucannot: what did I do this time? | |
| 14:26:04 | mriedem | alex_xu: bauzas: per https://review.openstack.org/#/c/554159/ - this is doing the same thing that 2.47 does with showing embedded flavors with extra specs in GET /servers/{server_id} | |
| 14:26:48 | fried_rice | mordred: Da boyz were noodling whether/how we could make certain jobs conditional. | |
| 14:27:01 | finucannot | mordred: It's mriedem you've to worry about. He's mad about not being able to override 'irrelevant-files' on our tox-py27 job | |
| 14:27:22 | fried_rice | mordred: E.g. https://review.openstack.org/#/c/561190/ changed only a UT file, so it would be nice if we didn't bother running all the tempest suites, and we could merge the sucker in 15 minutes instead of 2h. | |
| 14:27:38 | mordred | totallly understand | |
| 14:27:57 | bauzas | mriedem: so it's using a same policy rule for that ? | |
| 14:28:19 | mriedem | bauzas: see my reply to alex_xu in https://review.openstack.org/#/c/554159/ just now | |
| 14:28:27 | bauzas | lemme check the code but I need to disappear in like 2 mins | |
| 14:29:06 | bauzas | mriedem: okay, lemme change my vote, but it'll require a docfix | |
| 14:29:29 | bauzas | that's the other option I had in mind, so it was a soft -1 | |
| 14:29:37 | mordred | I would actually think that just adding a nova-specific openstack-tox-py35 variant with some additional irrelevant-files added would work | |
| 14:29:51 | mordred | I'm assuming y'all have tried that already? | |
| 14:30:41 | bauzas | mriedem: +W'd with a comment asking for a followup | |
| 14:32:58 | mriedem | bauzas: thanks | |
| 14:33:03 | mriedem | i replied to more of the inline stuff too | |
| 14:38:51 | mriedem | mordred: already talked about that above, and the implications of doing so | |