Skip to content

Tourofbeam+infrastructure - #24920

Closed
ruslan-ikhsan wants to merge 449 commits into
apache:masterfrom
akvelon:tourofbeam+infrastructure
Closed

Tourofbeam+infrastructure#24920
ruslan-ikhsan wants to merge 449 commits into
apache:masterfrom
akvelon:tourofbeam+infrastructure

Conversation

@ruslan-ikhsan

@ruslan-ikhsanruslan-ikhsan commented Jan 6, 2023

Copy link
Copy Markdown
Contributor

PR resolves (to be updated)
Purpose of PR: Tour of Beam infrastructure spin-up (both Backend and Frontend) using Terraform and Kotlin Gradle scripts

R: @olehborysevych

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
  • Update CHANGES.md with noteworthy changes.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels
Python tests
Java tests
Go tests

See CI.md for more information about GitHub Actions CI.

@codecov

codecovBot commented Jan 10, 2023

Copy link
Copy Markdown

Codecov Report

Merging #24920 (98da673) into master (48c143d) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@ Coverage Diff @@## master #24920 +/- ##
==========================================
- Coverage 72.82% 72.81% -0.02% 
==========================================
Files 775 775 Lines 102948 102936 -12 ==========================================
- Hits 74974 74953 -21 - Misses 26520 26528 +8 - Partials 1454 1455 +1 
FlagCoverage Δ
go53.95% <100.00%> (-0.02%)⬇️
python81.95% <100.00%> (-0.02%)⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted FilesCoverage Δ
...dks/python/apache_beam/options/pipeline_options.py93.97% <ø> (ø)
sdks/go/pkg/beam/runners/dataflow/dataflow.go60.76% <100.00%> (-1.10%)⬇️
...apache_beam/runners/dataflow/internal/apiclient.py76.80% <100.00%> (-0.23%)⬇️

... and 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment threadlearning/tour-of-beam/terraform/variables.tf Outdated
dependabotBotand others added 9 commits March 10, 2023 17:12
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.7.0 to 0.8.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](golang/net@v0.7.0...v0.8.0)
---
updated-dependencies:
- dependency-name: golang.org/x/net
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Use credential for mongodb performance tests
* Set cron job freq to twice a day as for other performance tests
Line 59 is using the flag `--inputText` which is different from the actual code repository which uses `--input-text`. Running the code with `--inputText` returns an error message `flag provided but not defined: -inputText` while running the code with `--input-text` returns the expected output
```markdown
go run main.go --inputText="Greetings"
```
This is inconsistent with the code which named the flag as `input-text` instead of `inputText` as shown in the snippet found in [Line 24 of main.go](https://github.com/apache/beam-starter-go/blob/main/main.go#L24)
```go
var (
input_text = flag.String("input-text", "default input text", "Input text to print.")
)
```
Update README.md
Update build.gradle.kts
Update build.gradle.kts
Update build.gradle.kts
Update build.gradle.kts
Update build.gradle.kts
Update variables.tf
Update main.tf
Transforming to variable
Update variables.tf
Update variables.tf
Update variables.tf
Update main.tf
Update main.tf
Debugging gcloud command output to variable
Update variables.tf
Update variables.tf
Update variables.tf
Updating local exec to iam resource
Update .gitignore
PR comments related updates
Add space at the end of TF files
PR comments related updates
Update unit_content.g.dart
Update unit_content.g.dart
Update unit_content.g.dart
Delete .firebaserc
Update .gitignore
@ruslan-ikhsan

Copy link
Copy Markdown
ContributorAuthor

Closed as new duplicated PR has been created to maintain clean commit history
New PR: #25793
CC: @olehborysevych

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@ruslan-ikhsan@olehborysevych@lostluck@damccorm@Abacn@johannaojeling@johnjcasey@riteshghorse@nickuncaged1201@robertwb@aromanenko-dev@rohdesamuel@Juta@aaltay@vachan-shetty@AnandInguva@yananhao12@yuvipanda@reuvenlax@ahmedabu98