| *** tetsuro has joined #openstack-placement | 01:45 | |
| *** tetsuro has quit IRC | 01:57 | |
| *** tetsuro has joined #openstack-placement | 02:00 | |
| *** Nel1x has joined #openstack-placement | 02:45 | |
| *** fried_rice has quit IRC | 02:53 | |
| *** fried_rice has joined #openstack-placement | 02:54 | |
| *** Nel1x has quit IRC | 03:09 | |
| *** Nel1x has joined #openstack-placement | 03:09 | |
| *** tetsuro has quit IRC | 03:18 | |
| *** Nel1x has quit IRC | 04:47 | |
| *** tetsuro has joined #openstack-placement | 06:30 | |
| *** rubasov has joined #openstack-placement | 06:37 | |
| *** tetsuro has quit IRC | 07:08 | |
| *** rubasov has quit IRC | 07:13 | |
| *** rubasov has joined #openstack-placement | 07:13 | |
| *** tetsuro has joined #openstack-placement | 08:09 | |
| *** tetsuro has quit IRC | 08:13 | |
| *** tetsuro has joined #openstack-placement | 08:20 | |
| *** takashin has left #openstack-placement | 08:22 | |
| *** tetsuro has quit IRC | 08:27 | |
| *** e0ne has joined #openstack-placement | 08:45 | |
| *** cdent has joined #openstack-placement | 09:57 | |
| *** rubasov has quit IRC | 10:37 | |
| * cdent waves | 10:50 | |
| cdent | gibi: if you get a chance to look at the database stuff at https://review.openstack.org/#/c/611441/ (and above) that would be great. I'm inclined to move it forward asap, as fix things as they break rather than belabor it. It's going to be a learning process for us all. | 11:06 |
|---|---|---|
| *** e0ne has quit IRC | 11:25 | |
| *** e0ne has joined #openstack-placement | 11:34 | |
| *** rubasov has joined #openstack-placement | 11:42 | |
| *** edleafe has joined #openstack-placement | 12:29 | |
| openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement generic HW offloads for os-vif https://review.openstack.org/607610 | 12:57 |
| *** e0ne has quit IRC | 12:58 | |
| *** cdent has quit IRC | 13:03 | |
| *** e0ne has joined #openstack-placement | 13:19 | |
| *** mriedem has joined #openstack-placement | 13:19 | |
| *** e0ne has quit IRC | 13:24 | |
| *** takashin has joined #openstack-placement | 13:25 | |
| *** e0ne has joined #openstack-placement | 13:30 | |
| *** takashin has quit IRC | 13:57 | |
| fried_rice | n-sch meeting now in #openstack-meeting-alt | 13:59 |
| *** fried_rice is now known as efried | 13:59 | |
| *** takashin has joined #openstack-placement | 14:00 | |
| *** SteelyDan is now known as dansmith | 14:16 | |
| *** jaypipes_ is now known as jaypipes | 14:17 | |
| *** cdent has joined #openstack-placement | 14:20 | |
| efried | Okay, so this red hat thing. | 14:58 |
| efried | Can anyone explain to me why, for this news as any other, good bad or indifferent, IBM's stock plummets? | 14:58 |
| openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | 15:00 |
| cdent | efried: are you expecting rational behavior from the stock market ? | 15:01 |
| edleafe | Looks like IBM is down 2 point. Plummet? | 15:02 |
| efried | okay, forgive the hyperbole, being IBM the volatility is constrained to more conservative swings, but seemingly down, always down. | 15:03 |
| *** takashin has left #openstack-placement | 15:04 | |
| efried | oh, and I looked right when the market opened, before it had bounced back | 15:04 |
| edleafe | efried: knee jerk reactions are to be expected | 15:04 |
| efried | so more like $5 | 15:04 |
| efried | yeah, but the knee jerk reaction to red hat stock was violently the opposite. | 15:04 |
| edleafe | efried: because of the offering price from IBM | 15:05 |
| edleafe | if a company's stock is at $X and another company offers to buy it at $X+50, the stock will very quickly jump to $X+50 | 15:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/placement master: Publish placement documents https://review.openstack.org/613953 | 15:07 |
| *** irclogbot_2 has joined #openstack-placement | 15:08 | |
| cdent | efried: do you consider https://bugs.launchpad.net/nova/+bug/1781008 still an issue? If so, can we make it something other than "new"? | 15:10 |
| openstack | Launchpad bug 1781008 in OpenStack Compute (nova) "Possible race updating consumer's proj/user" [Undecided,New] - Assigned to Jay Pipes (jaypipes) | 15:10 |
| efried | ... | 15:11 |
| efried | cdent: What status would you propose? I don't think "New" means it just arrived in an Amazon box on your doorstep; I think it just means we haven't triaged, confirmed, invalidated, fixed, etc. | 15:14 |
| efried | which we haven't | 15:14 |
| cdent | well either confirmed or triaged would be more appropriate, unless it is impossible to confirm. If it is impossible to confirm I would think it must be invalid | 15:15 |
| cdent | bugs that sit there for months doing nothing aren't particularly useful | 15:15 |
| *** cdent has quit IRC | 15:22 | |
| efried | I'm gonna say this one is doing exactly what it was intended to do, which is have someone notice it so it's not forgotten about. TODOs in code seem to have a lower noticeability. | 15:24 |
| mriedem | recruiter email sent | 15:28 |
| efried | Thanks mriedem | 15:35 |
| *** cdent has joined #openstack-placement | 15:48 | |
| * cdent turns on the big computer | 16:15 | |
| *** finucannot is now known as stephenfin | 16:17 | |
| cdent | efried, stephenfin you guys happy to make another pass at reviewing the tox cleanup: https://review.openstack.org/#/c/611719/ ? | 16:31 |
| *** e0ne has quit IRC | 16:36 | |
| efried | cdent: Yeah, another one that's been on my list. I was going to do all the local things again, which was going to take some time, so it wasn't a quick approve for me. | 16:38 |
| cdent | cool, thanks | 16:39 |
| openstackgerrit | Chris Friesen proposed openstack/nova-specs master: Add support for emulated virtual TPM https://review.openstack.org/571111 | 16:40 |
| *** N3l1x has joined #openstack-placement | 16:47 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 17:26 |
| cdent | edleafe: ^ replaces the table create WIP with something a bit more official | 17:27 |
| edleafe | cdent: cool. I'm still digging into the tests. Seems like not all are being run. | 17:28 |
| cdent | naming weirdness potentially? | 17:33 |
| cdent | edleafe: or perhaps because you don't have the "opportunistic setup"? | 17:34 |
| edleafe | Not that I can see. It's a mishmash of a ton of mixins that's confusing | 17:34 |
| cdent | edleafe: when I run 'tox -epy36 test_migrations' it discovers 16 tests but skips 12 | 17:40 |
| edleafe | I'm seeing the same thing running the ironic tests. | 17:41 |
| edleafe | jroll: around? | 17:43 |
| edleafe | cdent: it's skipping all the migration checks: | 17:45 |
| edleafe | "ironic.tests.unit.db.sqlalchemy.test_migrations.TestMigrationsMySQL.test_upgrade_and_create_schema ... SKIPPED: Backend 'mysql+pymysql' is unavailable: Could not connect | 17:45 |
| mriedem | jaypipes: if chef has you down, you can always try your luck as the browns new OC or HC | 17:45 |
| cdent | edleafe: right those are the "opportunistic checks" | 17:45 |
| cdent | that's expected unless you set up the right databases, as described in the docsgtring of the test_migrations.py file | 17:46 |
| edleafe | yeah, it doesn't feel very unit test-y | 17:47 |
| cdent | it's not | 17:48 |
| jaypipes | mriedem: how'd that Saints game go? | 17:49 |
| jaypipes | :P | 17:50 |
| mriedem | jaypipes: just like i expected | 18:02 |
| mriedem | except i had the over on cousins turning it over 2.5 times | 18:02 |
| openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | 18:19 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Added alembic environment https://review.openstack.org/611441 | 20:44 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Clean up the old migrations and models https://review.openstack.org/611440 | 20:44 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Add a document for creating DB revisions https://review.openstack.org/614024 | 20:44 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: WIP - Show an alembic migration https://review.openstack.org/614025 | 20:44 |
| edleafe | cdent: Added a doc and a DNM revision | 20:44 |
| cdent | yay! | 20:44 |
| openstackgerrit | Chris Dent proposed openstack/placement master: DNM: Demonstrate online migrations https://review.openstack.org/614030 | 21:17 |
| cdent | edleafe: that ^ shows some additional info we might want to consider with regard to the "true powers of alembic" or something | 21:17 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 21:17 |
| efried | cdent: How will we know if https://review.openstack.org/#/c/613953/ is working? IIRC, this is one of those situations where we don't know until it merges? | 21:23 |
| cdent | efried: I don't really know. I assume so, yes. | 21:25 |
| cdent | probably worth a closer look at what that template does | 21:25 |
| * cdent looks | 21:25 | |
| efried | cdent: fwiw, the nova .zuul.yaml uses the 'publish' rather than the 'build'. | 21:27 |
| cdent | efried: looking at, I think we can assume it works because we already have openstack-tox-docs up and running | 21:27 |
| efried | cdent: So the next question is, do we want to start publishing now, or wait until the content is prettier? | 21:27 |
| cdent | efried: you probably want an additional opinion, perhaps mriedem, because I tend to release early and often | 21:28 |
| cdent | as a general policy | 21:28 |
| efried | I mean, is it going to be linked from anywhere? | 21:28 |
| cdent | but there may be reasons of some kind why it shouldn't happen | 21:28 |
| cdent | no that requires a second step, lemme link you | 21:28 |
| efried | cdent: Waitwait, do placement docs currently publish under /nova/? | 21:29 |
| cdent | yes, | 21:29 |
| *** e0ne has joined #openstack-placement | 21:29 | |
| cdent | but from openstack/nova | 21:29 |
| efried | right, okay, they're just pages, there's not like a central index. | 21:30 |
| efried | I mean https://docs.openstack.org/nova/latest/user/placement.html is the central index. | 21:30 |
| efried | so point is, we wouldn't be conflicting with ^ if we started publishing to https://docs.openstack.org/placement/latest/* | 21:30 |
| cdent | efried: see http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/project-templates.yaml#n80 about needing to add a link | 21:31 |
| cdent | well yeah, that was my initial concern as well: maybe we should depend-on that stuff being removed from nova | 21:31 |
| cdent | but then I thought if we don't do the linking as describe in the project template, we'll be publishing, but not making visible to indexers | 21:32 |
| cdent | in which case we'll need to remember to do the exposing | 21:32 |
| efried | that seems like a reasonable sequence of events to me. | 21:32 |
| cdent | I'm not confident that the "conflict" is a big issue, but happy to go with whatever flow we like | 21:32 |
| efried | 1) Publish, so we can see what things look like on an ongoing basis as we merge patches to 2) Make pretty, then when we're happy, 3) flip the governance switch to link and 4) merge a patch to remove from nova. | 21:33 |
| efried | I don't think I care for 3 and 4 to be interdependent or in that order. | 21:33 |
| efried | but we can burn that bridge when we cross it. | 21:33 |
| efried | do you agree with 1 and 2? | 21:34 |
| cdent | 3 isn't about governance, placement is already an official deliverable, it's about making a link in the page that is the index for all the docs | 21:34 |
| cdent | 1 isn't really necessary since http://logs.openstack.org/53/613953/1/check/openstack-tox-docs/9f26ec2/html/ is the same stuff | 21:34 |
| efried | sorry by governance I just meant that that patch is in the governance repository, right? | 21:34 |
| cdent | it's merely convenient | 21:34 |
| cdent | which patch do you mean? the patch I'm talking about is described by https://docs.openstack.org/doc-contrib-guide/doc-index.html | 21:35 |
| efried | openstack-manuals repo? | 21:36 |
| *** e0ne has quit IRC | 21:36 | |
| efried | Okay, I wasn't reading very closely, I thought I saw something that talked about a governance repo, but I guess that's just the thing that calls placement an official project, which as you say is already done. | 21:37 |
| efried | As for 1, I agree we can use the in-flight product in the patches, but those are ephemeral. If we were, say, sending out a call for volunteers to help port/prettify documentation (cause that's how things get done around here), it would be more convenient to have a permanent URL to reference. | 21:38 |
| cdent | yes, agree | 21:38 |
| efried | so | 21:38 |
| efried | shall we merge this sucker? | 21:38 |
| cdent | we've given 11 minutes for mriedem to reject our plans, so yeah, I say we go for it | 21:39 |
| efried | We can always roll it back I guess, if something goes seriously pear-shaped. | 21:40 |
| cdent | aye | 21:40 |
| cdent | i've given you the pleasure | 21:41 |
| efried | writing up a summary, then will pull trigger. | 21:41 |
| cdent | I linked to this discussion, as figured it was short enough, but summary is nice | 21:41 |
| mriedem | we're all gonna die | 21:42 |
| cdent | yay! | 21:42 |
| cdent | i've been waiting so long | 21:42 |
| mriedem | need to go mow before (1) the sun goes down and (2) it snows | 21:42 |
| *** mriedem is now known as mriedem_away | 21:42 | |
| cdent | my nephew has moved to be neighbors with mriedem_away | 21:43 |
| cdent | gonna set up a fresh thyme market or something or other | 21:43 |
| mriedem_away | fresh thyme, for 2 months out of the year | 21:44 |
| mriedem_away | sounds lucrative | 21:44 |
| efried | it's 82 degrees here in central texas. | 21:48 |
| efried | you can keep your snow. | 21:48 |
| efried | cdent: 11 minutes, that's agile dude. | 21:48 |
| cdent | I feel pretty bad about this integration testing stuff I want to add. It makes the tests about 30 minutes long instead of 6 or less | 21:50 |
| efried | Good thing they're asynchronous. | 21:51 |
| efried | That 6 minute shit was way too good to last, and you know it. | 21:51 |
| cdent | meh, I'm not convinced the integration tests add that much additional confidence | 21:52 |
| cdent | only because placement is such an HTTP API and not much else | 21:52 |
| cdent | if it had more pieces working with more other pieces, then sure | 21:52 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Initial integration test using gabbits https://review.openstack.org/601614 | 22:03 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use gabbi-tempest job from gabbi-tempest plugin https://review.openstack.org/607508 | 22:03 |
| openstackgerrit | Merged openstack/placement master: Publish placement documents https://review.openstack.org/613953 | 22:04 |
| cdent | i've been at this too long today, goodnight | 22:13 |
| *** cdent has quit IRC | 22:13 | |
| *** takashin has joined #openstack-placement | 23:37 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!