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

CMDB日志查询的ciName和ciTypeName没有内容 #144

Closed
yolanda0209 opened this issue Sep 8, 2019 · 3 comments
Closed

CMDB日志查询的ciName和ciTypeName没有内容 #144

yolanda0209 opened this issue Sep 8, 2019 · 3 comments
Assignees
Labels
bug Something isn't working PBR Needed

Comments

@yolanda0209
Copy link

描述您遇到的bug
CMDB日志查询的ciName和ciTypeName没有内容

如何重现

  1. 登录 http://111.230.161.237:8080/cmdb
  2. 打开系统》日志查询页面,ciName和ciTypeName没有内容

预期行为
CMDB日志查询的ciName和ciTypeName显示相应的内容

截图
image

附加

@yolanda0209 yolanda0209 added the bug Something isn't working label Sep 8, 2019
@SimonKwok
Copy link
Contributor

cmdb日志只记录调用参数里有的信息,调用参数里缺失的信息是不会去query数据库的(AOP切面功能应尽量不影响主服务线程的性能)。这个逻辑之前跟@chaneyliu 讨论过,当时的结论是先这么做;如果现在重新检讨,那么解决这个问题有两个方案:
A,在日志功能里根据ciTypeId去查一次db,获取ciTypeName,缺点是会让正常的业务流程多一步DB操作
B,修改接口,让接口多接收一个ciTypeName的参数,然后让调用者传过来,缺点是api的提供方(cmdb)和使用方(wecube)都需要同步改。

@chaneyliu
Copy link

先这样处理就可以。

@jordanzhangsz jordanzhangsz added this to Not Plan in CMDB开源版本1.2 & hot fix via automation Sep 22, 2019
@jordanzhangsz jordanzhangsz moved this from Not Plan to To Do in CMDB开源版本1.2 & hot fix Sep 23, 2019
@jordanzhangsz
Copy link
Collaborator

跟超哥同步过,解决方案如下:

  1. 暂时去掉ciName和ciTypeName两列(数据库和页面展示)。
  2. 日志内容支持模糊查询,一般用户都是通过他们自己录入的数据关键字进行模糊查询的,基本能满足大部分用户的查询需求。

@irvinezhao irvinezhao moved this from To Do to In Progress in CMDB开源版本1.2 & hot fix Sep 24, 2019
irvinezhao added a commit that referenced this issue Sep 24, 2019
…s_of_log

#144 don't popular fields without log data for now
CMDB开源版本1.2 & hot fix automation moved this from In Progress to Done Sep 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working PBR Needed
Projects
No open projects
Development

No branches or pull requests

5 participants