Skip to content

Improve Slack Notification Step For Adhoc #277

@stanbrub

Description

@stanbrub

As of now, there is a slack notification step in the remote-benchmark.yml workflow. Some categories like nightly use it to publish tables to slack at the end of a run. Others like adhoc disable it by supplying no slack token in the scale-benchmark.properties file.

Notes:

  • The slack notification step is not used for adhoc runs
  • Deephaven is installed and started whether or not the publish actually uses it

Improvements:

  • Cancel the slack publish step entirely if a category doesn't use it
  • Only install Deephaven if the specific publish uses it
    • For example, 2 categories may publish to slack but only one needs Deephaven

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions