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

[BUG] Doc: update server URL port to 443 #2439

Closed
bk201 opened this issue Jul 4, 2022 · 3 comments
Closed

[BUG] Doc: update server URL port to 443 #2439

bk201 opened this issue Jul 4, 2022 · 3 comments
Assignees
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue priority/0 Must be fixed in this release require/doc Improvements or additions to documentation
Milestone

Comments

@bk201
Copy link
Member

bk201 commented Jul 4, 2022

Describe the bug

Since #1830, server_url must be https://VIP:443.
We need to revise the server URL port from 8443 to 443 in the docs. Especially for PXE page and the configuration page.

@bk201 bk201 added kind/bug Issues that are defects reported by users or that we know have reached a real release require/doc Improvements or additions to documentation priority/0 Must be fixed in this release labels Jul 4, 2022
@bk201 bk201 added this to the v1.0.3 milestone Jul 4, 2022
@bk201 bk201 assigned bk201 and Vicente-Cheng and unassigned bk201 Jul 4, 2022
@harvesterhci-io-github-bot

Pre Ready-For-Testing Checklist

  • If labeled: require/HEP Has the Harvester Enhancement Proposal PR submitted?
    The HEP PR is at:

  • Where is the reproduce steps/test steps documented?
    The reproduce steps/test steps are at:

  • Is there a workaround for the issue? If so, where is it documented?
    The workaround is at:

  • Have the backend code been merged (harvester, harvester-installer, etc) (including backport-needed/*)?
    The PR is at:

    • Does the PR include the explanation for the fix or the feature?

    • Does the PR include deployment change (YAML/Chart)? If so, where are the PRs for both YAML file and Chart?
      The PR for the YAML change is at:
      The PR for the chart change is at:

  • If labeled: area/ui Has the UI issue filed or ready to be merged?
    The UI issue/PR is at:

  • If labeled: require/doc, require/knowledge-base Has the necessary document PR submitted or merged?
    The documentation/KB PR is at:

  • If NOT labeled: not-require/test-plan Has the e2e test plan been merged? Have QAs agreed on the automation test case? If only test case skeleton w/o implementation, have you created an implementation issue?

    • The automation skeleton PR is at:
    • The automation test case PR is at:
  • If the fix introduces the code for backward compatibility Has a separate issue been filed with the label release/obsolete-compatibility?
    The compatibility issue is filed at:

@harvesterhci-io-github-bot

Automation e2e test issue: harvester/tests#397

@bk201 bk201 added the not-require/test-plan Skip to create a e2e automation test issue label Jul 5, 2022
@bk201
Copy link
Member Author

bk201 commented Jul 5, 2022

Verified the new doc is published.

@bk201 bk201 closed this as completed Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue priority/0 Must be fixed in this release require/doc Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants