Skip to content

Conversation

@snohio
Copy link
Contributor

@snohio snohio commented Oct 20, 2025

Signed-off-by: Michael Butler michael.butler@progress.com

Oracle updated their endpoint for attached volumes to include all attached volumes and not just multipath. This update moves from collecting volume information from attachedVolumes to the allAttachedVolumes API endpoint.

Description

Changes collecting OCI volume information from attachedVolumes API endpoint to a new one that they created called allAttachedVolumes for a specfic shared customer.

This also fixed the output of the information as it was improperly formatted prior to this change.

Related Issue

Fixes #1896

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

  • I have read the CONTRIBUTING document.
  • I have run the pre-merge tests locally and they pass.
  • I have updated the documentation accordingly. (there is none)
  • I have added tests to cover my changes. (Additional change not needed)
  • If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
  • All new and existing tests passed.
  • All commits have been signed-off for the Developer Certificate of Origin.

snohio added 7 commits October 3, 2025 14:34
Signed-off-by: Michael Butler <michael.butler@progress.com>
Modernized Ruby syntax.
Signed-off-by: Michael Butler <michael.butler@progress.com>
Modernized Ruby syntax to pass cookstyle --chefstyle.
Signed-off-by: Michael Butler <michael.butler@progress.com>
Modernized Ruby syntax to pass cookstyle --chefstyle.
Signed-off-by: Michael Butler <michael.butler@progress.com>
Signed-off-by: Michael Butler <michael.butler@progress.com>
@snohio snohio requested review from a team and jaymzh as code owners October 20, 2025 13:39
@sonarqubecloud
Copy link

@tpowell-progress tpowell-progress merged commit 80ed5f7 into chef:main Oct 21, 2025
45 checks passed
tpowell-progress pushed a commit that referenced this pull request Oct 29, 2025
* Added support for Windows Platform for oci metadata capturing.
* Updating allAttachedVolumes metadata for OCI instances

---------

Signed-off-by: Michael Butler <michael.butler@progress.com>
tpowell-progress added a commit that referenced this pull request Oct 29, 2025
* Oci-metadata-volumefix (#1897)

* Added support for Windows Platform for oci metadata capturing.
* Updating allAttachedVolumes metadata for OCI instances

---------

Signed-off-by: Michael Butler <michael.butler@progress.com>

* Style corrections

Signed-off-by: Thomas Powell <thomas.powell@progress.com>

* Appease Sonarqube

Signed-off-by: Thomas Powell <thomas.powell@progress.com>

* fix nesting

Signed-off-by: Thomas Powell <thomas.powell@progress.com>

* const not working

Signed-off-by: Thomas Powell <thomas.powell@progress.com>

* Now it's complaining about a different string

Signed-off-by: Thomas Powell <thomas.powell@progress.com>

---------

Signed-off-by: Michael Butler <michael.butler@progress.com>
Signed-off-by: Thomas Powell <thomas.powell@progress.com>
Co-authored-by: Mike Butler <24615387+snohio@users.noreply.github.com>
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.

update OCI medatat gathering from attachedVolumes endpoint to allAttachedVolumes

3 participants