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

Fix e2e job NATS Install step #379

Merged
merged 2 commits into from
Jan 11, 2024
Merged

Conversation

muralov
Copy link
Contributor

@muralov muralov commented Jan 10, 2024

@muralov muralov requested review from the1bit and a team as code owners January 10, 2024 14:29
@muralov muralov requested a review from marcobebway January 10, 2024 14:29
@kyma-bot kyma-bot added area/ci Issues or PRs related to CI related topics cla: yes Indicates the PR's author has signed the CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 10, 2024
@kyma-bot kyma-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 10, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label Jan 10, 2024
@kyma-bot kyma-bot merged commit a4bceab into kyma-project:main Jan 11, 2024
8 checks passed
@muralov muralov deleted the fix-e2e-job branch January 11, 2024 08:26
@@ -210,8 +210,10 @@ func (m *EventMeshMock) handleMessaging() func(w http.ResponseWriter, r *http.Re
if err == nil {
break
}
if i < 2 { // Don't sleep after the last attempt
time.Sleep(3 * time.Second)
two := 2
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe attempts is a better name.

@friedrichwilken
Copy link
Contributor

Can we have a better description here the next time? A reason like "The job x is broken (link) because of reasons. To fix that this PR does this and that?"

@muralov
Copy link
Contributor Author

muralov commented Jan 16, 2024

Can we have a better description here the next time? A reason like "The job x is broken (link) because of reasons. To fix that this PR does this and that?"

yes, I'll try to add more details next time.

friedrichwilken pushed a commit to friedrichwilken/eventing-manager that referenced this pull request Jan 16, 2024
* Fix e2e job NATS Install step

* Fix magic number linting errors
@grischperl grischperl modified the milestones: 1.0.3, 1.1.0 Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues or PRs related to CI related topics cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants