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

Increase pod replicas for each deployment to 2 (ACM) #139

Open
2 tasks
tobiscr opened this issue Jan 5, 2024 · 4 comments
Open
2 tasks

Increase pod replicas for each deployment to 2 (ACM) #139

tobiscr opened this issue Jan 5, 2024 · 4 comments
Assignees
Labels
area/application-connector Issues or PRs related to application connectivity bv/reliability Business Value: Reliability (see ISO 25010) kind/bug Categorizes issue or PR as related to a bug.

Comments

@tobiscr
Copy link
Contributor

tobiscr commented Jan 5, 2024

Description

To ensure HA, we have to configure for each deployment at least 2 replicas

AC:

@tobiscr tobiscr added kind/bug Categorizes issue or PR as related to a bug. area/application-connector Issues or PRs related to application connectivity bv/reliability Business Value: Reliability (see ISO 25010) labels Jan 5, 2024
@Disper Disper self-assigned this Mar 13, 2025
@Disper
Copy link
Member

Disper commented Mar 17, 2025

Before making the new release, we want to merge library bumps, which was not done in a long time.
Following problematic PRs should be merged first:

Edit: we decided to skip those two Renovate PRs for now.

@Disper
Copy link
Member

Disper commented Mar 20, 2025

While working on this I've discovered that our pipelines are porting false positives in CRA/Gateway/Validator components:

I've confirmed that the issue can be reproduced locally with a simple make test in those components directories.

I checked some older commits

  • git checkout 62ccf1f51506d90deeb6ecc970c8d1c0d7452417 (from Wed Nov 13 16:54:52 2024 +0100) - Failing 🔴
  • git checkout dd78f57(fromThu Aug 15 18:26:54 2024 +0200`) - OK 🟢

@Disper
Copy link
Member

Disper commented Mar 20, 2025

I was bisecting the failing tests in validator and it seems that they're failing since September 2024 and this commit...

@Disper
Copy link
Member

Disper commented Mar 21, 2025

The Validator test failures are connected with following test cases.

  • Removing those test cases makes the tests green
  • Moving those test cases, makes other tests fail
		{
			caseDescription: "Application not managed by Compass Runtime Agent without group and tenant",
			certInfoHeader: `Hash=f4cf22fb633d4df500e371daf703d4b4d14a0ea9d69cd631f95f9e6ba840f8ad;Subject="CN=test-application,OU=OrgUnit,O=Organization,L=Waldorf,ST=Waldorf,C=DE";` +
				`URI=,By=spiffe://cluster.local/ns/kyma-system/sa/default;` +
				`Hash=6d1f9f3a6ac94ff925841aeb9c15bb3323014e3da2c224ea7697698acf413226;Subject="";` +
				`URI=spiffe://cluster.local/ns/istio-system/sa/istio-ingressgateway-service-account`,
			expectedStatus: http.StatusOK,
			application:    applicationNotManagedByCompass,
		},
		{
			caseDescription: "Application not managed by Compass Runtime Agent with group and tenant",
			certInfoHeader: `Hash=f4cf22fb633d4df500e371daf703d4b4d14a0ea9d69cd631f95f9e6ba840f8ad;Subject="CN=test-application,OU=group,O=tenant,L=Waldorf,ST=Waldorf,C=DE";` +
				`URI=,By=spiffe://cluster.local/ns/kyma-system/sa/default;` +
				`Hash=6d1f9f3a6ac94ff925841aeb9c15bb3323014e3da2c224ea7697698acf413226;Subject="";` +
				`URI=spiffe://cluster.local/ns/istio-system/sa/istio-ingressgateway-service-account`,
			expectedStatus: http.StatusOK,
			application:    applicationNotManagedByCompass,
		},

@Disper Disper assigned koala7659 and VOID404 and unassigned Disper Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/application-connector Issues or PRs related to application connectivity bv/reliability Business Value: Reliability (see ISO 25010) kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

4 participants