Friday, 2016-10-28

*** VW has quit IRC00:04
*** VW has joined #craton00:11
*** VW has quit IRC00:31
*** VW has joined #craton01:31
*** VW has quit IRC01:36
*** VW has joined #craton02:33
*** VW has quit IRC02:37
*** VW has joined #craton03:34
*** VW has quit IRC03:39
*** Mudpuppy has quit IRC04:05
*** Mudpuppy has joined #craton04:05
*** Mudpuppy has quit IRC04:10
*** syed_ has joined #craton04:36
*** Mudpuppy has joined #craton04:38
*** Mudpuppy has quit IRC04:43
*** VW has joined #craton05:36
*** VW has quit IRC05:41
*** VW has joined #craton06:38
*** VW has quit IRC06:42
*** syed_ has quit IRC06:55
*** harlowja has joined #craton07:17
*** VW has joined #craton07:38
*** VW has quit IRC07:46
*** VW has joined #craton08:11
*** VW has quit IRC08:15
*** VW has joined #craton08:30
*** VW has quit IRC08:33
*** VW has joined #craton08:47
*** VW has quit IRC08:52
*** VW has joined #craton08:56
*** VW has quit IRC08:59
*** VW has joined #craton08:59
*** larsbutler has joined #craton09:16
larsbutlerjimbaker: https://review.openstack.org/#/c/391384/09:16
jimbakerlarsbutler, thanks09:17
jimbakerhttps://github.com/openstack/craton/blob/master/craton/api/v1/schemas.py#L31709:36
jimbakerlarsbutler, for https://bugs.launchpad.net/craton/+bug/1611425, we need to augment the schema09:38
openstackLaunchpad bug 1611425 in craton "Allow query by host variables" [High,In progress]09:38
*** jimbaker changes topic to "Summit talk: https://www.youtube.com/watch?v=Q-sf12SDR3M || Logs: http://eavesdrop.openstack.org/irclogs/%23craton/latest.log.html || Mon 1500 UTC in #openstack-meeting-4 || Client/ecosystem Tues 1600 UTC || Core Thur 1700 UTC || Resources: https://etherpad.openstack.org/p/Fleet_Management"09:45
*** VW has quit IRC09:47
*** VW has joined #craton09:47
*** larsbutler has quit IRC09:51
*** larsbutler has joined #craton09:51
*** antongribok has joined #craton09:52
*** harlowja has quit IRC09:55
*** harlowja has joined #craton09:55
jimbakerlarsbutler, https://gist.github.com/jimbaker/ffa82e8c8c20e2371979f15a4e4a9f28 for a simple script to let you interact with the craton model10:20
jimbakerwe may want to add something like this as a command10:20
*** VW has quit IRC10:29
*** antongribok has quit IRC10:33
*** Mudpuppy has joined #craton10:40
*** larsbutler has quit IRC10:45
*** Mudpuppy has quit IRC10:45
*** harlowja has quit IRC11:04
*** larsbutler has joined #craton11:31
*** harlowja has joined #craton12:08
*** antongribok has joined #craton12:10
*** VW has joined #craton12:12
*** harlowja has quit IRC12:33
*** VW has quit IRC12:40
*** Mudpuppy has joined #craton12:44
*** Mudpuppy has quit IRC13:04
*** valw has joined #craton13:25
*** harlowja has joined #craton13:26
*** larsbutler has quit IRC13:26
*** valw_ has joined #craton13:34
*** valw has quit IRC13:35
*** antongribok has quit IRC13:58
*** Tamayo has joined #craton14:12
*** Mudpuppy has joined #craton14:21
*** harlowja has quit IRC14:22
*** harlowja has joined #craton14:33
*** harlowja has quit IRC14:35
*** harlowja has joined #craton14:38
*** harlowja has quit IRC14:42
*** jovon has joined #craton15:00
*** harlowja has joined #craton15:01
*** syed_ has joined #craton15:02
*** harlowja has quit IRC15:07
*** larsbutler has joined #craton15:10
larsbutlerAnyone know why https://review.openstack.org/#/c/391384/ is getting a -1 from Jenkins? All of the listed Jenkins tests are green, so it’s not clear why Jenkins isn’t happy.15:29
sulolarsbutler: its failing requirements http://logs.openstack.org/84/391384/1/check/gate-craton-requirements/ddd2841/console.html15:31
suloaccording to global req, coverage should be coverage>=4.0  # Apache-2.015:33
larsbutlerokay15:36
larsbutleralright, it should be good now. let’s see.15:39
*** VW has joined #craton15:44
*** valw_ is now known as valw15:45
sigmaviruschrisspencer: why does do_cells_update require --name?15:47
chrisspencersigmavirus: ah crap, that might have been a merge issue15:48
chrisspencerit shouldn't15:49
chrisspencernice find15:50
*** larsbutler has quit IRC15:51
sigmaviruschrisspencer: I'll fix it up15:59
sigmavirusno worries15:59
sigmavirusI'm working on the cells_shell module and adding unit tests for it15:59
sigmavirusThanks for confirming my suspicion (that it shouldn't have been required)16:00
jovonalso in DefinitionRegion project_id should be required16:07
sigmavirusjovon: why?16:07
sigmavirusjovon: we've done work to eliminate the requirement that the user specify project_id16:07
sigmavirusCraton now pulls that from the context to avoid users spoofing project_ids and creating cells in projects they're not authorized to create them in16:08
sigmavirus(Or list them there for that matter)16:08
jovonso there is no X-AUTH-Project?16:10
sigmavirusjovon: opposite way around16:11
sigmavirusX-Auth-Project (when using local craton auth) is required, but project_id in the bodies of requests is not16:11
sigmavirusin fact, we don't even consider it in the body any longer, you should get a 422 from our schema validation layer16:11
chrisspencersigmavirus: jovon that means we should be able to remove those from the existing entities on create, delete, list, etc.  I think the only one we probably need to leave is update in case we want to allow users to move regions, cells,  etc to different projects.   If we don't want to allow that we can remove there as well16:13
sigmaviruschrisspencer: I thought I already did that16:14
jovoni see what you're saying sigmavirus16:17
sigmavirusjovon: if you're using keystone for auth, x-auth-project isn't sent by the user, just x-auth-token and in that case, the keystonemiddleware adds the auth_ref to the wsgi environ so we can get the project-id out of there16:18
chrisspencersigmavirus: did you already do that in the client?16:20
jovoneliminating our need to validate. right?16:20
sigmavirusjovon: eliminating the risk that user@project1 and create something in project2 without being a user in that project16:21
sigmaviruschrisspencer: I thought I had, but I could be wrong16:21
*** Mudpuppy_ has joined #craton16:24
*** Mudpuppy has quit IRC16:28
*** Mudpuppy_ has quit IRC16:29
*** valw has quit IRC16:33
*** valw has joined #craton16:34
*** jovon has quit IRC16:39
*** sballe_ has quit IRC16:40
*** serverascode has quit IRC16:40
*** jovon has joined #craton16:41
*** sballe_ has joined #craton16:42
*** serverascode has joined #craton16:43
*** VW has quit IRC16:53
*** VW has joined #craton17:20
*** valw has quit IRC17:31
*** valw has joined #craton17:33
*** Mudpuppy has joined #craton17:43
syed_sigmavirus: do you have some time ?17:44
sigmavirussyed_: I'm around, do you want to video call or something?17:44
syed_that would be good as well, i can share my screen so we can try looking at the keystone auth issue i am having17:45
syed_if its possible for you17:45
syed_sigmavirus: or if later anytime works best for you17:46
*** Mudpuppy_ has joined #craton17:48
sigmavirusI just need 5/10 min, if that's okay with you17:49
syed_sounds good to me17:49
syed_thanks17:49
*** Mudpuppy has quit IRC17:51
*** Mudpuppy_ is now known as Mudpuppy17:52
sigmavirussyed_: I'm ready when you are17:58
sigmavirussyed_: hop into the craton room?17:59
syed_sigmavirus: sounds good17:59
*** VW has quit IRC19:19
*** lcastell has quit IRC20:20
*** Tamayo has quit IRC20:38
*** Tamayo has joined #craton20:38
*** Tamayo has quit IRC20:41
*** valw has quit IRC20:49
*** jovon has quit IRC21:27
*** Mudpuppy has quit IRC22:48
*** valw has joined #craton23:09
*** valw has quit IRC23:31

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!