Skip to content

[skip ci] Configure ROBO nightlies with Group19-DRS-Disabled#7876

Merged
anchal-agrawal merged 1 commit intovmware:masterfrom
anchal-agrawal:7662-robo-nightly
May 3, 2018
Merged

[skip ci] Configure ROBO nightlies with Group19-DRS-Disabled#7876
anchal-agrawal merged 1 commit intovmware:masterfrom
anchal-agrawal:7662-robo-nightly

Conversation

@anchal-agrawal
Copy link
Contributor

This commit adds test scripts for running a nightly Jenkins job to
run tests in a non-DRS environment. This change runs the
Group19-DRS-Disabled suite against a pre-deployed, static vCenter
ELM testbed. The corresponding secrets file will be committed to
the internal repo.

Future changes will include adding more tests and using a testbed
deployed with a service account.

Fixes #7662

@@ -0,0 +1,72 @@
#!/bin/bash
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This file is similar to tests/nightly/jenkins-nightly-run.sh except at the bottom, where we're running a different test suite.

@@ -0,0 +1,56 @@
#!/bin/bash
Copy link
Contributor Author

@anchal-agrawal anchal-agrawal May 1, 2018

Choose a reason for hiding this comment

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

This file is similar to tests/nightly/upload-logs.sh except that the log artifact names and storage buckets are different.

@codecov-io
Copy link

codecov-io commented May 1, 2018

Codecov Report

Merging #7876 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7876   +/-   ##
=======================================
  Coverage   26.34%   26.34%           
=======================================
  Files          37       37           
  Lines        5189     5189           
=======================================
  Hits         1367     1367           
  Misses       3715     3715           
  Partials      107      107

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 70e4c2b...9393407. Read the comment docs.

Copy link
Contributor

@cgtexmex cgtexmex left a comment

Choose a reason for hiding this comment

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

Only nit is the use of ROBO in some names...that will lose meaning as time goes by and doesn't match the test suite, etc..

Copy link
Contributor

@mhagen-vmware mhagen-vmware left a comment

Choose a reason for hiding this comment

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

lgtm, just a couple of changes I would like to see


pybot --removekeywords TAG:secret -d robo tests/manual-test-cases/Group19-DRS-Disabled > robo/stdout.txt

cat robo/stdout.txt | grep '::' | grep -E 'PASS|FAIL' > console.log
Copy link
Contributor

Choose a reason for hiding this comment

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

Use the latest grep from vic-product (change initial grep to grep -E '::|...'):
https://github.com/vmware/vic-product/blob/master/tests/vic-product-nightly.sh#L52

echo "Tarball extraction passed, Running nightlies test.."
fi

pybot --removekeywords TAG:secret -d robo tests/manual-test-cases/Group19-DRS-Disabled > robo/stdout.txt
Copy link
Contributor

Choose a reason for hiding this comment

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

run this in a container to be more jenkins workspace compatible:
https://github.com/vmware/vic-product/blob/master/tests/vic-product-nightly.sh#L51

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure I'm following. The jenkins job runs this script in a docker container

#!/bin/bash
docker run -v $PWD/vic:/go --env-file <secrets file path> gcr.io/eminent-nation-87317/vic-integration-test:1.48 tests/robo/jenkins-robo-run.sh

whereas the vic-product-nightly job runs the script directly.

Copy link
Contributor

Choose a reason for hiding this comment

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

right, I want to move to the latter approach across the board

Copy link
Contributor

Choose a reason for hiding this comment

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

vic-engine nightlies aren't fully there, but product and longevity have already been converted. I would prefer you start from where we are now otherwise I have to go in an convert it later again. But I am not blocking your PR on this either way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done - thanks! Please give this script another look. Key changes:

  • the script runs on the executor directly
  • the pabot command runs in a container
  • the log upload script runs in a second container
  • all paths/references to robo updated to drs-disabled, per @cgtexmex's feedback

@anchal-agrawal
Copy link
Contributor Author

@cgtexmex Thanks - updated all paths/references to robo with drs-disabled.

@anchal-agrawal anchal-agrawal changed the title Configure ROBO nightlies with Group19-DRS-Disabled [skip ci] Configure ROBO nightlies with Group19-DRS-Disabled May 2, 2018
Copy link
Contributor

@mhagen-vmware mhagen-vmware left a comment

Choose a reason for hiding this comment

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

lgtm

This commit adds test scripts for running a nightly Jenkins job to
run tests in a non-DRS environment. This change runs the
Group19-DRS-Disabled suite against a pre-deployed, static vCenter
ELM testbed. The corresponding secrets file will be committed to
the internal repo.

Future changes will include adding more tests and using a testbed
deployed with a service account.

Fixes vmware#7662
@anchal-agrawal anchal-agrawal merged commit fe040e0 into vmware:master May 3, 2018
zjs pushed a commit to zjs/vic that referenced this pull request Jul 19, 2018
…7876)

This commit adds test scripts for running a nightly Jenkins job to
run tests in a non-DRS environment. This change runs the
Group19-DRS-Disabled suite against a pre-deployed, static vCenter
ELM testbed. The corresponding secrets file will be committed to
the internal repo.

Future changes will include adding more tests and using a testbed
deployed with a service account.

Fixes vmware#7662

(cherry picked from commit fe040e0)
zjs pushed a commit to zjs/vic that referenced this pull request Jul 27, 2018
…are#8151)

This commit adds test scripts for running a nightly Jenkins job to
run tests in a non-DRS environment. This change runs the
Group19-DRS-Disabled suite against a pre-deployed, static vCenter
ELM testbed. The corresponding secrets file will be committed to
the internal repo.

Future changes will include adding more tests and using a testbed
deployed with a service account.

Fixes vmware#7662

(cherry picked from commit fe040e0)
zjs pushed a commit that referenced this pull request Jul 27, 2018
This commit adds test scripts for running a nightly Jenkins job to
run tests in a non-DRS environment. This change runs the
Group19-DRS-Disabled suite against a pre-deployed, static vCenter
ELM testbed. The corresponding secrets file will be committed to
the internal repo.

Future changes will include adding more tests and using a testbed
deployed with a service account.

Fixes #7662

(cherry picked from commit fe040e0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants