Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

[JENKINS-29326] Don't add duplicate BuildData (downstream) #271

Merged
merged 5 commits into from Dec 16, 2015

Conversation

abayer
Copy link
Member

@abayer abayer commented Dec 11, 2015

Marked as @ignore until
jenkinsci/git-plugin#372 is merged and
released, at which point we'll need to bump the git plugin version as well.

Marked as @ignore until
jenkinsci/git-plugin#372 is merged and
released, at which point we'll need to bump the git plugin version as well.
@abayer
Copy link
Member Author

abayer commented Dec 11, 2015

cc @reviewbybees

@ghost
Copy link

ghost commented Dec 11, 2015

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@abayer abayer closed this Dec 11, 2015
@abayer abayer reopened this Dec 11, 2015
@abayer
Copy link
Member Author

abayer commented Dec 11, 2015

reopening for a new run...

@abayer
Copy link
Member Author

abayer commented Dec 11, 2015

Stupid nio channel failures! Unrelated, obviously.

@@ -162,4 +166,30 @@
assertFalse(iterator.hasNext());
}

// This test is currently disabled until the fix for JENKINS-29326 is released
// in the git plugin, in 2.4.1 or later.
Copy link
Member

Choose a reason for hiding this comment

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

You can use this text as the @Ignore value parameter I think.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done!

@jglick jglick closed this Dec 15, 2015
@jglick jglick reopened this Dec 15, 2015
@amuniz
Copy link
Member

amuniz commented Dec 15, 2015

🐝

@@ -162,4 +166,28 @@
assertFalse(iterator.hasNext());
}

@Ignore("This test is currently disabled until the fix for JENKINS-29326 is released in the git plugin 2.4.1 or later.")
Copy link
Member

Choose a reason for hiding this comment

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

TODO

" }\n" +
"}"));
WorkflowRun b = r.assertBuildStatusSuccess(p.scheduleBuild2(0));
assertEquals(1, b.getActions(BuildData.class).size());
Copy link
Member

Choose a reason for hiding this comment

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

🐜 also check GitTagData.

Copy link
Member

Choose a reason for hiding this comment

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

(resolved)

@jglick jglick changed the title Test for JENKINS-29326. [JENKINS-29326] Don't add duplicate BuildData (downstream) Dec 15, 2015
@jglick
Copy link
Member

jglick commented Dec 15, 2015

🐝

@abayer abayer closed this Dec 15, 2015
@abayer abayer reopened this Dec 15, 2015
abayer added a commit that referenced this pull request Dec 16, 2015
[JENKINS-29326] Don't add duplicate BuildData (downstream)
@abayer abayer merged commit 3840d8c into jenkinsci:master Dec 16, 2015
@jglick
Copy link
Member

jglick commented Dec 16, 2015

Now see if you can revert jenkinsci/bitbucket-plugin#20. @fbelzunc wrote no test so it will be a little work to verify.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants