This message was deleted.
# opals
Slackbot
03/10/2023, 3:43 PMThis message was deleted.
o
Or Weis
03/10/2023, 3:49 PMHi @loan75.
Yes of course we test OPAL before releasing a version, but we do admit there's much room to improve that current test suite that we have.
We'd love your (and anyone elses) help with it.
Would you be willing to write some tests? 🙏
Not sure what you mean by very hidden documentation: it's hosted and navigable at docs.opal.ac
OPAL_DATA_CONFIG_SOURCES is not optional,
l
loan75
03/10/2023, 3:51 PMOPAL_DATA_CONFIG_SOURCES is not optional but it can be empty. All policies do not need to rely on data.
loan75
03/10/2023, 3:53 PM(also, some data can be loaded directly from the policies)
o
Or Weis
03/10/2023, 3:54 PMYeah, I agree that there's a regression here, though we (and most users of OPAL) always load some data as well, so I guess @Ori Shavit and @Asaf Cohen overlooked the scenario.
We can look at providing a fix here, perhaps in the interim as a stop gap solution, load some dummy data?
l
loan75
03/10/2023, 3:56 PMwell i am hosting like 5 OPAL cluster. I agree that all have data except one than only rely on static data that are imported by the policies.
🤘 1
loan75
03/10/2023, 4:08 PMwell i put a dummy configSources as a work around for now.
loan75
03/10/2023, 4:09 PMdataConfigSources:
config:
entries: [
{
"url": "",
"topics": ["policy_data"],
"data": {},
"dst_path": "/dummy"
}
]
o
Or Weis
03/10/2023, 4:10 PMQuite literal :)
p
Philip Claesson
03/14/2023, 1:20 PMFWIW, I also got confused by this when I was deploying OPAL from the helm chart the first time (had to set flags
OPAL_DATA_UPDATER_ENABLED=0OPAL_OPA_HEALTH_CHECK_POLICY_ENABLED=1r
Ro'e Katz
03/20/2023, 4:40 PMquick update regarding those issues around the health check: https://github.com/permitio/opal-helm-chart/issues/31#issuecomment-1476554198
👍 2
Ro'e Katz
03/28/2023, 8:12 AMClosed the issue: https://github.com/permitio/opal-helm-chart/issues/31#issuecomment-1486399982
Bottom lines:
1. You don’t have to set
OPAL_OPA_HEALTH_CHECK_POLICY_ENABLED: TrueOPAL_DATA_CONFIG_SOURCESOPAL_DATA_UPDATER_ENABLED: False