Project

Profile

Help

Issue #5687

closed

Cannot set aws_default_acl to None using the Ansible installation

Added by joeydumont over 4 years ago. Updated almost 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
Installer - Moved to GitHub issues
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Documentation
Sprint:
Sprint 70
Quarter:

Description

I have installed pulp3 using the Ansible role. However, when trying to set AWS_DEFAULT_ACL = None (in order to configure django-storages properly) through Ansible, I have stumbled into issues.

Writing aws_default_acl: None in the pulp_settings variable gets rendered to AWS_DEFAULT_ACL = "None", not AWS_DEFAULT_ACL = None. My Ansible playbook runs fine, but when I try to sync a repo, the ACL seems to get passed as a file location to django-storages, yielding the somewhat cryptic error for the sync task

"An error occurred (InvalidArgument) when calling the PutObject operation: None"

I have also tried setting aws_default_acl to ~, null and leaving it empty. All of these get rendered as AWS_DEFAULT_ACL = null, which yields an error when trying to run the Ansible playbook:

TASK [pulp-database : Run database auth migrations] ***************************************************************************************************************************************************************
fatal: [xx.xx.xx.xx]: FAILED! => 
</code>
PLAY RECAP ********************************************************************************************************************************************************************************************************
xx.xx.xx.xx             : ok=49   changed=0    unreachable=0    failed=1    skipped=20   rescued=0    ignored=0
Actions #1

Updated by fao89 over 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 61
Actions #2

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 61 to Sprint 62
Actions #3

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 62 to Sprint 63
Actions #4

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 63 to Sprint 64
Actions #5

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 64 to Sprint 65
Actions #6

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 65 to Sprint 66
Actions #7

Updated by dkliban@redhat.com about 4 years ago

  • Description updated (diff)
  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com

Added by dkliban@redhat.com about 4 years ago

Revision 63f1ca23 | View on GitHub

Adds link to Dynaconf syntax for specifying pulp settings

re: #5687 https://pulp.plan.io/issues/5687

Added by dkliban@redhat.com about 4 years ago

Revision 63f1ca23 | View on GitHub

Adds link to Dynaconf syntax for specifying pulp settings

re: #5687 https://pulp.plan.io/issues/5687

Actions #8

Updated by dkliban@redhat.com about 4 years ago

  • Tags Documentation added

After digging into this on the mailing list, we determined that this is a documentation problem[0]. The value for None should be specified as a string "@none None"[1].

Here is the ansible-pulp docs PR[2]. I'll add another one for pulpcore.

[0] https://www.redhat.com/archives/pulp-dev/2020-February/msg00070.html [1] https://dynaconf.readthedocs.io/en/latest/guides/environment_variables.html#precedence-and-type-casting [2] https://github.com/pulp/ansible-pulp/pull/228

Actions #9

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 66 to Sprint 67
Actions #10

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 67 to Sprint 68
Actions #11

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 68 to Sprint 69
Actions #12

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 69 to Sprint 70

Added by Fabricio Aguiar almost 4 years ago

Revision dba0c77c | View on GitHub

Changing S3 example to use dynaconf casting

https://pulp.plan.io/issues/5687 [noissue]

Actions #14

Updated by dkliban@redhat.com almost 4 years ago

  • Status changed from ASSIGNED to MODIFIED
Actions #15

Updated by mdepaulo@redhat.com almost 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #16

Updated by bmbouter almost 4 years ago

  • Category set to Installer - Moved to GitHub issues
  • Tags deleted (Pulp 3 installer)

Also available in: Atom PDF