Skip to content

Commit 5bae969

Browse files
fix(validation): remove residual NTP references from scripts and docs
Cleans up stale NTP prompt, config override, synopsis comment, and doc rows left after the Cat-3 NTP cut. time.windows.com in endpoints.json is retained — it is a valid Azure Local firewall endpoint (OS time sync), not a Beacon pre-OS check. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013AZySkzowyq4Ne2hrSRRAL
1 parent d9b8dd1 commit 5bae969

6 files changed

Lines changed: 9 additions & 16 deletions

File tree

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ Or download the full driver pack for your hardware and point `-DriverPath` at th
5252

5353
### 1. Populate config
5454

55-
Copy `src/config/validation-config.example.json` to `src/config/validation-config.json` and fill in your deployment values (DNS, NTP, AD domain, node IPs, gateway, IP pool range).
55+
Copy `src/config/validation-config.example.json` to `src/config/validation-config.json` and fill in your deployment values (DNS, AD domain, node IPs, gateway).
5656

5757
Regenerate the Azure endpoint list if needed:
5858

docs/validation/active-directory.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ Select **option 1** from the Beacon main menu to validate readiness for an AD-jo
1010
| Domain controller IP(s) | `10.10.0.10, 10.10.0.11` | Yes |
1111
| Target OU Distinguished Name | `OU=AzureLocal,DC=corp,DC=improbability,DC=cloud` | Optional |
1212
| Deployment prefix (≤ 8 chars) | `azl` | Optional (default: `azl`) |
13-
| Primary NTP/time source | `10.10.0.1` | Optional (default: `time.windows.com`) |
1413

1514
## Tests run
1615

docs/validation/network-firewall.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ This path runs everything except the AD port tests (Category 4). It's ideal for:
88

99
- Verifying firewall policy before any identity configuration
1010
- Checking endpoint reachability from the management VLAN
11-
- Confirming DNS and NTP are reachable
11+
- Confirming DNS is reachable
1212

1313
## What you are prompted for
1414

src/Build-WinPEImage.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
The image is stage 1 of the 5-stage Azure Local validation lifecycle. See
2828
docs/index.md for the full coverage matrix.
2929
30-
All environment values (DNS, NTP, domain, node IPs) are supplied via the config folder
30+
All environment values (DNS, domain, node IPs) are supplied via the config folder
3131
copied into the image -- no values are hardcoded in this script.
3232
3333
.PARAMETER WorkspacePath

src/Start-AzlBeacon.ps1

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -201,20 +201,15 @@ function Invoke-ADMenu {
201201
$ouDn = Prompt-Optional 'Target OU Distinguished Name (e.g. OU=AzureLocal,DC=corp,DC=contoso,DC=com)'
202202
$deployPrefix = Prompt-Optional 'Deployment prefix (up to 8 chars)' -Default 'azl'
203203
Write-BeaconLine ''
204-
$ntpPrimary = Prompt-Optional 'Primary NTP/time source IP or FQDN' -Default 'time.windows.com'
205-
206204
Write-BeaconLine ''
207205
Write-BeaconLine ' Configuration collected. Updating validation config...' -Color Cyan
208206

209207
$overrides = @{
210-
adDomainFqdn = $domainFqdn
211-
dcIps = $dcIps
212-
deploymentPrefix = $deployPrefix
213-
}
214-
if ($ouDn) { $overrides['adOuDn'] = $ouDn }
215-
if ($ntpPrimary) {
216-
$overrides['ntpServers'] = @{ primary = $ntpPrimary; secondary = 'time.windows.com' }
208+
adDomainFqdn = $domainFqdn
209+
dcIps = $dcIps
210+
deploymentPrefix = $deployPrefix
217211
}
212+
if ($ouDn) { $overrides['adOuDn'] = $ouDn }
218213
Write-ValidationConfigOverrides -Overrides $overrides
219214

220215
Write-BeaconLine ''
@@ -298,7 +293,7 @@ function Invoke-LocalIdentityMenu {
298293
function Invoke-NetworkFirewallMenu {
299294
Write-BeaconHeader 'Networking and Firewall Validation'
300295
Write-BeaconLine ' Validates physical network, endpoint reachability (Azure Local + Arc + Dell),' -Color White
301-
Write-BeaconLine ' DNS, NTP, and the Microsoft Environment Checker.' -Color White
296+
Write-BeaconLine ' DNS and the Microsoft Environment Checker.' -Color White
302297
Write-BeaconLine ''
303298

304299
$gwIp = Prompt-Optional 'Management gateway IP (leave blank to use DHCP-detected)'

src/Start-AzlValidation.ps1

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,7 @@
6565
Prerequisites:
6666
- config\validation-config.json and config\endpoints.json must exist.
6767
- curl.exe must be on PATH for HTTPS GET probes (built-in to WinPE and Win10+).
68-
- w32tm.exe must be on PATH for NTP checks (built-in to WinPE).
69-
- AzStackHci.EnvironmentChecker required for Cat-6/7 (bundled at ISO build time).
68+
- AzStackHci.EnvironmentChecker required for Cat-5/6 (bundled at ISO build time).
7069
PSScriptAnalyzer: passes at Warning/Error severity.
7170
Compatibility: PowerShell 5.1 (WinPE) and PowerShell 7. No PS7-only syntax used.
7271
#>

0 commit comments

Comments
 (0)