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.

Sign up for GitHub

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

Jump to bottom

Containerize cri tests #107

Merged
merged 18 commits into from
Feb 15, 2021
Merged

Containerize cri tests #107

merged 18 commits into from
Feb 15, 2021

Conversation

shyamjesal
Copy link
Collaborator

@shyamjesal shyamjesal commented Feb 4, 2021 •
edited
Loading

  • add dockerfile to create cri test runner container
  • add cri runner container setup script
  • update cri_test makefile
  • removed rnnserving from the test as it fails to deploy on cri test container

To start the runner:

  1. clone the repo
  2. run scripts/github_runner/setup_runner_host.sh
  3. start runners using scripts/github_runner/start_runners.sh <number> <access_key> cri

Signed-off-by: Shyam Jesalpura s.jesalpura@gmail.com

@shyamjesal shyamjesal force-pushed the master branch 20 times, most recently from 45cc766 to 9250cf8 Compare February 9, 2021 14:38
ustiugov
ustiugov requested changes Feb 9, 2021
View reviewed changes
cri/Makefile Outdated Show resolved Hide resolved
ustiugov
ustiugov requested changes Feb 15, 2021
View reviewed changes
Copy link
Member

@ustiugov ustiugov left a comment

Choose a reason for hiding this comment

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

please move the copyrights above /bin/bash in all scripts

docs/developers_guide.md Outdated Show resolved Hide resolved
scripts/github_runner/Dockerfile.cri_dev_env Show resolved Hide resolved
scripts/github_runner/Dockerfile.github_runner_cri Show resolved Hide resolved
scripts/github_runner/setup_cri_dev_env.sh Outdated Show resolved Hide resolved
scripts/github_runner/setup_runner_cri.sh Outdated Show resolved Hide resolved
scripts/github_runner/setup_runner_host.sh Show resolved Hide resolved
scripts/github_runner/start_runners.sh Show resolved Hide resolved
@shyamjesal
Containerize cri tests
159ebbf 
add dockerfile to create cri runner container
add cri runner container setup script
update cri_test makefile
removed rnnserving from the test as it fails to deploy on cri test container

Signed-off-by: Shyam Jesalpura <s.jesalpura@gmail.com>
@shyamjesal
create cri test github action
70ef8bd 
Signed-off-by: Shyam Jesalpura <s.jesalpura@gmail.com>
@shyamjesal
add cri runner type in start_runners.sh
d73c85b 
Signed-off-by: Shyam Jesalpura <s.jesalpura@gmail.com>
@shyamjesal
add cri runners to setup_runner_cri.sh
a2e8b7f 
Signed-off-by: Shyam Jesalpura <s.jesalpura@gmail.com>
@shyamjesal
moved apt installations to cri dockerfile
5b73f91 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
add cri cleaning script
b849de2 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
rename setup_containerd.sh to setup_firecracker_containerd.sh
5c15358 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
remove travis test
e85cd5b 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
enable verbose output and add comments
2335bee 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
restored btrfs-tools package
5264c87 
btrfs-tools was replaced for testing. Now restored

Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
fix runner startup scripts for cri type runners
b3eb94d 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
add tiny delay in Makefile
1beaa54 
calls to containerd and firecracker-containerd were async. Added 1s
delay between them to make sure they are up and running in cri-test
Makefile

Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
update ease-lab/kind branch
1198dc3 
start_runners.sh now pulls from custom_docker_params_for_vHive branch
from ease-lab/kind

Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
create separate cri developer environment container
91345ea 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
bugfixes and refactor
e6746b3 
fix cri cleaning script to end gracefully when bridges not found
add integration test label to all test files
update start_runners.sh to include repo URL as an argument

Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
update docs
2f454d0 
update developers guide to include kind developer container
add CI guide

Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
minor fixes
d86f352 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@shyamjesal
update wordlist + MIT license added/fixed + added changelog
d4aa6c8 
added repo, cd, integ, cri, ps, params, dev, env and vhiveease to wordlist.

Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
ustiugov
ustiugov approved these changes Feb 15, 2021
View reviewed changes
Copy link
Member

@ustiugov ustiugov left a comment

Choose a reason for hiding this comment

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

please merge when ready

@ustiugov ustiugov merged commit 15ebaba into vhive-serverless:master Feb 15, 2021
HermioneKT pushed a commit that referenced this pull request Jan 31, 2024
@KTCyber @HermioneKT
# This is a combination of 154 commits.
6674807 
# This is the 1st commit message:

refactor

# This is the commit message #2:

modify powerNodeSelector

# This is the commit message #3:

modify powerNodeSelector

# This is the commit message #4:

modify powerNodeSelector

# This is the commit message #5:

modify namespace

# This is the commit message #6:

modify namespace

# This is the commit message #7:

add namespace

# This is the commit message #8:

init power manager script

# This is the commit message #9:

add go code

# This is the commit message #10:

test

# This is the commit message #11:

test

# This is the commit message #12:

test

# This is the commit message #13:

test

# This is the commit message #14:

go.sum

# This is the commit message #15:

go sum

# This is the commit message #16:

go mod tidy

# This is the commit message #17:

test

# This is the commit message #18:

test

# This is the commit message #19:

test

# This is the commit message #20:

test

# This is the commit message #21:

test

# This is the commit message #22:

test

# This is the commit message #23:

test

# This is the commit message #24:

test

# This is the commit message #25:

test

# This is the commit message #26:

test

# This is the commit message #27:

test

# This is the commit message #28:

test

# This is the commit message #29:

print output

# This is the commit message #30:

test

# This is the commit message #31:

test

# This is the commit message #32:

test

# This is the commit message #33:

test

# This is the commit message #34:

test

# This is the commit message #35:

test

# This is the commit message #36:

test

# This is the commit message #37:

test

# This is the commit message #38:

test

# This is the commit message #39:

test

# This is the commit message #40:

refactor

# This is the commit message #41:

change val

# This is the commit message #42:

test

# This is the commit message #43:

test

# This is the commit message #44:

test

# This is the commit message #45:

exp1

# This is the commit message #46:

add url

# This is the commit message #47:

test

# This is the commit message #48:

test

# This is the commit message #49:

test auth

# This is the commit message #50:

test

# This is the commit message #51:

test

# This is the commit message #52:

test

# This is the commit message #53:

test

# This is the commit message #54:

test

# This is the commit message #55:

fix bug

# This is the commit message #56:

remove log

# This is the commit message #57:

test

# This is the commit message #58:

test

# This is the commit message #59:

test

# This is the commit message #60:

fix loop

# This is the commit message #61:

increase invocation

# This is the commit message #62:

test

# This is the commit message #63:

test

# This is the commit message #64:

test

# This is the commit message #65:

test

# This is the commit message #66:

test

# This is the commit message #67:

test

# This is the commit message #68:

test

# This is the commit message #69:

Test

# This is the commit message #70:

test

# This is the commit message #71:

test

# This is the commit message #72:

test

# This is the commit message #73:

test

# This is the commit message #74:

test

# This is the commit message #75:

test

# This is the commit message #76:

test

# This is the commit message #77:

test

# This is the commit message #78:

test

# This is the commit message #79:

test

# This is the commit message #80:

test

# This is the commit message #81:

test

# This is the commit message #82:

test

# This is the commit message #83:

test

# This is the commit message #84:

test

# This is the commit message #85:

test

# This is the commit message #86:

test

# This is the commit message #87:

test

# This is the commit message #88:

test

# This is the commit message #89:

test

# This is the commit message #90:

test1

# This is the commit message #91:

test

# This is the commit message #92:

test

# This is the commit message #93:

test

# This is the commit message #94:

test

# This is the commit message #95:

test

# This is the commit message #96:

test

# This is the commit message #97:

test

# This is the commit message #98:

test

# This is the commit message #99:

test

# This is the commit message #100:

test

# This is the commit message #101:

test

# This is the commit message #102:

test

# This is the commit message #103:

test

# This is the commit message #104:

test

# This is the commit message #105:

test

# This is the commit message #106:

test

# This is the commit message #107:

test

# This is the commit message #108:

test

# This is the commit message #109:

test1

# This is the commit message #110:

test

# This is the commit message #111:

test

# This is the commit message #112:

Test

# This is the commit message #113:

test

# This is the commit message #114:

test

# This is the commit message #115:

test

# This is the commit message #116:

test

# This is the commit message #117:

test

# This is the commit message #118:

test

# This is the commit message #119:

Test

# This is the commit message #120:

Test

# This is the commit message #121:

test

# This is the commit message #122:

test

# This is the commit message #123:

Test

# This is the commit message #124:

test

# This is the commit message #125:

test

# This is the commit message #126:

test

# This is the commit message #127:

test

# This is the commit message #128:

test

# This is the commit message #129:

test

# This is the commit message #130:

test

# This is the commit message #131:

test

# This is the commit message #132:

test

# This is the commit message #133:

test

# This is the commit message #134:

test

# This is the commit message #135:

test

# This is the commit message #136:

test

# This is the commit message #137:

test

# This is the commit message #138:

test

# This is the commit message #139:

test

# This is the commit message #140:

test

# This is the commit message #141:

test

# This is the commit message #142:

test

# This is the commit message #143:

Test

# This is the commit message #144:

Test

# This is the commit message #145:

Test

# This is the commit message #146:

Test

# This is the commit message #147:

test

# This is the commit message #148:

test

# This is the commit message #149:

test

# This is the commit message #150:

test

# This is the commit message #151:

test

# This is the commit message #152:

test

# This is the commit message #153:

test

# This is the commit message #154:

Test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ustiugov ustiugov ustiugov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shyamjesal @ustiugov