Skip to content
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

Fix health interop issues with Azure Stack Hub #384

Merged
merged 2 commits into from
Feb 6, 2025
Merged

Conversation

arudell
Copy link
Member

@arudell arudell commented Feb 6, 2025

Description

Summary of changes:

  • Removed a couple of health tests that rely on NC URI API call
  • Updated the callers to not pass rest credentials
  • Fixed class call where the backticks were commented out

Change type

  • Bug fix (non-breaking change)
  • Code style update (formatting, local variables)
  • New Feature (non-breaking change that adds new functionality without impacting existing)
  • Breaking change (fix or feature that may cause functionality impact)
  • Other

Checklist:

  • My code follows the style and contribution guidelines of this project.
  • I have tested and validated my code changes.

@arudell arudell marked this pull request as ready for review February 6, 2025 22:55
@arudell arudell requested a review from a team as a code owner February 6, 2025 22:55
@arudell arudell enabled auto-merge February 6, 2025 22:55
@arudell arudell added this pull request to the merge queue Feb 6, 2025
Merged via the queue into main with commit 00b9e7b Feb 6, 2025
4 of 5 checks passed
@arudell arudell deleted the hub-interop-fixes branch February 6, 2025 22:57
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.

2 participants