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

Incorrect version comparison #66

Closed
ralphbean opened this issue Dec 17, 2014 · 7 comments
Closed

Incorrect version comparison #66

ralphbean opened this issue Dec 17, 2014 · 7 comments
Labels

Comments

@ralphbean
Copy link
Contributor

I just saw this message go by the fedmsg bus, which seems incorrect.

@ralphbean
Copy link
Contributor Author

We need to write a unit test that duplicates this scenario before fixing it, so we can be sure we have the case isolated.

@pypingou
Copy link
Member

pypingou commented Mar 4, 2015

The problem:
"old_version": "1.3.0", "upstream_version": "1.0.3",
A possible cause is that upstream finally uploaded an older release to pypi (that was missing before)

Another case where we would see this: upstream pushes an older tag to the git repo

In these case, anitya will announce the new release and will say on the website that something odd was found since the new version seems older than the previous latest version.

@ralphbean
Copy link
Contributor Author

So, is any fix needed in anitya? Or... should we just fix the language generated by fedmsg_meta_fedora_infrastructure?

@pypingou
Copy link
Member

pypingou commented Mar 4, 2015

We can go both ways (but I think I'd like to keep it in anitya, this feature is helpful when a project has bugfix releases for older version, think 1.2.x while 2.y is out), a third option could be to publish a message saying version.new instead of version.update when the ordering seems odd.

@ralphbean
Copy link
Contributor Author

Yeah, either option sounds good to me. To restate, either:

  • Modify fedmsg_meta_fedora_infrastructure to have a nicer message when old bugfix releases are discovered.
  • Modify anitya to send a different kind of message when such bugfix releases are discovered.

(honestly, I prefer option 1 since it will have the least amount of impact on other systems that might already be subscribed to anitya's message stream)

@pypingou
Copy link
Member

+1 on the first option for me as well :-)

-------- Message d'origine --------
De : Ralph Bean notifications@github.com
Date :10/03/2015 02:33 (GMT+01:00)
À : fedora-infra/anitya anitya@noreply.github.com
Cc : Pierre-Yves Chibon pingou@pingoured.fr
Objet : Re: [anitya] Incorrect version comparison (#66)

Yeah, either option sounds good to me. To restate, either:

Modify fedmsg_meta_fedora_infrastructure to have a nicer message when old bugfix releases are discovered.
Modify anitya to send a different kind of message when such bugfix releases are discovered.
(honestly, I prefer option 1 since it will have the least amount of impact on other systems that might already be subscribed to anitya's message stream)


Reply to this email directly or view it on GitHub.

pypingou added a commit that referenced this issue Mar 25, 2015
By odd I mean that the new version found appears to be older than the
last version we had (ie: upstream released a version 2 after having
released a version 3).

This should help fixing #66
@ralphbean
Copy link
Contributor Author

Noting that this only requires a change to fedmsg_meta now to use that new field to make the description more nice.

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

No branches or pull requests

2 participants