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

add script to push pipelines to glassflow #38

Closed
wants to merge 4 commits into from

Conversation

PabloPardoGarcia
Copy link
Member

@PabloPardoGarcia PabloPardoGarcia commented Jan 13, 2025

Add a script (glassflow-push-pipelines) to use during CI/CD to push changes made in pipelines to GlassFlow.

Arguments:

  • --pipelines-dir: Directory where the pipelines are defined in your repository (default: pipelines/)
  • -a or --files-changed: List of changed files that need to be pushed to GlassFlow (.yaml, .py or requirements.txt)
  • -d or --files-deleted: List of deleted files that need to be pushed to GlassFlow (.yaml, .py or requirements.txt)
  • -t or --personal-access-token: Personal access token used to interact with GlassFlow API
  • -y or --auto-approve: Automatically approve pipeline updates

TODO:

  • Add support for secrets in env variables and connections

Copy link

github-actions bot commented Jan 13, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py70100% 
api_client.py41392%84–86
client.py371267%98, 147, 168, 172–173, 178–181, 183, 185, 209
config.py80100% 
pipeline.py1811492%77, 87, 111, 236, 239, 246, 409–410, 412, 414, 485, 493, 509, 529
pipeline_data.py88792%52, 129, 146–147, 183, 198–199
space.py38392%109–110, 112
models
   __init__.py00100% 
models/api
   __init__.py20100% 
   api.py3440100% 
models/errors
   __init__.py30100% 
   clienterror.py35780%47–49, 51, 106–107, 119
   error.py10190%25
models/operations
   __init__.py100100% 
   access_token.py100100% 
   artifact.py100100% 
   base.py320100% 
   consumeevent.py180100% 
   consumefailed.py18194%64
   function.py280100% 
   pipeline.py500100% 
   publishevent.py140100% 
   space.py290100% 
utils
   __init__.py20100% 
   utils.py38823040%48, 53, 56–57, 61–66, 72–77, 79–80, 86–91, 93, 95–99, 103, 106, 122, 150, 153–155, 157, 161, 170–171, 176, 182, 205, 207–209, 211, 217, 219–220, 222–227, 229–231, 233–236, 238, 244, 248, 252, 255–258, 260–263, 265, 269, 271, 275, 278, 282, 284–285, 287, 321, 335, 346, 367–372, 374, 395, 397–399, 401–404, 406–407, 409–412, 414, 416, 420, 424–427, 434, 436–440, 442–444, 446, 448–449, 451–452, 463–466, 468, 470, 474, 476–477, 479, 495, 497–501, 503–505, 507–508, 510, 512–513, 515–518, 520–521, 523, 525–526, 528, 530–532, 534–537, 539, 541–542, 554–557, 559–560, 562–564, 566–568, 570–571, 573–574, 576–577, 579, 581–583, 585–586, 588–589, 591–592, 594, 596–598, 600, 602–603, 607, 615–616, 625, 639–641, 643–644, 646–647, 652, 659, 661, 675–680, 687
   yaml_models.py81810%5, 7–8, 10, 13–14, 17–21, 23–24, 27, 29–31, 33–35, 37–38, 40, 43–46, 48–52, 55–57, 59–63, 66–68, 70–74, 77–80, 83–88, 91–96, 98–100, 105–106, 109–114, 116–118, 123–124, 127
   yaml_operations.py1331330%1–5, 7, 9–11, 13–14, 17, 20–21, 23, 26, 32–33, 36–38, 40–42, 44–45, 47, 49–50, 52–54, 56–57, 61, 76, 82, 84–91, 93, 95, 98, 100–107, 109, 111–113, 115–118, 121, 123–126, 129–136, 138, 140–146, 148, 151, 156–160, 162, 170, 177–180, 183–184, 187, 194–195, 201, 203–204, 207, 212–213, 215–220, 222–223, 225–226, 229–230, 242–244, 252, 260, 266, 272, 280, 282–283, 292–293
TOTAL161749269% 

Tests Skipped Failures Errors Time
59 0 💤 0 ❌ 0 🔥 41.762s ⏱️

ashish-bagri
ashish-bagri previously approved these changes Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants