Skip to content

Unskip tests related to files_primary_s3 ceph as the issue has been fixed - #35601

Merged
DeepDiver1975 merged 1 commit into
masterfrom
unskip-primary-s3-tests
Jun 20, 2019
Merged

Unskip tests related to files_primary_s3 ceph as the issue has been fixed#35601
DeepDiver1975 merged 1 commit into
masterfrom
unskip-primary-s3-tests

Conversation

@dpakach

@dpakach dpakach commented Jun 19, 2019

Copy link
Copy Markdown
Contributor

Description

Because of owncloud/files_primary_s3#128 some core tests were skipped in ceph. But not that the issue has been fixed explained here. These tests can be run again in files_primary_s3 with ceph storage.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

Open tasks:

  • Backport (if applicable set "backport-request" label and remove when the backport was done)

@dpakach

dpakach commented Jun 19, 2019

Copy link
Copy Markdown
Contributor Author

Backport #35602

@phil-davis

Copy link
Copy Markdown
Contributor

@DeepDiver1975 @patrickjahns no codecov arrived. Please merge for us.

@phil-davis
phil-davis force-pushed the unskip-primary-s3-tests branch 2 times, most recently from 00fb6cd to 6524563 Compare June 20, 2019 12:08
@phil-davis
phil-davis force-pushed the unskip-primary-s3-tests branch from 6524563 to 7aca120 Compare June 20, 2019 14:19
@DeepDiver1975
DeepDiver1975 merged commit 371499c into master Jun 20, 2019
@delete-merged-branch
delete-merged-branch Bot deleted the unskip-primary-s3-tests branch June 20, 2019 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

files are uploaded to ceph even when wrong checksum given

4 participants