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

Adapt to core and plugin changes #763

Merged
merged 5 commits into from May 2, 2022
Merged

Conversation

timja
Copy link
Member

@timja timja commented Apr 20, 2022

@jtnord jtnord mentioned this pull request Apr 20, 2022
@@ -16,7 +16,7 @@
/**
* @author Kohsuke Kawaguchi
*/
@Describable("hudson.plugins.sshslaves.SSHLauncher")
@Describable({"Launch agents via SSH", "hudson.plugins.sshslaves.SSHLauncher"})
Copy link
Member

Choose a reason for hiding this comment

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

this is likely going to be an issue as there are multiple ways to "launch agents via SSH" (one is cloudbees specific with non blocking, and the other is the OSS ssh-slaves plugin. we should always be able to use the class name of the describable to not be dependant on any naming changes. has the UI changed such that the describable is no longer picked up correctly (should be ok as without it databinding will likely not work)

Copy link
Member Author

Choose a reason for hiding this comment

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

@jtnord
Copy link
Member

jtnord commented Apr 20, 2022

the majority of remaining failures seem to be the inability to select credentials for the SSH agents

Unable to locate By.xpath: //*[@id='credentials-dialog-form']//*[@path='/'] in http://127.0.0.1:62883/computer/createItem
Stacktrace
org.openqa.selenium.NoSuchElementException: Unable to locate By.xpath: //*[@id='credentials-dialog-form']//*[@path='/'] in http://127.0.0.1:62883/computer/createItem
	at org.jenkinsci.test.acceptance.po.CapybaraPortingLayerImpl.find(CapybaraPortingLayerImpl.java:199)
	at org.jenkinsci.test.acceptance.po.Control.resolve(Control.java:75)
	at org.jenkinsci.test.acceptance.po.Control.select(Control.java:286)
	at org.jenkinsci.test.acceptance.plugins.ssh_credentials.SshCredentialDialog$1.resolve(SshCredentialDialog.java:36)
	at org.jenkinsci.test.acceptance.po.CapybaraPortingLayerImpl$Resolver.find(CapybaraPortingLayerImpl.java:532)
	at org.jenkinsci.test.acceptance.po.CapybaraPortingLayerImpl.findCaption(CapybaraPortingLayerImpl.java:483)
	at org.jenkinsci.test.acceptance.plugins.ssh_credentials.SshCredentialDialog.select(SshCredentialDialog.java:33)

@timja
Copy link
Member Author

timja commented Apr 24, 2022

Credentials plugin issue caused by jenkinsci/jenkins#6464

@timja
Copy link
Member Author

timja commented May 1, 2022

The other test failing is:
plugins.NodeLabelParameterPluginTest#run_on_several_online_and_offline_slaves

Bisect shows:

900374e12421204f9ad76ac84397e96b9399cf00 is the first bad commit
commit 900374e12421204f9ad76ac84397e96b9399cf00
Author: Jan Faracik <43062514+janfaracik@users.noreply.github.com>
Date:   Fri Apr 15 21:59:48 2022 +0100

    [JENKINS-68129] Fix dropdown chevrons not separating links and overflowing (#6481)

 war/src/main/less/modules/breadcrumbs.less | 175 ++++++++++++++++-------------
 war/src/main/less/modules/page-header.less |  13 ++-
 war/src/main/less/modules/table.less       |   6 +
 3 files changed, 113 insertions(+), 81 deletions(-)

cc @janfaracik

haven't confirmed yet if regression or test needs changing - While it doesn't look ideal I don't think it's a regression, there's just a newline in the DOM now

2.343:

image

2.344:
image

timja added 4 commits May 1, 2022 21:47
Test was timing out sometimes because of the huge number of deps to
install.

Test didn't require this anyway so this is a much quicker better test
@timja timja marked this pull request as ready for review May 2, 2022 05:42
@timja timja merged commit 8f03d31 into jenkinsci:master May 2, 2022
@timja timja deleted the start-fixing-ath branch May 2, 2022 05:42
@NotMyFault
Copy link
Member

The checks are passing here but failing again on master, huh?

@timja
Copy link
Member Author

timja commented May 2, 2022

The checks are passing here but failing again on master, huh?

yes because jenkinsci/jenkins#6519 is not released yet.

I used the incremental from master branch to test this in the meantime

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.

None yet

3 participants