FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added corbench code without manifest files by TheKongers · Pull Request #2 · cortexproject/test-infra · GitHub

Added corbench code without manifest files - #2

Open
TheKongers wants to merge 2 commits into
cortexproject:mainfrom
TheKongers:add-corbench-code-without-manifests
Open

TheKongers wants to merge 2 commits into
cortexproject:mainfrom
TheKongers:add-corbench-code-without-manifests

Conversation

Copy link
Copy Markdown
Contributor

Moved over all corbench code from https://github.com/TheKongers/Corbench without the manifest files.

NOTE: the files under the directories /corbench/infra and /corbench/pkg are directly from prombench, but trimmed down to only keep the EKS functionality to make it more lightweight.

TheKongers force-pushed the add-corbench-code-without-manifests branch 2 times, most recently from 9ccc8e7 to d9df051 Compare August 8, 2025 06:32
Comment thread corbench/Dockerfile
@@ -0,0 +1,16 @@
FROM --platform=linux/amd64 golang:1.15-alpine

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Can we use newer version of golang?

name: gp2-csi
provisioner: ebs.csi.aws.com
parameters:
type: gp2

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Why not GP3?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I could use gp3, but i wonder if it will break, i will give it a test

Comment thread corbench/infra/Dockerfile
@@ -0,0 +1,18 @@
# Build stage
FROM golang:1.15-alpine AS builder

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Use go 1.24.x


type Resource = provider.Resource

type eksCluster struct {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Do we really need these code? Can we just import prombench as a library

"github.com/cortexproject/test-infra/corbench/pkg/provider"
)

func init() {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

ditto. If we cannot reuse prombench CLI, let's try to use its library instead of copy its code directly.

Comment thread corbench/setup-ebs-csi.sh
set -e

# Disable AWS CLI pager to prevent interactive prompts
export AWS_PAGER=""

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

It is ok to keep this script for now but we should try to just configure EBS CSI addon in the same eks cluster yaml

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I will add a note

Signed-off-by: Eric Kong <erikong@amazon.com>
Signed-off-by: Eric Kong <erikong@amazon.com>
TheKongers force-pushed the add-corbench-code-without-manifests branch from 7b7312f to 4b48209 Compare August 8, 2025 22:46

Copy link
Copy Markdown
Contributor Author

Hi ben! I would like to continue getting this in the repo when I have some spare time, but I don't have resources for testing changes anymore, Is there some way I could get aws credits?

Copy link
Copy Markdown
Contributor Author

@yeya24 anything blocking us from merging this code for now?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL