Skip to content

Add initial Dashboards Job Scheduler plugin - #2

Merged
cwperks merged 2 commits into
mainfrom
initial-code-transfer
Jun 16, 2026
Merged

cwperks merged 2 commits into
mainfrom
initial-code-transfer

Conversation

@cwperks

@cwperks cwperks commented Apr 30, 2026 •

Copy link
Copy Markdown
Member

Description

This PR is the initial commit to add code from https://github.com/cwperks/dashboards-job-scheduler to the repo under opensearch-project.

The plugin now links job history by both job index name and job ID, using query params that match the companion Job Scheduler history API:

  • GET /_plugins/_job_scheduler/api/history
  • GET /_plugins/_job_scheduler/api/history?job_index_name=<job_index_name>&job_id=<job_id>

Note: This plugin is still under active development with some features dependent on APIs that need to be added in job-scheduler repo: opensearch-project/job-scheduler#937

There should also be some additional work to make the authorization of job-scheduler APIs similar to authorization of security APIs, but for that to take place there needs to be a new marker on RestHandler to mark APIs as Sensitive. Similar to Sensitive settings introduced in this Core PR there needs to be a marker for Rest APIs that security can hook into to authorize like security role-based API authorization rather than by transport action name.

Similarly, the job scheduler dashboards menu should only show if the logged in user has access to security APIs (similar logic to how security-dashboards-plugin renders).

Testing

mise exec node@22.22.0 -- yarn lint:es --fix
mise exec node@22.22.0 -- yarn lint:es
mise exec node@22.22.0 -- yarn lint:style
git diff --check

Issues Resolved

Related to opensearch-project/job-scheduler#776

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Craig Perkins <craig5008@gmail.com>
@cwperks
cwperks force-pushed the initial-code-transfer branch from a25b5a8 to 61d60d2 Compare April 30, 2026 03:09

@DarshitChanpura DarshitChanpura left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks for adding this new way of managing jobs.

Added a few comments.

Comment thread .github/ISSUE_TEMPLATE/config.yml Outdated
Comment thread .github/.dependabot.yml Outdated
Comment thread public/components/app.tsx Outdated
Comment thread public/components/app.tsx Outdated
Comment thread public/components/app.tsx Outdated
Comment thread public/components/app.tsx Outdated
Comment thread public/components/app.tsx Outdated
Comment thread public/index.scss Outdated
Comment thread public/plugin.ts
Comment thread opensearch_dashboards.json Outdated
Signed-off-by: Craig Perkins <craig5008@gmail.com>
@cwperks

cwperks commented Jun 12, 2026

Copy link
Copy Markdown
Member Author

@DarshitChanpura Addressed all comments. See the screenshot below for what it looks like with this initial commit. Keep in mind this is the initial seed and there would be follow-ups to this to setup Github workflows, expands on tests and wire in the history portion.

For the history portion I will resurrect opensearch-project/job-scheduler#823 which was essentially completed when it was closed.

Screenshot 2026-06-11 at 10 23 44 PM

@DarshitChanpura DarshitChanpura left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Lets ship it.

Comment thread .eslintrc.js
*/
`

const SPDX_LICENSE_HEADER = `

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we should only have this one and remove the above.. not a blocker for this PR.

@cwperks
cwperks merged commit 18fbd03 into main Jun 16, 2026
2 checks passed
@cwperks cwperks mentioned this pull request Jun 17, 2026
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.

2 participants