Skip to content

NLB-6321: R33 release announcement #440

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

NLB-6321: R33 release announcement #440

wants to merge 1 commit into from

Conversation

xynicole
Copy link
Contributor

@xynicole xynicole commented Apr 22, 2025 •
edited
Loading

Proposed changes

From 05 May 2025 to 09 May 2025, NGINXaaS deployment running on the Stable Upgrade Channel will be updated to NGINX Plus Release 33 (R33).
Closes #ISSUE

Checklist

Before merging a pull request, run through this checklist and mark each as complete.

  • I have read the contributing guidelines
  • I have signed the F5 Contributor License Agreement (CLA)
  • I have rebased my branch onto main
  • I have ensured my PR is targeting the main branch and pulling from my branch from my own fork
  • I have ensured that the commit messages adhere to Conventional Commits
  • I have ensured that documentation content adheres to the style guide
  • If the change involves potentially sensitive changes1, I have assessed the possible impact
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • I have ensured that existing tests pass after adding my changes
  • If applicable, I have updated README.md and CHANGELOG.md

Footnotes

  1. Potentially sensitive changes include anything involving code, personally identify information (PII), live URLs or significant amounts of new or revised documentation. Please refer to our style guide for guidance about placeholder content. ↩

@xynicole xynicole requested a review from a team as a code owner April 22, 2025 22:25
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginxaas NGINX as a Service for Azure labels Apr 22, 2025
@github-actions GitHub Actions
Copy link

✅ Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/440/

@xynicole xynicole marked this pull request as draft April 22, 2025 22:29
@xynicole xynicole requested a review from JTorreG April 22, 2025 22:29
JTorreG
JTorreG reviewed Apr 24, 2025
View reviewed changes
content/nginxaas-azure/changelog.md
NGINXaaS for Azure deployments using the **Stable** [Upgrade Channel]({{< ref "/nginxaas-azure/quickstart/upgrade-channels.md" >}}) have now been automatically upgraded to [NGINX Plus Release 33 (R33)]({{< ref "/nginx/releases.md#nginxplusrelease-33-r33" >}}). This upgrade also includes updates to the following NGINX Plus modules:
- nginx-plus-module-njs

For entitlement and visibility policy, check [the FAQs]({{< ref "/nginxaas-azure/faq.md" >}}) to address any additional questions you might have.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
For entitlement and visibility policy, check [the FAQs]({{< ref "/nginxaas-azure/faq.md" >}}) to address any additional questions you might have.
For entitlement and visibility policy, check [the FAQs]({{< ref "/nginxaas-azure/faq.md" >}}) to address any additional questions you might have.

We are trying to move the content of the FAQ to a more appropriate locations and remove this doc. Not sure if linking to it is the best practice, knowing that.

russokj
russokj reviewed Apr 24, 2025
View reviewed changes
content/nginxaas-azure/changelog.md
- nginx-plus-module-njs

For entitlement and visibility policy, check [the FAQs]({{< ref "/nginxaas-azure/faq.md" >}}) to address any additional questions you might have.
For a complete list of allowed directives, see the [Configuration Directives List]({{< ref "/nginxaas-azure/getting-started/nginx-configuration/nginx-configuration-portal.md#configuration-directives-list" >}}).
Copy link
Contributor

Choose a reason for hiding this comment

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

This link seems wrong. I think it needs to be something like nginxaas/azure/getting-started/nginx-configuration/overview/#configuration-directives-list

Copy link
Contributor

@JTorreG JTorreG Apr 24, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Suggested change
For a complete list of allowed directives, see the [Configuration Directives List]({{< ref "/nginxaas-azure/getting-started/nginx-configuration/nginx-configuration-portal.md#configuration-directives-list" >}}).
For a complete list of allowed directives, see the [Configuration Directives List]({{< ref "/nginxaas-azure/getting-started/nginx-configuration/overview.md#configuration-directives-list" >}}).

@russokj refs use repository paths (/nginxaas-azure/... instead of /nginxaas/azure/...) site paths
To link to an anchor use the document name including .md and then anchor: overview.md#configration-directive-list

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

@russokj russokj russokj left review comments

@JTorreG JTorreG JTorreG left review comments

@y82 y82 y82 approved these changes

@puneetsarna puneetsarna puneetsarna approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation product/nginxaas NGINX as a Service for Azure
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@xynicole @y82 @russokj @puneetsarna @JTorreG