Skip to content

Conversation

@idbeta
Copy link
Contributor

@idbeta idbeta commented Feb 2, 2021

What this PR does / why we need it:

test: modify apisix path for centos7 ci

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@idbeta idbeta changed the title test: modify apisix path for centos7 ci test: make centos7 ci easier to understand Feb 3, 2021
@idbeta idbeta marked this pull request as ready for review February 3, 2021 03:54
@idbeta
Copy link
Contributor Author

idbeta commented Feb 3, 2021

@spacewander Please review if you have time.

# install test::nginx
yum install -y cpanminus build-essential libncurses5-dev libreadline-dev libssl-dev perl
cp -r /tmp/apisix ./apisix
# cp -r /tmp/apisix ./
Copy link
Member

Choose a reason for hiding this comment

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

We can remove this comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my fault, fixed.

Copy link
Member

@membphis membphis left a comment

Choose a reason for hiding this comment

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

LGTM


# install dependencies
cd apisix
git clone https://github.com/iresty/test-nginx.git test-nginx
Copy link
Member

Choose a reason for hiding this comment

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

we can change it to api7 later

@spacewander spacewander merged commit 1e35cc0 into apache:master Feb 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants