Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tests-only] [full-ci] Bump CORE_COMMITID for tests #3728

Merged
merged 1 commit into from
May 7, 2022

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented May 7, 2022

Description

Gets test changes from owncloud/core#40057 running in CI.

Related Issue

owncloud/QA#738

How Has This Been Tested?

CI

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • 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:

@phil-davis
Copy link
Contributor Author

phil-davis commented May 7, 2022

https://drone.owncloud.com/owncloud/ocis/11554/37/5

cs3ApiTests-ocis failed - that should be unrelated to the core commit id.

--- Failed steps:

  Scenario: Change treesize of personal home and for only one subtree # /var/lib/cs3api-validator/etag-propagation.feature:60
    And for user "admin" the treesize of the resource with the alias "a-folder" should be 8 # /var/lib/cs3api-validator/etag-propagation.feature:66
      Error: 
	Error Trace:	assert.go:13
	            				steps.go:439
	            				value.go:556
	            				value.go:339
	            				stepdef.go:182
	            				suite.go:185
	            				suite.go:393
	            				suite.go:465
	            				run.go:119
	            				asm_amd64.s:1581
	Error:      	Not equal: 
	            	expected: 0x4
	            	actual  : 0x8

12 scenarios (11 passed, 1 failed)
123 steps (120 passed, 1 failed, 2 skipped)

I will restart CI.

@phil-davis
Copy link
Contributor Author

I am offline the rest of the weekend, so anyone feel free to adjust this PR, push, merge...

@sonarcloud
Copy link

sonarcloud bot commented May 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@micbar micbar merged commit bae3834 into master May 7, 2022
@delete-merged-branch delete-merged-branch bot deleted the bump-commit-id-20220507 branch May 7, 2022 13:38
ownclouders pushed a commit that referenced this pull request May 7, 2022
Merge: 031b326 13f5ab0
Author: Michael Barz <mbarz@owncloud.com>
Date:   Sat May 7 15:38:52 2022 +0200

    Merge pull request #3728 from owncloud/bump-commit-id-20220507

    [tests-only] [full-ci] Bump CORE_COMMITID for tests
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