Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-199529926
### ACS CI BVT Run
**Sumarry:**
Build Number 116
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=13
Skipped=4
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-188440358
merged based on testes and reviews
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1361
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-188040784
@nvazquez LGTM based on integration tests and code.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-187396165
Hi @rafaelweingartner @kishankavala I've run test_ssvm.py,
test_secondary_storage.py and test_vm_life_cycle.py
integration tests under real hardware. This are my r
Github user serg38 commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-185256350
If requirements to use particular NFS version is known beforehand it can
be specified during the image store creation using details argument
details[0].key=nfs.vers
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-185194859
Sure @kishankavala, to set NFS version for a store with id
STORE_ID you'll need to insert a record in
image_store_details_table which has:
* store_id = STORE_I
Github user kishankavala commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-185176731
@nvazquez
Can you please repond to my earlier question also.
>> How is the nfs.version set in image_store_details table?
---
If your project is set up
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-184915639
@serg38 , @nvazquez, we still need some other LGTM here. You could ask for
some help reviewing this PR at Slack or @dev. Even with other LGTM, I think it
Github user serg38 commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-184799968
Hi committers,
Can someone perform a second review and merge it at the earliest? We would
like this to be part of 4.9?
---
If your project is set up for i
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181909261
Thanks a lot @rafaelweingartner for your words and your help!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181819761
@nvazquez, luckily the Jenkins build finished with success now.
@kishankavala, for now, I believe the simplest solution is the better. When
more d
Github user kishankavala commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181771585
One more query:
How is the nfs.version set in image_store_details table?
---
If your project is set up for it, you can reply to this email and have your
re
Github user kishankavala commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181715514
@nvazquez
Apologies for reviewing it late.
1. Since version is fetched from image_store_details, can we send a map
with all details for the image_store
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181646997
Sure, I'll do it
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181646167
Since it timed out, what about squashing those 3 last commits into one, and
crossing the fingers hoping that jenkins succeeds.
---
If your project is set
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181644342
This time it failed for timeout
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project doe
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181571630
Thanks @DaanHoogland, I pushed again
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181569850
@nvazquez I think the job doesn't clean the prior classes well. I cleared
the workspace. Can you push again?
---
If your project is set up for it, you can re
Github user cristofolini commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181047258
@nvazquez That is very strange. As far as I can tell there isn't a file
with that name neither on the current master nor on your branch. There is
however a fil
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-181041714
Hi @DaanHoogland can you help us with Jenkins build? A test file
SecondaryStorageManagerTest.java is failing, but that file is not
in the branch. I could compile w
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-179310061
@nvazquez, if you have any doubt in some of the spring modules, just shoot
me an email (If I have the knowledge I will not hesitate to help you).
Now I
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-178346694
@rafaelweingartner thanks for your reply!
I'm familiar to Spring, I've worked with it, I know how it works but I
wouldn't say I know it in depth. If you do
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-178267025
Hi @nvazquez,
I see that you have created a spring bean as I said. However, I noticed
some things that may not be needed or that seemed out of place t
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-178190394
Hi @rafaelweingartner
Thanks for your comments! I've been working following your indications,
however I didn't create any test case.
I wanted to ask
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-176948581
It is better now.
I only have to point two things out.
1 â About the class âcom.cloud.storage.ImageStoreDetailsUtilâ that
you created. You ar
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-176752434
@rafaelweingartner I pushed some changes according to your comments. Is it
better like this?
---
If your project is set up for it, you can reply to this email and
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-175103859
Ok, @rafaelweingartner I'll work on refactoring this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-175093081
Ok, I understood that you want to maintain compatibility.
What I do not understand is that: If there is a problem with some NFS (they
do not support v
Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-175056442
Hi @rafaelweingartner,
Thanks for your review! I agree with @serg38, it was thought just to extend
the functionality by adding an entry on image_store_deta
Github user serg38 commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-174746870
I believe the main reason is to provide the most of backward compatibility.
image_store_details doesn't have NFS version in existing installations so after
the upgra
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-174734613
@serg38, I did not know that, giving your explanation I rest my arguments ;)
@nvazquez, you changed the method âgetMountPointâ to receive a NFS
Github user serg38 commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-174554271
Unfortunately NFS version negotiation doesn't work properly with all
storage vendors. Some vendors e.g. Tintri require that vers=3 is supplied in
mount command likel
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-174177981
Hi @ nvazquez,
Are you sure that the problem is caused because we are not telling the NFS
protocol version to the mount command?
If you a loo
34 matches
Mail list logo
