Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • slapos slapos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 129
    • Merge requests 129
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • slaposslapos
  • Merge requests
  • !1541

rapid-cdn: fix failing test

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Titouan Soulard requested to merge tsoulard/slapos:for-mr-rabbit into master Mar 07, 2024
  • Overview 5
  • Commits 1
  • Pipelines 2
  • Changes 1

logrotate test for the rapid-cdn SR have been failing for quite some time because of a squid.log file created (and henceforth rotated) on any slave instance of the CDN:

Traceback (most recent call last):
    self.assertEqual(
AssertionError: Items in the second set but not the first:
'squid.log_10.0.160.212.20240306.23h57m09s-20240307.00h00m02s.old.xz'

Since we do not want to test this file in that specific test, we instead loosely check the content of the rotated files directory. In other words, we check for the presence of the two files to be tested, but avoid failing if other files exist.

This goes in line with the two following lines of that same test, loosely checking for file absence instead of an empty directory.

/cc @tomo @luke

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: for-mr-rabbit
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7