Skip to content

Expand Azure Arc onboarding role to include Windows support#2170

Open
stefanoochoa wants to merge 3 commits into
ansible-collections:devfrom
stefanoochoa:stefano/arcOnboardingWindows
Open

Expand Azure Arc onboarding role to include Windows support#2170
stefanoochoa wants to merge 3 commits into
ansible-collections:devfrom
stefanoochoa:stefano/arcOnboardingWindows

Conversation

@stefanoochoa

Copy link
Copy Markdown
SUMMARY

Follow up to #2067

Expanding azure arc onboarding role to include Windows support.

ISSUE TYPE
  • New Role Pull Request
COMPONENT NAME

roles/azure_arc

Comment thread tests/integration/targets/azure_arc_integration/tasks/setup_windows.yml Outdated
@zunyangc

Copy link
Copy Markdown
Collaborator

Hi @stefanoochoa ,

PR #2166 was recently merged into main and fixes the failing pipeline.
Please update your branch (rebase or merge main) so the PR validation picks up the latest changes.

Thanks!

@stefanoochoa stefanoochoa force-pushed the stefano/arcOnboardingWindows branch from b82c3ec to 341a87d Compare February 20, 2026 16:48
@stefanoochoa

Copy link
Copy Markdown
Author

Hi @zunyangc , updated! Thanks

@zunyangc zunyangc added the working In trying to solve, or in working with contributors label Feb 23, 2026
@stefanoochoa stefanoochoa force-pushed the stefano/arcOnboardingWindows branch from 58a4f0d to aaf7af6 Compare March 5, 2026 20:21
@stefanoochoa stefanoochoa requested a review from zunyangc March 5, 2026 20:23
@sonarqubecloud

Copy link
Copy Markdown

@stefanoochoa stefanoochoa force-pushed the stefano/arcOnboardingWindows branch from 4155d78 to 1ffe2f7 Compare June 9, 2026 15:17
@zunyangc zunyangc added ready-for-review The PR is ready to be reviewed and merged. and removed working In trying to solve, or in working with contributors labels Jun 10, 2026
@zunyangc

Copy link
Copy Markdown
Collaborator

Hi @Vieran, pipeline test passed. Ready to merge, thanks!

@stefanoochoa

Copy link
Copy Markdown
Author
image local test succeeded

- name: Use azure arc agent to connect to azure
become: true
# This command produces output that includes an access token
no_log: true

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This PR is to support Windows. Why delete the no_log setting in Linux?

@@ -0,0 +1,16 @@
---
- name: Use azure arc agent to connect to azure

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should we add a no_log here, like the Linux does?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review The PR is ready to be reviewed and merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants