Skip to content

Added fixes needed for QE Jenkins#415

Open
sanjivanipatrax wants to merge 8 commits into
mainfrom
fix-qe-jenkins-ios
Open

Added fixes needed for QE Jenkins#415
sanjivanipatrax wants to merge 8 commits into
mainfrom
fix-qe-jenkins-ios

Conversation

@sanjivanipatrax

Copy link
Copy Markdown
Contributor

This PR adds in the other platforms to the Jenkinsfiles as well as fixes needed which were previously in qe-jenkins branch.

Features added:

  • Added all platforms to the C Jenkinsfile (in jenkins/pipelines/QE/c/Jenkinsfile).
  • The phone connected to mac-mini-CBL-Android1 node keeps intermittently disconnecting so changed it to use qe-mac-india node.
  • Changed path in iOS Jenkinsfile to fix the git-lfs issue.
  • Added post stage in iOS and .NET Jenkinsfile to get notified on regressions and successes (in jenkins/pipelines/QE/dotnet/Jenkinsfile and jenkins/pipelines/QE/ios/Jenkinsfile).

@sanjivanipatrax sanjivanipatrax self-assigned this May 19, 2026
@sanjivanipatrax sanjivanipatrax requested a review from borrrden May 19, 2026 12:36
steps {
echo "=== Run C Windows Tests"
withCredentials([string(credentialsId: 'mobile-qe', variable: 'KEYCHAIN_PASSWORD')]) {
sh 'security unlock-keychain -p ${KEYCHAIN_PASSWORD} ~/Library/Keychains/login.keychain-db'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not needed for macOS, I think. It's there in iOS to deal with code signing

Comment thread jenkins/pipelines/QE/dotnet/Jenkinsfile Outdated
steps {
echo "=== Run C Windows Tests"
timeout(time: 120, unit: 'MINUTES') {
powershell "jenkins\\pipelines\\QE\\c\\run_test.ps1 -Version ${params.CBL_VERSION} -SgwVersion ${params.SGW_VERSION} -DatasetVersion ${params.TS_DATASET_VERSION}"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The stage windows will fail, can you avoid modifying the part of the c-pipeline for windows, iOS and Android since I've made sure they run as expected. You can add the remaining platforms above or below these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants