-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add additional metadata #4
Comments
normally it would be enough to have it on package level. but if it can change on different versions, it needs to be on the release. i would prefer to have it on package level to reduce data size. |
I think I can normalize most tasks to have the metadata at the package level. Some tasks however, have been forked or transferred to a different publisher, in which case the metadata would have changed at some point. The flutter task is an example of that. It started out from: Then got forked and taken over by: This means that while the @viceice Can the SourceUrl or the Homepage be set at the package level and the only where it need to differ at the release level? Or, since this would be for older versions in almost all cases, simply follow the latest version? eg:
|
Here's a generated sample json file containing:
This amounts to +- 50MB of json. Without the downloadUrl it's about 20MB. https://github.com/renovatebot/azure-devops-marketplace/tree/features/preview-and-deprecation |
i think we don't need the download url |
what about creating a JSON file per taskid and a meta file with mapping only. then the renovate datasources first fetches the metadata file and the taskid files on demand via GitHub CDN. that way we don't need to update renovate to get newer data. |
Sure a possibility. We now have the data and can slice and dice it in any way you want :). |
@JamieMagee @rarkins WDYT? |
An even better option would be to switch over top an API based approach, the TFS/Azure DevOps API has all the information needed for installed tasks: On top of the data the server already knows, the marketplace data can help people find newer versions that weren't installed into their server. For the built-in tasks it usually doesn't make much sense to report availability of versions the server doesn't have, since they only get updated through server upgrades or Microsoft's provisioning process and there isn't much an admin or user can do unless they go the "I'll install them myself route". I have a another project to give admins of Azure DevOps Server the ability to overwrite tasks: |
@viceice I like that idea. We use GitHub pages as a JSON API. @jessehouwing we can still implement the API-based datasource, and prefer it whenever Renovate is running in an Azure DevOps context. For the case where code is hosted in GitHub, but builds are run on Azure DevOps (this is the case for almost all the Microsoft org on GitHub for compliance reasons) we can use the cached data from this repo. |
I was thinking we could add additional config to make the API work even in a GitHub context. The Docs for configuring renovateme on Azure DevOps also show how to add GitHub configuration to generate release notes, this would be similar but the other way round. Eg
|
Yeah, good point. That's another good way to provide an API token. |
So the idea is to generate a json file per identifier and stick the ReleaseRef object in that file, right? That should be relatively simple to do from where we are now. |
yes please. than update the renovate datasource to fetch from github cdn instead of buildin big fat json. |
I've created a feature issue on renovate side for required datasource changes. |
There were a few illegal characters in the identifiers, I solved that with:
for now. |
@JamieMagee any news on the front-end implementation of this? |
Currently the metadata only contains the versions for each task, but it would be relatively simple to add:
Since all this data is available.
This would look something like this:
@JamieMagee @viceice What do you think and am I correct in guessing this would need to fit the
ReleaseResult
structure?Would I need to add the metadata at the
Release
level, or can I just populate the repo and homepage data at theReleaseResult
level?The text was updated successfully, but these errors were encountered: