Skip to content

Issue 690#691

Merged
joewxboy merged 2 commits intomasterfrom
Issue-690
Mar 11, 2026
Merged

Issue 690#691
joewxboy merged 2 commits intomasterfrom
Issue-690

Conversation

@euconnor
Copy link
Copy Markdown
Contributor

Pull Request Template

Description

Fixes # 690.
Also fixes what Joe raised in closed open-horizon.github.io PR 686.
Links here are to new files and will only resolve when anax PR 4467 is merged.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Not possible to fully test until published.

Additional Context (Please include any Screenshots/gifs if relevant)

...

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • [n/a] I have made corresponding changes to the documentation
  • [n/a] My changes generate no new warnings
  • [n/a] I have added tests that prove my fix is effective or that my feature works
  • [n/a] New and existing unit tests pass locally with my changes
  • [n/a] Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings
  • I have tagged the reviewers in a comment below incase my pull request is ready for a review
  • [?] I have signed the commit message to agree to Developer Certificate of Origin (DCO) (to certify that you wrote or otherwise have the right to submit your contribution to the project.) by adding "--signoff" to my git commit command.

Will ask @joewxboy to review.

Signed-off-by: Eugene O'Connor <euconnor@ie.ibm.com>
Signed-off-by: Eugene O'Connor <euconnor@ie.ibm.com>
@euconnor euconnor requested a review from joewxboy March 11, 2026 13:21
Copy link
Copy Markdown
Member

@joewxboy joewxboy left a comment

Choose a reason for hiding this comment

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

LGTM - some product names in there that could be substitution strings.

@joewxboy joewxboy linked an issue Mar 11, 2026 that may be closed by this pull request
@joewxboy joewxboy merged commit a78683a into master Mar 11, 2026
4 checks passed
@joewxboy joewxboy deleted the Issue-690 branch March 11, 2026 18:03
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.

Documentation📄: Fix links to installing/edge_cluster_agent.md

2 participants