Project

Profile

Help

Issue #5183

docker ruby bindings remote list ignores arguments

Added by jsherril@redhat.com 4 months ago. Updated about 1 month ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Severity:
2. Medium
Version - Docker:
Platform Release:
Blocks Release:
Target Release - Docker:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Katello-P1
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 56

Description

When running the following ruby code:

require 'pulp_docker_client'

config = PulpDockerClient::Configuration.new do |c|
          c.username = 'admin'
          c.password = 'password'
          c.debugging = true
      c.host = 'localhost:24817'
end

api = PulpDockerClient::RemotesDockerApi.new(PulpDockerClient::ApiClient.new(config))
api.list(:name => 'foo')

the following request was previously generated:

GET https://localhost/pulp/api/v3/remotes/docker/docker/?name=foo

but now, starting with 4.0.0b6.dev01563975458 it generates this request:

GET https://localhost/pulp/api/v3/remotes/docker/docker/

which no longer filters on the name.

Here's the output from the latest gem showing it returning a remote that does not match:

=> #<PulpDockerClient::InlineResponse2004:0x00000000076fbd58 @count=1, @results=[#<PulpDockerClient::DockerRemote:0x00000000076fb998 @policy="immediate", @_href="/pulp/api/v3/remotes/docker/docker/db219d5c-fe24-4c30-9ee9-44a361ed14a7/", @_created=Wed, 24 Jul 2019 15:10:42 +0000, @_type="docker.docker", @name="Default_Organization-Test-busybox-library", @url="http://docker.io", @ssl_validation=true, @_last_updated=Wed, 24 Jul 2019 15:10:42 +0000, @download_concurrency=20, @upstream_name="busybox">]>

notice the name value it returned is not 'foo'

History

#1 Updated by jsherril@redhat.com 4 months ago

  • Description updated (diff)

#2 Updated by jsherril@redhat.com 4 months ago

  • Description updated (diff)

#3 Updated by dkliban@redhat.com 4 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
  • Triaged changed from No to Yes
  • Sprint set to Sprint 56

#4 Updated by dkliban@redhat.com 4 months ago

  • Status changed from ASSIGNED to MODIFIED

I've pushed a new openapi-generator-cli.jar to fedorapeople.org0. I'll update my openapi-generator PR1 with the change also.

[0] https://repos.fedorapeople.org/repos/pulp/pulp/openapi/openapi-generator-cli.jar
[1] https://github.com/OpenAPITools/openapi-generator/pull/3405/

#5 Updated by CarmenNishida about 1 month ago

This article teaches a lesson to spread love don’t hate anybody and don’t call anyone your enemy because us all have to take this initiative to spread love and stop our arguments but now I'm happy to visit https://edubirdie.net/howitworks in order to manage the task. So I really like the article because it has described some major facts that probably help us a lot. But really an eye catching up in the material used to write the article.

Please register to edit this issue

Also available in: Atom PDF