Skip to content
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.

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

doc: fixing broken link #1320

Merged
merged 3 commits into from
Jan 12, 2022
Merged

doc: fixing broken link #1320

merged 3 commits into from
Jan 12, 2022

Conversation

dewhurstwill
Copy link
Contributor

@dewhurstwill dewhurstwill commented Jan 11, 2022

Checklist

  • The title of the PR is clear and informative
  • If applicable, the changes made in the PR have proper test coverage
  • Issues addressed by the PR are mentioned in the description followed by Fixes.

Description

https://github.com/Azure/application-gateway-kubernetes-ingress/blob/master/crds/AzureIngressProhibitedTarget.yaml -> https://github.com/Azure/application-gateway-kubernetes-ingress/blob/master/crds/AzureIngressProhibitedTarget-v1-CRD-v1.yaml

Fixes

Fixing broken link

Copy link
Member

@akshaysngupta akshaysngupta left a comment

Choose a reason for hiding this comment

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

thanks!

@akshaysngupta akshaysngupta changed the title Fixing broken link doc: fixing broken link Jan 12, 2022
@akshaysngupta akshaysngupta merged commit 571f3d8 into Azure:master Jan 12, 2022
akshaysngupta added a commit that referenced this pull request Feb 1, 2022
Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>
@akshaysngupta akshaysngupta mentioned this pull request Feb 1, 2022
3 tasks
akshaysngupta added a commit that referenced this pull request Feb 1, 2022
Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>
akshaysngupta added a commit that referenced this pull request Feb 2, 2022
* doc changes (#1204)

Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>

* Add cookie-based-affinity-distinct-name annotation (#1280)

* Add cookie-based-affinity-distinct-name annotation

* Add ServiceFullNameHash in internalTypes.go

* cookie name test

* Add log

* Add functional testing for distinct cookie name

* Add documenation

* Prefix appgw-affinity- for distinct cookie names

* fix functional test

* fix functional test

Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>

* rewrite: Add support for rewrite rule set assignment (#1287)

* Add support for rewrite rule set assignment

So that rewrite rule sets predefined in the app gateway can be assigned
to routing rules by adding an annotation in the corresponding Ingress.

Fixes #1003.
Related to #462.

* remove listener ingress

Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>

* Fix typo Greenfeild (#1305)

Signed-off-by: KeisukeYamashita <19yamashita15@gmail.com>

* fix: indentation error in helm values file (#1301)

* fixed indentation error in helm values file

* applied change to values-template.yaml file

Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>

* doc: format future readings section in "How to setup networking between Application Gateway and AKS" doc (#1297)

Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>

* fix: port reference by name in ingress (#1322)

* template: update aks default version in greenfield deploy templates (#1319)

* update aks default version

* small docs updates

* small modifications

* update windows guide

* Fixing broken link (#1320)

Co-authored-by: Akshay Gupta <aksgupta@microsoft.com>

* networking/v1: Add ingress class support (#1329)

* support ingress class

* add unit tests

* add test for default

* add e2e test

* test: add e2e test for rewrite-rule-set annotation; fix helm test (#1330)

* test: add test for rewrite-rule-set annotation

* add install task

* use the latest helm to generate

* add comment

Co-authored-by: erickhernandezascencio <48027483+erickhernandezascencio@users.noreply.github.com>
Co-authored-by: Raphael Bickel <raphael.bickel@gmail.com>
Co-authored-by: Isabel Andrade <67916678+beandrad@users.noreply.github.com>
Co-authored-by: KeisukeYamashita <19yamashita15@gmail.com>
Co-authored-by: jiayang <79306996+jiayanginfor@users.noreply.github.com>
Co-authored-by: heoelri <heoelri@users.noreply.github.com>
Co-authored-by: Will Dewhurst <34936646+dewhurstwill@users.noreply.github.com>
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.

None yet

2 participants