Skip to content

Revert "refactor(agw): Minor improvements to the fabfiles (#15004)"#15030

Merged
nstng merged 1 commit intomagma:masterfrom
nstng:revert_15004
Feb 20, 2023
Merged

Revert "refactor(agw): Minor improvements to the fabfiles (#15004)"#15030
nstng merged 1 commit intomagma:masterfrom
nstng:revert_15004

Conversation

@nstng
Copy link
Copy Markdown
Contributor

@nstng nstng commented Feb 20, 2023

This reverts commit 96bd8e3.

Summary

#15004 broke the agw build, see https://github.com/magma/magma/actions/runs/4221022707/jobs/7327856886

Test Plan

CI

Additional Information

  • This change is backwards-breaking

@nstng nstng requested review from a team as code owners February 20, 2023 08:34
@nstng nstng requested a review from rsarwad February 20, 2023 08:34
@pull-request-size pull-request-size bot added the size/L Denotes a Pull Request that changes 100-499 lines. label Feb 20, 2023
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for opening a PR! 💯

A couple initial guidelines

Howto

  • Reviews. The "Reviewers" listed for this PR are the Magma maintainers who will shepherd it.
  • Checks. All required CI checks must pass before merge.
  • Merge. Once approved and passing CI checks, use the ready2merge label to indicate the maintainers can merge your PR.

More info

Please take a moment to read through the Magma project's

If this is your first Magma PR, also consider reading

@github-actions github-actions bot added component: agw Access gateway-related issue component: cwf component: feg FEG-gateway related issues labels Feb 20, 2023
Comment thread cwf/gateway/fabfile.py
Comment thread cwf/gateway/fabfile.py
Comment thread feg/gateway/fabfile.py
Comment thread feg/gateway/fabfile.py
Comment thread feg/gateway/fabfile.py
Comment thread feg/gateway/fabfile.py
Comment thread feg/gateway/fabfile.py
Comment thread feg/gateway/fabfile.py
Comment thread feg/gateway/fabfile.py
Comment thread lte/gateway/fabfile.py
@github-actions
Copy link
Copy Markdown
Contributor

FeG Lint & Test

    2 files  203 suites   40s ⏱️
374 tests 374 ✔️ 0 💤 0
388 runs  388 ✔️ 0 💤 0

Results for commit 77a0024.

@github-actions
Copy link
Copy Markdown
Contributor

DP Lint & Test

14 tests   14 ✔️  3m 4s ⏱️
  1 suites    0 💤
  1 files      0

Results for commit 77a0024.

@nstng nstng merged commit 4802e7c into magma:master Feb 20, 2023
MoritzThomasHuebner added a commit to MoritzThomasHuebner/magma that referenced this pull request Feb 21, 2023
MoritzThomasHuebner added a commit that referenced this pull request Feb 22, 2023
* Revert "Revert "refactor(agw): Minor improvements to the fabfiles (#15004)" (#15030)"

This reverts commit 4802e7c.

Signed-off-by: Moritz Huebner <[email protected]>

* chore(agw): Fixed hash/commit_hash clash

Signed-off-by: Moritz Huebner <[email protected]>

---------

Signed-off-by: Moritz Huebner <[email protected]>
lucasgonze pushed a commit to lucasgonze/magma that referenced this pull request Feb 29, 2024
lucasgonze pushed a commit to lucasgonze/magma that referenced this pull request Feb 29, 2024
* Revert "Revert "refactor(agw): Minor improvements to the fabfiles (magma#15004)" (magma#15030)"

This reverts commit 4802e7c.

Signed-off-by: Moritz Huebner <[email protected]>

* chore(agw): Fixed hash/commit_hash clash

Signed-off-by: Moritz Huebner <[email protected]>

---------

Signed-off-by: Moritz Huebner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: agw Access gateway-related issue component: cwf component: feg FEG-gateway related issues size/L Denotes a Pull Request that changes 100-499 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants