Skip to content

[SOAR-20820] JSM get on calls#3809

Merged
lcwiklinski-r7 merged 3 commits intojira_service_management-1.0.0-releasefrom
SOAR-20820-jsm-get-on-calls
Mar 24, 2026
Merged

[SOAR-20820] JSM get on calls#3809
lcwiklinski-r7 merged 3 commits intojira_service_management-1.0.0-releasefrom
SOAR-20820-jsm-get-on-calls

Conversation

@lcwiklinski-r7
Copy link
Copy Markdown
Collaborator

🎫 Ticket

https://rapid7.atlassian.net/browse/SOAR-20820

🧩 Type of Change

  • [ ✅ ] Feature
  • Bug fix
  • Other

🧠 Background & Motivation

This change introduces the get on calls functionality in Jira Service Management, migrated from Opsgenie as part of the ongoing scheduling capabilities transition

✨ What Changed

  • Added action
  • call to JSM api / get on calls
  • Requires just an ID / optionally flat, and date
  • The output is different since JSM does not come back with the estimated time and the request id

🧪 Testing

  • Unit tests
  • Manual testing steps - created schedule via JSM (manually) and get it via API

@lcwiklinski-r7 lcwiklinski-r7 requested a review from a team as a code owner March 18, 2026 14:43
@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Mar 18, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Comment thread plugins/jira_service_management/icon_jira_service_management/util/api.py Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Jira Service Management “Get On-Calls” action to support the migration from Opsgenie scheduling, exposing JSM’s /schedules/{id}/on-calls endpoint and updating the plugin’s schemas/spec/docs accordingly.

Changes:

  • Implemented get_on_calls API method and wired it into the get_on_calls action.
  • Updated action I/O schema, plugin spec, and help documentation for the new response shape.
  • Added unit test coverage and a mocked API response payload for the new action.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
plugins/jira_service_management/icon_jira_service_management/util/api.py Adds the JSM Ops API call for fetching on-call participants by schedule ID.
plugins/jira_service_management/icon_jira_service_management/actions/get_on_calls/action.py Wires action inputs to the new API method and returns data.
plugins/jira_service_management/icon_jira_service_management/actions/get_on_calls/schema.py Updates input/output schema to remove identifier-type and legacy output fields.
plugins/jira_service_management/plugin.spec.yaml Updates action spec to reflect the new inputs/outputs and examples.
plugins/jira_service_management/help.md Updates user-facing docs/examples for the new action behavior.
plugins/jira_service_management/unit_test/test_get_on_calls.py Adds a unit test for the action.
plugins/jira_service_management/unit_test/payloads/get_on_calls_request.json.resp Adds mocked response payload used by the unit test.
plugins/jira_service_management/.CHECKSUM Updates checksums for regenerated/changed artifacts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread plugins/jira_service_management/unit_test/test_get_on_calls.py Outdated
Comment thread plugins/jira_service_management/help.md Outdated
@@ -251,45 +251,53 @@ This action is used to get current on-call participants
|date|date|None|False|Starting date of the timeline that will be provided in format as (yyyy-MM-dd'T'HH:mm:ssZ) (e.g. 2017-01-15T08:00:00+02:00). Default date is the moment of the time that request is received|None|2017-01-15T08:00:00+02:00|None|None|
|flat|boolean|None|False|When enabled, retrieves user names of all on-call participants. Default value is false|None|False|None|None|
|scheduleIdentifier|string|None|True|Identifier of the schedule|None|ScheduleName|None|None|
Comment thread plugins/jira_service_management/help.md
Comment on lines 223 to 227
@@ -225,17 +225,6 @@ actions:
type: string
required: true
example: ScheduleName
@lcwiklinski-r7 lcwiklinski-r7 force-pushed the SOAR-20820-jsm-get-on-calls branch from 21a4e9b to 81167e1 Compare March 24, 2026 09:57
@lcwiklinski-r7 lcwiklinski-r7 merged commit dba3850 into jira_service_management-1.0.0-release Mar 24, 2026
11 checks passed
@lcwiklinski-r7 lcwiklinski-r7 deleted the SOAR-20820-jsm-get-on-calls branch March 24, 2026 15:30
lcwiklinski-r7 added a commit that referenced this pull request Mar 25, 2026
* get on calls action and unit test

* cr fixes

* cr fixes
lcwiklinski-r7 added a commit that referenced this pull request Mar 26, 2026
* get on calls action and unit test

* cr fixes

* cr fixes
lcwiklinski-r7 added a commit that referenced this pull request Mar 26, 2026
* [SOAR-20738] Jira Service Management template (#3749)

* JSM template

* align initial plugin with current standards

* static code analysis

* Update .CHECKSUM

* code review fixes

* Update api.py

* removing unused f string

* removing unused imports

* set default value for source

* [SOAR-20818] JSM create alert action (#3785)

* Create alert and basic token auth instead of oauth

* Update action

* function wrapper

* Update api.py

* Update email example

* cr fixes

* Unit tests and minor fixes

* Update util.py

* Updated example for alert id

* Unimplemented test cases

* cr fixes

* Unimplemented test cases

* cr fixes

* linter fix

* Add logging and error handling to create_alert API call

* [SOAR-20740] JSM close alert (#3805)

* close alert action and unit tests

* Update api.py

* Update action.py

* [SOAR-20819] JSM get alert (#3807)

* Get alert action and unit test

* cr fix

* [SOAR-20820] JSM get on calls (#3809)

* get on calls action and unit test

* cr fixes

* cr fixes

* connection version (#3817)
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.

4 participants