Hi Asif, Please create a JIRA for checking the apiList.js issue. Then we can add this known issue to the release note and address it in m2 release.
Willem Jiang Blog: http://willemjiang.blogspot.com (English) http://jnn.iteye.com (Chinese) Twitter: willemjiang Weibo: 姜宁willem On Thu, Mar 8, 2018 at 3:49 PM, Mohammad Asif Siddiqui < asifdxtr...@apache.org> wrote: > Thanks YangBo and Cuiyihua for the suggestion. > > We will move the frontend configurations to the app.conf and add the > frontend starting guide in the main Readme, so from here on there will be > only 1 Readme and 1 conf which can be used by both frontend and > service-center. > > Coming back to configuration of SC api in frontend we will definitely move > this to configuration to the app.conf and it can be dynamically replaced > when frontend will start, this needs some time of testing in windows system > as in linux it's quite easy but in some windows OS version it fails, so > that was the reason we kept this conf in apiList.js. We will take this > issues in m2 release. > > Regards > Asif > > > On 2018/03/08 07:38:32, "Sure" <sure_0...@qq.com> wrote: > > Suggestion: merge these configures in SC app.conf, and move the frontend > readme into main readme. > > > > > > > > > > ------------------ 原始邮件 ------------------ > > 发件人: "Yang Bo"<oaky...@gmail.com>; > > 发送时间: 2018年3月8日(星期四) 下午3:36 > > 收件人: "dev"<dev@servicecomb.apache.org>; > > > > 主题: Re: [VOTE] Release Apache ServiceComb Service-Center > (incubating)version 1.0.0-m1 > > > > > > > > Hi Asif, > > > > For the first issue, It's OK that we leave it as is in this release. But > in > > the long run, it's still better to put the configurations together(the > > frontend.conf file). > > > > > > On Thu, Mar 8, 2018 at 3:31 PM, Mohammad Asif Siddiqui < > > asifdxtr...@apache.org> wrote: > > > > > Hi YangBo, > > > > > > Thanks for your vote. > > > > > > We will fix the 2nd and 3rd Issue. > > > > > > For 1st Issue I would like to clear that the hardcode values are in js > > > files which can be modifies easily afterwards in the release, which we > have > > > already mentioned in our Readme over here [1] > > > > > > [1] https://github.com/apache/incubator-servicecomb-service- > > > center/tree/master/frontend#note > > > > > > Regards > > > Asif > > > > > > > > > On 2018/03/08 07:20:17, Yang Bo <oaky...@gmail.com> wrote: > > > > -1 Binding Vote. > > > > > > > > Below Checks Done: > > > > 1. Verified the md5sum, sha1sum and GPG signature for source and > binary > > > > release packages. > > > > 2. Verified the service-center linux release using Java Chassis BMI > > > sample. > > > > 3. Verified the frontend on both windows and linux. > > > > 4. Checked the bundled files, no fonts and images without proper > > > licensing > > > > bundled. > > > > 5. Checked the LICENSE/NOTICE, README, DISCLAIMER for both binary and > > > > source release > > > > 6. RAT on source release is OK > > > > > > > > Some issues: > > > > 1. The service-center's address/port is hardcoded in frontend's js > file, > > > > which cause the frontend to not functioning if service center is not > > > > running on 127.0.0.1:30100. > > > > Bug report: https://issues.apache.org/jira/browse/SCB-375 > > > > > > > > 2. The scripts in binary release does not contain the ASF header. > > > > start-frontend.sh > > > > start-service-center.sh > > > > stop-frontend.sh > > > > stop-service-center.sh > > > > > > > > 3. Scripts on source release have ASF header but the script shebang > is > > > not > > > > in the first line, which would cause problems if the script is not > called > > > > directy by bash. We should still put the shell shebang on the first > line > > > of > > > > scripts and followd by a blank line then the ASF header. > > > > For example: > > > > =================================== > > > > #!/bin/bash > > > > > > > > # Licensed to the Apache ... > > > > > > > > > > > > On Thu, Mar 8, 2018 at 2:15 PM, Sukesh A C <sukes...@huawei.com> > wrote: > > > > > > > > > +1 Binding Vote. > > > > > > > > > > Below Checks Done: > > > > > 1. Verify the file signature. > > > > > 2. Verified windows release for service center using java chassis > demo. > > > > > 3. Ran the Service-Center and Frontend, verified basic > functionality of > > > > > service center and frontend. > > > > > > > > > > Thanks, > > > > > Sukesh. > > > > > > > > > > -----Original Message----- > > > > > From: Mohammad Asif Siddiqui [mailto:asifdxtr...@apache.org] > > > > > Sent: 07 March 2018 23:11 > > > > > To: dev@servicecomb.apache.org > > > > > Subject: [VOTE] Release Apache ServiceComb Service-Center > (incubating) > > > > > version 1.0.0-m1 > > > > > > > > > > Hi All, > > > > > > > > > > This is a call for Vote to release Apache ServiceComb > Service-Center > > > > > (Incubating) version 1.0.0-m1 (second release candidate). > > > > > > > > > > Release Notes : https://github.com/apache/ > > > incubator-servicecomb-service- > > > > > center/blob/master/docs/release/releaseNotes.md > > > > > > > > > > Release Candidate : https://dist.apache.org/repos/ > dist/dev/incubator/ > > > > > servicecomb/incubator-servicecomb-service-center/1.0.0-m1/ > > > > > > > > > > Release Tag : https://github.com/apache/ > incubator-servicecomb-service- > > > > > center/releases/tag/1.0.0-m1 > > > > > > > > > > Release CommitID : cb48d5b1a84bfae3b111a34955664e30cb4b9f3f > > > > > > > > > > Keys to verify the Release Candidate : > https://dist.apache.org/repos/ > > > > > dist/dev/incubator/servicecomb/KEYS > > > > > > > > > > Release Source : https://dist.apache.org/repos/dist/dev/incubator/ > > > > > servicecomb/incubator-servicecomb-service-center/1.0.0-m1/src/ > > > > > > > > > > Guide to build the release from source : > https://github.com/apache/ > > > > > incubator-servicecomb-service-center/tree/master/scripts/release > > > > > > > > > > Voting will start now ( Wednesday, 7th March, 2018) and will remain > > > open > > > > > for next 72 hours, Request all PPMC members to give their vote. > > > > > > > > > > [ ] +1 Release this package as 1.0.0-m1. > > > > > [ ] +0 No Opinion. > > > > > [ ] -1 Do not release this package because.... > > > > > > > > > > Regards > > > > > Asif > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Best Regards, > > > > Yang. > > > > > > > > > > > > > > > -- > > Best Regards, > > Yang. >