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

GitHub action build #6

Closed
wants to merge 47 commits into from
Closed

GitHub action build #6

wants to merge 47 commits into from

Conversation

c-po
Copy link
Owner

@c-po c-po commented Jul 4, 2024

Change Summary

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Related PR(s)

Component(s) name

Proposed changes

How to test

Smoketest result

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • I have run the components SMOKETESTS if applicable
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

alainlamar and others added 30 commits June 27, 2024 20:48
…ounters (vyos#3681)

* T6488: firewall: extend op-mode command to show global state-policy counters.
wireless: T6320: Add 802.11ax at 6GHz to VyOS
Add Loki plugin to telegraf

set service monitoring telegraf loki url xxx
pppoe-server: T5710: Add option permit any-login
T6477: Add telegraf loki output plugin
utils: T6530: add a helper for easily calling iproute2 commands
* T6452: Add QoS Op Commands
Added the following commands:

show qos shaping
show qos shaping detail
show qos shaping interface <int name>
show qos shaping interface <int name> detail
show qos shaping interface <int name> class <class name>
show qos shaping interface <int name> class <class name> detail
show qos cake interface <int name>
op-mode: T6524: rewrite "release dhcp(v6) interface" to new op-mode format
T6534: fix incorrect imports in vyos-config-encrypt.py
If a firewall is not configured there is no reason to get and
execute telegraf firewall custom scripts as there are no nft
chain in the firewall nftables configuration
T6523: Telegraf use nft scripts only if the firewall configured
T6527: add legacy Vyatta interpreter files still in use
ssh: T5878: Allow changing the PubkeyAcceptedAlgorithms option
T6538: Add the ability to set GENEVE interfaces to VRF
…ch-support-report

op-mode: T6498: add machine-readable tech support report script
…orted in 1.3 <+> with character supported in latest version <*>
c-po and others added 16 commits July 3, 2024 16:32
T6536: nat: add migration script that replaces wildcard charater
syslog: T5366: remove reference to deprecated sysvinit rsyslog script
T6546: unused import check permission update
Sometimes we need a reliable way to lock an execution until some other operation
is not done.

This commit introduces locking util, which can be used as a common lock, even
between different processes.

Usage example:

```
from vyos.utils.locking import Lock

lock = Lock('my_lock_id')
lock.acquire(timeout=10)

print(f'Lock acquired: {lock.is_locked}')

lock.release()
```
Improvements in the `vyos_net_name`:

- Used a new locking system, to be sure that multiple running scripts will not
try to perform operations at the same time.
- Replace logging from a file to syslog. This is common with all the rest logs,
and additionally gives a better view of actions done during a boot.
- Small bug fix in `get_configfile_interfaces()`: exit with an error in case a
config file cannot be parsed. This resolves potentially an unbound `config` object.
- Minor formatting fixes to follow our requirements.
op-mode: T6537: include hostname in the reboot/shutdown warning message
Most likely b/c of our non signed Kernel binary we do not trust the Debian
signed wireless regulatory database. Fallback to the upstream database instead.
vyos_net_name: T6544: Updated the `vyos_net_name` script
op-mode: T6371: fix output of NAT rules with single port range
T6539: add logging options to load-balancer reverse-proxy
wireless: T4287: use upstream regulatory database due to kernel signing
Copy link

github-actions bot commented Jul 4, 2024


PR title 'GitHub action build' does not match the required format!. Valid title example: T99999: make IPsec secure

Copy link

github-actions bot commented Jul 4, 2024


warning: Unused cmd imported from vyos.utils.process in src/op_mode/powerctrl.py:27.

@c-po c-po force-pushed the github-action-build branch 4 times, most recently from 5aa556d to ac9ac29 Compare July 4, 2024 18:10
Copy link

github-actions bot commented Jul 4, 2024

CI integration ❌ failed!

Details

CI logs

  • CLI Smoketests ❌ failed
  • Config tests 👍 passed
  • RAID1 tests 👍 passed

GitHub performs loose equality comparisons.
If the types do not match, GitHub coerces the type to a number. GitHub casts
data types to a number using these conversions:

https://docs.github.com/en/actions/learn-github-actions/expressions
@c-po c-po closed this Jul 5, 2024
@c-po c-po deleted the github-action-build branch July 5, 2024 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.