Skip to content
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

针对nginx+keeplived的vip映射管理方案 #154

Merged
merged 4 commits into from
Aug 21, 2023
Merged

Conversation

quanbisen
Copy link
Contributor

@quanbisen quanbisen commented Aug 18, 2023

目前我们公司内部使用遇到了vip的使用场景,要求多环境的nginx配置操作需要同步,我目前的方案是在环境的管理页面添加该环境是否需要同步的开关,并配置需要同步的操作API正则表达式。当开关开启且操作的接口完全匹配上,就会把该操作也作用到该环境上,代码用了一个middleware实现了,修改了部分逻辑。作者评估一下该方案如何,如果有bug等问题也可以联系我帮忙修复,希望该merge能被合并到下一版本中去。

详细变更内容如下:

  1. 环境管理配置添加是否操作同步功能,自定义配置接口API正则,针对非GET请求的,如一些配置管理、证书管理和重载等操作应用到该环境上,主要业务场景是nginx配置了KeepAlive,做主备的,需要保持配置同步;
  2. Nginx操作restart,reload,check添加错误处理,如果命令不存在,原忽略错误提示会返回200;
  3. 修复Nginx运行状态显示错误,增加对nginx.pid文件的大小判断,目前发现systemctl stop nginx某些情况下会存在nginx.pid文件存在,大小为0的情况;
  4. 站点复制默认填充原配置名称,添加了逻辑,若配置本身是enabled,复制过去应该也是enabled,相当于配置下发,如在主节点上操作测试通过了,然后复制到其它环境,reload一次就可以了;
  5. 组件StdDataEntry添加了一个show属性回调入口,动态控制模态框的FormItem是否显示;

@0xJacky
Copy link
Owner

0xJacky commented Aug 18, 2023

好的,这两天我评估一下,感谢您的贡献!

@quanbisen
Copy link
Contributor Author

好的,这两天我评估一下,感谢您的贡献!

thanks

@0xJacky
Copy link
Owner

0xJacky commented Aug 21, 2023

我先合并到 dev,我想后面改成列表选择,替代掉正则表达式。 因为正则表达式可能对普通用户不太友好😂

@0xJacky 0xJacky merged commit 6ede443 into 0xJacky:dev Aug 21, 2023
10 checks passed
@quanbisen
Copy link
Contributor Author

列表选择确实会好一点,根据功能模块做成列表估计可行。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants