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

Release 2018.9 #1761

Closed
wants to merge 3 commits into from
Closed

Release 2018.9 #1761

wants to merge 3 commits into from

Conversation

uajain
Copy link

@uajain uajain commented Oct 22, 2018

No description provided.

@jlebon
Copy link
Member

jlebon commented Oct 22, 2018

Yeah, I suppose it's about that time! There's #1760 that seems low risk, and I'd like to get #1740 into this release as well if possible.

@mwleeds
Copy link
Member

mwleeds commented Oct 22, 2018

This will have to be redone if #1763 is merged. Sorry about that.

@uajain
Copy link
Author

uajain commented Oct 23, 2018

This will have to be redone if #1763 is merged. Sorry about that.

Looks like it's approved but not merged due to some internal error. I'll check again after some time.
@jlebon @cgwalters Do you have anything else(symbol-wise) before I update this release PR?

@rh-atomic-bot
Copy link

☔ The latest upstream changes (presumably ed41822) made this pull request unmergeable. Please resolve the merge conflicts.

@cgwalters
Copy link
Member

@jlebon @cgwalters Do you have anything else(symbol-wise) before I update this release PR?

Nothing from me.

@uajain
Copy link
Author

uajain commented Oct 23, 2018

Updated the PR.

@cgwalters
Copy link
Member

Thanks. @uajain what time zone are you in? Often jlebon and I set up an etherpad where we write release notes together. Both of us are in EST. Do you want to participate in that too? If so, would say 2-3pm EST tomorrow work?

@uajain
Copy link
Author

uajain commented Oct 23, 2018

Hey, @cgwalters I am in India (UTC +5:30) timezone, so I don't think that time suits me. However, I am happy to get involved asynchronously for any matter.

@cgwalters
Copy link
Member

Hm, some test flakes there. But let's do this:

@rh-atomic-bot r+ b0eedc9

We can get the release commit merged, and I'll post a link to an etherpad here for us to collaborate on release notes and then take care of pushing the tag.

@rh-atomic-bot
Copy link

⌛ Testing commit b0eedc9 with merge 77f172e...

rh-atomic-bot pushed a commit that referenced this pull request Oct 24, 2018
Closes: #1761
Approved by: cgwalters
rh-atomic-bot pushed a commit that referenced this pull request Oct 24, 2018
Closes: #1761
Approved by: cgwalters
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@jlebon
Copy link
Member

jlebon commented Oct 24, 2018

Fix in #1765.
Actually, maybe I should just inline it here. Will do that.

src/ostree/ot-main.c Outdated Show resolved Hide resolved
Wrap the `Version` key in the YAML-compatible output of
`ostree --version` with quotes so that it's parsed as a string. The
issues with the previous approach in a nutshell:

```
In [5]: yaml.load("asdf: 2018.10")
Out[5]: {'asdf': 2018.1}
```

It's treating the version number as a floating-point. Now, this is
technically a backwards incompatible change, but given that the previous
approach is inherently broken for our needs, I don't see a way around
breaking it now.
@cgwalters
Copy link
Member

@rh-atomic-bot r+

@rh-atomic-bot
Copy link

📌 Commit 362e7ea has been approved by cgwalters

@rh-atomic-bot
Copy link

⌛ Testing commit 362e7ea with merge 759b099...

rh-atomic-bot pushed a commit that referenced this pull request Oct 24, 2018
Closes: #1761
Approved by: cgwalters
rh-atomic-bot pushed a commit that referenced this pull request Oct 24, 2018
Wrap the `Version` key in the YAML-compatible output of
`ostree --version` with quotes so that it's parsed as a string. The
issues with the previous approach in a nutshell:

```
In [5]: yaml.load("asdf: 2018.10")
Out[5]: {'asdf': 2018.1}
```

It's treating the version number as a floating-point. Now, this is
technically a backwards incompatible change, but given that the previous
approach is inherently broken for our needs, I don't see a way around
breaking it now.

Closes: #1761
Approved by: cgwalters
@cgwalters
Copy link
Member

Hmm, although the commit is after the release commit right? Probably need to rebase it to before?

@jlebon
Copy link
Member

jlebon commented Oct 24, 2018

We could do that. Though note whatever out there was relying on this before should just keep working in this release because 2018.9 still parses fine. So it doesn't have to be in this release.

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: cgwalters
Pushing 759b099 to master...

@cgwalters
Copy link
Member

I didn't start on the rest of release yet (relnotes/tag etc)...will look at this tomorrow.

@cgwalters
Copy link
Member

Set up an etherpad for this release: https://public.etherpad-mozilla.org/p/ostree-2018.9

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

5 participants