8000 Tutorial for FCP by haywoodsh · Pull Request #3954 · nginx/kubernetes-ingress · GitHub
[go: up one dir, main page]

Skip to content

Conversation

haywoodsh
Copy link
Contributor
@haywoodsh haywoodsh commented Jun 2, 2023

Proposed changes

Tutorial to configure and deploy cluster-connector, and connect to NMS for usage reporting, as well as retrieving usage data from the NMS API.

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

@github-actions github-actions bot added the documentation Pull requests/issues for documentation label Jun 2, 2023
@codecov
Copy link
codecov bot commented Jun 2, 2023

Codecov Report

Merging #3954 (c7eb4b9) into main (6df24a9) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3954      +/-   ##
==========================================
- Coverage   51.85%   51.83%   -0.03%     
==========================================
  Files          59       59              
  Lines       16705    16705              
==========================================
- Hits         8663     8659       -4     
- Misses       7745     7747       +2     
- Partials      297      299       +2     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ADubhlaoich ADubhlaoich self-requested a review June 7, 2023 09:47
@brianehlert brianehlert added this to the v3.2.0 milestone Jun 20, 2023
haywoodsh and others added 20 commits June 20, 2023 15:33
Co-authored-by: Alan Dooley <ADubhlaoich@users.noreply.github.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Alan Dooley <ADubhlaoich@users.noreply.github.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Alan Dooley <ADubhlaoich@users.noreply.github.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Signed-off-by: Brian Ehlert <b.ehlert@f5.com>
Signed-off-by: Brian Ehlert <b.ehlert@f5.com>
Signed-off-by: Brian Ehlert <b.ehlert@f5.com>
edited rather than going back and forth

Signed-off-by: Brian Ehlert <b.ehlert@f5.com>
whoops fixes

Signed-off-by: Brian Ehlert <b.ehlert@f5.com>
@github-actions
Copy link
Contributor
github-actions bot commented Jun 20, 2023

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

@haywoodsh haywoodsh marked this pull request as ready for review June 20, 2023 14:35
@haywoodsh haywoodsh requested a review from a team as a code owner June 20, 2023 14:35
Copy link
Contributor
@jjngx jjngx left a comment

Choose a reason for hiding this comment

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

👍🏻

67E6
haywoodsh and others added 9 commits June 21, 2023 12:16
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Alan Dooley <ADubhlaoich@users.noreply.github.com>
This commit makes a number of changes to the frontmatter of the page and
the phrasing of specific sections for clarity and readability. It also
removes the caret character from the shell commands, which should make
it easier for the reader to copy & paste from the examples.
Copy link
Contributor
@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.

I have pushed a number of changes per the feedback: there is one lingering suggestion I made from a missed instance of product naming, and have flagged one other item to Haywood to review.

haywoodsh and others added 3 commits June 23, 2023 11:42
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Haywood Shannon <5781935+haywoodsh@users.noreply.github.com>
@haywoodsh haywoodsh merged commit 70d5574 into main Jun 26, 2023
@haywoodsh haywoodsh deleted the docs/fcp branch June 26, 2023 12:26
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0