Skip to content
This repository was archived by the owner on Jun 28, 2024. It is now read-only.

[DNM] dummy: clh: test PR #1099 from kata-containers/packaging#2733

Closed
likebreath wants to merge 1 commit into
kata-containers:masterfrom
likebreath:dummy_test_packging_pr_1099
Closed

[DNM] dummy: clh: test PR #1099 from kata-containers/packaging#2733
likebreath wants to merge 1 commit into
kata-containers:masterfrom
likebreath:dummy_test_packging_pr_1099

Conversation

@likebreath
Copy link
Copy Markdown
Contributor

@likebreath likebreath commented Jul 22, 2020

Again this is a dummy PR to the test repo, and is used to validate the
changes to another PR from the packaging PR related to
cloud-hypervisor. The following linked issue is just a place holder.

Depends-on: github.com/kata-containers/packaging#1099
Depends-on: github.com/kata-containers/runtime#2900

Fixes: #2546

Signed-off-by: Bo Chen chen.bo@intel.com

@likebreath likebreath added the do-not-merge PR has problems or depends on another label Jul 22, 2020
@likebreath likebreath changed the title clh: test PR #1099 from kata-containers/packaging [DNM] dummy: clh: test PR #1099 from kata-containers/packaging Jul 22, 2020
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 0b60037 to 8962f53 Compare July 22, 2020 21:50
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 8962f53 to 9fddc0e Compare July 22, 2020 23:01
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 9fddc0e to 0e125c0 Compare July 28, 2020 02:31
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath
Copy link
Copy Markdown
Contributor Author

Interesting CI failure:

19:42:18 Report Summary:
19:42:18 +-----+----------------------+---------------------+---------------+------+-----+-----+-----+-----+-----+-----+
19:42:18 | P/F |         NAME         |         FLR         |     MEAN      | CEIL | GAP | MIN | MAX | RNG | COV | ITS |
19:42:18 +-----+----------------------+---------------------+---------------+------+-----+-----+-----+-----+-----+-----+
19:42:18 | *F* | boot-times           | Failed to load JSON | exit status 2 |      |     |     |     |     |     |     |
19:42:18 | *F* | memory-footprint     | Failed to load JSON | exit status 2 |      |     |     |     |     |     |     |
19:42:18 | *F* | memory-footprint-ksm | Failed to load JSON | exit status 2 |      |     |     |     |     |     |     |
19:42:18 | *F* | blogbench            | Failed to load JSON | exit status 2 |      |     |     |     |     |     |     |
19:42:18 | *F* | blogbench            | Failed to load JSON | exit status 2 |      |     |     |     |     |     |     |
19:42:18 +-----+----------------------+---------------------+---------------+------+-----+-----+-----+-----+-----+-----+

Restart both CI jobs:
/test-clh-docker
/test-clh-metrics

@likebreath
Copy link
Copy Markdown
Contributor Author

The number is here. Surprisingly, the metrics CI is NOT failing on max memory footprint (10:32:27 time="2020-07-30T17:32:27Z" level=debug msg=" Check maxval (298119.885000 < 269135.350000)"), but it is failing on minimum memory footprint (10:32:27 time="2020-07-30T17:32:27Z" level=warning msg="Failed Minval ( 269727.515000(minimal mean expected) > 269135.350000(minimal mean on dataset)) for [memory-footprint]"). The same as we observed on the other PRs.

It looks like we will need to update the metrics CI so that it won't fail on minmum memory footprint (while keep reporting as a warning if we want this information). @amshinde @jcvenegas @GabyCT

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 0e125c0 to 28f93ca Compare August 13, 2020 17:15
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh-metrics

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 28f93ca to 7f799ee Compare August 13, 2020 23:36
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 7f799ee to b86098d Compare August 14, 2020 16:43
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh-metrics

@likebreath
Copy link
Copy Markdown
Contributor Author

Failed w/ 11:35:00 time="2020-08-14T18:35:00Z" level=warning msg="Failed Minval ( 269742.458500(minimal mean expected) > 269741.400000(minimal mean on dataset)) for [memory-footprint-ksm]". Log is here: http://jenkins.katacontainers.io/job/kata-metrics-tests-ubuntu-clh-18-04-PR/85/console

11:35:00 Report Summary:
11:35:00 +-----+----------------------+-------+--------+--------+-------+--------+--------+-------+------+-----+
11:35:00 | P/F |         NAME         |  FLR  |  MEAN  |  CEIL  |  GAP  |  MIN   |  MAX   |  RNG  | COV  | ITS |
11:35:00 +-----+----------------------+-------+--------+--------+-------+--------+--------+-------+------+-----+
11:35:00 | P   | boot-times           | 83.3% | 90.7%  | 116.7% | 33.3% | 82.3%  | 101.1% | 22.9% | 5.2% |  20 |
11:35:00 | P   | memory-footprint     | 95.0% | 95.0%  | 105.0% | 10.0% | 95.0%  | 95.0%  | 0.0%  | 0.0% |   1 |
11:35:00 | *F* | memory-footprint-ksm | 95.0% | 95.0%  | 105.0% | 10.0% | 95.0%  | 95.0%  | 0.0%  | 0.0% |   1 |
11:35:00 | P   | blogbench            | 80.0% | 112.1% | 120.0% | 40.0% | 112.1% | 112.1% | 0.0%  | 0.0% |   1 |
11:35:00 | P   | blogbench            | 80.0% | 105.4% | 120.0% | 40.0% | 105.4% | 105.4% | 0.0%  | 0.0% |   1 |
11:35:00 +-----+----------------------+-------+--------+--------+-------+--------+--------+-------+------+-----+

Restart the metrics CI as the difference of failed check is very small (e.g. 269742.458500(minimal mean expected) > 269741.400000(minimal mean on dataset)).
/test-clh-metrics

@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath
Copy link
Copy Markdown
Contributor Author

All CI jobs related to CLH are passing after the fix to the metrics CI, expect the fedora one (which is a known failure on the setup of docker/podman). I am restarting the CI jobs one more time again to test the packaging PR that is now rebased on its master branch.

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 46f219c to 8d777e0 Compare August 25, 2020 17:21
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from 8d777e0 to f8dc793 Compare August 25, 2020 21:05
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

@likebreath likebreath force-pushed the dummy_test_packging_pr_1099 branch from f8dc793 to 6c45a15 Compare August 25, 2020 21:16
@likebreath
Copy link
Copy Markdown
Contributor Author

/test-clh

Again this is a dummy PR to the test repo, and is used to validate the
changes to another PR from the packaging PR related to
cloud-hypervisor. The following linked issue is just a place holder.

Depends-on: github.com/kata-containers/packaging#1099
Depends-on: github.com/kata-containers/runtime#2900

Fixes: kata-containers#2546

Signed-off-by: Bo Chen <chen.bo@intel.com>
@likebreath
Copy link
Copy Markdown
Contributor Author

All CI jobs are passing w/ seccomp option disabled. Restart the CIs to see how stable they are.
/test-clh

@likebreath
Copy link
Copy Markdown
Contributor Author

For the second time, all CI jobs are passing w/ seccomp option disabled. The only exception is I needed to the CI job jenkins-ci-ubuntu-1804-cloud-hypervisor-minimal (log here) was failing on sudo -E PATH="$PATH" bash -c "make conformance", which passed after restarting the CI.

@likebreath
Copy link
Copy Markdown
Contributor Author

Restarting the CI jobs on CLH the last/third time.
/test-clh

@likebreath
Copy link
Copy Markdown
Contributor Author

All CI jobs related to cloud-hypervisor are passing 3 times in a roll w/ the static build PR, which proves the stability of using static build of clh. Closing the dummy test PR now.

@likebreath likebreath closed this Aug 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

do-not-merge PR has problems or depends on another

Projects

None yet

Development

Successfully merging this pull request may close these issues.

clh: Enable crio testing

1 participant