Anders Harrisson (c770abbf) at 28 Mar 15:58
Merge branch 'master' into 'master'
... and 1 more commit
Thanks for the review, @georgekontogiorgos.
Your questions (and confusion) are all valid. As with many other "standards", it may not always be completely consistent across all tools.
The Artifactory repository gpg key that you refer to, may fall under the "not always consistent", if the remote filename is interpreted literally. I think the file name "artifactory.gpg.public" is simply an auto generated string based on the file location (/artifactory/api/gpg/key/public). The file itself is generated by Artifactory and is an armored ASCII key, thus I write it to the filesystem with the .asc extension when downloaded. This is a crucial detail, because APT will treat the keyring files different depending on extension. Keyrings with .asc will automatically be dearmored by APT, whereas .gpg will not.
The Zabbix key comes from the upstream Zabbix repo that we proxy/cache on Artifactory, thus this filename is also out of our control. I do the only sane thing here and rename it to .asc as it's downloaded. Again, this is to not confuse APT.
So, the arguably "incorrect" filenames are not our doing and not something we can change, but what ends up on the actual hosts are .asc files, which I find correct. And if I understand you correctly, you agree with that.
Anders Harrisson (a78fdff6) at 28 Mar 14:45
Merge branch 'master' into 'master'
... and 2 more commits
We have occasionally seen situations where the alarm configuration contains duplicate Component or PV names, and this has caused failures when applied to the Alarm Server.
This MR is meant to solve this by catching these situations in the GitLab-CI validation job, thus failing the pipeline early if such incorrect files exists.
Anders Harrisson (21487a44) at 26 Mar 11:34
Disable AWX integration for netbox-02
Anders Harrisson (fecbc0be) at 26 Mar 08:44
Upgrade to Shifter v1.0.11
Anders Harrisson (40b61bb7) at 26 Mar 08:17
Upgrade to Shifter v1.0.10
Anders Harrisson (777580a2) at 26 Mar 08:14
Upgrade to Shifter v1.0.7 on production
Anders Harrisson (136e2dae) at 26 Mar 07:11
Update dependency ics-ans-role-repository to v0.14.0
... and 14 more commits
Anders Harrisson (96e0cd28) at 25 Mar 21:14
Set version to 16.9.1-1
Anders Harrisson (55a35e90) at 25 Mar 07:49
Merge branch 'ce-2528-fix-naming-help-page-use-this-one' into 'master'
... and 1 more commit
uses correct frontend variables for the naming-ui, and adds the missing support url var
Anders Harrisson (d10db4c3) at 18 Mar 13:35
Merge branch 'ics-ans-role-repository_v0.14.0' into 'master'
... and 2 more commits
Anders Harrisson (fa0f82c1) at 18 Mar 13:15
Run Molecule scenarios in parallel GitLab pipeline jobs
... and 4 more commits
Looks okay. Merging this would change the environment variables on the production container, thus causing a restart. If you just want this applied to the test instance (for now), then you should probably assign the variables as host-variables instead. I think production will be left untouched due to the omit declaration.
Anders Harrisson (b1a7584a) at 18 Mar 09:16
Merge branch 'extra_vars' into 'master'
The current ce-deploy backend sends a list of iocs to deploy/undeploy instead of using an ansible inventory. While this is not properly idiomatic, it is necessary for the time being. This is intended to be the minimal merge request to switch from using inventories to our mechanism of deploying IOCs, with the intent of being relatively easy to revert in the future.