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

OpenStack Integration - Compute Buttons #3747

Closed
djbrooke opened this issue Apr 4, 2017 · 16 comments
Closed

OpenStack Integration - Compute Buttons #3747

djbrooke opened this issue Apr 4, 2017 · 16 comments

Comments

@djbrooke
Copy link
Contributor

djbrooke commented Apr 4, 2017

For the Openstack summit in early May, we should allow users to access an OpenStack instance through Dataverse via a "Compute" button or other means (design will fill in detail here).

@djbrooke djbrooke added the ready label Apr 4, 2017
@djbrooke djbrooke changed the title OpenStack Integration OpenStack Integration - Compute Buttons Apr 5, 2017
pdurbin added a commit that referenced this issue Apr 6, 2017
This branch was created based on on 4.5-export-harvest-swift which
recently had code from ferrys:4.5-export-harvest-swift-update merged
into it. This new branch was not a straight merge, however, because
there was some cruft in it that we don't want in the git history. The
latest from develop was merged into this new
3747-swift-and-compute-button branch and a `git reset` was performed to
make a single commit with all of the changes (and only the changes) that
we think we want. This single commit is being made by @pdurbin but thank
you to @ferrys @anuj-rt and @landreev who did all the actual work!
@TaniaSchlatter
Copy link
Member

TaniaSchlatter commented Apr 12, 2017

Please see UI/UX guidance for the compute button in this document:
https://docs.google.com/document/d/16tFwds9qW3PX4lkuxiglbbqHiQrCeAim_O_evFMZY8w/edit

compute_dataset_round3

compute-file

@djbrooke
Copy link
Contributor Author

When this is picked up, work should be done on a different branch created from the Swift branch:

https://github.com/IQSS/dataverse/tree/3747-swift-and-compute-button

mheppler added a commit that referenced this issue Apr 14, 2017
…group to dataset pg. Moved dataset version number into label next to title on dataset and file pgs. [ref #3747]
mheppler added a commit that referenced this issue Apr 21, 2017
pdurbin added a commit that referenced this issue Apr 24, 2017
mheppler added a commit that referenced this issue Apr 26, 2017
…l, and other related action buttons on the file, dataset and dataverse pgs. [ref #3747]
mheppler added a commit that referenced this issue Apr 28, 2017
… from fragment with file Explore/Download buttons. [ref #3747]
mheppler added a commit that referenced this issue Apr 28, 2017
pdurbin added a commit that referenced this issue May 1, 2017
 #3747

This is a new branch based on 3747-swift-with-derivative-file-support

Conflicts (3747-swift-with-derivative-file-support wins):
doc/sphinx-guides/source/installation/config.rst
src/main/java/edu/harvard/iq/dataverse/dataaccess/DataAccess.java
src/main/java/edu/harvard/iq/dataverse/dataaccess/SwiftAccessIO.java
@pdurbin
Copy link
Member

pdurbin commented May 1, 2017

@ferrys said we should merge 3747-compute-button-UI into 3747-swift-with-derivative-file-support but I didn't want to trash 3747-swift-with-derivative-file-support so I created a new branch from 3747-swift-with-derivative-file-support called 2909-3747-swift-compute-button and merged both 3747-compute-button-UI and develop into it. That branch should be code reviewed. Here's a hand link: https://github.com/IQSS/dataverse/compare/2909-3747-swift-compute-button

I'm moved this issue to Code Review at https://waffle.io/IQSS/dataverse

@pdurbin
Copy link
Member

pdurbin commented May 1, 2017

@ferrys is going to look into publishing not working and will commit a fix to the new 2909-3747-swift-compute-button branch.

@mheppler
Copy link
Contributor

mheppler commented May 2, 2017

UI clean up and various other loose enough have been tied up. Documentation placeholder on the Dataset + File Management pg in the User Guide is ready when the documentation is.

@mheppler mheppler removed their assignment May 2, 2017
@mheppler
Copy link
Contributor

mheppler commented May 2, 2017

As part of the UI clean up on the dataset and file pg associated with this issue, we also resolved this issue, Version number display #3706.

@kcondon
Copy link
Contributor

kcondon commented May 5, 2017

A few issues, some found by Mike:

  1. Compute button appears to guest user but according to UX doc it should not. Need to verify requirement or fix
  2. help text on dataset pg needs a space between icon and first letter of text.
    fixed
  3. when you click the copy button (in FF, Chrome at least) you are bounced up to the top of the page, so the link needs a return:false; or some other JS in there.
    fixed
  4. Potentially change "To learn more about Cloud Storage Access, visit the Cloud Storage Access section of the User Guide." to be less redundant: to learn more, visit the Cloud Storage Access section...
    fixed
  5. Ton of logging every time you load the dataset page of the type: [2017-05-05T16:19:56.553-0400] [glassfish 4.1] [INFO] [] [edu.harvard.iq.dataverse.DatasetPage] [tid: _ThreadID=49 _ThreadName=jk-connector(2)] [timeMillis: 1494015596553] [levelValue: 800] [[
    metadatas [edu.harvard.iq.dvn.core.study.FileMetadata[id=809],
    especially when have lots of files (26), eg. test_files.zip the sample of files I typically use.

mheppler added a commit that referenced this issue May 5, 2017
…with help text in Cloud Storage Access block. [ref #3747]
@mheppler
Copy link
Contributor

mheppler commented May 5, 2017

Fixed jumping issue with Copy button, and spacing and grammar issues with help text in Cloud Storage Access block.

@kcondon
Copy link
Contributor

kcondon commented May 5, 2017

Yes, 2-4 are fixed, just whether compute button appears to guest and excessive logging on dataset load, happens with both local and swift files but more with swift. 13 cat images also a good test case ;)

dlmurphy added a commit that referenced this issue May 15, 2017
Wrote new sections on Cloud Dataverse for user guide and added a note
about file restrictions in Swift to the Installation Config guide.
@dlmurphy
Copy link
Contributor

Just added new material about Cloud Dataverse to the Installation and User guides.

@mheppler, I recommend checking the Guides link in the Cloud Storage Access help text on the dataset page to make sure it still correctly links to the Cloud Storage Access section of the Dataset Management section of the user guide. I tried to set it up so it'll still work, but I want to make sure.

dlmurphy added a commit that referenced this issue May 15, 2017
Added additional detail and emphasis on warnings related to restricted
file problems in Cloud Dataverse in both installation and user guides.
mheppler added a commit that referenced this issue May 15, 2017
…ile Management pg. Minor revisions to Configuration and Installation pgs in Installation Guide. [ref #3747]
dlmurphy added a commit that referenced this issue May 15, 2017
Fixed typos on Dataset Management page, put Cloud Computing section
before Cloud Storage Access, made assorted minor style edits.
mheppler added a commit that referenced this issue May 16, 2017
…ed download link for Dataverse Project logo to Foundations pg. [ref #3747]
@kcondon kcondon closed this as completed May 17, 2017
@pdurbin pdurbin added this to the 4.6.2 - Tabular Mapping milestone May 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants