This message was deleted.
# supports
Slackbot
12/28/2023, 5:40 AMThis message was deleted.
d
dhawal
12/28/2023, 8:53 AM@Vishnu Teja Vallala the port is getting configured as
%!s(float64=9000)dhawal
12/28/2023, 8:53 AMthis seems to be the error
dhawal
12/28/2023, 8:53 AMinvalid port \":%!s(float64=9000)\" after hosts
Srikanth Chekuri
12/29/2023, 2:29 AM@Vishnu Teja Vallala pass port as string instead in the values
v
Vishnu Teja Vallala
12/29/2023, 11:21 AMThanks @Srikanth Chekuri @dhawal will try this 😃
Vishnu Teja Vallala
01/02/2024, 7:09 AMThanks @Srikanth Chekuri @dhawal This worked. The documentation should be updated for this i guess.
https://signoz.io/docs/operate/clickhouse/external-clickhouse/
Also I faced one more issue where the signozspanmetrics/prometheus was replaced with signozspanmetrics/cumulative. We had this in our override_values.yaml, i replaced it and it worked.
But should these changes be also published in migration guide so that the upgrade would be easier.
Please let me know if this is already published.
s
Srikanth Chekuri
01/02/2024, 10:57 AMThe documentation should be updated for this i guess.Could you send a PR for this? This is the repo https://github.com/SigNoz/signoz.io
Also I faced one more issue where the signozspanmetrics/prometheus was replaced with signozspanmetrics/cumulative. We had this in our override_values.yaml, i replaced it and it worked.
But should these changes be also published in migration guide so that the upgrade would be easier.
Please let me know if this is already published.This was an overlooked aspect from our side. We didn't think about the override values situation. We will take care of such details next time. Thanks for pointing it out.
👍 1
v
Vishnu Teja Vallala
01/02/2024, 11:55 AMSure @Srikanth Chekuri here is the PR for the same
https://github.com/SigNoz/signoz.io/pull/1422
88 Views