google / saxml

Apache License 2.0
122 stars 29 forks source link

Can't run admin_config: build broken? #8

Open destijl opened 1 year ago

destijl commented 1 year ago

Hi, I was following the instructions in the readme and got blocked at the admin_config step.

gcastle@sax-admin:~/saxml$ git log -1
commit 7122f1e502fa5c6b012fa2dfa9ed4fa63c939f81 (HEAD -> main, origin/main, origin/HEAD)
Author: Daniel Freeman <cdfreeman@google.com>
Date:   Mon Aug 14 16:16:37 2023 -0700

    adds support for persistent compilation cache

    PiperOrigin-RevId: 556942452
    Change-Id: I7342061514e57fdea41b2bcb1eb82cfecd0389c3

gcastle@sax-admin:~/saxml$ bazel run saxml/bin:admin_config --   --sax_cell=/sax/test   --sax_root=gs://${GSBUCKET}/sax-root   --fs_root=gs://${GSBUCKET}/sax-fs-root   --alsologtostderr
ERROR: /home/gcastle_google_com/saxml/saxml/protobuf/BUILD:387:14: //saxml/protobuf:multimodal_proto: no such attribute 'use_java_stubby_library' in 'proto_library' rule
ERROR: /home/gcastle_google_com/saxml/saxml/common/BUILD:136:11: Target '//saxml/protobuf:admin_go_proto_grpc' contains an error and its package is in error and referenced by '//saxml/common:config'
ERROR: Analysis of target '//saxml/bin:admin_config' failed; build aborted: 
INFO: Elapsed time: 0.185s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded, 0 targets configured)
ERROR: Build failed. Not running target
zhihaoshan-google commented 1 year ago

Hi Greg, Could you try building with new HEAD (72697a00e09ecf117088943aaa9473ac7a6ccd98)? It should be fixed.