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

[Synthetics] Document how to set locale and timezone via PlaywrightOptions #3004

Closed
paulb-elastic opened this issue Jun 8, 2023 · 3 comments · Fixed by #3079
Closed

[Synthetics] Document how to set locale and timezone via PlaywrightOptions #3004

paulb-elastic opened this issue Jun 8, 2023 · 3 comments · Fixed by #3079
Assignees
Labels
Area:Synthetics Synthetics Docs Issue Request Team:Docs Label for the Observability docs team

Comments

@paulb-elastic
Copy link
Contributor

paulb-elastic commented Jun 8, 2023

Description

At the moment, neither the timezone or locale is set in the Synthetics Service, which will work for many sites, but some sites will not work properly without one or both of these being set.

The Service is going to be changed to set the timezone for each individual location, but the locale will not be set as this will be dependent on the use case the user is trying to achieve (sometimes the same locate will be desired from different locations, sometimes it will intentionally want to be different).

These can already be set on a per monitor or global basis using for example:

playwrightOptions: {
  locale: 'en-AU',
  timezoneId: 'Australia/Brisbane',
}

We need to make this clearer in the documentation but also clarify that the timezoneId will reflect the timezone the location is running in. Likewise we should clarify that playwrightOptions set in the script/configuration will take priority over the default timezoneId set in the service.

This should wait until https://github.com/elastic/synthetics-service/issues/1312 has been implemented and rolled out.

Resources

Need to work out the best location, maybe somewhere in https://www.elastic.co/guide/en/observability/current/synthetics-create-test.html (looking for guidance here from the Docs team)

Collaboration

The product team will provide the initial content.

Point of contact.

Main contact: @paulb-elastic

Stakeholders:

@colleenmcginnis
Copy link
Contributor

This should wait until https://github.com/elastic/synthetics-service/issues/1312 has been implemented and rolled out.

@paulb-elastic it looks like https://github.com/elastic/synthetics-service/issues/1312 has been bumped to 8.10. Should I hold on this issue as well?

@paulb-elastic
Copy link
Contributor Author

@colleenmcginnis I think this is useful info (I've referred others to it recently), but the specific messaging in this issue talks about saying "we set timezone in the service", but we don't yet until that issue is done. However, I wonder if it would still be useful to document how to do it (set the timezone and locale), and then when we do set the timezone in the service, we can update the docs to make that clear (but users can still override if they wanted to).

@paulb-elastic
Copy link
Contributor Author

Note that the service/agent change will now be done under elastic/beats#36117

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:Synthetics Synthetics Docs Issue Request Team:Docs Label for the Observability docs team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants