-
Notifications
You must be signed in to change notification settings - Fork 989
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
[bug] artifacts.properties doesn't apply to internal Artifactory files #8418
Comments
I believe this is because |
@memsharded do you think my assessment is correct? If so, then the only other course of action possible would be to suggest it as an artifactory feature request. |
Yes, correct. Those files are internal implementation details of the server side. They are not downloaded as part of the packages, nor uploaded. They are not known by Conan client at all, it is impossible that it will set properties. For cleanup policies, maybe another approach is necessary, it depends if those scripts use the JFrog-CLI or they can also use Conan or is pure API based. This (cleanup policies) is something we want to start implementing as a Conan command layer as soon as Conan 2.0 is in beta stage, but that will take some time. |
From perspective of Conan API integrity, the question here is: should the properties apply to all relevant Conan repository files? If yes, then this would be Artifactory request (how do you do that?) |
Probably not. They are still "metadata", not repository files, and they do not belong to the Conan API. The Conan API only define things at the package level, like The fact that they look like files on the server side is an implementation detail. Lets say that you try to use those properties to copy those files to another repository. You cannot copy those files to another repository, they will be generated automatically in the other repository when moving the Conan packages, but probably forcing a copy could be undefined behavior. Those files should most likely not be copied, removed, modified, or uploaded manually to Artifactory. Maybe Artifactory team could clarify. For Artifactory support and requests, it depends:
Are those scripts client side with JFrog-CLI? Or using the Artifactory API? Or are they Artifactory user plugins server side? |
@klimkin are you still working on or around this issue? If so, are you able to answer any of the questions from @memsharded ? |
This issue is outdated, closing it. artifacts.properties file has been removed in Conan 2.0 completely. |
artifacts.properties
doesn't apply to internal Artifactory files. This causes issues for Artifactory tools like cleanup scripts that scan properties for retention indication.Environment Details (include every applicable attribute)
Steps to reproduce:
.timestamp
andindex.json
files don't have the property setLogs
Conan trace log shows property
retention.days
is part of PUT header:On artifactory, property is set for uploaded files:
But not for internal files:
The text was updated successfully, but these errors were encountered: