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

DM-41934 Add the ability to explicitly set Sasquatch time #169

Merged
merged 1 commit into from Dec 11, 2023

Conversation

natelust
Copy link
Contributor

@natelust natelust commented Dec 5, 2023

Add the ability to explicitly set the time Sasquatch will use from the pipeline task or pipeline interfaces. This is done by setting the task config variable timestamp_version, or the pipeline parameter sasquatch_timestamp_version to a value of
explicit_timestamp:%Y%m%dT%H%M%S%z

@@ -212,7 +213,8 @@ class AnalysisBaseConfig(PipelineTaskConfig, pipelineConnections=AnalysisBaseCon
doc="Which time stamp should be used as the reference timestamp for a "
"metric in a time series database, valid values are; "
"reference_package_timestamp, run_timestamp, current_timestamp, "
"and dataset_timestamp",
"dataset_timestamp and explicit_timestamp:datetime where datetime is"
"is given in the form %Y%m%dT%H%M%S%z",
Copy link
Contributor

Choose a reason for hiding this comment

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

So, %z is the time zone in format "±HHMM". AFAICT, times uploaded to the Chronograf are in Zulu time, i.e., UTC+0, i.e., GMT. Would that equate to +0000 in this format?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You can do +0000 or z which is an explicit carve out, I think because it is such a commonly output format

@natelust natelust force-pushed the tickets/DM-41934 branch 2 times, most recently from b518f0c to 8837b75 Compare December 7, 2023 17:27
Add the ability to explicitly set the time Sasquatch will use from
the pipeline task or pipeline interfaces. This is done by setting
the task config variable timestamp_version, or the pipeline parameter
sasquatch_timestamp_version to a value of
explicit_timestamp:%Y%m%dT%H%M%S%z
@natelust natelust merged commit a5bbf57 into main Dec 11, 2023
8 checks passed
@natelust natelust deleted the tickets/DM-41934 branch December 11, 2023 16:57
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.

None yet

2 participants