Skip to content

Conversation

@shirady
Copy link
Contributor

@shirady shirady commented Sep 23, 2024

Explain the changes

  1. Updated the number of days as a temporary solution.

Issues:

  1. This issue was mentioned as a part of PR CI | Update Commit Hash in Ceph Tests #8271, updating by one commit results in many failed tests, probably due to changes in the s3tests.conf.SAMPLE file and how our script attaches the information to it.
  2. GAP - I opened a Jira ticket (MCGI-254) to investigate it, after updating the commit hash the number of days should be 180 again.
    Here are the logs after updating to commit 54c1488a4365afdbe7748eb31809bbb05fa25fb3 (PR 537)
    ceph_s3:

Sep-23 5:54:01.813 [test_ceph_s3/184] [INFO] CONSOLE:: CEPH TEST SUMMARY: Suite contains 860, ran 389 tests, Passed: 6, Skipped: 6, Failed: 377

nsfs_ceph_s3:

Sep-23 5:48:55.040 [test_ceph_s3/120] [INFO] CONSOLE:: CEPH TEST SUMMARY: Suite contains 860, ran 321 tests, Passed: 5, Skipped: 6, Failed: 310

Testing Instructions:

  1. none (tested through the CI).
  2. If you wish to run it locally:
    make test-cephs3 CONTAINER_PLATFORM=linux/arm64
    make test-nsfs-cephs3 CONTAINER_PLATFORM=linux/arm64
    (I'm using the flag CONTAINER_PLATFORM because I have MacOS M1).
  • Doc added/updated
  • Tests added

@shirady shirady changed the title CI | Update Ceph S3 Tests (Temporary Solution) CI | Update Ceph S3 Tests Days (Temporary Solution) Sep 23, 2024
@shirady shirady marked this pull request as ready for review September 23, 2024 06:09
@shirady shirady self-assigned this Sep 23, 2024
@shirady shirady merged commit a3b5ab8 into noobaa:master Sep 23, 2024
10 checks passed
@shirady shirady deleted the ceph-update-days branch September 23, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants