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

Update maas3 daily stream url, add maas3-release. #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smoser
Copy link
Owner

@smoser smoser commented Feb 17, 2022

The maas3 daily stream url changed, and these tools were broken
as a result. The bug really resided in the tools direct usage
of the maas images file rather than going through index.sjson.

@smoser
Copy link
Owner Author

smoser commented Feb 17, 2022

The failure that this is fixing is below.
The old url was:
http://images.maas.io/ephemeral-v3/daily/streams/v1/com.ubuntu.maas:daily:v3:download.sjson
current is:
http://images.maas.io/ephemeral-v3/daily/streams/v1/com.ubuntu.maas:stable:v3:download.sjson

$ snap list image-status
Name          Version  Rev  Tracking     Publisher  Notes
image-status  0.1      45   latest/edge  smoser     -

$ image-status maas3
Traceback (most recent call last):
  File "/snap/image-status/45/usr/bin/sstream-query", line 147, in <module>
    main()
  File "/snap/image-status/45/usr/bin/sstream-query", line 137, in main
    tmirror.sync(smirror, path)
  File "/snap/image-status/45/usr/lib/python3/dist-packages/simplestreams/mirrors/__init__.py", line 85, in sync
    content, payload = reader.read_json(path)
  File "/snap/image-status/45/usr/lib/python3/dist-packages/simplestreams/mirrors/__init__.py", line 41, in read_json
    with self.source(path) as source:
  File "/snap/image-status/45/usr/lib/python3/dist-packages/simplestreams/contentsource.py", line 74, in __enter__
    self.open()
  File "/snap/image-status/45/usr/lib/python3/dist-packages/simplestreams/contentsource.py", line 131, in open
    self.fd = self._open()
  File "/snap/image-status/45/usr/lib/python3/dist-packages/simplestreams/contentsource.py", line 127, in _open
    raise myerr
OSError: Unable to open http://images.maas.io/ephemeral-v3/daily/streams/v1/com.ubuntu.maas:daily:v3:download.sjson. mirrors=[]

bin/u-stool Outdated Show resolved Hide resolved
@smoser
Copy link
Owner Author

smoser commented Feb 18, 2022

@sparkiegeek thanks for reviewing. Can you verify that I did what you wanted ?

@sparkiegeek
Copy link

sparkiegeek commented Feb 23, 2022

The candidate image stream is found at http://images.maas.io/ephemeral-v3/candidate/streams/v1/com.ubuntu.maas:candidate:v3:download.sjson

The stable image stream is found at http://images.maas.io/ephemeral-v3/stable/streams/v1/com.ubuntu.maas:stable:v3:download.sjson

daily is redirected to stable, so there is no difference between them; but candidate represents what daily used to cover

I tried to suggest the fix to use candidate, but that's not what you picked up :/

The maas3 daily stream url changed, and these tools were broken
as a result.  The bug really resided in the tools direct usage
of the maas images file rather than going through index.sjson.

For 'daily', use the 'candidate' stream at path:

   candidate/streams/v1/com.ubuntu.maas:candidate:v3:download.sjson

For stable continue using stable.
@smoser
Copy link
Owner Author

smoser commented Feb 23, 2022

The candidate image stream is found at http://images.maas.io/ephemeral-v3/candidate/streams/v1/com.ubuntu.maas:candidate:v3:download.sjson

The stable image stream is found at http://images.maas.io/ephemeral-v3/stable/streams/v1/com.ubuntu.maas:stable:v3:download.sjson

daily is redirected to stable, so there is no difference between them; but candidate represents what daily used to cover

I tried to suggest the fix to use candidate, but that's not what you picked up :/

i think what i did was fail to commit what was in my local tree.

@sparkiegeek, This looks better now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants