14:02:08 <devkulkarni> #startmeeting Solum Team Meeting
14:02:09 <openstack> Meeting started Wed Jun 28 14:02:08 2017 UTC and is due to finish in 60 minutes.  The chair is devkulkarni. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:02:10 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:02:12 <openstack> The meeting name has been set to 'solum_team_meeting'
14:02:20 <zhurong_> hi devkulkarni
14:02:28 <devkulkarni> hi zhurong_
14:02:36 <devkulkarni> nice to see you
14:02:45 <zhurong_> nice to see you too
14:03:01 <devkulkarni> lets wait for a minute or so for others to join
14:03:12 <zhurong_> sure
14:04:51 <devkulkarni> zhurong_: lets begin
14:04:59 <zhurong_> okay
14:05:03 <devkulkarni> main thing today is the docs patches
14:05:17 <zhurong_> yep
14:05:40 <devkulkarni> thanks for submitting them. I had not noticed them before so ended up creating one of the patches. but I will abandon it
14:06:01 <zhurong_> never mind
14:06:38 <devkulkarni> looking through them now
14:07:24 <zhurong_> the switch openstack theme patch need further improve
14:07:39 <devkulkarni> right, I saw the comment
14:11:07 <devkulkarni> zhurong_: in this patch https://review.openstack.org/#/c/477049/2 why do we remove contributing, installation, and usage files?
14:11:53 <zhurong_> those files seems useless
14:14:48 <devkulkarni> I think the content in installation and usage can be useful to someone who does not know a whole lot about openstack and/or python. Anything that can help end users we should keep. I am fine deleting those files, but we should not remove that content. Could you move it to one of the other files (say, solum.rst)
14:15:38 <devkulkarni> may be you can keep this particular patch only for turning on the warning-is-error flag
14:15:55 <devkulkarni> and in another patch move the content..
14:17:09 <zhurong_> if we need add them back, we need add more details about those
14:18:53 <devkulkarni> sure..we can do that in a separate patch. If we don't loose the content then we will have a base to start from
14:19:33 <devkulkarni> I agree about removing the contributing.rst
14:19:42 <devkulkarni> we already have it at the top level
14:20:53 <zhurong_> yep
14:22:12 <devkulkarni> for installation.rst and usage.rst, we can just move that content for now inside either solum.rst or top-level README.rst
14:23:09 <zhurong_> OK, I will add this to the README.rst
14:24:09 <devkulkarni> cool. we can always work on improving and adding to that content with things that might help users.
14:24:53 <zhurong_> yep, agree with this
14:24:59 <devkulkarni> cool
14:25:06 <devkulkarni> other two patches looked fine to me
14:26:46 <devkulkarni> for the openstackdocstheme change patch, looks like the steps to follow are here: https://docs.openstack.org/openstackdocstheme/latest/
14:26:56 <devkulkarni> I am guessing you have already looked through them
14:27:50 <zhurong_> yep, I am try to do this
14:28:04 <devkulkarni> zhurong_: cool
14:28:29 <devkulkarni> the etherpad that lists all the steps is https://etherpad.openstack.org/p/doc-migration-tracking
14:29:00 <devkulkarni> *looking at the Directions (line 27 - 46) *
14:29:42 <zhurong_> about the soulm install guider, we do not have this
14:30:03 <devkulkarni> Step 1 requires us to have a patch that depends on the spec. It is this patch https://review.openstack.org/#/c/477058/3
14:30:21 <devkulkarni> yeah, we do not have install guide (or any other guides) yet.
14:30:31 <devkulkarni> for us the only thing that needs to be moved is the cli
14:31:03 <devkulkarni> for step 1, they had asked us to have a Depends-On for the spec
14:31:19 <zhurong_> this spec had already merged
14:31:29 <zhurong_> we don't need Depends-on this
14:31:54 <zhurong_> actually we need the install-guide to solum
14:32:14 <devkulkarni> good point
14:32:29 <devkulkarni> which line number you are referring to?
14:33:05 <zhurong_> https://docs.openstack.org/project-install-guide/messaging/ocata/
14:33:08 <zhurong_> like this
14:33:24 <devkulkarni> btw, look at 5.2 (line 35). You will need a Depends-On for the openstackdocstheme patch as well
14:33:57 <zhurong_> I will submit the openstackdocstheme patch right now
14:34:03 <devkulkarni> zhurong_: yep, agree we need something like this
14:34:24 <devkulkarni> zhurong_: but having a install guide is not part of the docs migration
14:34:34 <devkulkarni> once we have completed docs migration we can work on the install guide
14:34:39 <zhurong_> yep, I think so
14:35:19 <devkulkarni> about the openstackdocstheme patch, make sure you have the Depends-On as mentioned in point 5.2 (line 35)
14:35:25 <devkulkarni> if it is required
14:36:39 <zhurong_> which patch should I Depends-On?
14:36:55 <devkulkarni> Depends-On: Ifc5512c0e2373cf3387e0e0498268eab092e52bb
14:37:13 <devkulkarni> https://etherpad.openstack.org/p/doc-migration-tracking check line 35
14:37:23 <zhurong_> nop, already merged too
14:37:29 <devkulkarni> oh okay
14:37:32 <devkulkarni> then we don't need that
14:38:22 <devkulkarni> Do you understand point 6 (line 36)?
14:41:13 <zhurong_> not so clear
14:41:46 <devkulkarni> yeah.. also the link in point 6.2 is not working
14:41:56 <devkulkarni> I am getting 404
14:42:17 <zhurong_> me too
14:42:35 <zhurong_> https://docs.openstack.org/developer/oslo.config/reference/sphinxconfiggen.html
14:42:39 <zhurong_> maybe this one
14:43:46 <zhurong_> https://github.com/openstack/glance/blob/master/doc/source/conf.py#L55
14:44:16 <devkulkarni> probably you are right
14:44:31 <devkulkarni> those might have been correct links
14:44:36 <zhurong_> seems we doesn't need this, I am not sure
14:45:19 <devkulkarni> we do have solum.conf which is used by the server side
14:46:55 <devkulkarni> zhurong_: https://github.com/openstack/solum/blob/master/doc/source/conf.py
14:47:41 <zhurong_> yep, maybe solum need this, we need do more research
14:47:54 <devkulkarni> right
14:47:55 <zhurong_> And I have done the openstack theme patch
14:48:05 <devkulkarni> oh cool
14:49:09 <devkulkarni> looks fine to me
14:49:41 <devkulkarni> lets continue going through rest of the steps from that etherpad
14:49:51 <devkulkarni> - For step 6 (need more research)
14:50:07 <devkulkarni> How about step 7? (lines 40-41)?
14:50:28 <devkulkarni> I think we don't need to do step 7
14:50:42 <zhurong_> agree
14:50:50 <zhurong_> we don't have this
14:50:55 <devkulkarni> right
14:51:08 <devkulkarni> step 8.
14:51:16 <devkulkarni> not very clear to me
14:51:36 <devkulkarni> I guess this is a change to project-config
14:52:11 <zhurong_> https://review.openstack.org/#/c/477232/1/zuul/layout.yaml
14:52:14 <zhurong_> like this
14:52:31 <devkulkarni> yep
14:52:43 <zhurong_> https://review.openstack.org/477051 got Jenkins +1
14:53:16 <devkulkarni> cool. once andreas weighs in, we can merge it
14:53:27 <zhurong_> aj +1 too
14:54:00 <devkulkarni> approved
14:54:35 <zhurong_> thanks
14:54:41 <devkulkarni> so for step 8, we will have to change project-config for openstack/python-solumclient
14:54:46 <devkulkarni> how about step 9
14:55:15 <devkulkarni> what is the "new location" referred to in that line (line 44)
14:56:39 <zhurong_> https://review.openstack.org/#/c/476526/1/README.rst
14:56:42 <zhurong_> check this
14:57:03 <devkulkarni> I see
14:57:40 <zhurong_> https://review.openstack.org/#/c/477049/ we can merge this first, I will add the delete content to the README.rst
14:58:05 <devkulkarni> ok
14:58:31 <devkulkarni> done
14:58:49 <zhurong_> ths devkulkarni
14:59:01 <devkulkarni> its almost end of the meeting time
14:59:11 <zhurong_> yep
14:59:15 <devkulkarni> I think it was good discussion today
14:59:25 <devkulkarni> we are almost done with our docs migration
14:59:32 <zhurong_> i thinl so
14:59:40 <devkulkarni> only question is about point number 6 and then the patch to change project-config
14:59:57 <devkulkarni> we can work on that next
15:00:14 <zhurong_> sure
15:00:14 <devkulkarni> lets continue the discussion in solum channel
15:00:17 <devkulkarni> will end the meeting
15:00:22 <devkulkarni> #endmeeting