Skip to content

Add note to the Tech Specs about removal of archs #7782

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
May 14, 2025
Merged

Add note to the Tech Specs about removal of archs #7782

merged 3 commits into from
May 14, 2025

Conversation

@AlexFenlon
Copy link
Contributor

@AlexFenlon AlexFenlon commented May 14, 2025 •
edited
Loading

Proposed changes

  • Add note to the Tech Specs to notify users of the upcoming removal of 3 architechures: armv7, s390x & ppc64le

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork
@AlexFenlon AlexFenlon requested review from a team as code owners May 14, 2025 11:21
@AlexFenlon AlexFenlon added the needs cherry pick Cherry pick this PR into a release branch label May 14, 2025
@github-actions github-actions bot added the documentation Pull requests/issues for documentation label May 14, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 14, 2025

✅ Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/nginx-ingress-controller/7782/
ADubhlaoich
ADubhlaoich previously approved these changes May 14, 2025
View reviewed changes
Copy link
Member

@ADubhlaoich ADubhlaoich left a comment

Choose a reason for hiding this comment

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

Generally LGTM: suggestions for re-phrasing.
@AlexFenlon @ADubhlaoich
Update site/content/technical-specifications.md
2bed046 
Co-authored-by: Alan Dooley <a.dooley@f5.com> Signed-off-by: AlexFenlon <a.fenlon@f5.com>
@AlexFenlon
Copy link
Contributor Author

AlexFenlon commented May 14, 2025 •
edited
Loading

removing the release notes as we already have a notice here
@AlexFenlon
revert releases.md
98c9821 
Signed-off-by: AlexFenlon <a.fenlon@f5.com>
@AlexFenlon AlexFenlon changed the title Add note to release notes and Tech Specs about removal of archs Add note to the Tech Specs about removal of archs May 14, 2025
@AlexFenlon AlexFenlon merged commit a0fd424 into main May 14, 2025
30 checks passed
@AlexFenlon AlexFenlon deleted the docs/update-tech-spec-archs branch May 14, 2025 12:58
nginx-bot pushed a commit that referenced this pull request May 14, 2025
@AlexFenlon @web-flow
Add note to the Tech Specs about removal of archs (#7782)
a5c983c
haywoodsh pushed a commit that referenced this pull request May 15, 2025
@AlexFenlon @haywoodsh
Add note to the Tech Specs about removal of archs (#7782)
7a6d836
@AlexFenlon AlexFenlon removed the needs cherry pick Cherry pick this PR into a release branch label Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Pull requests/issues for documentation

5 participants

@AlexFenlon @ADubhlaoich @vepatel @pdabelf5