Skip to content

fix(dedicated): using iceberg utilities and fix iceberg call of stati… #16836

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

sefiani-ovh
Copy link
Contributor

fixing an issue in iceberg utilities! :(

@sefiani-ovh sefiani-ovh requested review from a team as code owners April 29, 2025 20:04
@sefiani-ovh sefiani-ovh requested review from tibs245, pauldkn, darsene, qpavy, chipp972 and mhelhali-soufien and removed request for a team April 29, 2025 20:04
@@ -339,14 +339,14 @@ class DedicatedCloudService {
}

getDatacenterInfoVm(serviceName, datacenterId, params = {}) {
return this.icebergQuery(
return this.icerbergUtils.icebergQuery(
Copy link
Contributor

Choose a reason for hiding this comment

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

The error is not here but on the initial name icerbergUtils

Sorry It's me I think the origin of this problem

@sefiani-ovh sefiani-ovh force-pushed the project/ms-license-iceberg branch from 69c95c7 to 2a5e6af Compare May 2, 2025 08:25
ref: #MANAGER-14723

Signed-off-by: ahmed sefiani <ahmed.sefiani.ext@corp.ovh.com>
@sefiani-ovh sefiani-ovh force-pushed the project/ms-license-iceberg branch from 2a5e6af to 87d20c6 Compare May 2, 2025 08:27
@sefiani-ovh sefiani-ovh closed this May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants