Skip to content

Commit 7b48b2b

Browse files
authored
fix: Try fix wrong success oidc auth (#516) (#517)
1 parent 4e2141d commit 7b48b2b

File tree

2 files changed

+14
-5
lines changed

2 files changed

+14
-5
lines changed

Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM epamedp/headlamp:0.22.40
1+
FROM epamedp/headlamp:0.22.41
22

33
COPY --chown=100:101 assets/ /headlamp/frontend
44
COPY --chown=100:101 dist/main.js /headlamp/plugins/edp/main.js

src/pages/pipeline-details/view.tsx

+13-4
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export const PageView = () => {
4040
const theme = useTheme();
4141

4242
const renderPageContent = React.useCallback(() => {
43-
const hasReserveLogs = !logs.isLoading && (!logs.error || !logs.data.all.length);
43+
const hasReserveLogs = !logs.isLoading && !logs.error && !!logs?.data?.all?.length;
4444

4545
if (pipelineRun.error) {
4646
return (
@@ -54,9 +54,18 @@ export const PageView = () => {
5454
fontSize={theme.typography.pxToRem(14)}
5555
color="#596D80"
5656
>
57-
{hasReserveLogs
58-
? 'No pipeline runs were found for the requested resource. Logs have been retrieved from OpenSearch.'
59-
: 'No logs were found for the requested pipeline run. This might have been caused by environment cleanup. Please ensure you have checked the correct resource.'}
57+
{hasReserveLogs ? (
58+
'No pipeline runs were found for the requested resource. Logs have been retrieved from OpenSearch.'
59+
) : (
60+
<>
61+
<p>
62+
No logs were found for the requested pipeline run. This might have been
63+
caused by environment cleanup. Please ensure you have checked the correct
64+
resource.
65+
</p>
66+
{logs.error && logs.error?.message}
67+
</>
68+
)}
6069
</Typography>
6170
</Stack>
6271
</Stack>

0 commit comments

Comments
 (0)