Friday, 2017-06-23

*** gongysh has joined #openstack-keystone00:05
*** gongysh has quit IRC00:06
* samueldmq 's back 00:12
*** openstackstatus has quit IRC00:13
*** openstackstatus has joined #openstack-keystone00:14
*** ChanServ sets mode: +v openstackstatus00:14
samueldmqlbragstad: you around ?00:21
*** jmlowe has joined #openstack-keystone00:30
samueldmqlbragstad: talked in -infra, I am finding my +2s sticky sometimes in gerrit00:31
samueldmqand other times it's really odd, like in
samueldmqI had +2ed, -1ed and finished with a +2 even if I didn't give a +2, that's odd00:31
samueldmqI was wondering if you (or any other cores) were seeing the same00:32
*** zhugaoxiao has quit IRC00:39
*** Administrator__ has quit IRC00:39
*** spzala has joined #openstack-keystone00:41
samueldmqah gerrit automatically adds your vote back if the patch comes to the same state of a previous patchset you had a vote on00:42
samueldmqthat's interesting00:42
*** liujiong has joined #openstack-keystone00:59
*** akrzos has quit IRC00:59
*** akrzos has joined #openstack-keystone01:06
samueldmqlbragstad: all patches in bug/1696574 will need updates in those policy in code descriptions, used to generate the sample file :)01:17
samueldmqlbragstad: I guess they can come as follow-up01:17
*** xuhaigang has joined #openstack-keystone01:22
samueldmqlbragstad:  we need stable cores to look at and
samueldmqis morgan a stable core? ^01:25
samueldmqor stevemar ?01:26
openstackgerritMerged openstack/keystone master: Use DocumentedRuleDefault for token operations
*** Shunli has joined #openstack-keystone01:31
*** hoonetorg has quit IRC01:35
openstackgerritMerged openstack/keystone master: Move related project information into main doc
openstackgerritSamuel de Medeiros Queiroz proposed openstack/keystone master: Add annotation about token authenticate
*** hoonetorg has joined #openstack-keystone01:52
*** dave-mccowan has joined #openstack-keystone02:03
*** shuyingya has joined #openstack-keystone02:06
*** shuyingya has quit IRC02:06
*** shuyingya has joined #openstack-keystone02:06
*** shuyingya has quit IRC02:07
*** shuyingya has joined #openstack-keystone02:07
*** zhurong has joined #openstack-keystone02:09
*** tobberydberg has joined #openstack-keystone02:11
*** shuyingya has quit IRC02:13
*** shuyingya has joined #openstack-keystone02:13
*** tobberydberg has quit IRC02:16
*** phalmos has quit IRC02:23
*** spzala has quit IRC02:41
*** spzala has joined #openstack-keystone02:42
*** spzala has quit IRC02:46
openstackgerritMerged openstack/keystone master: Split test_get_head_catalog_no_token
openstackgerritMerged openstack/keystoneauth master: Updated from global requirements
openstackgerritMerged openstack/keystonemiddleware master: Fix html_last_updated_fmt for Python3
openstackgerritMerged openstack/keystonemiddleware master: Removing double spaces
*** zhurong has quit IRC02:58
*** markvoelker has quit IRC02:59
*** zhurong has joined #openstack-keystone02:59
openstackgerritMerged openstack/keystonemiddleware master: Updated from global requirements
*** markvoelker has joined #openstack-keystone03:00
*** sbezverk has joined #openstack-keystone03:01
*** thorst has joined #openstack-keystone03:03
*** thorst has quit IRC03:04
*** markvoelker has quit IRC03:05
*** nicolasbock has joined #openstack-keystone03:16
*** edmondsw has quit IRC03:17
*** namnh has joined #openstack-keystone03:21
*** zhurong has quit IRC04:02
*** dave-mccowan has quit IRC04:05
*** markvoelker has joined #openstack-keystone04:14
*** links has joined #openstack-keystone04:16
*** tobberydberg has joined #openstack-keystone04:27
*** tobberydberg has quit IRC04:31
*** zhurong has joined #openstack-keystone04:35
*** markvoelker has quit IRC04:48
*** edmondsw has joined #openstack-keystone04:49
*** edmondsw has quit IRC04:54
*** spzala has joined #openstack-keystone04:55
*** spzala has quit IRC04:59
openstackgerritMerged openstack/keystone master: Ensure the endpoint filter API supports HEAD
openstackgerritMerged openstack/keystone master: Add HEAD API to domain config
openstackgerritMerged openstack/keystone master: Move domain config to DocumentedRuleDefault
openstackgerritMerged openstack/keystone master: Ensure the ec2 API supports HEAD
openstackgerritMerged openstack/keystone master: Ensure oauth API supports HEAD
openstackgerritMerged openstack/keystone master: Ensure the trust API supports HEAD requests
openstackgerritMerged openstack/keystone master: Ensure HEAD is supported with simple cert
openstackgerritMerged openstack/keystone master: Ensure the endpoint policy API supports HEAD
*** markvoelker has joined #openstack-keystone05:24
*** spzala has joined #openstack-keystone05:25
*** spzala has quit IRC05:30
*** markvoelker has quit IRC05:31
*** markvoelker has joined #openstack-keystone05:36
*** thorst has joined #openstack-keystone05:36
*** thorst has quit IRC05:40
*** markvoelker has quit IRC05:41
*** spzala has joined #openstack-keystone05:42
*** spzala has quit IRC05:46
*** markvoelker has joined #openstack-keystone05:52
morganYes I am a stable core05:55
-openstackstatus- NOTICE: unknown issue with the git farm, everything broken - we're investigating06:02
*** ChanServ changes topic to "unknown issue with the git farm, everything broken - we're investigating"06:02
*** spzala has joined #openstack-keystone06:05
*** gyee has quit IRC06:08
*** markvoelker has quit IRC06:08
*** spzala has quit IRC06:10
*** markvoelker has joined #openstack-keystone06:12
*** markvoelker has quit IRC06:27
*** rcernin_ has joined #openstack-keystone06:32
*** spzala has joined #openstack-keystone06:36
*** edmondsw has joined #openstack-keystone06:37
*** spzala has quit IRC06:40
*** edmondsw has quit IRC06:42
*** spzala has joined #openstack-keystone07:03
*** spzala has quit IRC07:08
*** pcaruana has joined #openstack-keystone07:18
*** markvoelker has joined #openstack-keystone07:21
*** markvoelker has quit IRC07:26
*** markvoelker has joined #openstack-keystone07:28
*** spzala has joined #openstack-keystone07:34
*** gongysh has joined #openstack-keystone07:35
*** markvoelker has quit IRC07:35
*** spzala has quit IRC07:39
*** tesseract has joined #openstack-keystone07:45
*** spzala has joined #openstack-keystone07:48
*** junbo has joined #openstack-keystone07:48
*** pooja_jadhav has quit IRC07:49
*** spzala has quit IRC07:52
*** f13o has joined #openstack-keystone07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-keystone08:00
*** admcleod_ has quit IRC08:00
-openstackstatus- NOTICE: ok is working again, you can recheck failed jobs08:02
*** ChanServ changes topic to "ok is working again, you can recheck failed jobs"08:02
*** markvoelker has joined #openstack-keystone08:09
*** cristicalin has joined #openstack-keystone08:14
*** markvoelker has quit IRC08:14
*** markvoelker has joined #openstack-keystone08:16
*** f13o has quit IRC08:16
*** markvoelker has quit IRC08:20
knikollao/ morning!08:23
openstackgerritVan Hung Pham proposed openstack/keystonemiddleware master: Using assertFalse(A) instead of assertEqual(False, A)
cmurphyknikolla: morning! welcome to the quiet side of the world08:23
knikollacmurphy: thanks!08:25
knikollacmurphy: how's germany?08:25
cmurphyknikolla: it is great, perfect biergarten weather08:25
cmurphyknikolla: where are you at?08:26
*** edmondsw has joined #openstack-keystone08:26
knikollacmurphy: visiting home in Albania.08:26
cmurphyknikolla: awesome, i've never been there08:27
knikollacmurphy: wouldn't really recommend it.  I was in Germany just a week prior to the summit though. :) Have friends in Karlsruhe and Darmstadt.08:27
cmurphyneat :)08:28
*** markvoelker has joined #openstack-keystone08:28
*** edmondsw has quit IRC08:30
*** f13o has joined #openstack-keystone08:32
*** shuyingya has quit IRC08:33
*** spzala has joined #openstack-keystone08:34
*** xuhaigang has quit IRC08:35
openstackgerritVan Hung Pham proposed openstack/keystoneauth master: Using assertFalse(A) instead of assertEqual(False, A)
*** markvoelker has quit IRC08:35
*** spzala has quit IRC08:38
*** f13o has quit IRC08:44
*** markvoelker has joined #openstack-keystone08:49
*** pnavarro has quit IRC08:52
*** henrynash has joined #openstack-keystone08:53
*** pnavarro has joined #openstack-keystone08:53
*** xuhaigang has joined #openstack-keystone08:53
*** markvoelker has quit IRC08:58
*** f13o has joined #openstack-keystone09:02
*** spzala has joined #openstack-keystone09:02
*** markvoelker has joined #openstack-keystone09:05
*** shuyingya has joined #openstack-keystone09:05
*** pnavarro has quit IRC09:05
*** pnavarro has joined #openstack-keystone09:06
*** spzala has quit IRC09:08
*** markvoelker has quit IRC09:09
*** ChanServ changes topic to "Pike release schedule: | Meeting agenda: | Bugs that need triaging:"09:10
*** aselius has quit IRC09:10
*** f13o has quit IRC09:14
*** henrynash has quit IRC09:15
*** markvoelker has joined #openstack-keystone09:21
*** markvoelker has quit IRC09:25
*** markvoelker has joined #openstack-keystone09:27
*** gongysh has quit IRC09:28
*** Shunli has quit IRC09:31
*** markvoelker has quit IRC09:33
*** spzala has joined #openstack-keystone09:33
*** zhurong has quit IRC09:37
*** spzala has quit IRC09:38
*** markvoelker has joined #openstack-keystone09:47
*** spzala has joined #openstack-keystone09:48
*** spzala has quit IRC09:52
*** cristicalin has quit IRC09:54
*** markvoelker has quit IRC09:55
*** chlong has quit IRC10:03
*** namnh has quit IRC10:03
*** liujiong has quit IRC10:04
*** henrynash has joined #openstack-keystone10:09
*** junbo has quit IRC10:10
*** junbo has joined #openstack-keystone10:13
*** edmondsw has joined #openstack-keystone10:14
*** edmondsw has quit IRC10:18
*** spzala has joined #openstack-keystone10:18
*** henrynash has quit IRC10:20
*** spzala has quit IRC10:23
*** markvoelker has joined #openstack-keystone10:25
*** ppiela has quit IRC10:27
*** ppiela has joined #openstack-keystone10:27
*** markvoelker has quit IRC10:29
*** markvoelker has joined #openstack-keystone10:31
*** spzala has joined #openstack-keystone10:34
*** spzala has quit IRC10:38
*** markvoelker has quit IRC10:40
*** markvoelker has joined #openstack-keystone10:42
*** sjain has joined #openstack-keystone10:44
*** markvoelker has quit IRC10:53
sjainasettle: are the OS specific changes in the openstack-manuals done, can I pull them to keystone directory?10:54
asettlesjain: lemme just have a peak :)10:54
asettleNot quite:
asettleThis one has gone thorugh though:
asettleI think you need this one too:
sjainokay cool I can work on these install guides for now10:56
sjainthanks :)10:56
sjainbtw asettle: I hope there are no more changes in install guides?10:57
sjain*upcoming changes10:57
asettlesjain: the master branch is frozen :)10:57
asettleUnless its migration related10:57
sjaincool, thanks!10:58
asettleYou're welcome :)10:58
*** raildo has joined #openstack-keystone10:58
*** spzala has joined #openstack-keystone11:03
*** spzala has quit IRC11:07
*** ducttape_ has joined #openstack-keystone11:09
*** shuyingya has quit IRC11:09
*** shuyingy_ has joined #openstack-keystone11:09
*** ducttape_ has quit IRC11:13
*** markvoelker has joined #openstack-keystone11:14
*** pnavarro has quit IRC11:15
*** sjain has quit IRC11:16
*** pnavarro has joined #openstack-keystone11:19
*** markvoelker has quit IRC11:21
*** markvoelker has joined #openstack-keystone11:24
*** sjain has joined #openstack-keystone11:26
*** markvoelker has quit IRC11:29
*** openstackgerrit has quit IRC11:33
*** spzala has joined #openstack-keystone11:34
*** spzala has quit IRC11:39
*** f13o has joined #openstack-keystone11:39
*** henrynash has joined #openstack-keystone11:40
*** sjain has quit IRC11:43
*** henrynash has quit IRC11:44
*** markvoelker has joined #openstack-keystone11:48
*** spzala has joined #openstack-keystone11:50
*** markvoelker has quit IRC11:53
*** markvoelker has joined #openstack-keystone11:54
*** spzala has quit IRC11:54
*** sjain_ has joined #openstack-keystone11:55
*** markvoelker has quit IRC11:58
*** shuyingy_ has quit IRC11:58
*** thorst has joined #openstack-keystone12:00
*** jistr is now known as jistr|errand12:00
*** markvoelker has joined #openstack-keystone12:11
*** pnavarro has quit IRC12:12
*** markvoelker has quit IRC12:15
*** markvoelker has joined #openstack-keystone12:18
*** dave-mccowan has joined #openstack-keystone12:19
*** edmondsw has joined #openstack-keystone12:21
*** markvoelker has quit IRC12:25
vladiskuzbreton: Do you know how to configure Ceph with Keystone using Fernet token?12:33
*** f13o has quit IRC12:36
*** f13o has joined #openstack-keystone12:44
bretonvladiskuz: nope12:45
bretonvladiskuz: maybe ceph folks know12:45
*** spzala has joined #openstack-keystone12:47
*** sjain_ has quit IRC12:50
*** spzala has quit IRC12:52
*** lucasxu has joined #openstack-keystone12:56
*** catintheroof has joined #openstack-keystone13:00
*** bknudson has joined #openstack-keystone13:01
*** slunkad_ has quit IRC13:02
*** Dinesh_Bhor has quit IRC13:10
*** spzala has joined #openstack-keystone13:18
hrybackilbragstad knikolla reminder that we make the call at 5PM EST tonight regarding: So far we have seven responses with the most agreed upon time block being Tuesday evenings. However it looks like cmurphy can't make that block so we may opt for another?13:20
*** spzala has quit IRC13:23
cmurphywell don't block it on account of me, i can work on bugs at other times13:24
cmurphyand my tuesday evening conflict is temporary13:24
*** jistr|errand is now known as jistr13:25
knikollahrybacki: ack.13:25
knikollaTuesday morning seems the other best day13:26
knikollaSince same day :P13:27
*** ducttape_ has joined #openstack-keystone13:32
hrybackiknikolla: and lines up nicely with our meeting :)13:33
*** spzala has joined #openstack-keystone13:37
*** jmlowe has quit IRC13:40
*** shuyingya has joined #openstack-keystone13:47
*** shuyingya has quit IRC13:51
*** chlong has joined #openstack-keystone13:56
*** bknudson1 has joined #openstack-keystone13:58
*** bknudson has quit IRC14:00
*** ducttape_ has quit IRC14:00
*** jmlowe has joined #openstack-keystone14:16
*** aojea has joined #openstack-keystone14:31
lbragstadknikolla: hrybacki that sounds good - i was looking at the responses yesterday14:35
*** openstackgerrit has joined #openstack-keystone14:36
openstackgerritLance Bragstad proposed openstack/keystone master: DO NOT MERGE: Testing rolling upgrades
hrybackiis there a way to see the self url for a project when using the openstack client?14:36
lbragstadhrybacki: i don't think so - or not that i know of14:36
hrybackiI know they are in the response body14:36
hrybackilbragstad: okay, thanks14:36
lbragstadnice work team!
hrybackilbragstad++ nice!14:42
lbragstadthis is also a good read -
knikollahrybacki: what do you mean by self url for a project?14:42
lbragstadandymccr: odyssey4me we still have a little bit of work to do there to make sure we test the stable/master case, right?14:42
andymccrlbragstad: i put a fix in for that already - lemme double check14:43
lbragstadi think it was if A is propose to stable and B is propose to master, the need to be tested at the same time14:43
hrybackiknikolla: specifically I'm trying to find out what OpenStack client is getting out of the resposne in links.projects[whatever].links.self14:44
hrybackimy bad:
andymccrlbragstad: hmm you mean you have a stable patch and a master patch - to test both? hmm not sure how we would achieve that14:45
knikollahrybacki: i usually do a `openstack token issue` and see whatever project that gives me.14:46
hrybackiknikolla: trying to figure out if OSC would return project links tied to the InternalURL rather than the PublicURL. In this case, W.X.Y.Z is the PublicURL14:46
odyssey4meandymccr lbragstad yeah, we'd have to work that out - although not sure what the urgency is at the moment14:46
lbragstadandymccr: yeah - it was a super interesting testing edge case that clarkb and fungi described14:46
knikollaYou can do that with --debug and see the response in the token. Maybe.14:46
odyssey4mewe can fence against that interfering for now by just ensuring that we block any stable branch zuul changes from being included14:46
andymccrhow do those repos get cloned?14:46
knikollahrybacki: oh i see.14:47
lbragstadodyssey4me: yeah - that's a good point, i was more of less just trying to figure out what the next steps were14:47
andymccrlike if i have a stable branch and a master branch - where do they go?14:47
andymccrif we know that it should be reasonably ok14:47
odyssey4meandymccr we'd need to set out two folders to scan, one with the old repositories (for the stable branch) and one for the new14:47
odyssey4methen ensure the clone map knows where to put which14:47
lbragstadsomething like /opt/stack/old/keystone and /opt/stack/new/keystone14:47
andymccrhmm yeah14:47
odyssey4me^ yeah, that's what devstack-gate does14:48
hrybackiknikolla: ack. I've modified my identity endpoint to use the machines loopback address as the InternalURL to help me figure it out14:48
andymccri fixed the case where stable/pike --> master will cause teh keystone package to be used by both since the package constraints file is name-spaced by version now14:48
lbragstadold being stable/ocata + any dependent patches and new being master + and patches in review14:48
odyssey4mewhich is why infra suggested we work out how to use devstack gate and switch to it instead of reinventong the wheel14:48
andymccrodyssey4me: yeah i thinkt hat is probably a better way to go14:48
lbragstadwell - the devstack gate tooling14:48
lbragstador job template?14:48
odyssey4melbragstad tooling14:48
odyssey4meit does a bunch of stuff we also do14:49
odyssey4meif we can converge, it'll be less burden for us all14:49
knikollahrybacki: i would assume the openstackclient ignores self links and instead rebuilds the link itself.14:49
odyssey4mebut unfortunately that also means it'll take time to figure out, because the paradigms are quite different14:49
lbragstadandymccr: regarding
knikollaWhen trying to access a resource.14:50
lbragstadandymccr:  another big advantage we saw with using osa over devstack was that we could do more with less gate resources14:50
*** sjain has joined #openstack-keystone14:50
andymccrlbragstad: ahh good point - i can add that in :)14:50
lbragstad(e.g. running the upgrade across containers instead of across vms, hopefully making it more friendly for infra)14:50
*** jistr is now known as jistr|mtg14:54
*** bknudson1 has quit IRC14:57
*** ducttape_ has joined #openstack-keystone15:00
*** rcernin_ has quit IRC15:01
*** bknudson has joined #openstack-keystone15:02
bretonwhat is today's way of having a superadmin user? Admin role in is_admin_project?15:09
lbragstadbreton: that works isn't finished15:10
lbragstadbreton: admin is still technically super admin15:10
bretonlbragstad: ok. I am planning to implement is_admin_project thingy in panko and was making sure.15:11
*** ducttape_ has quit IRC15:11
*** aselius has joined #openstack-keystone15:11
*** ducttape_ has joined #openstack-keystone15:11
lbragstadbreton: we do have this, too
lbragstadwhich was proposed partially because of
lbragstadbreton: these might give you more context
*** aojea has quit IRC15:15
lbragstadbreton: i plan on starting that implementation ^ next week15:16
lbragstador at least a PoC of it15:16
*** links has quit IRC15:16
bretonlbragstad: do you think i should wait with my implementation?15:18
lbragstadwell - you're going to be waiting on keystone for fixes regardless :/15:20
*** sjain has quit IRC15:20
bretonwhat fixes for example?15:21
lbragstadbreton: you're going to be waiting for is_admin_project to finish being implemented or global roles to be implemented15:21
bretonlbragstad: yeah, but what's left from is_admin_project?15:22
bretonlbragstad: i know that not all projects use it today15:22
lbragstadbreton: a few fixes in keystone, but quite a bit of work cross project15:22
lbragstadbreton: i listed the pros and cons of each in the ^ thread above15:23
bretonlbragstad: nah, i don't depend on cross-project work if i implement it in panko15:23
lbragstadsamueldmq: merged - so we can start merging patches that migrate things from openstack-manuals15:26
lbragstadsamueldmq: not sure if sjain has seen ^ that yet15:27
*** jistr|mtg is now known as jistr15:30
*** ducttape_ has quit IRC15:43
*** ducttape_ has joined #openstack-keystone15:47
*** ducttap__ has joined #openstack-keystone15:55
*** ducttape_ has quit IRC15:57
*** ducttape_ has joined #openstack-keystone15:57
*** ducttap__ has quit IRC16:00
*** sjain_ has joined #openstack-keystone16:07
sjain_@lbragstad, thanks for taking care of that related projects section :)16:09
sjain_I saw that other specs patch merged, I'm just waiting for other migration related patches to be merged16:10
sjain_some of them change the docs a bit, I'll address all the reviews while merging those new ones16:11
*** pcaruana has quit IRC16:18
*** links has joined #openstack-keystone16:18
*** lucasxu has quit IRC16:19
*** gyee has joined #openstack-keystone16:21
lbragstadsjain_: awesome - i'll be focusing on those reviews today16:26
lbragstadsjain_: i'd like to get as much of the migration done as possible, so we can spend next week removing duplicate documentation16:27
mordredmorgan, cmurphy, samueldmq: I'm adding the tests to the discovery patch series - and there's an interface that feels awkward using it in the tests16:27
mordredwe haven't cut a release with the first 3 patches of the series in it yet - so I'm inclined to change the interface16:28
morganGreat it is on my review list today16:28
morganLet's change it16:28
morganBut let's land that asap16:28
mordredcool - I'll have at least the first one done real soon16:28
sjain_@lbragstad: I checked with asettle, they are still working on improving certain docs, should I wait for them or start working on those patches16:28
morganI want to be super careful on landed interface changes n16:28
lbragstadsjain_: are there any sections they aren't working on that we can port over today?16:28
sjain_I can try and improve admin guides16:29
sjain_and configuration guides too16:30
*** rmascena has joined #openstack-keystone16:31
*** raildo has quit IRC16:34
*** spzala has quit IRC16:39
samueldmqmordred: morgan: great, looking forward to looking at the patches (and tests)16:43
*** jdennis has quit IRC16:46
lbragstadsjain_: sounds good16:46
samueldmqlbragstad: nice!16:46
samueldmqare there things we want to start merging already?16:46
lbragstadsjain_: let me know if you need anything - i'm going to start digging into doug's comment
samueldmqI saw there are a couple of docs that have been reworked by the docs team before we start migrating16:46
samueldmqI saw some patches from dhellmann16:47
lbragstadsamueldmq: that's what we need to figure out - it sounds like there are a few things that the docs team is still working on, so i'm not sure if they are ready to migrate yet16:47
*** sjain__ has joined #openstack-keystone16:51
*** sjain_ has quit IRC16:51
samueldmqlbragstad: knikolla: we doing bug smash today?16:54
sjain__@lbragstad: I had a look, The admin guides have not changed so I'll make a patch for that, install guides are currently under some change16:55
sjain__and I'm not sure how to address Doug's comment in, so for now I'll just merge changes I made initially by copying16:56
cmurphywow nova has 800 open bugs - we're not doing so bad :)16:56
lbragstadcmurphy: lol16:58
lbragstadsjain__: cool - let's try and get the admin-guides merged over today then16:59
lbragstadsamueldmq: I'm probably going to be focusing on docs most of the day with sjain__16:59
lbragstadsamueldmq: we have yet to release our new office hour time, too16:59
samueldmqlbragstad: gotcha17:00
lbragstadsamueldmq: per hrybacki's comment earlier17:00
samueldmqI've reviewed mostly of sjain__ 's work yesterday17:00
samueldmqwe had good comments from cmurphy there17:00
sjain__samueldmq: for the patches I uploaded yesterday, should I just abandon the first three?17:00
samueldmqsjain__: I think the second is fine, I +2ed it iirc17:01
sjain__okay, will the second one merge without the 1st?17:01
sjain__because you asked to revert the changes in 1st17:02
lbragstadi think the good thing about migrating the admin-guide first is that there is a *lot* of duplicate stuff in our tree that we can remove17:09
lbragstadonce the admin-guide is in keystone, we can start consolidating all those duplicate topics17:10
lbragstadwhich can be done in parallel17:10
*** tesseract has quit IRC17:11
*** f13o has quit IRC17:13
lbragstadsjain__: is in merge conflict - but it would be a good starting point17:15
lbragstadsjain__: would you like to perform the rebase or would you like me to?17:15
sjain__Thanks lbragstad, but I just removed that merge conflict :)17:15
*** links has quit IRC17:15
lbragstadsjain__: oh - awesome!17:16
lbragstadsjain__: you're two steps ahead of me17:16
sjain__I was just wondering should I apply your comments here or in next patch?17:16
sjain__right now I'm trying to make the structure consistent with that spec17:17
lbragstadsjain__: oh - we can address those in a follow on17:17
lbragstadsjain__: i can help with that17:17
sjain__sure, then I'll quickly push that change17:18
*** lucasxu has joined #openstack-keystone17:20
samueldmqmigrating the admin docs?17:22
lbragstadsamueldmq: yeah - i think we're going to start there because the docs team doesn't appear to be changing that content at the moment17:23
lbragstadaccording to the specification - the admin guide should live under doc/source/admin17:23
samueldmqlbragstad: sjain__  hmm yeah I guess that is fine now that is merged17:24
samueldmqwe'll probably need sjain__ to migrate them again, as changed things there17:24
sjain__oh but are any keystone docs changed17:25
sjain__I just checked, I couldn't find any17:25
*** f13o has joined #openstack-keystone17:27
samueldmqsjain__: there was a bunch of doc/admin-guide/source/identity-* that got removed in that patch17:28
samueldmqwouldnt that affect us?17:28
lbragstadsamueldmq: it would - but i think we want to pull that content over to our project17:28
sjain__those are all those docs that I migrated17:28
samueldmqah yes17:28
samueldmqI thought they were just cleaning up a couple of things they didnt want us to migrate17:29
lbragstadthe docs team is fulfilling their part of this spec
sjain__plus is this merged,
samueldmqbut that's what comes after the migration actually17:29
lbragstadit looks like everything sjain__ migrated was removed - so i think we're good17:30
sjain__I just did git pull in that repo, and didn't find any change in admin-guides at least17:30
samueldmqyeah I think we're fine17:30
lbragstadnot sure if that was suppose to be removed before all the projects had a change to migrate though17:30
samueldmqwhat are we waiting on then?17:30
sjain__@lbragstad: is that patch merged already?17:30
samueldmqlet's get that in :D17:30
samueldmqsjain__: no it's not, I guess just after all projects migrate17:30
lbragstadsjain__: not yet - it's going through the gate as we speak17:30
samueldmqlbragstad: ++ but the top parent has workflow-1, I guess they're waiting on the migrations to complete maybe17:31
samueldmqit wouldnt make sense to remove without migrating in first place17:31
lbragstadsamueldmq: oh - sur e17:31
openstackgerritSamriddhi proposed openstack/keystone master: Added keystone admin guides to documentation
lbragstadyep - ok17:31
lbragstadwe're good then17:31
lbragstadi didn't see the dependent patches17:31
sjain__this we will need to check for all the repos, that all the changes are done before I migrate them17:32
sjain__plus, ^^ admin guides done!17:33
*** spzala has joined #openstack-keystone17:33
lbragstadsjain__: yeah - the install and configuration guides are left17:33
sjain__should I do them now, or lets wait for the doc team, we can improve these admin guides now17:34
samueldmqsjain__: I think lbragstad is saying they will be kept there ?17:35
lbragstadno - we will have to migrate those, too17:35
samueldmqhm I thought I had seen some of the repos would be kept there17:35
lbragstadthese are all the open patches against the openstack-manuals project
lbragstadthe HA guide, security guide, operator guide, etc... are going to stay in openstack-manuals17:36
lbragstadthe installation guide, project installation guides, administrator guides, and configuration references are going to move into project rees17:37
samueldmqlbragstad: not sure about the Configuration Reference17:37
samueldmqI see in the spec they talking about some of that going to oslo.config, and other things17:37
samueldmqah ok: "The remainder will be removed, and replaced with new pages in the in-tree documentation built using oslo_config.sphinxext"17:38
lbragstadsamueldmq: dhellmann had a comment about that here -
samueldmqbut I guess we need to wait on first step "A few pages will move from openstack-manuals to the user-facing documentation in oslo.config"17:38
lbragstadso - our job would be to leverage the work done in oslo.config to generate the documentation for the configuration references17:38
sjain__can some please explain me this oslo_config part17:39
lbragstadthat way we don't have to maintain the configuration reference, instead it will be generated automatically17:39
samueldmqlbragstad: hmm that;s interesting17:39
lbragstadsjain__: oslo.config is a library keystone consumes to handle configuration needs for our project17:39
lbragstadsjain__: let me grab an example17:39
samueldmqhow will that be generated automatically?17:39
samueldmqwhere did they get the install instructions from?17:39
lbragstadsamueldmq: that's the part that needs attention ;)17:40
samueldmqlbragstad: so basically they want to create an standard to reprensent how to configure services?17:40
lbragstadsjain__: in keystone, we define the configuration options we want17:40
lbragstadsjain__: like this
samueldmqand then generate docs automatically from that?17:40
lbragstadsamueldmq: yep17:40
lbragstadsjain__: that uses the oslo.config library
samueldmqyeah, we will need to parallelize17:41
samueldmqthere is a LOT of work going on in docs17:41
sjain__ohkay I understood a bit17:42
lbragstadsjain__: that library let's us define help text, descriptions, and defaults for our configuration options. we can then reference it in other parts of our code where we have business logic that needs to use configuration17:42
samueldmqlbragstad: should be good to merge17:42
lbragstadsjain__: but - another cool oslo.config let's us do is generate sample configuration files based on what we define in code17:42
lbragstadanother cool thing*17:43
sjain__oh so it will automatically generate those files too17:43
lbragstadsjain__: so - for example, if you take the keystone project and run `tox -e genconfig`17:43
lbragstadyou can an updated version based on what is maintained and tested in code17:43
lbragstadi think what the documentation team is suggesting17:44
lbragstadis that we enhance the functionality of oslo.config's configuration generation bits to make it handle the generation of the configuration reference as well17:44
lbragstad(before oslo.config, each project had to manually update their sample configuration files - this is still the case with the configuration reference guide)17:45
lbragstadwhich means a lot of copy/paste17:45
sjain__oh, so for these config docs we need to connect it to oslo.config rather than simply copying the docs17:45
lbragstadand a lot of room for error17:45
lbragstadsjain__: yep - exactly17:45
lbragstadthe trick is that we don't know if oslo.config will render things nicely or exactly the way we expect the configuration reference to look17:45
lbragstadso - there might be work to improve *how* things render in oslo.config17:46
samueldmqlbragstad: is a +2 from me17:46
samueldmqcompared with docs in opentack-manuals, it's a really clean copy-paste from there17:46
lbragstadonce that is done, each project should create a configuration reference section in there doc/source tree and just tell oslo.config where the configuration module is and to generatea  configuration reference17:46
samueldmqshould be fine to get in17:46
lbragstadsamueldmq: cool - reviewing17:48
sjain__see this one
sjain__for install guides17:49
sjain__removing project specific docs17:50
sjain__that means may be this is complete17:50
lbragstadsjain__: yes - i would say so17:50
sjain__I was most concerned for, this is merged17:50
lbragstadsjain__: i'm looking through all the open openstack-manual patches, but i'm not seeing anything that is changing the content that we have to be worries about17:50
lbragstadsjain__: yeah - so long as we have copy that includes that patch, I think we're good to move that stuff over whenever17:51
sjain__these install docs have changed, from what I added in previous patch17:51
sjain__so I think I'll start with this one17:52
sjain__it will require some rearranging17:52
samueldmqit wouldn't hurt to double check with asettle and dhellmann what's ready for migration already17:52
samueldmqaside from the admin guides, just to make sure we are all in the same page17:53
samueldmqsjain__: but yeah, from the spec it's clear the Installation Guide wil lcome in-tree17:54
samueldmqwe would just need to wait for
samueldmqbut it's been merged17:55
samueldmqso you should be able to update your migration to keystone17:55
lbragstadsamueldmq: yeah - so we're good to move that to keystone whenever17:55
samueldmqlbragstad: ++17:55
sjain__yeah, so I'll migrate those then17:55
lbragstadi'm +2 on the admin guide17:55
lbragstadwaiting for jenkins to verify it and I'll kick it through17:56
sjain__thanks :)17:56
lbragstadlet's see if we can do the same with the install guide today17:56
lbragstadthen the only big piece left is working our the configuration reference bits with oslo.config17:56
samueldmq++ that'd be awesome17:59
sjain__one query: how are these install-guides different from the ones we right now have in getting-started/general info18:00
lbragstadsjain__: the install guides from openstack-manuals were the official installation guides18:01
lbragstadsjain__: more than likely the ones that operators reference when doing an installation18:02
lbragstadsjain__: the ones we have in tree are kind of a mish-mash of developer and operator installation techniques18:02
sjain__oh alright, but was adding those in general section18:03
sjain__should I add them to operator?18:03
lbragstadsjain__: the installation guide from openstack-manuals?18:03
lbragstadsjain__: i'd create a new place for it18:04
lbragstadsjain__: put them under doc/source/install18:04
sjain__no in the layout18:04
lbragstadsjain__: i'd make it its own top-level section18:05
lbragstadjust like `API Reference`18:05
lbragstadI'd make one called `Installation Guide` and have it render everything under doc/source/install18:05
sjain__ohkay, makes sense18:06
lbragstadsjain__: yep18:06
lbragstadsjain__: that way we keep the two sets of documentation separate for the time being18:06
*** aojea has joined #openstack-keystone18:06
lbragstadthen we can slowly go through all the installation docs *not* in doc/source/install and either move it to the appropriate section of doc/source/install or delete it18:06
lbragstadthat's going to be the fun part because we get to remove all the duplicate code18:08
*** aojea has quit IRC18:11
*** spzala has quit IRC18:16
*** spzala has joined #openstack-keystone18:17
*** spzala has quit IRC18:18
*** spzala has joined #openstack-keystone18:18
*** spzala has quit IRC18:23
lbragstadsjain__: one last comment on to be consistent with the installation guide18:31
*** spzala has joined #openstack-keystone18:40
sjain__@lbragstad: should I place it very top or before/after operator docs18:43
sjain__samueldmq: ^^18:43
openstackgerritLance Bragstad proposed openstack/keystone master: Add history behind why keystone has two ports
*** bknudson has quit IRC18:58
openstackgerritSamriddhi proposed openstack/keystone master: Added keystone admin guides to documentation
*** clayton has quit IRC19:19
*** MasterOfBugs has joined #openstack-keystone19:19
*** clayton has joined #openstack-keystone19:21
*** ducttape_ has quit IRC19:21
*** ducttape_ has joined #openstack-keystone19:22
lbragstadsjain__: what you have there is perfect,19:26
lbragstad samueldmq19:26
sjain__I'm almost done with install guides19:26
sjain__have a look at the last section of
*** dave-mccowan has quit IRC19:29
lbragstadsjain__: yep19:29
lbragstadsjain__: wondering what to do about the other projects referenced there?19:29
sjain__I removed those19:29
sjain__I just copied keystone docs19:29
lbragstad++ i think that's fine19:30
sjain__nd rearranged a little bit19:30
sjain__I'm getting some issues with rendering19:30
lbragstadsjain__: hmm19:30
sjain__which I'm not able to figure out19:30
sjain__I'll push the changes19:30
lbragstadsjain__: cool - do that and i'll see if i can recreate19:31
sjain__can you have a look at those pls19:31
sjain__thanks :)19:31
openstackgerritSamriddhi proposed openstack/keystone master: WIP: Added keystone installation guides
sjain__the issue is with 'Install and configure' page, there are multiple links appearing19:32
*** jmlowe has quit IRC19:39
lbragstadsjain__: this is what i'm getting locally when i run `tox -e docs`19:41
lbragstadfull trace -
sjain__strange, that file is uploaded here
lbragstadsjain__: ah - nevermind19:44
lbragstadsjain__: i have something wrong in my local env19:44
lbragstadrerunning now19:44
lbragstadsjain__: ok - that time it worked19:46
lbragstadsjain__: i see the Installation Guides section just under the Related Projects bit19:46
lbragstadwhich seems fine19:46
sjain__yeah navigate to any one19:47
sjain__and open install and configure19:47
lbragstadok - i'm in index-debian.html19:47
sjain__open contents in that19:49
lbragstadsjain__: well - there are four sections rendering, right?19:49
lbragstadsjain__: each section is for a different platform19:49
sjain__offside.. there was some text common in abstract which I brought on first rendering page19:49
sjain__yes right19:50
sjain__but in each section again they are rendering all the docs and not specific to that section19:50
lbragstadsjain__: i think that was something specific to the docs project that made less duplication for the maintainers of openstack-manuals?19:52
lbragstadasettle: would be the person to ask there though19:52
sjain__okay, looks like the issue is with :blob: directive19:53
sjain__I'll ask her tomorrow19:53
lbragstadyeah - i'd like to have her opinion on that change19:54
lbragstadit's obvious that all the installation stuff needs to go in doc/source/install19:54
lbragstadbut i wonder if we label each section according to the platform19:54
lbragstade.g. Ubuntu install and configure19:55
lbragstador how we go about that19:55
sjain__yeah, lets see what she says19:56
sjain__we will make the changes then19:56
lbragstaddoes someone else want to give this another look?
* lbragstad steps away for 15 minutes20:00
*** f13o has quit IRC20:05
*** sjain__ has quit IRC20:05
morganlbragstad: i'm happy to +2/+A it, but that is a brutally large patchset20:06
morganlbragstad: it's just a straight copy from the previous location(s), right?20:06
cmurphyyes, and it's identical to the openstack-manuals except for identity-management.rst changes the title20:08
cmurphylbragstad: is that one good to go or are we waiting on something from asettle?20:09
morgancmurphy: lbragstad +2'd it20:09
*** aojea has joined #openstack-keystone20:16
*** f13o has joined #openstack-keystone20:17
*** aojea has quit IRC20:20
lbragstadcmurphy: nope the admin guide is ready to go20:31
lbragstadcmurphy: we're going to wait on some clarification from asettle on the installation and configuration guides20:31
cmurphyokay cool20:31
*** ducttap__ has joined #openstack-keystone20:32
lbragstadwhich is fine because the admin guide has a ton of good information that we duplicated in our dev docs, so we can probably start working on removing that duplication once the admin-guide merges20:32
cmurphysamueldmq is on top of it20:33
samueldmqapproved :)20:33
*** ducttape_ has quit IRC20:36
*** f13o has quit IRC20:47
*** lucasxu has quit IRC20:53
*** ducttap__ has quit IRC20:53
*** nicolasbock has quit IRC20:58
*** catinthe_ has joined #openstack-keystone21:10
*** catintheroof has quit IRC21:12
openstackgerritMerged openstack/keystone master: Add HEAD APIs to federated API
openstackgerritSamuel Pilla proposed openstack/keystone master: WIP: Add project tags
*** spzala has quit IRC21:17
*** bknudson has joined #openstack-keystone21:18
*** pramodrj07 has joined #openstack-keystone21:18
*** ducttape_ has joined #openstack-keystone21:19
*** pramodrj07 has quit IRC21:19
*** pramodrj07 has joined #openstack-keystone21:19
*** ducttap__ has joined #openstack-keystone21:20
*** PramodJ has joined #openstack-keystone21:21
*** MasterOfBugs has quit IRC21:22
*** ducttape_ has quit IRC21:23
*** pramodrj07 has quit IRC21:25
openstackgerritMerged openstack/keystone master: Add annotation about token authenticate
* knikolla is wondering how old this note is
*** dave-mccowan has joined #openstack-keystone21:49
*** catinthe_ has quit IRC21:52
*** PramodJ has quit IRC21:57
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Move version discovery logic to
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add url manipulation and microversion collection
mordredmorgan, cmurph, samueldmq: first patch shouild be good - second patch still needs work but I figured I'd push it up so you could see where it's going22:04
mordredmorgan: there are literally zero things about hte tests in the second one that are going to make you happier to be alive22:04
mordredbecause, you know - to test this, it means constructing the evil catalog entries22:05
mordredBUT - it's a great exercise22:05
openstackgerritMerged openstack/keystone master: Added keystone admin guides to documentation
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add url manipulation and microversion collection
mordredcmurphy, morgan: actually, apparently just pushing it up allowed me to see the problem, so that's neat22:19
mordredalthough WOW I just found a fun new bog :)22:22
*** dave-mccowan has quit IRC22:24
*** chlong has quit IRC22:45
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add url manipulation and microversion collection
mordredcmurphy: I found that 3.20 was being parsed into (3, 2) - which isn't quite the expectation22:46
cmurphyi can imagine being really confused trying to use nova 2.20 and getting nova 2.222:48
morgancmurphy: oh man22:53
morganthat would be bad22:53
morganmordred: LOL22:53
morganmordred: yeah..... *whoopse*22:53
morganneed to do those as stings22:53
mordredit's otherwise VERY comprehensively tested22:53
morgannot floats22:53
mordredmorgan: be prepared for eye-gouging22:54
morganmordred: you know... if we handle all versions a tuples22:54
morgan(X, Y, ZZ)22:54
morganall the math works22:54
morganbecause             version = str(float(int(version))) is going to run into similar issues22:55
morgangod, how did that ever work22:55
*** eandersson has quit IRC22:56
*** eandersson has joined #openstack-keystone22:56
morganmordred: uh22:56
mordredmorgan: yah22:56
morganam i mis-reading this? we still have the 3.20->3.2 issue22:56
mordredwe don't - added test to prove22:56
mordredalthough looking at it I'm confused as to why it works22:57
morgansomething is wonky atm22:57
morganre-reading the code22:57
mordredoh - it works because int('3.20') throws22:57
mordredso that line ONLY works for strings that cleanly are single ints22:58
mordredand strings that are not pure ints fall through to the split line22:58
mordredthat at least warrants a commant22:58
morganoh gross22:58
morganit has to be an int, or it has some breaking logic22:59
morganyou know... i almost think we should move the if-checks into the try-except22:59
morganohw ait no.22:59
morganthis is ... trying really hard to normalize every possible form23:00
mordredyah. and it's got a bunchof backwards compats terror atached23:00
morganso i'd make the ifs Elifs23:00
morganit would make it a lot cleaner23:00
morganor at least easier to read23:00
mordredyes. I agree - we do not need fallthrough there23:01
morganif string: do X, elif float: do Y, elif int: do x23:01
morganit's fine as is23:01
morgani mean, not worth re-spinning on that alone but23:01
mordredwell - I think the one thing does need a comment23:03
mordredand you haven't read much more of that patch yet :)23:03
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add url manipulation and microversion collection
mordredmorgan: ^^ how's that look23:04
mordredelifs and more comments23:04
mordredand a pin, apparenlty23:04
*** ducttap__ has quit IRC23:04
morganwell then23:06
morgani was commenting! :P23:06
morganlooking it over again23:06
morgani think it'll be good23:06
mordredoh - I'll go read both versions23:06
morgannah, it's all good23:07
morganmy comments were nits at best23:07
*** bknudson has quit IRC23:08
*** bknudson has joined #openstack-keystone23:08
morgansee in-line nits23:09
morgan+1 until CI response23:09
morganthen will upgrade to +223:09
*** bknudson has quit IRC23:13
*** spzala has joined #openstack-keystone23:18
*** spzala has quit IRC23:22
cmurphygonna look at this again tomorrow morning, it's getting too late for my brain23:24
morgancmurphy: simple solution - 1) stare at it until brain bleeds, 2) ???, 3) merge code, 4) profit23:33
*** edmondsw has quit IRC23:47
*** edmondsw has joined #openstack-keystone23:48
*** edmondsw has quit IRC23:52
*** gyee has quit IRC23:52
mordredmorgan: that's my process most times23:55
morganmordred:  you also often include beer-mouth and poolside hacking23:56
mordredmorgan: I agree, btw, with one of your nits about the comment not being useful - it's definitely too snark. the thing I think it _should_ point out is that attaching a v2 version discovery document to a service called "volumev3" isn't a test bug - so maybe a followup with less snark :)23:57
mordredmorgan: and YES to beer-mouth and poolside hacking23:57
morganmordred: or smoking large cuts of meat23:57
morganyep, it's a nit23:57
morgannot worth another patch23:57
morganbut man... snaaaark23:57
* morgan beer mouths23:57
morganmordred: it's a legit test case23:58
morganmordred: next place i live (looking at options) will include a place to BBQ and a place to smoke meats23:58
morganand possibly a pool23:58

Generated by 2.15.3 by Marius Gedminas - find it at!