quickstart-ios icon indicating copy to clipboard operation
quickstart-ios copied to clipboard

Firestore Auto-update not happening

Open paulb777 opened this issue 4 years ago • 12 comments

How is the auto-update configured? It looks like it's not working for Firestore:

Screen Shot 2020-12-11 at 9 24 56 AM

paulb777 avatar Dec 11 '20 17:12 paulb777

@ulukaya can you take a look at this?

morganchen12 avatar Dec 13 '20 01:12 morganchen12

My script just finds "Podfile"s and updates them. https://github.com/GoogleCloudPlatform/repository-gardener/blob/master/use-latest-deps-ios.sh#L67 Somehow the latest Podfile update stopped it. I'll check what's going on.

ulukaya avatar Dec 13 '20 01:12 ulukaya

I don't see any errors but looks like it got timed out. I updated manually for now, that should make the next iteration work. I'll investigate further if it gets stuck again.

ulukaya avatar Dec 13 '20 02:12 ulukaya

https://github.com/firebase/quickstart-ios/commit/92f82d085c829271c8b2a812cd742e88c160bdae

ulukaya avatar Dec 13 '20 02:12 ulukaya

Reopening since this is still a problem as can be seen from repo's main page's directories' last update messages.

paulb777 avatar Jan 22 '21 17:01 paulb777

The culprit seems to be gRPC-C++:

Installing gRPC-C++ (1.28.2)
[!] /bin/bash -c 
set -e
sed -E -i '' 's;#include <openssl/(.*)>;#if COCOAPODS==1\
  #include <openssl_grpc/\1>\
#else\
  #include <openssl/\1>\
#endif;g' $(find src/core -type f \( -path '*.h' -or -path '*.cc' \) -print | xargs grep -H -c '#include <openssl_grpc/' | grep 0$ | cut -d':' -f1)
    find src/core/ third_party/upb/ -type f \( -name '*.h' -or -name '*.c' -or -name '*.cc' \) -print0 | xargs -0 -L1 sed -E -i'.grpc_back' 's;#include "upb/(.*)";#if COCOAPODS==1\
  #include  "third_party/upb/upb/\1"\
#else\
  #include  "upb/\1"\
#endif;g'
    find src/core/ third_party/upb/ -type f -name '*.grpc_back' -print0 | xargs -0 rm
    find src/core/ src/cpp/ -type f \( -name '*.h' -or -name '*.c' -or -name '*.cc' \) -print0 | xargs -0 -L1 sed -E -i'.grpc_back' 's;#include "(.*).upb.h";#if COCOAPODS==1\
  #include  "src/core/ext/upb-generated/\1.upb.h"\
#else\
  #include  "\1.upb.h"\
#endif;g'
    find src/core/ src/cpp/ -type f -name '*.grpc_back' -print0 | xargs -0 rm

sed: can't read s;#include <openssl/(.*)>;#if COCOAPODS==1\
  #include <openssl_grpc/\1>\
#else\
  #include <openssl/\1>\
#endif;g: No such file or directory

morganchen12 avatar Jun 14 '21 16:06 morganchen12

Maybe related to the wrong sed on the machine? https://github.com/firebase/firebase-ios-sdk/issues/736#issuecomment-362338481

paulb777 avatar Jun 14 '21 16:06 paulb777

Based on this flag (--platform=ubuntu) in the build logs, the kokoro jobs are running on ubuntu. Moving them to macOS will probably fix the issue.

morganchen12 avatar Jun 14 '21 21:06 morganchen12

Moved jobs to macOS in cl/379353433, cl/379513834. Will re-test and see if Firestore works now.

morganchen12 avatar Jun 15 '21 16:06 morganchen12

Kokoro is even more broken now. Going to roll the CLs back while I figure out what in the script is causing failures.

morganchen12 avatar Jun 16 '21 00:06 morganchen12

This has been fixed in the gRPC podspec, and will be fixed in Firestore whenver they update their gRPC dependency.

morganchen12 avatar Sep 10 '21 19:09 morganchen12

Will be fixed in M114.

morganchen12 avatar Mar 28 '22 22:03 morganchen12