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

Initial install test to validate correct version of SG/A has been installed and is running #1463

Closed
wants to merge 3 commits into from

Conversation

ajres
Copy link
Contributor

@ajres ajres commented Nov 17, 2017

fixes #1449
fixes #1450
fixes #1456
fixes #1457

@raghusarangapani
Copy link
Contributor

Hi Andy, in the test and the conftest, all parsing/references to liteserv-host, liteserv-port, liteserv-version, liteserv-platform, liteserv-storage-engine, server-upgraded-version, sync-gateway-upgraded-version and cbs-upgrade-toybuild can be removed. They are not used in this test.

@hrajput89
Copy link
Contributor

Hi Andy, in the test and the conftest, all parsing/references to liteserv-host, liteserv-port, liteserv-version, liteserv-platform, liteserv-storage-engine, server-upgraded-version, sync-gateway-upgraded-version and cbs-upgrade-toybuild can be removed. They are not used in this test.

These variables are needed in test_upgrade.

@sridevi-15
Copy link
Contributor

To me, this PR is not required. Install and verification of syncgateway version is being testing in our regression . We have upgrade test already under sync gateway which does same. I would say we have to remove this PR

@hrajput89 hrajput89 closed this Dec 7, 2018
@pasin pasin removed the review label Dec 7, 2018
@hrajput89 hrajput89 deleted the feature/issue_1449 branch December 7, 2018 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment