Skip to content

Log CI linter times

Furkan Ayhan requested to merge 349968-log-linter into master

What does this MR do and why?

Currently, the pipeline logger (!72996 (merged)) works only for pipeline creations. In this MR, we implement it to the CI linter.

It only works when the project is provided to the linter.

Related to #349968 (closed), #349971 (closed)

More context: #349968 (comment 839896508)

Screenshots or screen recordings

Feature.enable(:ci_pipeline_creation_logger)
tail -f log/application_json.log
{
  "severity": "INFO",
  "time": "2022-02-14T09:23:38.387Z",
  "correlation_id": "01FVVRPN8NFTE6S2MGCX1911GJ",
  "class": "Gitlab::Ci::Pipeline::Logger",
  "pipeline_creation_caller": "Gitlab::Ci::Lint",
  "project_id": 22,
  "pipeline_persisted": false,
  "pipeline_source": "unknown",
  "pipeline_creation_service_duration_s": 5.334216999821365,
  "config_build_variables_duration_s": {
    "count": 1,
    "min": 0.27937000012025237,
    "max": 0.27937000012025237,
    "avg": 0.27937000012025237
  },
  "config_build_context_duration_s": {
    "count": 1,
    "min": 0.280734000261873,
    "max": 0.280734000261873,
    "avg": 0.280734000261873
  },
  "config_yaml_load_duration_s": {
    "count": 1,
    "min": 0.005250999704003334,
    "max": 0.005250999704003334,
    "avg": 0.005250999704003334
  },
  "config_mapper_normalize_duration_s": {
    "count": 7,
    "min": 2.00001522898674e-06,
    "max": 1.5000347048044205e-05,
    "avg": 6.428653640406472e-06
  },
  "config_mapper_rules_duration_s": {
    "count": 7,
    "min": 1.000240445137024e-06,
    "max": 0.017646000254899263,
    "avg": 0.002525285751159702
  },
  "config_mapper_wildcards_duration_s": {
    "count": 7,
    "min": 0.0,
    "max": 0.12409100029617548,
    "avg": 0.017728285863995552
  },
  "config_mapper_variables_duration_s": {
    "count": 28,
    "min": 2.00001522898674e-06,
    "max": 0.0009200000204145908,
    "avg": 4.4035692034023146e-05
  },
  "config_mapper_verify_duration_s": {
    "count": 28,
    "min": 5.999580025672913e-06,
    "max": 5.7999975979328156e-05,
    "avg": 1.3357111518936498e-05
  },
  "config_mapper_select_duration_s": {
    "count": 28,
    "min": 0.001997999846935272,
    "max": 0.1404900001361966,
    "avg": 0.0199087143077382
  },
  "config_mapper_process_duration_s": {
    "count": 2,
    "min": 0.022078000009059906,
    "max": 0.6801499999128282,
    "avg": 0.35111399996094406
  },
  "config_external_verify_duration_s": {
    "count": 28,
    "min": 9.997747838497162e-07,
    "max": 2.4999957531690598e-05,
    "avg": 3.00002284348011e-06
  },
  "config_external_merge_duration_s": {
    "count": 28,
    "min": 4.00003045797348e-06,
    "max": 5.700020119547844e-05,
    "avg": 1.6535732096859386e-05
  },
  "config_external_process_duration_s": {
    "count": 1,
    "min": 0.6891349996440113,
    "max": 0.6891349996440113,
    "avg": 0.6891349996440113
  },
  "config_yaml_extend_duration_s": {
    "count": 1,
    "min": 0.07284900033846498,
    "max": 0.07284900033846498,
    "avg": 0.07284900033846498
  },
  "config_tags_resolve_duration_s": {
    "count": 1,
    "min": 0.0634129997342825,
    "max": 0.0634129997342825,
    "avg": 0.0634129997342825
  },
  "config_stages_inject_duration_s": {
    "count": 1,
    "min": 0.03369500022381544,
    "max": 0.03369500022381544,
    "avg": 0.03369500022381544
  },
  "config_expand_duration_s": {
    "count": 1,
    "min": 0.8830399997532368,
    "max": 0.8830399997532368,
    "avg": 0.8830399997532368
  },
  "config_compose_duration_s": {
    "count": 1,
    "min": 3.105483999941498,
    "max": 3.105483999941498,
    "avg": 3.105483999941498
  },
  "yaml_process_duration_s": {
    "count": 1,
    "min": 4.754203999880701,
    "max": 4.754203999880701,
    "avg": 4.754203999880701
  }
}

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Furkan Ayhan

Merge request reports