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.

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

Jump to bottom

fix(ci): fix cache reference during rpm delivery 2110 #564

Merged
merged 3 commits into from
Feb 13, 2023
Merged

fix(ci): fix cache reference during rpm delivery 2110 #564

merged 3 commits into from
Feb 13, 2023

Conversation

tuntoja
Copy link
Contributor

@tuntoja tuntoja commented Feb 10, 2023

Description

Please include a short resume of the changes and what is the purpose of PR. Any relevant information should be added to help:

  • QA Team (Quality Assurance) with tests.
  • reviewers to understand what are the stakes of the pull request.

Fixes # (issue)

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 21.10.x
  • 22.04.x
  • 22.10.x
  • 23.04.x (master)

How this pull request can be tested ?

Please describe the procedure to verify that the goal of the PR is matched. Provide clear instructions so that it can be correctly tested.

Any relevant details of the configuration to perform the test should be added.

Checklist

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have made corresponding changes to the documentation.
  • I have rebased my development branch on the base branch (master, maintenance).

@tuntoja tuntoja changed the title Fix cache rpm delivery 2110 fix(ci): fix cache reference during rpm delivery 2110 Feb 10, 2023
@tuntoja tuntoja merged commit 7009b55 into dev-21.10.x Feb 13, 2023
@tuntoja tuntoja deleted the fix-cache-rpm-delivery-2110 branch February 13, 2023 09:03
paloth added a commit that referenced this pull request Feb 20, 2023
@paloth @denliA
@bouda1 @jean-christophe81 @chgautier @sc979 @tuntoja @kduret
Release #18995#
b6d76f2 
* enh(broker): cbd with multiargs and robot tests (#306)

-enh(broker): cbd with multiargs and robot tests
-Validate with tests/broker/command-line.robot
-Update README.md
-Common.py : fix find in log with timeout

REFS: MON-13901

* fix(broker/bam): downtimes on kpi can be more than one

* enh(tests/bam): new tests to reproduce a bug
* chore(doc): CHANGELOG updated and version set to 21.10.3
* chore(tests): README updated

REFS: MON-14091

* fix(broker/engine): grpc api can be changed through configuration. Otherwise it is fixed to localhost (#320)

REFS: MON-13904

* fix(broker/rrd): rebuild fixed in 21.10 (#340)

* Also an issue due to conan fixed, python 3.8 is needed.

REFS: MON-14092

* fix(ci): issues with conan fixed

* fix(ci/scripts): conan bad path

* fix(broker/bam): overlapping downtimes on kpi service are well handled now. (#341)

REFS: MON-14091

* fix(ci/tests): missing dependencies for python38

* fix(broker/lua): lua stream connector accepts empty parameters(21.10) (#359)

REFS: MON-13875

* fix(broker/muxer): poller waits at most 5s to send goodbye to broker before shutdown (#360)

REFS: MON-14511

* fix(broker/config): if failovers are badly defined, error logs are raised but this does not stop broker (#365)

REFS: MON-6986

* chore(release):rebase dev-21.10.x on 21.10.x  (#388)

* chore(release): merge release 21.10.next into 21.10.x (#386)

* enh(broker): cbd with multiargs and robot tests (#306)

-enh(broker): cbd with multiargs and robot tests
-Validate with tests/broker/command-line.robot
-Update README.md
-Common.py : fix find in log with timeout

REFS: MON-13901

* fix(broker/bam): downtimes on kpi can be more than one

* enh(tests/bam): new tests to reproduce a bug
* chore(doc): CHANGELOG updated and version set to 21.10.3
* chore(tests): README updated

REFS: MON-14091

* fix(broker/engine): grpc api can be changed through configuration. Otherwise it is fixed to localhost (#320)

REFS: MON-13904

* fix(broker/rrd): rebuild fixed in 21.10 (#340)

* Also an issue due to conan fixed, python 3.8 is needed.

REFS: MON-14092

* fix(ci): issues with conan fixed

* fix(ci/scripts): conan bad path

* fix(broker/bam): overlapping downtimes on kpi service are well handled now. (#341)

REFS: MON-14091

* fix(ci/tests): missing dependencies for python38

* fix(broker/lua): lua stream connector accepts empty parameters(21.10) (#359)

REFS: MON-13875

* fix(broker/muxer): poller waits at most 5s to send goodbye to broker before shutdown (#360)

REFS: MON-14511

Co-authored-by: denliA <91119589+denliA@users.noreply.github.com>
Co-authored-by: David Boucher <dboucher@centreon.com>
Co-authored-by: jean-christophe81 <98889244+jean-christophe81@users.noreply.github.com>

* fix(broker/config): if failovers are badly defined, error logs are raised but this does not stop broker (#365)

REFS: MON-6986

Co-authored-by: denliA <91119589+denliA@users.noreply.github.com>
Co-authored-by: David Boucher <dboucher@centreon.com>
Co-authored-by: jean-christophe81 <98889244+jean-christophe81@users.noreply.github.com>

* fix(broker/config): file failover should not stop broker

* delete from hostgroups and servicegroups is delayed of one minute (#379)

REFS-MON-14908

* fix(broker/stats): center concurrent access changed to use mutexes (#412)

REFS: MON-15575

* fix(broker/bam): no need to read last_level in bam (#417)

It is not stored in kpis, just computed on demand and stored in db.

    REFS: MON-15622

* update credentials (#432)

* MON-15761 compiled are saved in bin, lib, tests directories (#431)

* MON-15761 compiled are saved in bin, lib, tests directories
* MON-15761 ci use new directories
* MON-15761 import 22.04 ci files
* MON-15761 import suite
* MON-15761 connector install correction
* MON-15761 use of WITH_PREFIX_LIB_CLIB and PREFIX_LIB
* MON-15761 clib include location
* MON-15761 engine
* MON-15761 modules
* MON-15761 cbd
* MON-15761 no pc and no duplicate engine log file
* MON-15761 no connector test
REFS:MON-15761

* enh(ci): workflow+actions added to develop

* fix(ci): docker builder improved

* enh(ci): build scripts added

* fix(ci): docker-builder is updated with password variable

* fix(ci): docker-builder fixed

* fix(ci): docker folder missing

* fix(ci): password fixed

* fix(ci): fail-fast set to false

* fix(ci): artifactory path updated

* fix(ci): registry and runners updated

* enh(ci): packaging updated

* fix(ci): username updated

* fix(ci): Jenkinsfiles removed

* feat(chore): update .gitignore and remove deprecated scripts (#437)

* fix(ci): new build workflow

* fix(ci): cmake.sh improved

* enh(cmake): cmake called as in 22.04, and newer versions of collect

* enh(cmake): cmake called as in 22.04, and newer versions of collect
* fix(cmake): bad paths on tests
* fix(cmake): the script could fail with devtool
* fix(ci): actions copied from develop branch
* fix(ci): collect tests script updated

REFS: MON-15860

* fix(ci): debian build was broken

* fix(ci): debian packaging had bad paths

* fix(ci): poller-module.json is a bad name

* fix(ci): bad path

* fix(ci): bad path in rpm packaging

* fix(ci): release simplified

* fix(connector): unit tests may fail on centos7

* fix(delivery): add delivery to old yum repo (#458)

* enh(ci): rebuild yum metadata only one time (#463)

* fix(ci): fix workflow trigger on push dev-21.10.x (#464)

* enh(packaging): add rpm sign to workflow 2110 (#470)

* fix(packaging): fix cmake parameter for broker config path (#467)

* fix(ci): remove debian related workflows and actions 2110 (#518)

* fix(ci): remove debian related workflows and actions 2110

* Cleanup workflows by removing debian related

* Cleanup docker registry urls

* MON-16310 help cmake to find protoc

* add delivery to stable

* fix get-packages action

* update robot nightly

* Update delivery

Co-authored-by: Jean Christophe Roques <jcroques@centreon.com>
Co-authored-by: chgautier <cgautier@centreon.com>

* enh(ci): remove paths to workflow files in workflows for components (#513) (#525)

* chore(license): update maintainer and email addresses (#530)

* Mon 16454 add alma 8 to centreon collect nightly tests (#506) (#521)

* enh(ci): Add alma8 to centreon-collect nightly

* Add job to robot-nightly workflow

* enable ci test branch

* Update robot nightly workflow

* add debug to get-packages

* Add release to github_env

* add delivery test

* Get pkg dir version name

* Fix url

* fix url

* make curl silent

* fix curl

* add mariadb client

* update get-packages

* disable nightly on alma 8

* update alma8 image

* Add MariaDB packages to alma8 dockerfile

* remove extra sudo

* update collect-test-robot.sh

* Remove duplicate Robot test job

* fix(ci/tests): attempt to upgrade grpc python library

* Install proper grpc version based on OS version id

* help cmake to find protoc

* disable delivery tests

* remove unused lines from actions

* Remove more unused lines

Co-authored-by: David Boucher <dboucher@centreon.com>

* chore(ci): Update delivery credentials (#543) (#546)

* chore(ci): Update delivery credentials (#543)

* Update delivery credentials

* Update delivery credentials and inputs

* Update deb delivery credentials

* Update .github/workflows/centreon-collect.yml

Co-authored-by: Kevin Duret <kduret@centreon.com>

* Update .github/workflows/centreon-collect.yml

Co-authored-by: Kevin Duret <kduret@centreon.com>

---------

Co-authored-by: Kevin Duret <kduret@centreon.com>

* MON-16922 bad parameters to configuration::kpi constructor (#563)

* fix(ci): fix cache reference during rpm delivery 2110 (#564)

* debug delivery rpm

* fix rpm delivery

* fix rpm delivery

* chore(ci): Add extra delivery to artifactory (#555)

* chore(release):set version to 21.10.5

* Update registry

* Update docker registry credentials

* Update failed credentials

* Remove gorgone from docker builder matrix

---------

Co-authored-by: denliA <91119589+denliA@users.noreply.github.com>
Co-authored-by: David Boucher <dboucher@centreon.com>
Co-authored-by: jean-christophe81 <98889244+jean-christophe81@users.noreply.github.com>
Co-authored-by: Charles Gautier <33026375+chgautier@users.noreply.github.com>
Co-authored-by: Stéphane Chapron <34628915+sc979@users.noreply.github.com>
Co-authored-by: tuntoja <58987095+tuntoja@users.noreply.github.com>
Co-authored-by: Kevin Duret <kduret@centreon.com>
Co-authored-by: Jean Christophe Roques <jcroques@centreon.com>
Co-authored-by: chgautier <cgautier@centreon.com>
Co-authored-by: Centreon <release@centreon.com>
Co-authored-by: tuntoja <tuntoja@centreon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bouda1 bouda1 bouda1 approved these changes

@jean-christophe81 jean-christophe81 jean-christophe81 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tuntoja @bouda1 @jean-christophe81