Skip to content

Revert "qa/workunits/cls: add executable bit to script"#40857

Closed
neha-ojha wants to merge 1 commit intoceph:masterfrom
neha-ojha:wip-50339
Closed

Revert "qa/workunits/cls: add executable bit to script"#40857
neha-ojha wants to merge 1 commit intoceph:masterfrom
neha-ojha:wip-50339

Conversation

@neha-ojha
Copy link
Copy Markdown
Member

This reverts commit 5242487.

test_cls_cas tests need to be fixed before running them consistently.

Fixes: https://tracker.ceph.com/issues/50339
Signed-off-by: Neha Ojha nojha@redhat.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

This reverts commit 5242487.

test_cls_cas tests need to be fixed before running them consistently.

Fixes: https://tracker.ceph.com/issues/50339
Signed-off-by: Neha Ojha <nojha@redhat.com>
@neha-ojha
Copy link
Copy Markdown
Member Author

@tchaikov
Copy link
Copy Markdown
Contributor

could you please check if this change breaks src/script/gen-corpus.sh?

@tchaikov
Copy link
Copy Markdown
Contributor

probably #40867 is a better fix.

@neha-ojha
Copy link
Copy Markdown
Member Author

probably #40867 is a better fix.

agreed!

@neha-ojha
Copy link
Copy Markdown
Member Author

Closing in favor of #40867

@neha-ojha neha-ojha closed this Apr 15, 2021
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