Skip to content

[ZTP] Do dhcp discovery regardless profile status#397

Merged
gord1306 merged 1 commit intoedge-core:202311.Xfrom
peter-yu:pr2
Apr 2, 2025
Merged

[ZTP] Do dhcp discovery regardless profile status#397
gord1306 merged 1 commit intoedge-core:202311.Xfrom
peter-yu:pr2

Conversation

@peter-yu
Copy link
Contributor

@peter-yu peter-yu commented Apr 1, 2025

Why I did it
After sonic-ztp: [ZTP] Use config db instead of ZTP configuration profile
this is required for dhcp discovery

How I did it

How I verified it

  1. Modify etc/default/ztp to use USE_DEFAULT_CONFIG="no"
  2. Config breakout for port and enable link up.
  3. config save
  4. 'ztp run' to start ZTP.
  5. breakout port should be 'Link up detected' and ZTP should start to downloading provisioning data from the port.

@peter-yu peter-yu force-pushed the pr2 branch 2 times, most recently from 1aeee59 to 1cd2420 Compare April 2, 2025 09:04
Why I did it
After sonic-ztp: [ZTP] Use config db instead of ZTP configuration profile
this is required for dhcp discovery

How I did it
Do dhcp discovery regardless profile status

How I verified it
1. Modify etc/default/ztp to use USE_DEFAULT_CONFIG="no"
2. Config breakout for port and enable link up.
3. config save
4. 'ztp run' to start ZTP.
5. breakout port should be 'Link up detected' and ZTP should start to downloading provisioning data from the port.
@peter-yu
Copy link
Contributor Author

peter-yu commented Apr 2, 2025

@gord1306 @chenkelly @linarnan Please help to review.

@gord1306 gord1306 merged commit 1e5f815 into edge-core:202311.X Apr 2, 2025
1 check passed
gord1306 pushed a commit to gord1306/ec-sonic-buildimage that referenced this pull request May 2, 2025
…tomatically (#19201)

#### Why I did it
src/sonic-linux-kernel
```
* f9bfb90 - (HEAD -> master, origin/master, origin/HEAD) [ci] Migrate to sonicbld1es agent pool. (edge-core#397) (15 minutes ago) [Liu Shilong]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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