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

make CRI test logs persistent #196

Merged
merged 2 commits into from
Mar 28, 2021
Merged

make CRI test logs persistent #196

merged 2 commits into from
Mar 28, 2021

Conversation

shyamjesal
Copy link
Collaborator

@shyamjesal shyamjesal commented Mar 26, 2021 •
edited by ustiugov
Loading

  • CRI test
  • Minio test
  • Stock containerd test
Run echo $HOSTNAME
cri-test-github-runner-iccluster128-1-control-plane
691114828

The logs are stored on iccluster128 at /tmp/ctrd-logs/691114828/

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

@shyamjesal shyamjesal requested a review from ustiugov March 26, 2021 09:39
ustiugov
ustiugov reviewed Mar 26, 2021
View reviewed changes
cri/Makefile Outdated
test:
sudo mkdir -p -m777 -p $(CTRDLOGDIR)
sudo containerd 1>$(CTRDLOGDIR)/ctrd.out 2>$(CTRDLOGDIR)/ctrd.err &
sleep 1s
sudo /usr/local/bin/firecracker-containerd --config /etc/firecracker-containerd/config.toml 1>$(CTRDLOGDIR)/fccd.out 2>$(CTRDLOGDIR)/fccd.err &
sleep 1s
sudo ./../vhive 1>$(CTRDLOGDIR)/orch.out 2>$(CTRDLOGDIR)/orch.err &
sudo ./../vhive -d 1>$(CTRDLOGDIR)/orch.out 2>$(CTRDLOGDIR)/orch.err &
Copy link
Member

Choose a reason for hiding this comment

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

must be -dbg if I remember correctly

ustiugov
ustiugov reviewed Mar 26, 2021
View reviewed changes
cri/Makefile Show resolved Hide resolved
@shyamjesal shyamjesal force-pushed the CRI_test_log_collection branch from 2b9439a to 83b192d Compare March 26, 2021 18:29
@ustiugov ustiugov changed the title make CRI test logs persistant make CRI test logs persistent Mar 26, 2021
@ustiugov ustiugov added the enhancement New feature or request label Mar 26, 2021
@shyamjesal shyamjesal force-pushed the CRI_test_log_collection branch from 83b192d to 82bfe68 Compare March 26, 2021 19:56
ustiugov
ustiugov previously approved these changes Mar 26, 2021
View reviewed changes
@ustiugov ustiugov enabled auto-merge (rebase) March 26, 2021 20:48
Made CRI test logs persistent.
71349a4 
Added hostname labels to the runner names.

Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@ustiugov ustiugov force-pushed the CRI_test_log_collection branch from 82bfe68 to 71349a4 Compare March 28, 2021 12:00
@shyamjesal
start_runners.sh refactor
a6d337f 
Signed-off-by: Shyam Jesal <s.jesalpura@gmail.com>
@ustiugov ustiugov merged commit 4c6dd04 into master Mar 28, 2021
@ustiugov ustiugov deleted the CRI_test_log_collection branch March 28, 2021 17:23
HermioneKT added a commit that referenced this pull request Jan 31, 2024
@HermioneKT
# This is a combination of 252 commits.
3c6919f 
parent 6674807
author HermioneKT <hermionegrangerkt@gmail.com> 1706694164 +0800
committer HermioneKT <hermionegrangerkt@gmail.com> 1706694164 +0800

test

# This is the commit message #157:

test

# This is the commit message #159:

test

# This is the commit message #160:

test

# This is the commit message #161:

test

# This is the commit message #162:

test

# This is the commit message #163:

test

# This is the commit message #164:

tesT

# This is the commit message #165:

test

# This is the commit message #166:

test

# This is the commit message #167:

test

# This is the commit message #168:

test

# This is the commit message #169:

test

# This is the commit message #170:

test

# This is the commit message #171:

Test

# This is the commit message #172:

test

# This is the commit message #173:

test

# This is the commit message #174:

test

# This is the commit message #175:

test

# This is the commit message #176:

test

# This is the commit message #177:

test

# This is the commit message #178:

test

# This is the commit message #179:

test

# This is the commit message #180:

test

# This is the commit message #181:

test

# This is the commit message #182:

test

# This is the commit message #183:

test

# This is the commit message #184:

test

# This is the commit message #185:

test

# This is the commit message #186:

test

# This is the commit message #187:

test

# This is the commit message #188:

test

# This is the commit message #189:

test

# This is the commit message #190:

Test

# This is the commit message #191:

Test

# This is the commit message #192:

test

# This is the commit message #193:

Test

# This is the commit message #194:

test

# This is the commit message #195:

test

# This is the commit message #196:

test

# This is the commit message #197:

test

# This is the commit message #198:

test

# This is the commit message #199:

Test

# This is the commit message #200:

test

# This is the commit message #201:

test

# This is the commit message #202:

Test

# This is the commit message #203:

test

# This is the commit message #204:

test

# This is the commit message #205:

test

# This is the commit message #206:

test

# This is the commit message #207:

test

# This is the commit message #208:

test

# This is the commit message #209:

test

# This is the commit message #210:

test

# This is the commit message #211:

Test

# This is the commit message #212:

test

# This is the commit message #213:

Test

# This is the commit message #214:

Test

# This is the commit message #215:

Test

# This is the commit message #216:

test

# This is the commit message #217:

Test

# This is the commit message #218:

test

# This is the commit message #219:

test

# This is the commit message #220:

test

# This is the commit message #221:

test

# This is the commit message #222:

test

# This is the commit message #223:

test

# This is the commit message #224:

test

# This is the commit message #225:

test

# This is the commit message #226:

test

# This is the commit message #227:

test

# This is the commit message #228:

test

# This is the commit message #229:

Test

# This is the commit message #230:

test

# This is the commit message #231:

test

# This is the commit message #232:

test

# This is the commit message #233:

test

# This is the commit message #234:

Test

# This is the commit message #235:

test

# This is the commit message #236:

test

# This is the commit message #237:

test

# This is the commit message #238:

test

# This is the commit message #239:

test

# This is the commit message #240:

test

# This is the commit message #241:

test

# This is the commit message #242:

test

# This is the commit message #243:

test

# This is the commit message #244:

test

# This is the commit message #245:

test

# This is the commit message #246:

test

# This is the commit message #247:

test

# This is the commit message #248:

test

# This is the commit message #249:

test

# This is the commit message #250:

test

# This is the commit message #251:

test

# This is the commit message #252:

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

@shyamjesal shyamjesal

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

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