X Tutup
Skip to content

[tests-only][full-ci]Skip migration status for app test on OC stable#40129

Merged
phil-davis merged 1 commit intomasterfrom
skipOccMigrationOnlatest
Jun 9, 2022
Merged

[tests-only][full-ci]Skip migration status for app test on OC stable#40129
phil-davis merged 1 commit intomasterfrom
skipOccMigrationOnlatest

Conversation

@SagarGi
Copy link
Member

@SagarGi SagarGi commented Jun 9, 2022

Description

This PR skips the occ migration status for app tests for latest stable OC as it is failing on nightly.
Related issue : owncloud/user_ldap#738

@SagarGi SagarGi added the QA:team label Jun 9, 2022
@SagarGi SagarGi self-assigned this Jun 9, 2022
@SagarGi SagarGi force-pushed the skipOccMigrationOnlatest branch from 35055ee to bb0b1a2 Compare June 9, 2022 09:37
@SagarGi SagarGi requested a review from phil-davis June 9, 2022 09:38
@phil-davis
Copy link
Contributor

Note: now I realize why we have to skip on 10.10.0 and earlier. As well as adding this test, I fixed the code of the command - see PR #40085

But that command code fix did not make it into 10.10.0. So the test passes on current master (daily tarball...), but will fail against 10.10.0 and earlier.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 9, 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

@phil-davis phil-davis merged commit 1834a8f into master Jun 9, 2022
@delete-merged-branch delete-merged-branch bot deleted the skipOccMigrationOnlatest branch June 9, 2022 10:22
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

X Tutup