Closed volkc-basf closed 4 years ago
Just to give you an intuition how the output of the above hcl script could look like ... ;-)
Hey @ttsangAtlassian,
may I ask you for an update? Please let me know if you do need any further help from my side.
Thanks, Chris
@volkc-basf apologies. I will try and take a look today. I have noticed that the formatting is failing to pass in the build test. Could you fix that up?
@jamestoyer Hope it's okay now.
@volkc-basf I've gone ahead and released v2.5.0 of the go-artifactory client library. Could you update to it and check that everything works as expected? 🙂
@jamestoyer: From my side, everything looks very good ;-)
Looking through this I think what you really want are two data sources. One for the File Info and one to get the file.
Do you have a use case for that, where someone is only interested in file metadata? I mean Terraform is built to deploy certain things, hence I believe people are more interested in the raw file rather than in its metadata. But of course, one thing we could discuss is to make the output_path property mandatory.
I can think of a few scenarios whereby you might only want one or the other
In both scenarios you don't want the file at first, but may want it later. Having an optional field which modifies the download behaviour of the file feels like it's making the process more opaque.
Okay, I don't mind. Added a second dataSource called _artifactoryfileinfo, that can be used to access the metadata only and in parallel I set the output_path in _artifactoryfile as required. Docs are also there now.
@jamestoyer: The build is failing with "Please check your payment method or billing status.", could you please check that?
😞
@ttsangAtlassian have actions been turned off for this repo?
It looks good. I'll just wait for @ttsangAtlassian to check what's happened to the GitHub actions then we can get this merged.
Perfect, thank you for your help!
I have a bad feeling we are being bitten by this https://github.community/t5/GitHub-Actions/GitHub-Actions-workflows-can-t-be-executed-on-this-repository/td-p/38153 It looks like when actions went GA on Nov 14th, legacy plans had actions disabled for all plans including OSS. I'll try and chase this up
I have a bad feeling we are being bitten by this https://github.community/t5/GitHub-Actions/GitHub-Actions-workflows-can-t-be-executed-on-this-repository/td-p/38153 It looks like when actions went GA on Nov 14th, legacy plans had actions disabled for all plans including OSS. I'll try and chase this up
@ttsangAtlassian: How is it? Hope you have some positive news for me l;-)
@volkc-basf I'm sorry we are still trying to work out what is happening with our licenses on our end. For now I am happy to remove the build checks to get this merged.
This PR implements https://github.com/atlassian/terraform-provider-artifactory/issues/63, means it will add support to download artifacts using terraform. It won't compile as long as https://github.com/atlassian/go-artifactory/pull/19 is not merged.
Best, Chris