Skip to content

Conversation

dconeybe
Copy link
Contributor

I had originally thought that the issue was specific to Android, because the bug only surfaced on Android; however, it indeed affects all platforms.

I had originally thought that the issue was specific to Android, because the bug only surfaced on Android; however, it indeed affects all platforms.
@dconeybe dconeybe merged commit 29089a7 into main Nov 22, 2021
@jonsimantov jonsimantov deleted the dconeybe/FixReleaseNotesEntryFrom747 branch November 22, 2021 21:53
@github-actions github-actions bot added the tests: in-progress This PR's integration tests are in progress. label Nov 22, 2021
@github-actions
Copy link

github-actions bot commented Nov 22, 2021

❌  Integration test FAILED

Requested by @dconeybe on commit 29089a7
Last updated: Mon Nov 22 16:26 PST 2021
View integration test log & download artifacts

Failures Configs
firestore [TEST] [FAILURE] [Android] [windows] [android_target]
(2 failed tests)  FirestoreIntegrationTest.TestCanDeleteAFieldWithAnUpdate
  QueryTest.TestQueriesCanUseNotEqualFiltersWithObject
messaging [TEST] [ERROR] [Android] [macos] [android_target]

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

@github-actions github-actions bot added the tests: failed This PR's integration tests failed. label Nov 23, 2021
@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Nov 23, 2021
DellaBitta added a commit that referenced this pull request Dec 1, 2021
* Fix test on emulator workflow failures (#734)

* If simulator  install ios app failed, reset simulator and try again (#733)

* Trigger workflow move github api cod to github.py (#746)

* Fix a data race that could manifest as null pointer dereference in FutureBase::Release() (#747)

* Cancel callbacks for messaging (#745)

* Cancel callbacks for messaging

util::Terminate is referenced counted som when there ar more APIs than messaging active the callbacks will not be canceled until later and still cause a NULL ref due to the FutureData being destroyed now.

* Cancel callback earlier

* Update readme

* Remove "Android" tag from the release notes entry for #747 (#749)

* Remove calls to LogInfo, LogError, LogDebug during obj-c +load. (#706)

* Remove calls to LogInfo, LogError, LogDebug during obj-c +load.

This could be causing an issue in C++ as global class constructors have not yet been run.

* Add Objective-C/C++ and Java to code formatter script; format those files. (#755)

* Allow format_code to format .m/.mm files; clang-format already knows how.

* Run format_code.py on all objective-c/objective-c++ files.

* Add Java file extensions to format_code.py

* Format all Java source files.

* Remove check for objc header, as they are now supported.

* Format objective-c .h files.

* Don't let lint comment on line length any more; code formatting will report that.

* Messaging crash during initialization (#760)

* Messaging crash during initialization

* Update readme

* Don't redeclare inherited state in CredentialsProviderDesktop (#731)

* Reduce disk space usage when packaging the built SDK (#763)

Remove intermediate build files during desktop packaging step.

This should reduce the disk space usage, as those files (*.o and *.obj) are not required when merging libraries.

* Workaround for Linux x86 build:  downgrade libraries on GitHub runners (#764)

When installing 32-bit Linux dependencies on GitHub runners, downgrade libpcre2-8-0 to an earlier version to ensure compatibility with the i386 version of the package. This is something that should be fixed in a subsequent Ubuntu release and so is a temporary workaround.

This also adds checks to the various prerequisite commands run by build_desktop.py, which was previously just silently ignoring errors (making this much harder to track down). Now it will error out as soon as a command fails.

Co-authored-by: Mou Sun <[email protected]>
Co-authored-by: Denver Coneybeare <[email protected]>
Co-authored-by: Tobias Barendt <[email protected]>
Co-authored-by: Jon Simantov <[email protected]>
Co-authored-by: Sebastian Schmidt <[email protected]>
@firebase firebase locked and limited conversation to collaborators Dec 23, 2021
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.

2 participants