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

Tutorial for FCP #3954

Merged
merged 41 commits into from
Jun 26, 2023
Merged

Tutorial for FCP #3954

merged 41 commits into from
Jun 26, 2023

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
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Show resolved Hide resolved
@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 <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Co-authored-by: Alan Dooley <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Co-authored-by: Alan Dooley <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Signed-off-by: Brian Ehlert <[email protected]>
Signed-off-by: Brian Ehlert <[email protected]>
Signed-off-by: Brian Ehlert <[email protected]>
edited rather than going back and forth

Signed-off-by: Brian Ehlert <[email protected]>
whoops fixes

Signed-off-by: Brian Ehlert <[email protected]>
@github-actions
Copy link

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.

👍🏻

docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
haywoodsh and others added 9 commits June 21, 2023 12:16
Co-authored-by: Luca Comellini <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Co-authored-by: Luca Comellini <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Co-authored-by: Luca Comellini <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Co-authored-by: Luca Comellini <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Co-authored-by: Luca Comellini <[email protected]>
Signed-off-by: Haywood Shannon <[email protected]>
Co-authored-by: Luca Comellini <[email protected]>
Signed-off-by: Alan Dooley <[email protected]>
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.

docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
docs/content/usage-reporting.md Outdated Show resolved Hide resolved
@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