[Bug][opsgenie-plugin] Only 20 services are in scope config. Paging is not working

[antarasevich]

Search before asking

• I had searched in the issues and found no similar issues.

What happened

opsgenie plugin list first 20 service while scope configuring

1 Wrong parameters are passed to v1/services of opsgenie API.
correct one are offset and limit while page per_page are passed.
Example
https://api.opsgenie.com/v1/services?offset=285&limit=5

2 Only first 20 are returned back to UI (default paging of opsgenie)
No code in backend to process

What do you expect to happen

opsgenie plugin list all services available in opsgeneie for my user

How to reproduce

1 Please config more then 20 services in opsgenie
2 try to config scope

Anything else

No response

Version

v1.0.2-beta3

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[dosubot]

Hello @antarasevich! I'm here to help you with any bugs, questions, or contributions you have in mind. Let's tackle this issue together!

To address the issue of the Opsgenie plugin only listing the first 20 services, you need to modify the API request parameters from page and per_page to offset and limit, as these are the correct parameters for the Opsgenie API.

Here's a general approach to fix this:

1. Update the API Request Parameters: Change the parameters in the listOpsgenieRemoteScopes function to use offset and limit instead of page and per_page.

2. Adjust the Pagination Logic: Implement logic to calculate the offset based on the current page and the number of services you want to retrieve per request (limit).

Here's a conceptual example of how you might adjust the code:

if page.Page == 0 {
    page.Page = 1
}
if page.PerPage == 0 {
    page.PerPage = 100
}

offset := (page.Page - 1) * page.PerPage

query := url.Values{
    "offset": []string{fmt.Sprintf("%v", offset)},
    "limit":  []string{fmt.Sprintf("%v", page.PerPage)},
}

res, err := apiClient.Get("v1/services", query, nil)

3. Test the Changes: After making these changes, test the plugin to ensure it correctly lists all services by iterating through the pages using the offset and limit parameters.

By implementing these changes, the Opsgenie plugin should be able to list all available services for your user, rather than just the first 20.

To continue talking to Dosu, mention @dosu.

---

^{Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other}

[klesh]

Thanks for reporting the issue, looking forward o your PR. 🌹