Add directory option for receptors data

[AaronH88]

SUMMARY

Adding option for setting receptors datadir option which will allow users to set the location of their receptor data. The default is /tmp/receptor/ which may not be the best option for some customers.

ISSUE TYPE

• New or Enhanced Feature

ADDITIONAL INFORMATION

Tested locally by editing my awx-demo.yml with the following var receptor_datadir under spec

---
apiVersion: awx.ansible.com/v1beta1
kind: AWX
metadata:
  name: awx-demo
spec:
  service_type: nodeport
  receptor_datadir: /tmp/aarontest2/

[TheRealHaoLiu]

its not persistent since /var/run/receptor is an empty dir mount

[TheRealHaoLiu]

this will make it that by default we log into the directory?

[AaronH88]

not logs, but it will make us by default save all the contents of the /tmp/receptor/NODEID/WORKID/* to /var/run/receptor instead

[TheRealHaoLiu]

what goes into the datadir? is it just the log?

by default on kube i think receptor logs to STDOUT which can be accessed via kubectl log

having a log dir with no rotation can potentially eat up storages

the way that the PR is written there's no way for people to NOT use the datadir

also there's no persistent storage backing the directory which mean that data will be wiped out if the pod is deleted

[AaronH88]

what goes into the datadir? is it just the log?

by default on kube i think receptor logs to STDOUT which can be accessed via kubectl log

having a log dir with no rotation can potentially eat up storages

the way that the PR is written there's no way for people to NOT use the datadir

also there's no persistent storage backing the directory which mean that data will be wiped out if the pod is deleted

Thanks for the feedback!

The receptor data dir is the location where receptor stores all data it is currently using, as you know (because you taught me) it is currently set to /tmp/receptor/ so the idea here is to make this configurable. I am told some customers would like this 😄

My description on persistent storage is incorrect and I will change it now, thanks for pointing that out.

[TheRealHaoLiu]

O that tmp dir 🤦‍♂️

The description of the PR mention receptor log

[AaronH88]

outside the scope of what is needed, containerised is not needed right now, closing