Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix stop.sh execute failed in macos (#3187) #3188

Merged
merged 26 commits into from
Apr 3, 2022
Merged

Conversation

IceFoxs
Copy link
Contributor

@IceFoxs IceFoxs commented Apr 2, 2022

fix stop.sh exxcute failed in macos(#3187)

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

�optimize start.sh ,support different jdk configure JAVA_OPTS parameter
fix start.bat [shenyu-bootstrap-dist missing agent parameter in start.bat script (Issue apache#2814)]
@codecov-commenter
Copy link

codecov-commenter commented Apr 2, 2022

Codecov Report

Merging #3188 (a4934b9) into master (35e9f98) will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3188      +/-   ##
============================================
+ Coverage     57.47%   57.52%   +0.05%     
- Complexity     4526     4528       +2     
============================================
  Files           730      730              
  Lines         20818    20818              
  Branches       1915     1915              
============================================
+ Hits          11965    11976      +11     
+ Misses         7803     7792      -11     
  Partials       1050     1050              
Impacted Files Coverage Δ
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 63.82% <0.00%> (+1.06%) ⬆️
...henyu/admin/service/impl/UpstreamCheckService.java 70.00% <0.00%> (+7.14%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35e9f98...a4934b9. Read the comment docs.

@@ -22,7 +22,7 @@ cd `dirname $0`
cd ..
DEPLOY_DIR=`pwd`

PIDS=`ps -eo pid,cmd |grep -v tail|grep -v grep|grep "$DEPLOY_DIR"|awk '{print $1}'`
PIDS=`ps ax |grep -v tail|grep -v grep|grep "$DEPLOY_DIR"|awk '{print $1}'`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

support linux?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I already verified it in ubuntu and centos.

image

@yu199195 yu199195 added type: bug Something isn't working type: build labels Apr 3, 2022
@yu199195 yu199195 added this to the 2.4.3 milestone Apr 3, 2022
@yu199195 yu199195 merged commit 427d0c9 into apache:master Apr 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working type: build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants