Skip to content

nexedi / slapos.core

Go to a project
Merge request !175 opened by  

testing/testcase: include partition dump when setup class fail

Sometimes instanciation fail because some service do not start and the "check port listening" promises fail. In this case, it's very important to also have the partition content to see what's in the log files.

{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Jérome Perrin @jerome

    mentioned in merge request slapos!681 (merged)

    mentioned in merge request slapos!681 (merged)

    Toggle commit list
  • Jérome Perrin @jerome

    added 1 commit

    • 62678c9a - testing/testcase: include partition dump when setup class fail

    Compare with previous version

    added 1 commit

    • 62678c9a - testing/testcase: include partition dump when setup class fail

    Compare with previous version

    Toggle commit list
  • Jérome Perrin @jerome

    mentioned in commit fea3bd23

    mentioned in commit fea3bd23

    Toggle commit list
  • Jérome Perrin @jerome

    merged

    merged

    Toggle commit list
  • Jérome Perrin @jerome commented
    Owner

    I did not check test results carefully enough, there's an error in caddy-frontend:

    OSError: [Errno 17] File exists: '/srv/slapgrid/slappart9/srv/testnode/cqg/inst/test0-0/tmp/snapshots/caddy-frontend/setUpClass._cleanup leaked_partitions/inst/etc/supervisord.conf.d'

    I'm looking at this now /cc @luke

  • Jérome Perrin @jerome commented
    Owner

    I think this was not related, slapos!677 (merged) was not correct in case of error.

Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备14008524号