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

travis: disable integration #5807

Merged
merged 1 commit into from
Jul 6, 2018
Merged

Conversation

joshmoore
Copy link
Member

gh-5800 is currently failing on the integration step. gh-5787 started
trying to fix this but is not yet green. Disabling for the moment.

cc: @sbesson @mtbc @jburel

omegh-5800 is currently failing on the integration step. omegh-5787 started
trying to fix this but is not yet green. Disabling for the moment.
@manics
Copy link
Member

manics commented Jul 6, 2018

From #5800: E ln: failed to create hard link '/OMERO/ManagedRepository/11a98b27-9736-488b-bb49-d2c79f903764_2/2018-07/06/07-51-57.322/import_fake_file_1vWFp8d&series=1&extra_args=[\'--transfer=ln\'].fake' => '/opt/omero/server/omero/tmp/omero_omero-server/919/import_fake_file_1vWFp8d&series=1&extra_args=[\'--transfer=ln\'].fake': Invalid cross-device link

Potential fixes:

  • Run df in travis to get a list of mounts (maybe they moved /opt onto a separate partition?)
  • Move the fake file to under /OMERO/tmp instead to guarantee it's on the same device as /OMERO

@joshmoore
Copy link
Member Author

Thanks, @manics. Linked to your comment from omero-test-infra, but for the moment since there are other change too, I'd rather just get this in so PRs go green again.

@sbesson
Copy link
Member

sbesson commented Jul 6, 2018

Thanks all. Agreed with everything above. Merging this immediately to prevent PRs from blocking ongoing PRs to be excluded because of upstream changes. I understand ways to address this problem are investigate independently. Once done, we can look into re-enabling these integration tests.

@sbesson sbesson merged commit d61e7b8 into ome:develop Jul 6, 2018
@joshmoore joshmoore deleted the disable-integration branch July 6, 2018 12:19
@mtbc
Copy link
Member

mtbc commented Jul 10, 2018

Move the fake file to under /OMERO/tmp

looks like a good idea. Coming back to this issue is captured on a card or something?

@joshmoore
Copy link
Member Author

ome/omero-test-infra#20

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.

5 participants