Skip to content
This repository was archived by the owner on Oct 10, 2020. It is now read-only.

[merged] update default trust policy file#822

Closed
aweiteka wants to merge 1 commit intoprojectatomic:masterfrom
aweiteka:update-default-policy
Closed

[merged] update default trust policy file#822
aweiteka wants to merge 1 commit intoprojectatomic:masterfrom
aweiteka:update-default-policy

Conversation

@aweiteka
Copy link
Contributor

Installed default policy is https://raw.githubusercontent.com/projectatomic/skopeo/master/default-policy.json Match that file if it doesn't exist.

@rhatdan
Copy link
Member

rhatdan commented Jan 10, 2017

I am not crazy about duplicating this code in two different places, but I don't see an option. Will
atomic trust reset
Rewrite this file?

@aweiteka
Copy link
Contributor Author

I am not crazy about duplicating this code in two different places, but I don't see an option.

I agree, although there's no new functionality here. I'm just updating the default and making it a bit cleaner.

Will atomic trust reset Rewrite this file?

Yes

@aweiteka aweiteka force-pushed the update-default-policy branch 3 times, most recently from 35b2705 to c8f1b93 Compare January 12, 2017 21:45
@rhatdan
Copy link
Member

rhatdan commented Jan 13, 2017

@rh-atomic-bot restest

@rhatdan
Copy link
Member

rhatdan commented Jan 13, 2017

@rh-atomic-bot test

Signed-off-by: Aaron Weitekamp <aweiteka@redhat.com>
@aweiteka aweiteka force-pushed the update-default-policy branch from 49628c5 to a5a884f Compare January 13, 2017 19:41
@aweiteka
Copy link
Contributor Author

@rhatdan should be all set now.

@aweiteka aweiteka mentioned this pull request Jan 13, 2017
@rhatdan
Copy link
Member

rhatdan commented Jan 16, 2017

@rh-atomic-bot r+

@rh-atomic-bot
Copy link

📌 Commit a5a884f has been approved by rhatdan

@rh-atomic-bot
Copy link

⌛ Testing commit a5a884f with merge ef8b07e...

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: rhatdan
Pushing ef8b07e to master...

@rh-atomic-bot rh-atomic-bot changed the title update default trust policy file [merged] update default trust policy file Jan 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants