-
Notifications
You must be signed in to change notification settings - Fork 137
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
Raw ActiveGate image support #544
Conversation
3b7b4ec
to
d063c19
Compare
d063c19
to
a6a80fe
Compare
src/controllers/activegate/reconciler/statefulset/statefulset_test.go
Outdated
Show resolved
Hide resolved
src/controllers/activegate/reconciler/statefulset/statefulset_test.go
Outdated
Show resolved
Hide resolved
log.Info("tenant has no endpoints") | ||
} | ||
|
||
return tenantInfo, nil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm missing tenantInfo.UUID
validation here, what if it's empty or whitespace-only?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dtclient
should be refactored - for example generated with swagger. I don't want to manually provide sophisticated logic here because it requires maintenance.
If tenantInfo.UUID
is empty, or whitespace-only, we can do nothing except printing warning. I see no value for user here because pod creation is failing anyway with obvious reason.
5112370
to
e147c35
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure that for the entrypoint.sh script settings read from Volumes have higher priority than read from entrypoint.properties file.
src/controllers/activegate/reconciler/statefulset/statefulset.go
Outdated
Show resolved
Hide resolved
96a8537
to
7d2ed58
Compare
7d2ed58
to
220b60a
Compare
220b60a
to
017c36f
Compare
Operator fetches
tenant uuid
,tenant token
andconnection endpoints
for givenapiUrl
and applies it to AG via environment variables and mounted secrets.Operator supports "configured" images as well as "raw" images. Container created using configured image is overridden with parameters as if it was raw.
Added reconciler for parameters mentioned above. Dybakube's
activegate.Reconciler
maintains<dynakube-name>-activegate-tenant-secret
secret.Known limitations:
dtclient
used by operator is not networkzone aware, soconnection endpoints
may be invalid. It's not a regression, though.