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

kustomize deploy #104

Merged
merged 6 commits into from
Sep 5, 2024
Merged

Conversation

warjiang
Copy link
Contributor

@warjiang warjiang commented Sep 4, 2024

What type of PR is this?
/kind feature

What this PR does / why we need it:
add kustomize style of deploy, support mode of nodePort and ingress

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

- support deploy mode of nodePort and ingress

@karmada-bot karmada-bot added the kind/feature Categorizes issue or PR as related to a new feature. label Sep 4, 2024
@karmada-bot karmada-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 4, 2024
artifacts/overlays/nodeport-mode/dashboard-config.yaml Outdated Show resolved Hide resolved
artifacts/overlays/ingress-mode/ingress.yaml Outdated Show resolved Hide resolved
artifacts/overlays/ingress-mode/dashboard-config.yaml Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

Why the .gitignore live in this path?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will merge this .gitignore file. The reason is that for convinient 😆

@warjiang warjiang force-pushed the feature/kustomize-deploy branch from f01759a to 4446ac0 Compare September 4, 2024 07:32
@warjiang
Copy link
Contributor Author

warjiang commented Sep 4, 2024

@RainbowMango The branch already be rebased, and related problem has been resolved.

Comment on lines +37 to +39
env:
- name: GIN_MODE
value: release
Copy link
Member

Choose a reason for hiding this comment

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

What's the environment used for? I searched over the code base but didn't find any reference to it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's the env for gin, more details can refer: https://github.com/gin-gonic/gin/blob/master/mode.go

artifacts/dashboard/karmada-dashboard-web.yaml Outdated Show resolved Hide resolved
@@ -0,0 +1 @@
.dockerconfigjson
Copy link
Member

Choose a reason for hiding this comment

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

no end of line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I merge the inner .gitignore to the outer one

warjiang and others added 6 commits September 4, 2024 21:22
Signed-off-by: warjiang <1096409085@qq.com>
Signed-off-by: warjiang <1096409085@qq.com>
Signed-off-by: warjiang <1096409085@qq.com>
Signed-off-by: warjiang <1096409085@qq.com>
Signed-off-by: warjiang <1096409085@qq.com>
Co-authored-by: Hongcai Ren <renhongcai@huawei.com>
Signed-off-by: warjiang <1096409085@qq.com>
@warjiang warjiang force-pushed the feature/kustomize-deploy branch from 51beb7e to 41de190 Compare September 4, 2024 13:22
@RainbowMango RainbowMango added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Sep 5, 2024
Copy link
Member

@RainbowMango RainbowMango left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@karmada-bot karmada-bot added the lgtm Indicates that a PR is ready to be merged. label Sep 5, 2024
@karmada-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: RainbowMango

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@karmada-bot karmada-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 5, 2024
@karmada-bot karmada-bot merged commit 33a1275 into karmada-io:main Sep 5, 2024
4 checks passed
chouchongYHMing pushed a commit to chouchongYHMing/dashboard that referenced this pull request Sep 9, 2024
* deploy: add gin env & update image tag

Signed-off-by: warjiang <1096409085@qq.com>

* deploy: use kustomize for nodePort mode

Signed-off-by: warjiang <1096409085@qq.com>

* deploy: use kustomize for ingress mode

Signed-off-by: warjiang <1096409085@qq.com>

* chore: reformat config file

Signed-off-by: warjiang <1096409085@qq.com>

* chore: merge ingress-mode/.gitignore into outside .gitignore

Signed-off-by: warjiang <1096409085@qq.com>

* Update artifacts/dashboard/karmada-dashboard-web.yaml

Co-authored-by: Hongcai Ren <renhongcai@huawei.com>
Signed-off-by: warjiang <1096409085@qq.com>

---------

Signed-off-by: warjiang <1096409085@qq.com>
Co-authored-by: Hongcai Ren <renhongcai@huawei.com>
Signed-off-by: chouchongYHMing <121396869+chouchongYHMing@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants