[go: up one dir, main page]

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

Update armv7 base images for alpine:3.18 #7247

Closed

Conversation

yanggangtony
Copy link
Contributor
@yanggangtony yanggangtony commented Jan 30, 2023

Signed-off-by: yanggang gang.yang@daocloud.io

/kind cleanup

/docs-not-required

/release-note-not-required

@yanggangtony yanggangtony requested a review from a team as a code owner January 30, 2023 07:02
@marvin-tigera marvin-tigera added this to the Calico v3.26.0 milestone Jan 30, 2023
@marvin-tigera marvin-tigera added docs-pr-required Change is not yet documented release-note-required Change has user-facing impact (no matter how small) labels Jan 30, 2023
@mgleung mgleung added docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact labels Feb 2, 2023
@marvin-tigera marvin-tigera removed release-note-required Change has user-facing impact (no matter how small) docs-pr-required Change is not yet documented labels Feb 2, 2023
@mgleung
Copy link
Contributor
mgleung commented Feb 2, 2023

/sem-approve

@mgleung
Copy link
Contributor
mgleung commented Feb 2, 2023

@yanggangtony is there something we should be aware of in the newest alpine image? I know we also pull some libs from older alpine images for calicoctl, node, and apiserver. Do we need to update those as well?

@yanggangtony
Copy link
Contributor Author
yanggangtony commented Feb 2, 2023

@mgleung
Yes , you are right..
But i search of keyword 'alpine:' , found these base image have a lot of different versions..
like the pic below.
image

Should we unify all of these to the latest?
In my opinion , alpine's latest version maybe the good choice for develop or product environments.
what do you think about that?

@caseydavenport
Copy link
Member

Should we unify all of these to the latest?

It would be nice to use a consistent alpine version, yeah

@yanggangtony yanggangtony changed the title Update armv7 base images for alpine:3.17.1 Update armv7 base images for alpine:3.17.2 Feb 27, 2023
@yanggangtony
Copy link
Contributor Author

Update to latest version : 3.17.2

@mgleung
Copy link
Contributor
mgleung commented Apr 24, 2023

/sem-approve

@caseydavenport
Copy link
Member

Build is failing:

 > [2/5] RUN apk add --no-cache     python     netcat-openbsd:
#0 0.120 fetch https://dl-cdn.alpinelinux.org/alpine/v3.17/main/x86_64/APKINDEX.tar.gz
#0 0.254 fetch https://dl-cdn.alpinelinux.org/alpine/v3.17/community/x86_64/APKINDEX.tar.gz
#0 0.501 ERROR: unable to select packages:
#0 0.531   python (no such package):
#0 0.531     required by: world[python]
------

@yanggangtony
Copy link
Contributor Author

the pr 's time is a lot months ago.
Now the alpine is 3.17.3 and 3.18.
alpine releases
😂😂

@caseydavenport
Copy link
Member

Yeah, sorry about the long cycle time on this one.

@yanggangtony
Copy link
Contributor Author

Yeah, sorry about the long cycle time on this one.

@caseydavenport
Thanks for review.
So , should i change it to the latest one 3.18?
Or should i keep the same now, but figure out the details of the py-error.

@caseydavenport
Copy link
Member

I am happy with either approach - whichever is easiest for you. If we stick with v3.17, we can always do another PR after to upgrade to v3.18.

Signed-off-by: yanggang <gang.yang@daocloud.io>
@yanggangtony
Copy link
Contributor Author

@caseydavenport
Yep . I will go the 3.18.

@yanggangtony yanggangtony changed the title Update armv7 base images for alpine:3.17.2 Update armv7 base images for alpine:3.18 May 23, 2023
@caseydavenport
Copy link
Member

/sem-approve

@yanggangtony
Copy link
Contributor Author

@caseydavenport
Thanks for trigger the ci.
But i see the ci is error .
I can not access it .
Is there a method to get the detail of the ci 's result?
image
image

@caseydavenport
Copy link
Member

@yanggangtony I've uploaded the test output here: https://gist.github.com/caseydavenport/65c6adddc3c7be7cdd743b84932748ba

It's not immediately obvious how it's related to your changes, but it does seem to be happening consistently.

@yanggangtony
Copy link
Contributor Author

@caseydavenport
Thanks for the log share.
Can not figure out the immediate error hints.
This time , the error

 > [2/5] RUN apk add --no-cache     python     netcat-openbsd:
#0 0.120 fetch https://dl-cdn.alpinelinux.org/alpine/v3.17/main/x86_64/APKINDEX.tar.gz
#0 0.254 fetch https://dl-cdn.alpinelinux.org/alpine/v3.17/community/x86_64/APKINDEX.tar.gz
#0 0.501 ERROR: unable to select packages:
#0 0.531   python (no such package):
#0 0.531     required by: world[python]
------

have no appear , because i update the apk add , the alpine 3.17 or 3.18 is adapted to the py3.

@marvin-tigera marvin-tigera removed docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact labels Oct 15, 2023
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.

4 participants