Skip to content

add exclusions#4516

Merged
basil merged 4 commits intojenkinsci:masterfrom
darinpope:add-exclusions
Feb 20, 2025
Merged

add exclusions#4516
basil merged 4 commits intojenkinsci:masterfrom
darinpope:add-exclusions

Conversation

@darinpope
Copy link
Contributor

Pulling over the exclusions added when testing 2.492.2-rc. AWS seems to be running a little slower than Azure, so these tests sometimes work and sometimes don't because their timeouts are kicking in.

Testing done

Tested in #4471

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

Pulling over the exclusions added when testing 2.492.2-rc. AWS seems to be running a little slower than Azure, so these tests sometimes work and sometimes don't because their timeouts are kicking in.
@darinpope darinpope requested a review from a team as a code owner February 20, 2025 18:58
@darinpope darinpope enabled auto-merge (squash) February 20, 2025 18:58
@basil basil disabled auto-merge February 20, 2025 19:17
hudson.plugins.build_timeout.operations.AbortAndRestartOperationTest#abortAndRestartTwice

# TODO times out since move to AWS. Remove after 2.462.x falls off
io.jenkins.plugins.pipelinegraphview.utils.PipelineStepApiTest#gh362_stepsBeforeStepBlockGetValidStatus No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

I can't find a PR for this one, so after 2.462.x falls off would this just start failing again?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, you're right. I'll put in a PR for that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

darinpope and others added 3 commits February 20, 2025 15:07
Copy link
Member

@basil basil left a comment

Choose a reason for hiding this comment

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

Thanks!

@basil basil merged commit 3794bfd into jenkinsci:master Feb 20, 2025
5 checks passed
@basil
Copy link
Member

basil commented Feb 20, 2025

jenkinsci/pipeline-graph-analysis-plugin#149 likely fits in the same category.

@darinpope darinpope deleted the add-exclusions branch February 20, 2025 22:49
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