Point the telemetry service to telemetry.trento.suse.com #640
Conversation
web/telemetry/publisher.go
Outdated
| @@ -40,7 +40,7 @@ func (tp *TelemetryPublisher) Publish(telemetryName string, installationID uuid. | |||
| return nil | |||
| } | |||
|
|
|||
| var apiHost = "https://telemetry.trento-project.io" // this service still does not exist | |||
| var apiHost = "https://telemetry.trento.suse.com" // this service still does not exist | |||
There was a problem hiding this comment.
Hey, @dottorblaster ,
I would remove the comment besides the new domain, as now it will exists hehe
There was a problem hiding this comment.
By the way, now I notice, apiHost looks a quite inaccurate name for this, doesn't it? XD
There was a problem hiding this comment.
Bad naming here is my fault 😅
Feel free to change or not to something like telemetryApiHost telemetryEndpoint or whatever makes sense.
There was a problem hiding this comment.
Comment removed, thanks for that! apiHost was quite clear to me but feel free to suggest something else, I think this line is going to be edited a lot to allow configuration via build flags in the future so... 😄
There was a problem hiding this comment.
👍🏼 for renaming. I suggest telemetryServiceUrl.
dottorblaster
force-pushed
the
telemetry-host
branch
from
7eae998
to
375eb4d
Compare
As above 😅