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

disable tag.gpgsign as it could be active for some systems #1553

Merged
merged 6 commits into from
Mar 17, 2024

Conversation

olamy
Copy link
Member

@olamy olamy commented Feb 16, 2024

Some test cases related to tag git command can fail in case of the following ~/.gitconfig configuration

[tag]
       gpgSign = true

Checklist

  • I have read the CONTRIBUTING doc
  • I have referenced the Jira issue related to my changes in one or more commit messages
  • I have added tests that verify my changes
  • Unit tests pass locally with my changes
  • I have added documentation as necessary
  • No Javadoc warnings were introduced with my changes
  • No spotbugs warnings were introduced with my changes
  • Documentation in README has been updated as necessary
  • Online help has been added and reviewed for any new or modified fields
  • I have interactively tested my changes
  • Any dependent changes have been merged and published in upstream modules (like git-client-plugin)

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply. Delete the items in the list that do not apply

  • Dependency or infrastructure update

@github-actions github-actions bot added the tests Automated test addition or improvement label Feb 16, 2024
@github-actions github-actions bot added the dependencies Dependency related change label Feb 19, 2024
pom.xml Outdated
@@ -102,6 +102,7 @@
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>git-client</artifactId>
<version>4.6.1-rc3521.3dcd9ff316b_4</version>
Copy link
Member Author

Choose a reason for hiding this comment

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

temporary to test with jenkinsci/git-client-plugin#1107

Copy link
Contributor

Choose a reason for hiding this comment

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

The git client plugin dependency on the master branch is now updated to git client plugin 4.7.0 that includes the test fixes for autosign. Thanks again for making these improvements!

Copy link
Member Author

Choose a reason for hiding this comment

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

@MarkEWaite yeah. thanks for the ping ;)
I update that here. But I still have one issue with with auto sign I need to figure out.

Expected: is <SUCCESS>
     but: was <FAILURE>
[ERROR] Errors: 
[ERROR]   CredentialsUserRemoteConfigTest.checkoutWithValidCredentials:273->Object.wait:328->Object.wait:-2 » TestTimedOut test timed out after 180 seconds

Copy link
Member Author

Choose a reason for hiding this comment

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

well this looks to be a flaky test

This comment was marked as off-topic.

@olamy olamy force-pushed the tag-gpg-auto-sign branch from 4bae916 to 0bac973 Compare March 16, 2024 23:56
@olamy olamy marked this pull request as ready for review March 17, 2024 09:25
@olamy olamy requested a review from a team as a code owner March 17, 2024 09:25
@olamy olamy changed the title [WIP] disable tag.gpgsign as it could be active for some systems disable tag.gpgsign as it could be active for some systems Mar 17, 2024
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Thanks very much for your work on this change!

pom.xml Outdated
@@ -102,6 +102,7 @@
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>git-client</artifactId>
<version>4.6.1-rc3521.3dcd9ff316b_4</version>

This comment was marked as off-topic.

@MarkEWaite MarkEWaite enabled auto-merge (squash) March 17, 2024 15:44
@MarkEWaite MarkEWaite removed the dependencies Dependency related change label Mar 17, 2024
@MarkEWaite MarkEWaite merged commit 7f8afeb into jenkinsci:master Mar 17, 2024
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Automated test addition or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants