-
Notifications
You must be signed in to change notification settings - Fork 510
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[operator] ao-logs going to go away - find replacement #2611
Comments
from slack:
|
The way the ansible operator base image logs its output changed. See: operator-framework/operator-sdk#2321 (went in 0.14.0) This is because the ao-logs are not available in RHEL8/UBI8 - see operator-framework/operator-sdk#2007 The ao-logs script is still available in 0.16.0 but we don't need it with the way 0.16.0 outputs the ansible logs now. The currently productized ansible base images are well-behind 0.16.0, though. We can move ahead with the implementation of this in the community builds, but not for the productized builds because we need 0.16.0 in order to replace the log output we lose when not using ao-logs. As of this writing: |
PR #2644 is ready but we will need a second PR that removes the ao-logs container from the kiali-ossm metadata - this can only happen when a productized version of the ansible base image is up to at least 0.16.0. |
Re-opening because we need to do something similar for kiali-ossm metadata. |
The latest 4.4 also is not at the version we need as of right now. See:
|
Base image 4.5 is now at 0.16.0 which provides the alternative to the ao-logs. We can implement this now. |
this also fixes some other metadata for the next release based on the new sdk
* remove ao-logs - this fixes: kiali/kiali#2611 this also fixes some other metadata for the next release based on the new sdk * convert to using the new k8s_info. fixes: kiali/kiali#2073
the ansible operator uses the ao-logs script to log the ansible messages in the ansible container.
However, when you start the operator now, the first line that spits out is:
So we need to find out what the alternative is and switch to that.
The text was updated successfully, but these errors were encountered: