glance-simplestreams-sync doesn't support proxying

Bug #1399622 reported by JuanJo Ciarlante
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Glance-Simplestreams-Sync Charm
Fix Released
Wishlist
Frode Nordahl
glance-simplestreams-sync (Juju Charms Collection)
Invalid
Wishlist
Unassigned

Bug Description

glance-simplestreams-sync charm installs a symlink as:
/etc/cron.daily/glance-simplestreams-sync.py -> /usr/share/glance-simplestreams-sync/glance-simplestreams-sync.py
, voiding any http_proxy setup done by juju.

FYI this works:
ubuntu@unit$ cat /etc/cron.daily/glance-simplestreams-sync-with-proxy
#!/bin/bash
test -f /home/ubuntu/.juju-proxy && source /home/ubuntu/.juju-proxy
exec /usr/share/glance-simplestreams-sync/glance-simplestreams-sync.py

JuanJo Ciarlante (jjo)
tags: added: canonical-bootstack canonical-is
James Page (james-page)
Changed in glance-simplestreams-sync (Juju Charms Collection):
importance: Undecided → Wishlist
status: New → Triaged
Revision history for this message
Mike McCracken (mikemc) wrote :
Revision history for this message
JuanJo Ciarlante (jjo) wrote :
James Page (james-page)
Changed in charm-glance-simplestreams-sync:
importance: Undecided → Wishlist
status: New → Triaged
Changed in glance-simplestreams-sync (Juju Charms Collection):
status: Triaged → Invalid
Revision history for this message
Frode Nordahl (fnordahl) wrote :
Changed in charm-glance-simplestreams-sync:
status: Triaged → In Progress
assignee: nobody → Frode Nordahl (fnordahl)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-glance-simplestreams-sync (master)

Reviewed: https://review.openstack.org/636898
Committed: https://git.openstack.org/cgit/openstack/charm-glance-simplestreams-sync/commit/?id=663a15c5fe77bb75c646981f378e71f9a0b81195
Submitter: Zuul
Branch: master

commit 663a15c5fe77bb75c646981f378e71f9a0b81195
Author: Frode Nordahl <email address hidden>
Date: Thu Feb 14 11:11:15 2019 +0100

    Fix sourcing of proxy related environment variables

    Juju has changed where it stores proxy related environemnt
    variables on units:
    https://github.com/juju/juju/commit/223b8db274268e2ae655a71d4596f2a8f981344a

    Update helper script to source from the correct location.

    Change-Id: I506de446df05051acc7844323e5b5d5a5495f9bf
    Closes-Bug: #1399622

Changed in charm-glance-simplestreams-sync:
status: In Progress → Fix Committed
James Page (james-page)
Changed in charm-glance-simplestreams-sync:
milestone: none → 19.04
David Ames (thedac)
Changed in charm-glance-simplestreams-sync:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.