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.

Sign up for GitHub

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

Jump to bottom

vpc access support #387

Merged
merged 15 commits into from
Jun 20, 2024
Merged

vpc access support #387

merged 15 commits into from
Jun 20, 2024

Conversation

mridulgain
Copy link
Contributor

@mridulgain mridulgain commented Jun 13, 2024 •
edited
Loading

Description

Fixes

  • app pod reconciliation logic update
  • app pod watch bug fix
  • vpc access support

How Has This Been Tested?

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have ran go fmt
  • I have updated the helm chart as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit test cases.
  • I have verified the E2E test cases with new code changes.
  • I have added all the required E2E test cases.

Does this PR introduce a breaking change?

bharath-avesha and others added 13 commits May 10, 2024 17:11
@bharath-avesha
fix(): Fix app pod reconciliation
6f8142c 
Signed-off-by: Bharath Horatti <[email protected]>
@mridulgain
create vpc egress ns in all workers
92bff1b 
Signed-off-by: Mridul Gain <[email protected]>
@bharath-avesha
fix(): Modified logs and loglevel
4869495 
Signed-off-by: Bharath Horatti <[email protected]>
@bharath-avesha
fix(): Updated monitoring pkg version
8c2e6a6 
Signed-off-by: Bharath Horatti <[email protected]>
@mridulgain
fix(): watch for workload changes
a3ca88d 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain
check for empty labels
455e66c 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain
Merge pull request #386 from kubeslice/hotfix-watch
9aeb171 
fix(): watch for workload changes
@mridulgain
api canges for slice & sliceconfigs
9da1b72 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain
Merge branch 'master' into feature-vpc-egress
6cc6274 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain
gateway types update
89e4ff3 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain
service export should be functional from vpc access ns
a1ed47e 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain
Merge branch 'fix-app-update' into feature-vpc-egress
a20b5b0 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain
import fix
8767ad0 
Signed-off-by: Mridul Gain <[email protected]>
@mridulgain mridulgain self-assigned this Jun 13, 2024
bharath-avesha
bharath-avesha reviewed Jun 20, 2024
View reviewed changes
go.mod Outdated Show resolved Hide resolved
@mridulgain mridulgain merged commit cb0480a into master Jun 20, 2024
11 checks passed
@mridulgain mridulgain deleted the feature-vpc-egress branch July 8, 2024 12:12
@mridulgain mridulgain restored the feature-vpc-egress branch July 8, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bharath-avesha bharath-avesha bharath-avesha approved these changes

@narmidm narmidm Awaiting requested review from narmidm narmidm is a code owner

@richiesebastian richiesebastian Awaiting requested review from richiesebastian richiesebastian is a code owner

@Rahul-D78 Rahul-D78 Awaiting requested review from Rahul-D78 Rahul-D78 is a code owner

@gourishkb gourishkb Awaiting requested review from gourishkb gourishkb is a code owner

Assignees

@mridulgain mridulgain

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mridulgain @bharath-avesha