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

[Proposal] openyurt application delivery feature #879

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

huiwq1990
Copy link
Member

Signed-off-by: huiwq1990 [email protected]

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


other Note

@openyurt-bot
Copy link
Collaborator

@huiwq1990: GitHub didn't allow me to assign the following users: your_reviewer.

Note that only openyurtio members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

Signed-off-by: huiwq1990 [email protected]

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


other Note

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openyurt-bot openyurt-bot added kind/design kind/design size/L size/L: 100-499 labels Jun 11, 2022
@rambohe-ch
Copy link
Member

@huiwq1990 please fix markdown check error.

@huiwq1990 huiwq1990 force-pushed the feat-app-delivery branch 3 times, most recently from 6a2cb4c to 6b776e3 Compare June 14, 2022 01:35
@openyurt-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: huiwq1990, rambohe-ch

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

@openyurt-bot openyurt-bot added the approved approved label Jun 14, 2022
@openyurt-bot openyurt-bot added size/XL size/XL: 500-999 and removed size/L size/L: 100-499 labels Jul 4, 2022

## Subset定义

对于多区域管理,业界已经有定义模型,如:openyurt的appset,opencruise的UnitedDeployment。
Copy link
Member

Choose a reason for hiding this comment

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

opencruise --> openkruise




### opencruise模型
Copy link
Member

Choose a reason for hiding this comment

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

ddiot

@huiwq1990
Copy link
Member Author

Kubevela community invite us to share the proposal at July 12, 2022 Community Meeting. Do you have time for this meeting?
kubevela/kubevela#4326

@huiwq1990 huiwq1990 force-pushed the feat-app-delivery branch 3 times, most recently from e3529d8 to 45c61b4 Compare July 6, 2022 01:43
@huiwq1990 huiwq1990 force-pushed the feat-app-delivery branch from f849ecf to f703b29 Compare July 6, 2022 01:56
properties:
clusters: ["local"]
namespace: "default"
- name: default-subsets

Choose a reason for hiding this comment

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

    - name: replication
      type: replication
      properties:
        keys: ["beijing","hangzhou"]

-> render 2 times, first time carries context.replicaKey: "beijing", context.name will become "\(context.name)-\(context.replicaKey)"

@stale
Copy link

stale bot commented Oct 10, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Oct 10, 2022
@stale stale bot closed this Oct 17, 2022
@rambohe-ch rambohe-ch reopened this Oct 18, 2022
@codecov
Copy link

codecov bot commented Oct 18, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@c298bb8). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #879   +/-   ##
=========================================
  Coverage          ?   48.96%           
=========================================
  Files             ?       96           
  Lines             ?    13053           
  Branches          ?        0           
=========================================
  Hits              ?     6392           
  Misses            ?     6130           
  Partials          ?      531           
Flag Coverage Δ
unittests 48.96% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved approved kind/design kind/design pinned size/XL size/XL: 500-999
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants