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

Hackdna/update file storage #3269

Merged
merged 6 commits into from Mar 21, 2019
Merged

Hackdna/update file storage #3269

merged 6 commits into from Mar 21, 2019

Conversation

hackdna
Copy link
Member

@hackdna hackdna commented Mar 21, 2019

Toward #3052:

  • set default data volume size to 1GB and type to gp2
  • simplify init.pp
  • simplify file store and solr data dir config
  • create file store dir using Puppet
  • create file import dir only in Vagrant

@codecov
Copy link

codecov bot commented Mar 21, 2019

Codecov Report

Merging #3269 into develop will decrease coverage by 1.26%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #3269      +/-   ##
===========================================
- Coverage    66.63%   65.37%   -1.27%     
===========================================
  Files          386      383       -3     
  Lines        26635    25153    -1482     
  Branches       976      946      -30     
===========================================
- Hits         17749    16444    -1305     
+ Misses        8886     8709     -177
Impacted Files Coverage Δ
refinery/file_store/tasks.py 58.37% <ø> (ø) ⬆️
refinery/file_store/test_utils.py 100% <ø> (ø) ⬆️
refinery/file_store/models.py 79.08% <ø> (-4.63%) ⬇️
refinery/file_store/utils.py 68.93% <ø> (ø) ⬆️
refinery/ui/source/js/commons/services/group.js 66.66% <0%> (-22.23%) ⬇️
.../ui/source/js/data-set-about/ctrls/details-ctrl.js 42.46% <0%> (-14.02%) ⬇️
refinery/core/serializers.py 81.25% <0%> (-10.99%) ⬇️
refinery/core/views.py 54.32% <0%> (-10.09%) ⬇️
...urce/js/file-browser/ctrls/user-perms-icon-ctrl.js 93.33% <0%> (-1.67%) ⬇️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a671b1...c8fa9cb. Read the comment docs.

@hackdna hackdna merged commit 35a4421 into develop Mar 21, 2019
@hackdna hackdna deleted the hackdna/update-file-storage branch March 21, 2019 17:48
@hackdna hackdna moved this from In Progress to QA in AWS S3 as a storage backend Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants