Skip to content

Commit b3019c4

Browse files
authored
[Job graph] Include respository from pipeline selector in filtering (#29191)
## Summary & Motivation as titled ## How I Tested These Changes Logged into an organization where job names are not unique across deployments and saw that we correctly filtered assets to the right job by including the repository location and name.
1 parent 38495c3 commit b3019c4

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

Diff for: js_modules/dagster-ui/packages/ui-core/src/asset-graph/useAssetGraphData.tsx

+7-1
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,13 @@ export function useAssetGraphData(opsQuery: string, options: AssetGraphFetchScop
156156
const repoFilteredNodes = useMemo(() => {
157157
let matching = allNodes;
158158
if (pipelineSelector) {
159-
matching = matching.filter((node) => node.jobNames.includes(pipelineSelector.pipelineName));
159+
matching = matching.filter((node) => {
160+
return (
161+
node.jobNames.includes(pipelineSelector.pipelineName) &&
162+
node.repository.name === pipelineSelector.repositoryName &&
163+
node.repository.location.name === pipelineSelector.repositoryLocationName
164+
);
165+
});
160166
}
161167
if (groupSelector) {
162168
matching = matching.filter((node) => {

0 commit comments

Comments
 (0)