Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add documentation about building docker image #549

Merged
merged 2 commits into from
May 9, 2024

Conversation

0405ysj
Copy link
Collaborator

@0405ysj 0405ysj commented May 8, 2024

No description provided.

@0405ysj 0405ysj requested review from k311093, ser-io and ikicha and removed request for adelva1984, rmuthiah, Databean, jmacnak and jemoreira May 8, 2024 02:26
@0405ysj 0405ysj added the kokoro:run Run e2e tests. label May 8, 2024
@0405ysj 0405ysj force-pushed the docker_documentation branch from 2373d92 to 496cdc5 Compare May 8, 2024 02:29
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label May 8, 2024
README.md Show resolved Hide resolved
@0405ysj 0405ysj force-pushed the docker_documentation branch 2 times, most recently from 26dac85 to 024561d Compare May 8, 2024 06:01
@0405ysj 0405ysj added the kokoro:force-run Trigger a presubmit build unconditionally. label May 8, 2024
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label May 8, 2024
@0405ysj 0405ysj requested a review from ikicha May 8, 2024 06:03
@0405ysj 0405ysj added the kokoro:force-run Trigger a presubmit build unconditionally. label May 8, 2024
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label May 8, 2024
README.md Outdated Show resolved Hide resolved
@0405ysj 0405ysj force-pushed the docker_documentation branch from adf8ea6 to e3347cf Compare May 9, 2024 01:21
@0405ysj 0405ysj requested a review from ser-io May 9, 2024 01:22
@0405ysj 0405ysj added the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
Copy link
Collaborator

@ikicha ikicha left a comment

Choose a reason for hiding this comment

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

Please update our internal guide accordingly because it changes docker image name

@0405ysj 0405ysj added the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
@0405ysj 0405ysj force-pushed the docker_documentation branch from e64c07b to 26cd912 Compare May 9, 2024 01:42
@0405ysj 0405ysj added the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
@0405ysj 0405ysj force-pushed the docker_documentation branch from 26cd912 to 5f109e3 Compare May 9, 2024 04:18
@0405ysj 0405ysj added the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label May 9, 2024
@Nirzak
Copy link

Nirzak commented May 9, 2024

The is not issue reporing section, so I am reporting in here. The docker build option is giving error during building docker images on arm64 instance. here what I did:

ran ./build.sh under arm-server section.

Here's the errors what came

mk-build-deps: Unable to install cuttlefish-common-build-deps-depends at /usr/bin/mk-build-deps line 459.
mk-build-deps: Unable to install all build-dep package

=> ERROR [11/17] RUN apt install -y -f /root/cuttlefish-base_.deb /root/cuttlefish-user_.deb /root/cuttlefish-orch 1.0s

@0405ysj 0405ysj merged commit 4132cc9 into google:main May 9, 2024
6 checks passed
@0405ysj
Copy link
Collaborator Author

0405ysj commented May 9, 2024

Hello Nirzak,

Sorry, but I need more information to help the issue you've faced at.

  • Machine information ran ./build.sh in docker/orchestration(docker/arm-server in the past) : Does the OS equal to Linux? Is the arch in x86_64 or arm64?
  • Which version or commit of this github repo you've used?
  • What was the result for below? Would debian packages including cuttlefish-base be made well?
cd docker/debs-builder-docker
./main.sh

@Nirzak
Copy link

Nirzak commented May 9, 2024

May be I found the error. My arch is arm64 linux. so it actually giving the following error

Correcting dependencies...Starting pkgProblemResolver with broken count: 1
Starting 2 pkgProblemResolver with broken count: 1
Investigating (0) cuttlefish-common-build-deps-depends:arm64 < 0.9.29 @iU K Nb Ib >
Broken cuttlefish-common-build-deps-depends:arm64 Depends on bazel:arm64 < none @un H >
  Removing cuttlefish-common-build-deps-depends:arm64 because I can't find bazel:arm64
Done

May be this one is leading to the other errors. I have checked the codes and here is what I have found

deb [arch=amd64 signed-by=/usr/share/keyrings/bazel-archive-keyring.gpg] https://storage.googleapis.com/bazel-apt stable jdk1.8" on build-hostpkg.sh file

isn't here the arch should be aarch64 for arm64 architecture?

@Nirzak
Copy link

Nirzak commented May 9, 2024

Tried arm64 anf aarch64 on the arch section of bazel repo. but seems like it couldn't find bazel arm64 deb package from repo.

but they actually supports arm64 linux here https://github.com/bazelbuild/bazel/releases/download/7.1.2/bazel-7.1.2-linux-arm64

@Nirzak
Copy link

Nirzak commented May 9, 2024

I have tried to include the following lines in the build-hostpkg.sh file to make it compatible with arm64. but looks like they are not executing or something. I can't see any info about the in the log.

    curl -o bazel https://releases.bazel.build/7.1.2/release/bazel-7.1.2-linux-arm64
    chmod +x bazel
    sudo mv bazel /bin

@0405ysj
Copy link
Collaborator Author

0405ysj commented May 9, 2024

Thanks for the investigation! We're currently looking at the same page, and checked bazel is not installable by apt in aarch64/arm64 architecture.

@jemoreira Could you look at here? Currently cuttlefish-base is not built in arm64, because of lack of bazel.

@ikicha
Copy link
Collaborator

ikicha commented May 9, 2024

@ser-io can we use bazelisk? https://bazel.build/install/bazelisk

@Nirzak
Copy link

Nirzak commented May 9, 2024

The problem is bazelisk is also not available as a APT package. whenever we are trying to install cuttlefish-common-base deb it by default is checking the bazel in the installed packages. so installing by other means will not reflect if we don't ignore the check from dependency checks or cuttlefish.

@Nirzak
Copy link

Nirzak commented May 9, 2024

Please allow us to create issues on this repository. Thanks

I have set up cuttlefish successfully using docker on my amd machine after getting unsuccessful attempts on arm. the docker image was built without any errors. and also the container started without any error. but the problem is the web GUI isn't showing any devices. but almost 40-50 ports are open for devices also I can see network interfaces for them inside the container.. then what is this issue? isn't there should be a device running already inside the container? or do we have to manually download the Android image and cvd package and then run it inside the docker? @0405ysj @ikicha
Thanks

image

@0405ysj 0405ysj deleted the docker_documentation branch July 16, 2024 05:34
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.

5 participants