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

[FEATURE] Support timestamp fields in workflows #548

Closed
ohltyler opened this issue Feb 29, 2024 · 1 comment · Fixed by #551
Closed

[FEATURE] Support timestamp fields in workflows #548

ohltyler opened this issue Feb 29, 2024 · 1 comment · Fixed by #551
Assignees
Labels
enhancement New feature or request v2.13.0 Issues targeting release v2.13.0

Comments

@ohltyler
Copy link
Member

ohltyler commented Feb 29, 2024

A common field for plugin resources include timestamps indicating when a resource is created / updated / deployed. This is also a requirement based on the latest @opensearch-project/opensearch-ux mocks for the frontend (refer to mock in opensearch-project/dashboards-flow-framework#65). I propose the following added fields:

  • last_update_time in the workflows system index, and updating using the current timestamp when a POST/PUT request is made.
  • last_launched_time (no strong opinion on the name) in the workflows state index indicating when a provision/launch was last executed. This could be updated the same time the provisioning progress field is updated, for example.
@ohltyler ohltyler added the enhancement New feature or request label Feb 29, 2024
@ohltyler
Copy link
Member Author

Update: from searching through the codebase, I do see a LAST_UPDATED_TIME_FIELD, but I don't see it being used/indexed in the workflows system indices anywhere, and it is not included in any search API responses as a result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v2.13.0 Issues targeting release v2.13.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants