Skip to content

Conversation

@geremyCohen
Copy link
Contributor

@geremyCohen geremyCohen commented Nov 4, 2025

Before submitting a pull request for a new Learning Path, please review Create a Learning Path

  • I have reviewed Create a Learning Path

Please do not include any confidential information in your contribution. This includes confidential microarchitecture details and unannounced product information.

  • I have checked my contribution for confidential information

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the Creative Commons Attribution 4.0 International License.

geremyCohen and others added 30 commits October 9, 2025 14:09
- Fixed get_request() to show actual response instead of filtering for nginx title
- Updated pod detection to match service endpoint IP to pod IP directly
- Ensures each architecture service correctly identifies its corresponding pod
- Intel service now served by intel pod, AMD by amd pod, ARM by arm pod
- Reorder chapters with ascending weights (10, 20, 30, 40, 50, 60, 70)
- Provide 10-point increments for future insertions
- Maintain logical flow: cluster setup → test utility → deployments → performance
- Replace inline YAML with curl download from nginxOnAKS repository
- Maintain same explanatory text and deployment details
- Consistent pattern with Intel deployment chapter
- Centralize YAML management in nginxOnAKS repository
…ate outputs, add service selector explanation
…d service selector explanation, remove AMD references
…es/3 services, update to JSON response format
…ences, and add wrk and btop usage instructions
…ences, and add wrk and btop usage instructions
…ences, and add wrk and btop usage instructions
geremyCohen and others added 11 commits October 29, 2025 10:54
- Update error message to show 'curl', 'wrk', 'put', 'login'
- Change ConfigMap name from nginx-intel-config to nginx-config
- Remove ConfigMap from ARM deployment output
- Remove ARM-specific optimization bullet point
- Update ConfigMap verification to show only nginx-config
Remove 'Now that you have your Intel nginx deployment running' as nginx
hasn't been deployed yet at this point in the tutorial.
@jasonrandrews jasonrandrews added ACM Arm Cloud Migration awaiting_tech_review labels Nov 5, 2025
@jasonrandrews jasonrandrews self-assigned this Nov 5, 2025
@jasonrandrews jasonrandrews merged commit 7d082cf into ArmDeveloperEcosystem:main Nov 5, 2025
3 checks passed
@madeline-underwood madeline-underwood added ready_to_publish Ready for final review and publish and removed editorial_review labels Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ACM Arm Cloud Migration ready_to_publish Ready for final review and publish

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants