Skip to content

feat: support pipeline variable import/export #4081

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.

Sign up for GitHub

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

Merged
luobily merged 4 commits into from
Jul 23, 2026
Merged

feat: support pipeline variable import/export #4081

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
27905a4
feat: support pipeline variable import export
ruoqianfengshao Jul 6, 2026
0b01419
feat: add msp route preview locale
ruoqianfengshao Jul 7, 2026
46bfc21
Revert "feat: add msp route preview locale"
ruoqianfengshao Jul 9, 2026
c67143b
fix: avoid importing encrypted pipeline variables
ruoqianfengshao Jul 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 26 additions & 1 deletion shell/app/modules/application/pages/settings/components/app-variable-config.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,14 @@ const configTypeMap = {
deploy: 'DeployConfig',
};

const convertExportConfigs = (configs: PIPELINE_CONFIG.ConfigItem[] = []) =>
configs
.filter(({ encrypt }) => !encrypt)
.map(({ key, value }) => ({
key,
value,
}));

interface IProps {
envToNs: Obj<string>;
configs: PIPELINE_CONFIG.ConfigItemMap;
Expand Down Expand Up @@ -370,7 +378,7 @@ const VariableConfig = ({
<Button type="primary" className="mb-3" ghost onClick={() => openModal(null, env)}>
{i18n.t('dop:Add Variable')}
</Button>
{configType === configTypeMap.deploy && (
{importConfig && exportConfig && (
<>
<Button
type="primary"
Expand Down Expand Up @@ -509,6 +517,23 @@ export const PipelineConfig = () => {
addConfig={configStore.addConfigs}
updateConfig={configStore.updateConfigs}
deleteConfig={configStore.removeConfigWithoutDeploy}
importConfig={({ query, configs }) => {
const encryptedKeys = new Set(
fullConfigs[query.namespace_name]?.filter(({ encrypt }) => encrypt).map(({ key }) => key),
);
return configStore.updateConfigs({
query: { ...query, encrypt: false },
configs: JSON.parse(configs)
.filter(({ key }: PIPELINE_CONFIG.ConfigItem) => !encryptedKeys.has(key))
.map((item: PIPELINE_CONFIG.ConfigItem) => ({
...item,
encrypt: false,
type: item.type || typeMap.kv,
})),
batch: true,
});
}}
exportConfig={({ namespace_name }) => Promise.resolve(convertExportConfigs(fullConfigs[namespace_name]))}

@luobily luobily Jul 16, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

导入可能将加密变量覆盖为明文：
配置以 decrypt: false 获取，导入窗口却预填全部变量；提交时再统一设置 encrypt: false。用户只需打开导入窗口并确认，就可能用未解密的返回值覆盖密文，同时取消加密属性。

@ruoqianfengshao ruoqianfengshao Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修复：pipeline 变量导出/导入弹窗预填时会过滤掉 encrypt=true 的变量；导入提交时也会根据当前 namespace 的已有配置跳过加密 key，避免打开弹窗直接确认时覆盖加密变量或取消加密属性。

uploadTip={i18n.t('dop:upload-file-tip')}
/>
);
Expand Down
Loading