Skip to content

Add nOps.io block integration#1403

Merged
rita-gorokhod merged 11 commits intoopenops-cloud:mainfrom
andremilk:adding-nops-integration
Oct 7, 2025
Merged

Add nOps.io block integration#1403
rita-gorokhod merged 11 commits intoopenops-cloud:mainfrom
andremilk:adding-nops-integration

Conversation

@andremilk
Copy link
Copy Markdown
Contributor

@andremilk andremilk commented Oct 2, 2025

This pull request is to add an nOps.io integration to OpenOps. nOps.io is a Multi-cloud visibility platform with numerous SaaS integrations, a robust FinOps Agent, and automated optimization for AWS commitments and EKS.

Fixes OPS-2770

@andremilk andremilk changed the title Adding nops integration Add nOps.io block integration Oct 2, 2025
@andremilk andremilk force-pushed the adding-nops-integration branch from 3d13414 to c8a26fb Compare October 2, 2025 18:23
@andremilk andremilk force-pushed the adding-nops-integration branch from af9e47f to 5655dbe Compare October 2, 2025 18:27
description: 'Your nOps API key',
required: true,
}),
signature: Property.SecretText({
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@andremilk could you please re-check that everything works? I moved the signature here.

…rom actions and updating API request handling. Adjusted documentation for clarity and ensured proper header management in HTTP client functions.
…ling

- Updated nOps authentication to use only API key, removing optional signature.
- Refactored HTTP client methods to accept API key directly instead of nOpsAuth object.
- Cleaned up unnecessary code and improved header management in HTTP requests.
…ties

- Updated test assertions to access action properties using bracket notation for consistency and clarity.
- Ensured that the tests remain functional while adhering to best practices.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Oct 7, 2025

@rita-gorokhod rita-gorokhod merged commit d8ed67f into openops-cloud:main Oct 7, 2025
17 checks passed
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.

3 participants