Skip to content

Conversation

@lucacome
Copy link
Contributor

@lucacome lucacome commented Aug 30, 2022

Adds the information about the platform to the list of labels

@roidelapluie
Copy link
Member

I have ping other people to get an opinion on this.

@lucacome lucacome force-pushed the add-platform-info branch 2 times, most recently from c9770e1 to d4d71e3 Compare November 11, 2022 03:34
@beorn7
Copy link
Member

beorn7 commented Nov 14, 2022

Looks good to me.

The tests seem to fail because of a problem upstream in prometheus/common. Maybe a rebase will help?

@lucacome
Copy link
Contributor Author

yeah until either #407 or #410 go in the CI is going to fail.

Looks like the only maintainer in the project is @roidelapluie ?

@roidelapluie
Copy link
Member

Note that prometheus is already displaying this:

prometheus, version 2.37.4 (branch: release-2.37, revision: a9a24ba58c901a0972e4c38d5b0d2502951de16d)
  build user:       roidelapluie@nixos
  build date:       20221206-08:16:58
  go version:       go1.19
  platform:         linux/amd64

Signed-off-by: Luca Comellini <luca.com@gmail.com>
@juliusv
Copy link
Member

juliusv commented Dec 10, 2022

Thanks! 👍

@juliusv juliusv merged commit 1bc7f65 into prometheus:main Dec 10, 2022
@lucacome lucacome deleted the add-platform-info branch December 10, 2022 09:35
philipgough added a commit to philipgough/prometheus-operator that referenced this pull request Dec 22, 2022
radek-ryckowski pushed a commit to goldmansachs/common that referenced this pull request May 18, 2023
Signed-off-by: Luca Comellini <luca.com@gmail.com>

Signed-off-by: Luca Comellini <luca.com@gmail.com>
radek-ryckowski pushed a commit to goldmansachs/common that referenced this pull request May 30, 2023
Signed-off-by: Luca Comellini <luca.com@gmail.com>

Signed-off-by: Luca Comellini <luca.com@gmail.com>
radek-ryckowski pushed a commit to goldmansachs/common that referenced this pull request May 21, 2025
Signed-off-by: Luca Comellini <luca.com@gmail.com>

Signed-off-by: Luca Comellini <luca.com@gmail.com>
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