Skip to content

Adding changes to support s390x#53

Merged
sdurrheimer merged 2 commits intoprometheus:masterfrom
linux-on-ibm-z:golang-builder-s390x
Sep 19, 2018
Merged

Adding changes to support s390x#53
sdurrheimer merged 2 commits intoprometheus:masterfrom
linux-on-ibm-z:golang-builder-s390x

Conversation

@duane-ibm
Copy link
Copy Markdown
Contributor

Added changes to create golang-builder-z docker image for s390x.
Can someone please review the changes?

Copy link
Copy Markdown
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Minor nit, otherwise LGTM.

@@ -0,0 +1,15 @@
# Copyright 2016 The Prometheus Authors
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please update the date for new files.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated.

@duane-ibm duane-ibm force-pushed the golang-builder-s390x branch 2 times, most recently from 6f07d5a to 459af02 Compare September 14, 2018 06:44
Copy link
Copy Markdown
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

LGTM

@SuperQ
Copy link
Copy Markdown
Member

SuperQ commented Sep 14, 2018

Strange, the build is failing due to OSXCROSS_SDK_URL being missing.

@gouthamve
Copy link
Copy Markdown
Member

@SuperQ See: #52

@sdurrheimer
Copy link
Copy Markdown
Contributor

Why the z naming ?

@SuperQ
Copy link
Copy Markdown
Member

SuperQ commented Sep 15, 2018

z comes from IBM's Z-Series branding. I would probably stick to matching the goarch, which is s390x.

@duane-ibm
Copy link
Copy Markdown
Contributor Author

Should i change the z to s390x?

@SuperQ
Copy link
Copy Markdown
Member

SuperQ commented Sep 18, 2018

Yes, please name the files s390x to match the Go architecture name.

@duane-ibm
Copy link
Copy Markdown
Contributor Author

Naming conventions changed to use s390x instead of z.

Copy link
Copy Markdown
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Thanks!

@sdurrheimer
Copy link
Copy Markdown
Contributor

@duane-ibm With #51, we have replaced Go 1.9 with Go 1.11. Could you edit your commit to reflect this change ?

@duane-ibm
Copy link
Copy Markdown
Contributor Author

@sdurrheimer will do it.

@simonpasquier
Copy link
Copy Markdown
Member

LGTM

@sdurrheimer sdurrheimer merged commit 985dca4 into prometheus:master Sep 19, 2018
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