Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jan 31 2019 14:55
    theopenlab-ci[bot] commented #488
  • Jan 31 2019 14:46
    vadimstroganov starred fog/fog-openstack
  • Jan 31 2019 14:41
    vadimstroganov commented #487
  • Jan 31 2019 14:40
    vadimstroganov commented #487
  • Jan 31 2019 14:38
    vadimstroganov opened #488
  • Jan 31 2019 14:33
  • Jan 31 2019 12:24
    vadimstroganov opened #487
  • Jan 31 2019 09:36
    aufi labeled #486
  • Jan 31 2019 09:35

    aufi on master

    Created explicit mock and real … Merge pull request #486 from gi… (compare)

  • Jan 31 2019 09:35
    aufi closed #486
  • Jan 30 2019 16:38
    gildub closed #482
  • Jan 30 2019 16:38

    gildub on master

    Add multiattach parameter to vo… Merge pull request #482 from ma… (compare)

  • Jan 30 2019 16:38
    gildub commented #482
  • Jan 30 2019 16:23
    theopenlab-ci[bot] commented #486
  • Jan 30 2019 16:07
    gildub synchronize #486
  • Jan 30 2019 15:54
    gildub review_requested #486
  • Jan 30 2019 15:53
    gildub opened #486
  • Jan 30 2019 15:22

    gildub on master

    Fixed a couple of path to clear… Merge pull request #485 from gi… (compare)

  • Jan 30 2019 15:22
    gildub closed #485
  • Jan 30 2019 12:06
Jason Frey
@Fryguy
what are you trying to encode/decode @ioggstream
Bill Wei
@bzwei
can you download the file using "http%3A%2F%2Fciao%2Ftmp%2Fa+b.txt”?
Jason Frey
@Fryguy
you might be better off using CGI.escape/unescape only on the query_params part
ioggstream
@ioggstream
@bzwei not with open-uri
Jason Frey
@Fryguy
so split the URI with URI.parse, and only encode the parts you care about
ioggstream
@ioggstream
@Fryguy yes but an overkill as I'd like to delegate a lib
so that the lib will work around corner cases
Jason Frey
@Fryguy
what are you trying to solve though...like what is the actual issue?
ioggstream
@ioggstream
I want to replace spaces in http:// uris with %20
actually I wanted to retain the old
URI.encode(URI.decode()) behavior
Jason Frey
@Fryguy
in any part of the URI, or only the query params or the path?
ioggstream
@ioggstream
as they are heat templates, the goal is to make it work with static files
and probably serving "foo.com/a heat template with spaces.yaml" is not a great deal too
Jason Frey
@Fryguy
i see...so there are potential spaces in the filename which you want to ensure work through a URI
if so, it sounds like the path portion of the URI...but you could just do uri.gsub(" ", "%20")
to use a library method though URI.encode/decode is what you want
ioggstream
@ioggstream
@Fryguy said you have
;)
ioggstream
@ioggstream
@bzwei @Fryguy I think we could either:
1- remove the whitespace check
2- stick with URI.encode(URI.decode(
Bill Wei
@bzwei
or simply do like @Fryguy said search/replace
ioggstream
@ioggstream
@bzwei ok for me
:D
ioggstream
@ioggstream
ok, pushing...
ioggstream
@ioggstream
I wrote a small docker-compose for developing fog-openstack
it's useful to test with different ruby versions
is it worth to be added to the project?
Bill Wei
@bzwei
@ioggstream I approved it. Thanks!
ioggstream
@ioggstream
@bzwei Great! :DDDDD
Thank you very much1
!
@bzwei should we get in touch with @Ladas or @seanhandley for the merge?
Bill Wei
@bzwei
yes
ioggstream
@ioggstream
Let me know if I should rebase everything in a single patch or if some sort of history is needed
Bill Wei
@bzwei
I think it is good to rebase because many of the commits were temporary and got overriden with later commits. You may decide to have a few meaningful commits rather than a big one.
ioggstream
@ioggstream
@bzwei @Ladas @seanhandley rebased with 6 commits. Let me know pls!
Ladislav Smola
@Ladas
@ioggstream merged
ioggstream
@ioggstream
@Ladas thx for the merge!
Divya K Konoor
@dikonoor
Hi , As per https://github.com/fog/fog-openstack/blob/master/supported.md, the highest supported version of cinder is v2 (even though the current version of cinder is v3) and the highest version of nova is v2, even though v2.1 is the current in community. Are there any plans to upgrade the support to the latest/current ones ?
ioggstream
@ioggstream
@dikonoor I think we should update the v. There are features in microversions that are very useful like soft-anti-affinity...
Divya K Konoor
@dikonoor
@ioggstream Perhaps I should could open a bug so that it gets attention
ioggstream
@ioggstream
@dikonoor you should open a bug explaining how to riproduce the issue
did you wrote a patch?
Divya K Konoor
@dikonoor
@ioggstream :) well..there are no specific scenarios I have tried yet to be able to reproduce an issue as such but the versions mentioned in that support.cmd obviusly looks outdated. Also, I am still a MIQ end user / operator at this point. Haven't got my hands dirty yet
ioggstream
@ioggstream
@dikonoor is there a BZ or support case related?
I saw a related PR fog/fog-openstack#339