Skip to content

Conversation

@tupyy
Copy link
Collaborator

@tupyy tupyy commented Feb 2, 2026

Signed-off-by: Cosmin Tupangiu [email protected]

@tupyy tupyy force-pushed the e2e-followup-1 branch 6 times, most recently from 76fd4f0 to 48173a7 Compare February 2, 2026 17:20
Name string `json:"name"`
}

func (a *Actioner) CreateSource(name string) (string, error) {
Copy link
Collaborator

@AvielSegev AvielSegev Feb 2, 2026

Choose a reason for hiding this comment

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

https://github.com/kubev2v/migration-planner/blob/main/test/e2e/service/source.go#L19
Can we reuse the code from the planner?
I mean moving part of the files from test/e2e into pkg and then use them in both places?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

right. I've forgotten about that one. We don't need to move it to pkg. I'll try to use it.

)

BeforeAll(func() {
var err error
Copy link
Collaborator

Choose a reason for hiding this comment

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

any reason for not putting it in the var group above?

@tupyy tupyy force-pushed the e2e-followup-1 branch 9 times, most recently from 69e66f9 to dd29a92 Compare February 3, 2026 16:03
add building blocks for tests:
- proxy which acts as a reverse proxy between agent and backend
- observer to count the requests between agent and backend
- actioner to act as a user on backend or to create jwt for agent
- dbReadWriter to act directly on the db to check or create resources
  like private keys for agent authentication

Signed-off-by: Cosmin Tupangiu <[email protected]>
@tupyy tupyy merged commit 48022f4 into main Feb 4, 2026
8 checks passed
@tupyy tupyy deleted the e2e-followup-1 branch February 4, 2026 09:59
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.

3 participants