[ https://issues.apache.org/jira/browse/YUNIKORN-928?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ryan Lo reassigned YUNIKORN-928: -------------------------------- Assignee: (was: Ryan Lo) > refactor admission controller code > ---------------------------------- > > Key: YUNIKORN-928 > URL: https://issues.apache.org/jira/browse/YUNIKORN-928 > Project: Apache YuniKorn > Issue Type: Improvement > Components: shim - kubernetes > Reporter: Wilfred Spiegelenburg > Priority: Major > > During the logging review in YUNIKORN-927 I noticed a possible nil pointer > and a number of sub optimal code paths: > * in the serve function we check for a nil {{AdmissionReview.Request}} this > is after all the calls are made that access the same object without nil > checks. We either do not need this or have possible nil pointers > * we should unwrap the {{AdmissionReview}} and use the {{AdmissionRequest}} > in our internal calls > * we create {{AdmissionResponse}} objects all over the code (9 places) we > need to refactor that into a single function that takes the fields: uid, > resultMessage and patch and returns the correct object -- This message was sent by Atlassian Jira (v8.20.1#820001) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@yunikorn.apache.org For additional commands, e-mail: issues-h...@yunikorn.apache.org