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

Upgrade from Debian Bullseye to Bookworm for /build #3574

Closed
wants to merge 15 commits into from

Conversation

Kalaiselvi84
Copy link
Contributor

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug

/kind cleanup

/kind documentation
/kind feature
/kind hotfix
/kind release

What this PR does / Why we need it:

Which issue(s) this PR fixes:

Work on #3570

Special notes for your reviewer:

@github-actions github-actions bot added kind/cleanup Refactoring code, fixing up documentation, etc size/S labels Jan 8, 2024
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: da082bb4-0b9a-4aef-9480-75a55f2b6294

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@Kalaiselvi84 Kalaiselvi84 marked this pull request as draft January 8, 2024 20:23
@Kalaiselvi84 Kalaiselvi84 changed the title Bw build Upgrade from Debian Bullseye to Bookworm for /build Jan 8, 2024
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 536e12b5-c1aa-49a5-87b9-959efaf42b20

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@markmandel
Copy link
Collaborator

Looks like it's complaining that python doesn't exist as a package here:

https://github.com/googleforgames/agones/pull/3574/files#diff-686b06cb4b89c224b0df5f76fa2bbc7b5757477074e6917a21b8040a9e6857a4R20

...
3.470 Reading state information...
3.476 Package python is not available, but is referred to by another package.
3.476 This may mean that the package is missing, has been obsoleted, or
3.476 is only available from another source
3.476 However the following packages replace it:
3.476   python-is-python3 2to3
...

I actually don't think we need it explicitly defined in the apt-get install command, so my suggestion would be to drop it entirely, so make that line instead

apt-get install -y build-essential gnupg curl git wget psmisc rsync make bash-completion

And see if that solves the issue.

@Kalaiselvi84
Copy link
Contributor Author

Log of make build-images && make push && make install - https://gist.github.com/Kalaiselvi84/98b1c859563e817f1c58ce2d7bcdff2c

I am not fully convinced about this PR. I think I may have made some mistakes while creating it. Will submit another PR for this.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: b27efbe3-a63a-48f8-8680-e72c7ed61133

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@markmandel
Copy link
Collaborator

I see the same in the CI build

google-cloud-cli-app-engine-python : Depends: python2.7 but it is not installable

I don't think you did anything wrong, but I have a good idea on how to fix this - I think we're tryng to install the old version of the gcloud command line client, and didn't update the apt repositories.

@markmandel
Copy link
Collaborator

I had assumed that gcloud would install the dependency.

Let's try one more thing:

apt-get install -y build-essential gnupg curl git wget psmisc python2.7 rsync make bash-completion

See how that goes if we install python2.7 directly.

@markmandel
Copy link
Collaborator

Huh, this is interesting - seeing the same issue over here: GoogleCloudPlatform/cloud-sdk-docker#345

@markmandel
Copy link
Collaborator

Yep - looks like this is a legitimate bug with the gcloud sdk and Debian Bookworm!

https://issuetracker.google.com/issues/265528492

So we should pause this PR until I do some hunting internally, see what I can find when it's going to get fixed.

@Kalaiselvi84
Copy link
Contributor Author

Kalaiselvi84 commented Jan 9, 2024

Log suggested - python-is-python3 2to3 - https://gist.github.com/Kalaiselvi84/f120f58d2f79f1c024bf2659367623b9

Tried replacing python to python3 - Log- https://gist.github.com/Kalaiselvi84/fd5d6e62992d805b19401e476b8c9ba9. Error - 6.200 google-cloud-cli-app-engine-python : Depends: python2.7 but it is not installable

This is only line we have that uses google-cloud-cli-app-engine-go

apt-get install google-cloud-cli google-cloud-cli-gke-gcloud-auth-plugin google-cloud-cli-app-engine-go -y && \

We have python3 in /build-image dir -

&& apt-get install -y gcc python3-dev python3-setuptools python3-pip \

apt-get install -y python3-pip && \

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 43a21987-68fe-4ef0-b130-0d159746fb82

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@Kalaiselvi84
Copy link
Contributor Author

Including python3 and excluding google-cloud-cli-app-engine-go seems resolves the google-cloud-cli-app-engine-python : Depends: python2.7 but it is not installable.However, I'm not sure if this is the right approach, and we are now facing a different error:

9.426 error: externally-managed-environment
9.426 
9.426 × This environment is externally managed
9.426 ╰─> To install Python packages system-wide, try apt install
9.426     python3-xyz, where xyz is the package you are trying to
9.426     install.

Log - https://gist.github.com/Kalaiselvi84/bcfa4659bcf02334a688b776ef4018cf

@markmandel
Copy link
Collaborator

Unfortunately we need google-cloud-cli-app-engine-go - it's how we deploy the Agones website. So looks like we'll need to wait on https://issuetracker.google.com/issues/265528492 to be resolved.

@Kalaiselvi84
Copy link
Contributor Author

Log for make build-images with python3, without google-cloud-cli-app-engine-go, and install yamllint using apt-get install - https://gist.github.com/Kalaiselvi84/56844929164d8f981e208028eafa5b0d

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 34533557-9754-44d9-82eb-4bc8db00423a

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@markmandel
Copy link
Collaborator

New issue (these would all need to get fixed anyway)!

#10 1.178 E: Package 'openjdk-11-jre' has no installation candidate

Let's replace openjdk-11-jre with openjdk-23-jre` which is in this version of Debian.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 885ddb61-06d8-46d7-b76a-4c251163d72f

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@markmandel
Copy link
Collaborator

markmandel commented Jan 23, 2024

New issue (these would all need to get fixed anyway)!

#10 1.178 E: Package 'openjdk-11-jre' has no installation candidate

Let's replace openjdk-11-jre with openjdk-23-jre` which is in this version of Debian.

Whoops, my bad. My local machine is one debian version ahead of bookworm.

Let's do openjdk-17-jre, since that actually exists on Bookworm:

https://packages.debian.org/search?suite=bookworm&section=all&arch=any&searchon=names&keywords=openjdk

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: d888a425-2bfa-4257-a11d-481d5f043930

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@Kalaiselvi84
Copy link
Contributor Author

Kalaiselvi84 commented Jan 24, 2024

@markmandel, we are waiting for the completion of this ticket to resolve our build issue. Please correct me if i am wrong.

@markmandel
Copy link
Collaborator

@markmandel, we are waiting for the completion of this ticket to resolve our build issue. Please correct me if i am wrong.

Yeah, that's a blocker at the moment.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 61b8428b-ec7a-409b-a120-43e8fdcc2caa

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@Kalaiselvi84
Copy link
Contributor Author

Kalaiselvi84 commented Feb 5, 2024

The command make build-images && make gcloud-auth-docker && make push && make install has successfully passed locally with the workaround mentioned in this ticket. Fixed the Gem Command not found issue with ruby-full.

@Kalaiselvi84 Kalaiselvi84 requested a review from gongmax February 5, 2024 22:50
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: df7e2fdf-bfcf-4903-88b5-c08dbf588f32

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 37ab750e-de3b-499c-b337-5981a7538b4a

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@markmandel
Copy link
Collaborator

Any objection to closing this now? I'm pretty sure #3621 covers all of this now.

@Kalaiselvi84
Copy link
Contributor Author

Any objection to closing this now? I'm pretty sure #3621 covers all of this now.

No objection from me!

@Kalaiselvi84 Kalaiselvi84 deleted the bw-build branch March 15, 2024 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Refactoring code, fixing up documentation, etc size/S size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants