FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update FTL devices to run tests by sunmou99 · Pull Request #757 · firebase/firebase-cpp-sdk · GitHub

Update FTL devices to run tests - #757

Merged
jonsimantov merged 1 commit into
mainfrom
bugfix/update-ftl-devices
Nov 30, 2021
Merged

Update FTL devices to run tests#757
jonsimantov merged 1 commit into
mainfrom
bugfix/update-ftl-devices

Conversation

sunmou99 commented Nov 23, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Current android-latest devices not responding. https://github.com/firebase/firebase-cpp-sdk/runs/4298317233?check_suite_focus=true
Update FTL *-latest devices to the latest. Also, updated the android-target device.


Testing

Tested manually: https://github.com/firebase/firebase-cpp-sdk/actions/runs/1495928800
Also, tested with label: see comment below.


Type of Change

Place an x the applicable box:

  • Bug fix. Add the issue # below if applicable.
  • New feature. A non-breaking change which adds functionality.
  • Other, such as a build process or documentation change.

google-cla Bot added the cla: yes label Nov 23, 2021
sunmou99 added the tests-requested: full Trigger a FULL set of integration tests (uses expanded test matrix). label Nov 23, 2021
github-actions Bot added tests: in-progress This PR's integration tests are in progress. and removed tests-requested: full Trigger a FULL set of integration tests (uses expanded test matrix). labels Nov 23, 2021

github-actions Bot commented Nov 23, 2021
edited
Loading

Copy link
Copy Markdown

❌  Integration test FAILED

Requested by @jonsimantov on commit d878458
Last updated: Tue Nov 30 15:20 PST 2021
View integration test log & download artifacts

Failures Configs
firestore [TEST] [ERROR] [Android] [ubuntu] [emulator_target]
[TEST] [ERROR] [Android] [windows, macos] [android_target]

Add flaky tests to go/fpl-cpp-flake-tracker

github-actions Bot added the tests: failed This PR's integration tests failed. label Nov 23, 2021
firebase-workflow-trigger Bot removed the tests: in-progress This PR's integration tests are in progress. label Nov 23, 2021
sunmou99 removed the tests: failed This PR's integration tests failed. label Nov 30, 2021
sunmou99 enabled auto-merge (squash) November 30, 2021 19:58
jonsimantov disabled auto-merge November 30, 2021 20:15
jonsimantov merged commit d878458 into main Nov 30, 2021
github-actions Bot added tests: in-progress This PR's integration tests are in progress. tests: failed This PR's integration tests failed. labels Nov 30, 2021
firebase-workflow-trigger Bot removed the tests: in-progress This PR's integration tests are in progress. label Nov 30, 2021
jonsimantov added a commit that referenced this pull request Dec 4, 2021
* Fix link in readme (#768)

* update test devices (#757)

* Timeout and Retry FTL failures and mark flakiness

* Update C++ SDK version to 8.8.0.

* Add Analytics header update to iOS SDK update. (#775)

* Add Analytics header update to iOS SDK update.

* Update Android dependencies - Wed Dec 01 2021 (#773)

### Android

- com.google.firebase.firebase_database → 20.0.3
- com.google.firebase.firebase_perf → 20.0.4

> Created by [Update Android and iOS dependencies workflow](https://github.com/firebase/firebase-cpp-sdk/actions/runs/1527401903).

Co-authored-by: firebase-workflow-trigger-bot <firebase-workflow-trigger-bot@google.com>

* Fix typo in log output.

* Change Xcode version from 12.2 to 12.4 to ensure GitHub support. (#774)

* Change Xcode version from 12.2 to 12.4 to ensure GitHub support.

* Fix integration tests workflow to respect xcode_version in matrix.

* Add extra newline to header comment.

* Update iOS dependencies - Thu Dec 02 2021 (#779)

* Update iOS dependencies - Thu Dec 02 2021

### iOS

- Firebase/Analytics → 8.10.0
- Firebase/Auth → 8.10.0
- Firebase/Core → 8.10.0
- Firebase/Crashlytics → 8.10.0
- Firebase/Database → 8.10.0
- Firebase/DynamicLinks → 8.10.0
- Firebase/Firestore → 8.10.0
- Firebase/Functions → 8.10.0
- Firebase/Installations → 8.10.0
- Firebase/Messaging → 8.10.0
- Firebase/RemoteConfig → 8.10.0
- Firebase/Storage → 8.10.0

> Created by [Update Android and iOS dependencies workflow](https://github.com/firebase/firebase-cpp-sdk/actions/runs/1528664697).

* Add missing copyright lines.

* Add Firestore release note.

Co-authored-by: firebase-workflow-trigger-bot <firebase-workflow-trigger-bot@google.com>
Co-authored-by: Jon Simantov <jsimantov@google.com>

* Properly convert milliseconds to seconds (#767)

* Properly convert milliseconds to seconds

Currently, a value expressed in milliseconds is directly assigned to a variable
which is supposed to store seconds and it's later converted to mills again.
Thus, convert properly upon assignment

* Update readme.md

* Change LogDebug to NSLog in app's swizzling code. (#783)

* Change LogDebug to NSLog.

* Change log message.

* Add comment.

* Add copyright line when importing Analytics iOS headers, if missing (#780)

* Add copyright line to iOS headers

* Make copyright conditional to whether the file already has one.

* Also set Xcode version explicitly in Android workflow. (#786)

* Timeout & retry "install integration test deps" step (#785)

* Format code.

Co-authored-by: Борис Рязанцев <88411548+borisssoft@users.noreply.github.com>
Co-authored-by: Mou Sun <69009538+sunmou99@users.noreply.github.com>
Co-authored-by: firebase-workflow-trigger[bot] <80733318+firebase-workflow-trigger[bot]@users.noreply.github.com>
Co-authored-by: firebase-workflow-trigger-bot <firebase-workflow-trigger-bot@google.com>
Co-authored-by: pawelsnk <pawelsnk@o2.pl>
sunmou99 deleted the bugfix/update-ftl-devices branch December 7, 2021 00:22
firebase locked and limited conversation to collaborators Dec 31, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes tests: failed This PR's integration tests failed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL