When the set doesn't exist osbuildapi-update-nft.sh give nice errors, and a big comment. #2538

Merged
ryanlerch merged 1 commit from nftables into main 2025-04-11 00:31:15 +00:00
Contributor
No description provided.
First-time contributor
Build succeeded. https://fedora.softwarefactory-project.io/zuul/buildset/259ff02ee55f4d2b8da6324fade961b9 - [fi-ansible-lint-diff ](https://fedora.softwarefactory-project.io/zuul/build/4405dfc0d5cc438d9ca3227c90d00ae2) : SUCCESS in 2m 49s - [fi-yamllint-diff ](https://fedora.softwarefactory-project.io/zuul/build/959f065dda6142caad4a58a1e0d6dc19) : SUCCESS in 2m 09s
Contributor

Hum, so it's actually not occasional. It's 100% of the time, we just have a nag-once 1d in there, so it only emails once a day.

This is the buildvm-a64 and buildvm-ppc64le in stg. I'm not sure whats different about them from the rest of the builders.

Any ideas?

Hum, so it's actually not occasional. It's 100% of the time, we just have a nag-once 1d in there, so it only emails once a day. This is the buildvm-a64 and buildvm-ppc64le in stg. I'm not sure whats different about them from the rest of the builders. Any ideas?
Author
Contributor

Huh, so it's all the non-x86_64 builders? And the firewall looks the same on all the arches?

The patch here won't hurt, but the comment is def. wrong and I doubt it fixes anything.

Feel free to ping me today on matrix, when you have some spare minutes ... I've no idea why arch would matter (but only tested on local x86).

Huh, so it's all the non-x86_64 builders? And the firewall looks the same on all the arches? The patch here won't hurt, but the comment is def. wrong and I doubt it fixes anything. Feel free to ping me today on matrix, when you have some spare minutes ... I've no idea why arch would matter (but only tested on local x86).
Author
Contributor

rebased onto b7c82cbb47

rebased onto b7c82cbb4701a07fba8578fc51df7320cf3ee4af
Author
Contributor

Note that this is now just a cleanup, and comment patch. The real fix for the cron errors is in: https://pagure.io/fedora-infra/ansible/pull-request/2550

Note that this is now just a cleanup, and comment patch. The real fix for the cron errors is in: https://pagure.io/fedora-infra/ansible/pull-request/2550
First-time contributor
Build succeeded. https://fedora.softwarefactory-project.io/zuul/buildset/152a0d8c8bde437ea7dec7cee9e490af - [fi-ansible-lint-diff ](https://fedora.softwarefactory-project.io/zuul/build/9c246db1f7fe4113a61ddde998e1cf13) : SUCCESS in 2m 42s - [fi-yamllint-diff ](https://fedora.softwarefactory-project.io/zuul/build/f62fab2ac0c341669d08fa38129f5089) : SUCCESS in 2m 19s
First-time contributor
Build succeeded. https://fedora.softwarefactory-project.io/zuul/buildset/a0f795ecf3954ec99c265fd709a0ef2e - [fi-ansible-lint-diff ](https://fedora.softwarefactory-project.io/zuul/build/a150d59e9e374b58bb9f67d11f40f5c8) : SUCCESS in 2m 40s - [fi-yamllint-diff ](https://fedora.softwarefactory-project.io/zuul/build/00f185889eb7496c9159841ddce29acf) : SUCCESS in 2m 18s
Contributor

rebased onto b9553f3378

rebased onto b9553f33784665d424a817add8e3f49f5b93837b
Contributor

rebased onto b9553f3378

rebased onto b9553f33784665d424a817add8e3f49f5b93837b
Contributor

This only should affect the staging builders (we haven't switched prod to nftables yet). So merging.

This only should affect the staging builders (we haven't switched prod to nftables yet). So merging.
Contributor

Pull-Request has been merged by kevin

Pull-Request has been merged by kevin
First-time contributor
Build succeeded. https://fedora.softwarefactory-project.io/zuul/buildset/5e196079e68748e69e36a07526294019 - [fi-ansible-lint-diff ](https://fedora.softwarefactory-project.io/zuul/build/e036abb3eff14347aecd623ce1a4b34b) : SUCCESS in 2m 35s - [fi-yamllint-diff ](https://fedora.softwarefactory-project.io/zuul/build/83c816678b464f5fb78aea8b9868801a) : SUCCESS in 2m 01s
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Infrastructure/ansible#2538
No description provided.