text:
Add functional smoke tests and automated testing workflow

🗣 Description

Added smoke tests (/Testing/Functional/SmokeTests/smoke_test.py) to verify the following:

scubagoggles output (all files/directories are created)
ScubaResults.json (determine if there are API exceptions or other errors produced)
Parent/Individual html reports (navigation links, agency data, baseline report data, and policy tables display as expected)

Added an automated testing workflow (.github/workflows/run_smoke_test.py) that runs the smoke tests against Windows/macOS and across different python versions. The workflow runs when a PR is opened, reopened, synchronized (a new commit is added), and when a commit is pushed into main.
The workflow has been tested against Python versions ranging from 3.9 - 3.12. Due to limitations with the numpy package testing is limited < v3.9. (numpy v1.24.4 is the highest supported version for Python v3.8.4 so there's issues with downgrading to numpy v1.26.4)
💭 Motivation and context

We manually test scubagoggles output, python min/max supported versions, etc. which is resource intensive. The purpose of these smoke tests and automated workflows is to ease the burden of testing when new PRs are opened, changes are added into main, and prior to releases.
closes #171
Create future todos:

[x] Use Selenium to check for the correct baseline and tool version in the agency table (e.g. 0.2, 0.2.0) - #339
[x] Update run_smoke_test.py to take in input when running the workflow manually. This will allow our developers to input specific Python versions as opposed to hardcoded values.  - #340
[x] Update docs on how to run smoke tests - #341

🧪 Testing

pip install pytest selenium

pytest -s -vvv ./Testing/Functional/SmokeTests/ --subjectemail="subjectemail@mail.com" --domain="domain@domain.com"

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] If applicable, All future TODOs are captured in issues, which are referenced in the PR description.
[x] The relevant issues PR resolves are linked preferably via closing keywords.
[x] All relevant type-of-change labels have been added.
[x] I have read and agree to the CONTRIBUTING.md document.
[x] These code changes follow cisagov code standards.
[ ] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge Checklist

[ ] This PR has been smoke tested to ensure main is in a functional state when this PR is merged.
[ ] Squash all commits into one PR level commit using the Squash and merge button.

✅ Post-merge Checklist

[ ] Delete the branch to clean up.
[ ] Close issues resolved by this PR if the closing keywords did not activate.

Huge step in the right direction. And it already found a bug. Looks good. I'm going to go ahead and approve but my asks would be to create an issue (or issues) for the various TODO items you identified and to make the Pylinter happy.

Thanks! Issues have been created for the various todo items and the Pylint check is passing.

Great job Mitchel!
When I try to run the tests, I am getting an error:

I tried adding the relative path, but it still complains. Did I miss any step?

Thanks Sheetal! The import for gws_products() changed because I rebased after Roy's PR was merged into main. Should resolve after running git pull --rebase origin 171-functional-smoke-test then rerunning the test suite.

Great job Mitchel!
When I try to run the tests, I am getting an error:

I tried adding the relative path, but it still complains. Did I miss any step?

Thanks Sheetal! The import for gws_products() changed because I rebased after Roy's PR was merged into main. Should resolve after running git pull --rebase origin 171-functional-smoke-test then rerunning the test suite.

Now its failing on the Chrome driver:

Running Through Workflow on Mac. Out of the box, I get this error:

Will troubleshoot and report findings

Let me document this since I didn't earlier. I also ran the smote test manually, and it worked for me. I tested on a Windows.

Running Through Workflow on Mac. Out of the box, I get this error:
Will troubleshoot and report findings

@LaurenBassett That looks like the Pandas/numpy interrogability error we've been struggling with elsewhere

Running Through Workflow on Mac. Out of the box, I get this error:
Will troubleshoot and report findings

@LaurenBassett That looks like the Pandas/numpy interrogability error we've been struggling with elsewhere

Yup. Just fixed it and ran again. I figured I'd document every piece of the flow though, just for documentation's sake. I downgraded numpy, I know we had discussed other ways to fix in our sync yesterday though.

Seeing this message now, it may be user error on my part?

WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a2822a0>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/redacted
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a283020>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/redacted
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a283200>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/readacted
============================================================= short test summary info ==============================================================
FAILED Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_output - Failed: An error occurred, Command 'scubagoggles gws --subjectemail redacted@redacted.org --quiet' returned non-zero exit status 127.
FAILED Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_report - Failed: An error occurred, Expected the reports table to have a length of 10
=========================================================== 2 failed, 1 passed in 1.08s ============================================================

Seeing this message now, it may be user error on my part?

WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a2822a0>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/redacted
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a283020>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/redacted
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a283200>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/readacted
============================================================= short test summary info ==============================================================
FAILED Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_output - Failed: An error occurred, Command 'scubagoggles gws --subjectemail redacted@redacted.org --quiet' returned non-zero exit status 127.
FAILED Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_report - Failed: An error occurred, Expected the reports table to have a length of 10
=========================================================== 2 failed, 1 passed in 1.08s ============================================================

Hmm interesting. I think the urllib3 Errno 61 error is caused when selenium driver.quit() is called prematurely. That would explain the test_scubagoggles_report error because the report does not contain all baselines. That points to the root cause being in test_scubagoggles_output. Can you successfully create a report when just running scubagoggles gws --subjectemail redacted@redacted.org --quiet?

Seeing this message now, it may be user error on my part?

WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a2822a0>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/redacted
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a283020>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/redacted
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11a283200>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/readacted
============================================================= short test summary info ==============================================================
FAILED Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_output - Failed: An error occurred, Command 'scubagoggles gws --subjectemail redacted@redacted.org --quiet' returned non-zero exit status 127.
FAILED Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_report - Failed: An error occurred, Expected the reports table to have a length of 10
=========================================================== 2 failed, 1 passed in 1.08s ============================================================

This looks like a network issue to me. Maybe your corporate proxy is blocking the download?

@LaurenBassett I would also check that the credentials.json is in the correct format from the Using a Service Account steps.

@LaurenBassett I would also check that the credentials.json is in the correct format from the Using a Service Account steps.

Somehow missed the part about the service account. That was a huge blocker for sure.
Running it again, the scubagoggles gws --subjectemail redacted@redacted.org --quiet command works just fine. I'm still running into issues with the full command.
"serviceaccount@projectname.iam.gserviceaccount.com"
=============================================================== test session starts ================================================================
platform darwin -- Python 3.12.4, pytest-8.3.2, pluggy-1.5.0 -- .../CISA_SCuBA/.venv/bin/python
cachedir: .pytest_cache
rootdir: ../CISA_SCuBA/ScubaGoggles
configfile: pytest.ini
collected 3 items

Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_output Password:
PASSED
Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubaresults PASSED
Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_report FAILED

===================================================================== FAILURES =====================================================================
________________________________________________________ SmokeTest.test_scubagoggles_report ________________________________________________________

self = <smoke_test.SmokeTest object at 0x113a85730>, browser = <SmokeTests.selenium_browser.Browser object at 0x113a6cf50>
domain = 'serviceaccount@projectname.iam.gserviceaccount.com'

    def test_scubagoggles_report(self, browser, domain):
        """
        Test if the generated baseline reports are correct,
        i.e. BaselineReports.html, CalendarReport.html, ChatReport.html
        """
        try:
            output_path: str = get_output_path()
            report_path: str = prepend_file_protocol(os.path.join(output_path, BASELINE_REPORTS))
            browser.get(report_path)
>           run_selenium(browser, domain)

Testing/Functional/SmokeTests/smoke_test.py:69:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
Testing/Functional/SmokeTests/smoke_test_utils.py:154: in run_selenium
    verify_tenant_table(browser, domain, True)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

browser = <SmokeTests.selenium_browser.Browser object at 0x113a6cf50>, domain = 'serviceaccountname@projectname.iam.gserviceaccount.com'
parent = True

    def verify_tenant_table(browser, domain, parent):
   ...
        customer_domain = tenant_table_rows[1].find_elements(By.TAG_NAME, "td")[0].text
>       assert customer_domain == domain
E       AssertionError

Testing/Functional/SmokeTests/smoke_test_utils.py:259: AssertionError

During handling of the above exception, another exception occurred:

self = <smoke_test.SmokeTest object at 0x113a85730>, browser = <SmokeTests.selenium_browser.Browser object at 0x113a6cf50>
domain = 'serviceaccoint@projectname.iam.gserviceaccount.com'

    def test_scubagoggles_report(self, browser, domain):
       ...
        except (ValueError, AssertionError, Exception) as e:
            browser.quit()
>           pytest.fail(f"An error occurred, {e}")
E           Failed: An error occurred,

Testing/Functional/SmokeTests/smoke_test.py:72: Failed
-------------------------------------------------------------- Captured log teardown ---------------------------------------------------------------
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x127ecbe60>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/8bbf2c22ca41e8c27b1ab39281b802b5
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x113bce750>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/8bbf2c22ca41e8c27b1ab39281b802b5
WARNING  urllib3.connectionpool:connectionpool.py:870 Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPConnection object at 0x127ecbe00>: Failed to establish a new connection: [Errno 61] Connection refused')': /session/8bbf2c22ca41e8c27b1ab39281b802b5
============================================================= short test summary info ==============================================================
FAILED Testing/Functional/SmokeTests/smoke_test.py::SmokeTest::test_scubagoggles_report - Failed: An error occurred,
=========================================================== 1 failed, 2 passed in 59.22s =========================================

Update: disconnecting from the vpn help resolve the Chrome driver issue
Regular reports using the service account as the auth method works
Running the smoke tests: my results are similar to what @LaurenBassett posted (the latest screencapture)

Somehow missed the part about the service account. That was a huge blocker for sure.
Running it again, the scubagoggles gws --subjectemail redacted@redacted.org --quiet command works just fine. I'm still running into issues with the full command.

No worries, will be sure to reference the service account setup when building out the smoke test documentation.
When running pytest -s -vvv ./Testing/Functional/SmokeTests/ --subjectemail="subjectemail@mail.com" --domain="domain@domain.com" the value passed into the --domain parameter must match the Customer Domain column in the reports.

Somehow missed the part about the service account. That was a huge blocker for sure.
Running it again, the scubagoggles gws --subjectemail redacted@redacted.org --quiet command works just fine. I'm still running into issues with the full command.

No worries, will be sure to reference the service account setup when building out the smoke test documentation.
When running pytest -s -vvv ./Testing/Functional/SmokeTests/ --subjectemail="subjectemail@mail.com" --domain="domain.com" the value passed into the --domain parameter must match the Customer Domain column in the reports, otherwise the assert customer_domain == domain check fails.

That did it! I have it running successfully on Mac now. I guess I got a little confused with the service account vs. domain vs. admin account.

That did it! I have it running successfully on Mac now. I guess I got a little confused with the service account vs. domain vs. admin account.

Awesome! Thanks for documenting the setup process, a lot of good feedback here. Are there areas which can be more clear, perhaps changing domain to customerdomain?

That did it! I have it running successfully on Mac now. I guess I got a little confused with the service account vs. domain vs. admin account.

Awesome! Thanks for documenting the setup process, a lot of good feedback here. Are there areas which can be more clear, perhaps changing the domain parameter to customerdomain?

customerdomain would make it consistent with scubagogles parameter customerid

That did it! I have it running successfully on Mac now. I guess I got a little confused with the service account vs. domain vs. admin account.

Awesome! Thanks for documenting the setup process, a lot of good feedback here. Are there areas which can be more clear, perhaps changing the domain parameter to customerdomain?

I think there are a few bits that would be helpful in the documentation:

I was unclear what subject email was referring to; my scuba domain email address, or the service account's email address.
Having the example be subject@domain.com, or even user@domain.com may be clearer
I'd make a stronger note about the need for a service account and to ensure credentials are updated
The instructions for what to replace are confusing:

A service account is required to run the smoke tests. Replace "subjectemail@mail.com" with your subject email, then replace "domain@domain.com" with the domain you're running scubagoggles against.

I was using myuser@domain.com for the domain, not the basic domain, which was causing that last error

authors: LaurenBassett adhilto mitchelbaker-cisa snarve

repo: cisagov/ScubaGoggles


text:
update vincent dev hostname to point to cloudfront

🗣 Description
These changes are to point the existing FQDN for VINCE-NT DEV at the test Cloudfront distribution for connectivity testing.
💭 Motivation and context
This is to support testing Cloudfront and preparing to move VINCE-NT to staging for the next steps in deployment.
🧪 Testing
Cloudfront distribution FQDN has been verified.

✅ Pre-approval checklist

[X] This PR has an informative and human-readable title.
[X] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[X] I have read the CONTRIBUTING document.
[X] These code changes follow cisagov code standards.
[ ] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

@jwoytek-cisa - I have applied your change.  Please test and let me know if things seem to be working as expected.

I also rebased your commit onto the latest from upstream develop.  You were 150+ commits behind.

Looks good! Thank you! And apologies for the rebase issue--my fork was out of date but I missed it when I was making this update.

@jwoytek-cisa - Now that you have two approving reviews your PR should merge automatically once the checks are able to run.  I'll keep an eye on it and reach out if there are any issues.

Thank you!

source: gharchive/pull-request

license_type: permissive

license_source: github-api


text:
Migration

Description

[x] Workflows #3
[x] Issue templates #3
[x] #81
[x] README #54
[x] Poetry #3
[x] Tests
[x] Codebase #3
[x] Publish on PyPI https://pypi.org/project/vmngclient/

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/issues/4

source: gharchive/issue

repo: cisco-open/cisco-catalyst-wan-sdk


text:
Split person/entity field sets more practically

Spawned from https://github.com/citation-file-format/citation-file-format/pull/155#issuecomment-842456834.
Currently, fields that are used in both person and entity definitions - a lot of them simple strings - each have their own definition, which isn't very nice and unnecessarily blows up the file.
Instead, we could group them into three sets, e.g., uniqueToEntity, uniqueToPerson, and shared (refs #103), and then the definition could be
"entity": {
    "allOf": [
        {"$ref": "#/definitions/uniqueToEntity"},
        {"$ref": "#/definitions/shared"}
    ]
},
"person": {
    "allOf": [
        {"$ref": "#/definitions/uniqueToPerson"},
        {"$ref": "#/definitions/shared"}
    ]
}

In this example, shared was the original suggestion in the comment, although I'd like to make it more semantically unambiguous by naming the definition personOrEntity or similar.

So, this won't work with draft 7, as allOf cannot be used for real inheritance in combination with "additionalProperties": false and/or "required": [].
The only way one can make this work is by introducing empty property entries for all properties in both person and entity, which sort of defies the reason for splitting those up in the first place.
I'll have a look at using the if-else syntax to make this work in a sensible manner, but am not hopeful.

@jspaaks If you know of or can find another way of implementing this, please let me know. Otherwise I'd vote to close this.


text:
Add extra labels to servicemonitors

Hello!
Can you add possibility to add custom extra labels to service monitor when exporter enabled?
This feature need for target autodiscovering when prometheus operator is using.
I can write simple patch to helm chart if needed

Hi @toggetit thanks for the suggestion we will review this and get back.

Hi @toggetit thanks for the suggestion we will review this and get back.

I mean changes in citrix-ingress-controller.
There
https://github.com/citrix/citrix-helm-charts/blob/7014c386813c81ff8f1ccb8b25f445293c1e1b26/citrix-ingress-controller/templates/citrix-k8s-ingress.yaml#L226

hi @toggetit
In recent release we have added the support for extra labels to service monitor. Please try the latest charts.
https://github.com/citrix/citrix-helm-charts/releases/tag/cic-v1.34.16

authors: ankits123 subashd toggetit

repo: citrix/citrix-helm-charts


text:
Remove blue outline on search field

Commit bc0bb4e removes the blue border that's generated by mouseclicks, but not those from keyboard input. This lets us get the classy look without hurting accessibility. BUT, since the first input field gets focused on programmatically and not via mouse, it's still showing up there. Should we prevent that field from being selected on page load?

authors: LindseyWagner danjmoore

repo: cityofsomerville/cornerwise

license: mit

license_source: bigquery


text:
Tag Creator - Restore Tag Creator output when an affiliated organization is specified

Overview
Currently, Tag Creator fails to return results for most attempts to specify a project URL if an Affiliated Organization is selected from the dropdown menu. Hack for LA Brigade -> CTI-website-frontend and Hack for LA Brigade -> CTI-website-backend are the only combinations that correctly find a project and render topic tags.
Action Items
[ ] - Troubleshoot to determine why Civic Tech Index frontend and backend work, while other pages/organizations do not.
[ ] - Resolve the functionality issue in code.
Resources/Instructions

After some hunting around, I've found at least one repo that yields results:

The fetch code in Tag Creator may be working, after all.

Here is another one ( CodeForAfrica / HealthTools.KE-theStarHealth ):

I am closing this one now, as it does not now seem to be an issue.

authors: JafarIronclad kevinashworth

repo: civictechindex/CTI-website-frontend

license: MIT


text:
Hide header logo on mobile

Description
In North Star, hide the CE logo in the header on mobile following the UX mocks.
Release notes
n/a
Checklist
General
Read the full guidelines for PRs here

[x] Added the correct label: < feature | enhancement | bug | under-development | dependencies | infrastructure | ignore-for-release | database >
[x] Assigned to a specific person, civiform/developers, or a more specific round-robin list
[x] Added an additional reviewer from outside your organization as FYI (if the primary reviewer is in the same organization as you)
[x] Created unit and/or browser tests which fail without the change (if possible)
[x] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[x] Extended the README / documentation, if necessary

User visible changes

[x] Followed steps to internationalize any new strings

[x] Added context strings to new messages
[x] Didn't use a message in applicant facing code that isn't translated yet (unless behind a flag)

[x] Wrote browser tests using the validateAccessibility method
[x] Tested on mobile view. See mobile device mode
[x] Manually tested at 200% size
[x] Manually evaluated tab order

Instructions for manual testing
Use mobile view in Inspector
Issue(s) this completes
#7895

#7961  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ssandbekkhaug and the rest of your teammates on  Graphite

Looks good. Would you mind adding a browser test?

A couple questions:

Are you thinking something like "Show the header on mobile, assert the logo is not visible"?

Would the existing tests be enough? They have screenshots on mobile that include the header.

(I have no opposition to adding this test, just want to make sure I'm testing the right thing)

Looks good. Would you mind adding a browser test?

A couple questions:

Are you thinking something like "Show the header on mobile, assert the logo is not visible"?
Would the existing tests be enough? They have screenshots on mobile that include the header.

(I have no opposition to adding this test, just want to make sure I'm testing the right thing)

How about adding an assertion to an existing test to check that the logo isn't visible?  That would be enough IMO.

authors: melanie-exygy ssandbekkhaug

repo: civiform/civiform


text:
adds search label toggler

toggles between 'cultural organizations' and 'schools' in search results label

Hi--Not sure why I'm still getting these, I thought you took me off civil services project management app, no?
Allison Wyper
RHIZOMATIC ARTS
http://rhizomaticarts.com
http://allisonwyper.com
On Mar 7, 2015, at 9:31 PM, Chris Cuellar notifications@github.com wrote:
toggles between 'cultural organizations' and 'schools' in search results label
You can view, comment on, or merge this pull request online at:
https://github.com/civil-services/artsforla-asset-map/pull/14
Commit Summary
adds search label toggler
File Changes
M js/app.js (21)
M js/min/app-min.js (2)
Patch Links:
https://github.com/civil-services/artsforla-asset-map/pull/14.patch
https://github.com/civil-services/artsforla-asset-map/pull/14.diff
—
Reply to this email directly or view it on GitHub.

authors: ChrisC anwyper

repo: civil-services/artsforla-asset-map


text:
use gtests framework

Complete missing tests
Update test with Google Tests Framework

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (88b4dd1) 35.20% compared to head (9f5387a) 35.61%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
+ Coverage   35.20%   35.61%   +0.40%
==========================================
  Files          14       14
  Lines        2701     2701
  Branches     1166     1166
==========================================
+ Hits          951      962      +11
+ Misses       1121     1114       -7
+ Partials      629      625       -4

Files
Coverage Δ

clic/src/cudabackend.cpp
6.15% <ø> (ø)

clic/src/tier1.cpp
35.39% <0.00%> (-0.10%)
:arrow_down:

... and 5 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: StRigaud codecov-commenter


text:
No semantic token for member of anonymous struct in templated base class

In the following code:
template <typename T>
struct Base {
  struct {
    int waldo;
  };
};

template <typename T>
struct Derived : Base<T> {
  using Base<T>::waldo;

  void foo() {
    waldo;  // <--
  }
};

the waldo token on the indicated line does not get a semantic token.

Proposed fix: https://github.com/llvm/llvm-project/pull/80588

repo: clangd/clangd


text:
Add intraline mode to Diff

Diff should support an intraline mode where differences can be highlighted at a granularity finer than full lines. These differences should be easily processable as Hunk objects and should be serializable using a format similar to the unified diff format. git's porcelain format for word diffs seems like a good starting place, but we want to make sure our diffs unambiguously describe every byte that's different, including all whitespace. The diffs we generate should be usable as patches, and applying the patch should always produce the exact string that was diffed and be reversible.
This issue effectively replaces issue #2, which was a sloppier means to the same end.

This change has been implemented.

repo: clascade/clascade


text:
Add support for Tilt based dev

/kind feature
kamaji-provider should provide tilt-provider.json file to make use of Tilt based dev and test.
Providers such as CAPO support this feature.
https://cluster-api.sigs.k8s.io/developer/tilt.html
related issue

https://github.com/kubernetes-sigs/cluster-api-provider-openstack/issues/649

kamaji-provider needs the provider name such as CAPO and Label such as cluster.x-k8s.io/provider=infrastructure-openstack

https://cluster-api.sigs.k8s.io/clusterctl/provider-contract.html (Labels)

Tilt dashboard will show something like below

@jds9090 may I ask you if #22 is addressing the labels requirement you mentioned?

@jds9090 may I ask you if #22 is addressing the labels requirement you mentioned?

Yes! :)


text:
DndContext  make render to each columns

I create a Kanban board using dndkit when click on any task card all columns rerender
LINK video of the issue
https://jam.dev/c/457aa8ce-afd9-4054-8613-eeb2aecb50b1

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

what this ....!

repo: clauderic/dnd-kit


text:
BaseMainDbActivity: onConnectionFailed:null

Hi,
I can't test your sample because the App show "BaseMainDbActivity: onConnectionFailed:null" when connecting to Google Services.
Any idea ?

You have stacktrace ?
Il 10 mag 2017 4:47 PM, "Zozor54" notifications@github.com ha scritto:

Hi,
I can't test your sample because the App show "BaseMainDbActivity:
onConnectionFailed:null" when connecting to Google Services.
Any idea ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/claudiodegio/dbsync/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3-D3UJdEJ0eQ3sur5gbmv3VM_N5wjqks5r4c4AgaJpZM4NWxmN
.

I don't have any error actually but your sample app doesn't work.
I can't connect to my google account on the account selection activity...

You use the sample application from store or from github ?
Il 11 mag 2017 09:04, "Zozor54" notifications@github.com ha scritto:
I don't have any error actually but your sample app doesn't work.
I can't connect to my google account on the account selection activity...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/claudiodegio/dbsync/issues/1#issuecomment-300701398,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3-D_hhtEQJSCyUsWbZ6Z_NDH68xBRYks5r4rL9gaJpZM4NWxmN
.

I use sample from github.

OK probably don't work because you dont have registred the application on
Google Admin Console per GDrive
read the page
https://developers.google.com/drive/android/auth
On Fri, May 12, 2017 at 10:17 AM, Zozor54 notifications@github.com wrote:

I use sample from github.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/claudiodegio/dbsync/issues/1#issuecomment-301014333,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3-Dxl2XhjkqCrtfMteKe7UjQo8i41sks5r5BWugaJpZM4NWxmN
.

--
Claudio Degioanni
Skype: claudiodegio

What is the name of your application in the store ?

Hi,
You can use my applicazione name because the marcket don't allow 2
application with same name the easy way is:
change the my applicazione name in sample app e configure the new name in
Drive console
regs
On Fri, May 12, 2017 at 1:27 PM, Zozor54 notifications@github.com wrote:

What is the name of your application in the store ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/claudiodegio/dbsync/issues/1#issuecomment-301052569,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3-D93EvRnUBGqkPF1O0jlAXHHLV4LHks5r5EIvgaJpZM4NWxmN
.

--
Claudio Degioanni
Skype: claudiodegio

repo: claudiodegio/dbsync


text:
🛑 livevault is down

In bcf5179, livevault (https://livevault.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: livevault is back up in 1ca9841.

repo: claytonplong/backup-uptime


text:
🛑 livevault is down

In f292469, livevault (https://livevault.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: livevault is back up in 63e688d.


text:
🛑 www.livevault is down

In cf87bcc, www.livevault (https://www.livevault.com) was down:

HTTP code: 503
Response time: 229 ms

Resolved: www.livevault is back up in 412a34c.


text:
🛑 livevault is down

In ca4e26c, livevault (https://livevault.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: livevault is back up in 5f8a5ca.


text:
shorthand issue with border

It seems that borders shorthand does not work, even when mergeIntoShorthands is true.
But it works for border radius
Environment

clean-css version - npm ls clean-css: 4.1.11
node.js version - node -v: 9.4.0
operating system: ubuntu 16.04.4

Configuration options
var CleanCSS = require('clean-css');
new CleanCSS({
  2: {
            mergeAdjacentRules: false, // controls adjacent rules merging; defaults to true
            mergeIntoShorthands: true, // controls merging properties into shorthands; defaults to true
            mergeMedia: false, // controls `@media` merging; defaults to true
            mergeNonAdjacentRules: false, // controls non-adjacent rule merging; defaults to true
            mergeSemantically: false, // controls semantic merging; defaults to false
            overrideProperties: true, // controls property overriding based on understandability; defaults to true
            removeEmpty: true, // controls removing empty rules and nested blocks; defaults to `true`
            reduceNonAdjacentRules: true, // controls non-adjacent rule reducing; defaults to true
            removeDuplicateFontRules: true, // controls duplicate `@font-face` removing; defaults to true
            removeDuplicateMediaBlocks: true, // controls duplicate `@media` removing; defaults to true
            removeDuplicateRules: true, // controls duplicate rules removing; defaults to true
            removeUnusedAtRules: false, // controls unused at rule removing; defaults to false (available since 4.1.0)
            restructureRules: true, // controls rule restructuring; defaults to false
            skipProperties: [] // controls which properties won't be optimized, defaults to `[]` which means all will be optimized (since 4.1.0)
        }
})

Input CSS
{
  border-left: 5px solid red;
  border-top: 5px solid red;
  border-bottom: 5px solid red;
  border-right: 5px solid red;
}

Actual output CSS
{
  border-left: 5px solid red;
  border-top: 5px solid red;
  border-bottom: 5px solid red;
  border-right: 5px solid red;
}

Expected output CSS
{
  border: 5px solid red;
}

Same problem with overflow:
@jakubpawlowicz If there is a simple way to add some rules parsing I could do a pull request for missing shorthand compression each time I see one.
Actual output CSS
{
  overflow-x: hidden;
  overflow-y: hidden;
}

Expected output CSS
{
  overflow: hidden;
}

Won't fix as clean-css is going into maintenance mode.

authors: gamegee jakubpawlowicz

repo: clean-css/clean-css


text:
rpm: Don't compress non-man files

The previous patterns like ".1" or ".3" are too rough, they also match
libruby.so.2.7.1 unexpectedly.

I'll check this.

The previous patterns like ".1" or ".3" are too rough, they also match
libruby.so.2.7.1 unexpectedly.

Oh. I'd understood!! This causes libruby dependency on td-agent4 release candidate branch!!!!!

I'd confirmed that unintended libruby removing is fixed this patch.
[vagrant@localhost Packages]$ sudo yum install td-agent-4.0.0.rc1-1.el7.x86_64.rpm
Loaded plugins: fastestmirror
Examining td-agent-4.0.0.rc1-1.el7.x86_64.rpm: td-agent-4.0.0.rc1-1.el7.x86_64
Marking td-agent-4.0.0.rc1-1.el7.x86_64.rpm as an update to td-agent-3.7.1-1.el7.x86_64
Resolving Dependencies
--> Running transaction check
---> Package td-agent.x86_64 0:3.7.1-1.el7 will be updated
---> Package td-agent.x86_64 0:4.0.0.rc1-1.el7 will be an update
--> Finished Dependency Resolution

Dependencies Resolved

============================================================================================================================================
 Package                   Arch                    Version                          Repository                                         Size
============================================================================================================================================
Updating:
 td-agent                  x86_64                  4.0.0.rc1-1.el7                  /td-agent-4.0.0.rc1-1.el7.x86_64                   59 M

Transaction Summary
============================================================================================================================================
Upgrade  1 Package

Total size: 59 M
Is this ok [y/d/N]: y
Downloading packages:
Running transaction check
Running transaction test
Transaction test succeeded
Running transaction
  Updating   : td-agent-4.0.0.rc1-1.el7.x86_64                                                                                1/2
prelink detected. Installing /etc/prelink.conf.d/td-agent-ruby.conf ...
  Cleanup    : td-agent-3.7.1-1.el7.x86_64                                                                                2/2
  Verifying  : td-agent-4.0.0.rc1-1.el7.x86_64                                                                                1/2
  Verifying  : td-agent-3.7.1-1.el7.x86_64                                                                                2/2

Updated:
  td-agent.x86_64 0:4.0.0.rc1-1.el7

Complete!

Thanks!

repo: clear-code/td-agent-builder


text:
Build a curation community

Now that we have an alpha version of the software and understand the general design and philosophy around ClearlyDefined, we want to build the curation processes and community. I believe we likely need:

[x]  A proposal with some "invisible stakes in the ground" that puts forward ideas around how the curation process might work, how the community will form around it, and what we are building. This document will be very flexible and expected to change, but will give people a place to start from.
[x] A meeting with people interested in this topic to discuss the proposal and make changes.
[ ] Potentially the same meeting or a different meeting from the above of organizations that have curators they would be willing to have work in the public doing work similar to what they've done in the past for their organizations internally.

I will focus on building this and look forward to the discussion around it to come. Will update this issue as things evolve.

I've created the following doc for the proposal: https://docs.google.com/document/d/13o45CNtxo_Ca234x0NvyIVHiYQu_di1dJtmqQp769mU/edit#
We have met once and reviewed. We plan to meet again and then agree on when and how often we want to meet in the future.

repo: clearlydefined/clearlydefined


text:
OrganizationSwitcher UI suggestions

Preliminary Checks

[X] I have reviewed https://clerk.com/docs for existing features that would solve my problem

[X] I have searched for existing feature requests: https://github.com/clerkinc/javascript/issues

[X] This issue is not a question, general help request, or anything other than a feature request directly related to Clerk. Please ask questions in our Discord community: https://clerk.com/discord.

Description
Noticed a couple spots the switcher can be improved. Feel free to close if this isn't the correct forum to post something like this.

Remove grey divider under "Manage Organization" when there's nothing to display under that divider.

Vertically center the switcher <button/> inside the parent <div/>.

Affected Packages
@clerk/clerk-js

Hi, I would like to work on it, can you assign it to me

@Alone-Y154 - you got it! appreciate the willingness to jump in. Please let us know if you need any help/guidance 😁

authors: Alone-Y154 jescalan nick-kang

repo: clerkinc/javascript


text:
Model config json is not loaded correctly

In Config class, FileNotFoundError is not caught in load_json method, resulting in a crash when file is written to results. Here's a possible fix: stack post

Exception is not caught, indeed. But when it is in load_json then it cannot happen when it is written to results? Is it because the json does not exist or there is a missing folder?

authors: clessig kacpnowak

repo: clessig/atmorep


text:
Some small fixes.

Just some general cross are fixes.

A new inspection was created.

A new inspection was created.

A new inspection was created.

A new inspection was created.

The inspection completed: 85 new issues, 96 updated code elements

A new inspection was created.

The inspection completed: 85 new issues, 96 updated code elements

authors: clickalicious scrutinizer-notifier

repo: clickalicious/Doozr


text:
Commit/connect transaction may lock datastore with no info or method to break it

Transactions that connect or commit may leave the transaction without releasing a datastore lock.
This lock is not bound to a client, but to a transaction that is already finished.

Find (all) ways in which transactions may end without releasing lock
Design a way of knowing which transaction locks datastore
Make a command that breaks the lock (previously https://github.com/clicon/clixon-controller/issues/81)

Refactored code to identify all places where a transaction terminates to eliminate the issue
Added CLI command transaction unlock to break the lock
Added FAQ item to describe howto

repo: clicon/clixon-controller


text:
🛑 GEMS (API) is down

In 1252c3f, GEMS (API) (https://dev.gems.climateseed.com/api/k8/liveness) was down:

HTTP code: 0
Response time: 0 ms

Resolved: GEMS (API) is back up in c6d4fcf after 11 minutes.

repo: climateseed/services-health


text:
cacli info not returning anything

When running cacli info <MODEL-ID> I don't get a return.
It would be great to get model information like number of samples, runs, what type of model etc.

Realised info isn't a valid command... log probably covers the important parameters, otherwise it can be opened to be a feature request

cacli info isn't a command yet, but it's on the roadmap to give some more internal WML information like model local in the object storage bucket

repo: cloud-annotations/training


text:
Dashboards automation tracker

Issue to track dashboards automation. Will be raising incremental PRs for syncing dashboards with our grafana.
Current Dashboards Migration Status
├── api-performance-overview.jsonnet
├── cilium-k8s-perf.jsonnet
├── etcd-on-cluster-dashboard.jsonnet
├── hypershift-performance.jsonnet
├── k8s-perf.jsonnet
├── kube-burner.jsonnet
├── ocp-performance.jsonnet - Done
├── ovn-dashboard.jsonnet
├── pgbench-dashboard.jsonnet
├── uperf-perf.jsonnet
├── vegeta-wrapper.jsonnet
└── ycsb.jsonnet
New Dashboards

Kube-burner Report - OCP Wrapper Dashboard - Done

List of PRs:

https://github.com/cloud-bulldozer/performance-dashboards/pull/78
https://github.com/cloud-bulldozer/performance-dashboards/pull/79
https://github.com/cloud-bulldozer/performance-dashboards/pull/80

Closing this as the migration is handed over to supraja.

repo: cloud-bulldozer/performance-dashboards


text:
Customer Surveys

Analyze customer surveys to see how we can best modify services to address customer issues.

closing due to age

authors: ArsHaider wz-gsa

repo: cloud-gov/product


text:
Rename "ngw_id" output to "ids"

There are a number of reasons for this change:

As a design point, we shouldn't prefix attributes of the module. e.g. this is the nat gateway module, the value should be the id
The output value is actually an array of ids and the name should reflect that
We want to be able to use this module interchangably with the internet gateways module so we want a name that generically represents the ouptut

Have added ids as output.  Will cleanup once the entire architecture is tested with BOM.

authors: indirakalagara seansund


text:
Issues with umlauts

Hi there, not sure where to start looking at. When I add any name or alias with umlauts, like "Peter Müller", I get an Unprocessable Entity Exception.

@pgtaboada can you provide some code to reproduce it?

@pgtaboada if you are alive please reopen this issue and provide a sample usage of the client that produces the Unprocessable Entity Exception.

authors: johnou pgtaboada

repo: cloudbees/zendesk-java-client


text:
🛑 cbn.id is down

In 4902e67, cbn.id (https://cbn.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: cbn.id is back up in 3ca0301.

repo: cloudbip/upptime


text:
🛑 biznetgio.com is down

In a4de13a, biznetgio.com (https://biznetgio.com) was down:

HTTP code: 504
Response time: 10287 ms

Resolved: biznetgio.com is back up in d99a811 after 1 hour, 51 minutes.


text:
🛑 biznetgio.com is down

In e50595e, biznetgio.com (https://biznetgio.com) was down:

HTTP code: 504
Response time: 10325 ms

Resolved: biznetgio.com is back up in c7b3ed3 after 53 minutes.


text:
🛑 my.domainesia.com is down

In f20992f, my.domainesia.com (https://my.domainesia.com) was down:

HTTP code: 403
Response time: 293 ms

Resolved: my.domainesia.com is back up in 73059ba after 18 minutes.


text:
🛑 mncplay.id is down

In a4d01a7, mncplay.id (https://mncplay.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: mncplay.id is back up in 88aff19.


text:
🛑 mncplay.id is down

In 09d2947, mncplay.id (https://mncplay.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: mncplay.id is back up in 62f8829.


text:
🛑 puskomedia.id is down

In f96bf91, puskomedia.id (https://puskomedia.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: puskomedia.id is back up in b2638a6.


text:
🛑 cbn.id is down

In 5c394fa, cbn.id (https://cbn.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: cbn.id is back up in 08f5ed3.


text:
🛑 cbn.id is down

In ef01ab1, cbn.id (https://cbn.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: cbn.id is back up in d3d11ce.


text:
🛑 puskomedia.id is down

In 0332a18, puskomedia.id (https://puskomedia.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: puskomedia.id is back up in 0a3083e.


text:
Eloquent Adapter: HasMany sync/detach doesn't work with MorphMany relationships

The documentation says about Eloquent morphMany relationships that they are handled by the JSON API relationship type hasMany. This has the effect that the following exception is thrown when trying to update a resource with a  morphMany relationship:
Argument 1 passed to CloudCreativity\LaravelJsonApi\Eloquent\HasMany::sync() must be an instance of Illuminate\Database\Eloquent\Relations\HasMany, instance of Illuminate\Database\Eloquent\Relations\MorphMany given

The reason for this error is the type hinting to Relations\HasMany in the argument declaration of the sync and detach methods of the CloudCreativity\LaravelJsonApi\Eloquent\HasMany class.
@lindyhopchris Was this done by intention because the sync/detach only works with hasMany relationships or is it a bug?

@flofloflo no this should be working.
I had thought it was tested, but looking at the test that's only testing read situations, not write:
https://github.com/cloudcreativity/laravel-json-api/blob/master/tests/lib/Integration/Eloquent/MorphManyTest.php
I think probably what's happened here is I've forgotten to update the tests with the write scenarios when I was dev'ing the new relationship stuff.

Fixed and will be in the 1.0.0-beta.1 release.

authors: flofloflo lindyhopchris

repo: cloudcreativity/laravel-json-api


text:
Make get_current_user() use the user_info set by the AuthenticationMi…

…ddleware

Codecov Report

Merging #63 into master will decrease coverage by 0.15%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   81.83%   81.67%   -0.16%
==========================================
  Files          41       41
  Lines        5405     5409       +4
==========================================
- Hits         4423     4418       -5
- Misses        982      991       +9

Impacted Files
Coverage Δ

endpoints/users_id_token.py
88.84% <60%> (-0.59%)
:arrow_down:

endpoints/test/test_util.py
69.86% <0%> (-9.59%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bafa10f...2e9af30. Read the comment docs.

authors: bradfriedman codecov-io

repo: cloudendpoints/endpoints-python


text:
How to build impala and hdfs images for helm charts

I was hoping to use the helm charts in tools/kubernetes/helm to get me started with a simple impala cluster. However, it doesn't look like the images are available publicly anywhere; nor can I find the Dockerfiles to build them from source.
Any clues?

Unfortunately those are out of scope of Hue for now. Would recommend to ask on each project community list instead.

repo: cloudera/hue


text:
I met an error about submiting an oozie coordinator from HUE UI.

Hi friends:
submit Coordinator cron-testoozieshell-ssh-cnsh231375 - hue  error
E1003: Invalid coordinator application attributes, parameter [start] = [2015-08-14T06:30Z] must be Date in GMT+08:00 format (yyyy-MM-dd'T'HH:mm+0800). Parsing error java.text.ParseException: Could not parse [2015-08-14T06:30Z] using [yyyy-MM-dd'T'HH:mm+0800] mask
I have selected the timezone America/Los_Angeles.
Please.

Can you provide the date for the 'start' attribute with a forma similar to yyyy-MM-dd'T'HH:mm+0800?
What does your Coordinator config look like?

Very probably the same issue as https://github.com/cloudera/hue/issues/227

authors: phoenixhadoop romainr


text:
hue-plugins mvn build

make install-ing failed, ending at

After trying the link, I found the problem occurred because the dependency was no longer at the cloudera repository. I was able to download the jar and manually install it and successfully continue with the make.
Please update hue-plugin's pom.xml to pull the jackson-mapper jar from an existing location.

Yes, it is unfortunate but the repo has some hiccups from time to time, usually retrying a bit afterwards should work (we have hourly build that checks that).

authors: maggie-lagos romainr


text:
why do not you provide pre-built hue?

I want to know why do not you provide pre-built hue?I built the hue with some errors.I build failed all the time.

Tarball releases are a little bit easier to compile http://gethue.com/hue-4-1-is-out/, if not have a look at the CDH package or CM cf. gethue.com or https://www.cloudera.com/documentation/enterprise/latest/topics/cdh_ig_cdh_hue_configure.html

authors: dahaian romainr


text:
Ensure that getZoneSetting() returns a value key before attempting to access it

I received this error on my site:

Trying to access array offset on value of type null

Because CF\WordPress\Hooks::zoneSettingAlwaysUseHTTPSEnabled() got back a null from $this->api->getZoneSetting($zoneTag, "always_use_https") and then tried to access ["value"] on that null.
This adds some sanity checks on the value. Note that this will default the return value to false if the zone setting comes back as null — I think that's a reasonable assumption.
Using older PHP syntax for this check to retain maximum compatibility (WordPress unfortunately still runs on PHP 5.6).

thanks! i've swapped out the check to use !empty which covers the two existence checks in one but seems good to me.

authors: jacobbednarz markjaquith

repo: cloudflare/Cloudflare-WordPress


text:
Update workers.yaml

Update changelog.

Waiting on the preview service to update before moving this out of draft.

Just learned I should have had the write permissions to merge from @jp4a50, which I don't. CC: @irvinebroque @deadlypants1973

repo: cloudflare/cloudflare-docs


text:
Update shopify.mdx to identify plan restriction and enablement flag

Summary
Shopify currently does not support O2O without a feature flag that Support/SE must set. Due to this restriction, O2O to Shopify is currently restricted to Enterprise plan levels.
Documentation checklist

[X] The documentation style guide has been adhered to.

Hi @reichbc, thanks for raising this. I've checked with the PM but didn't get sign off on that. Please feel free to circle back internally if further consideration is needed.

authors: RebeccaTamachiro reichbc


text:
CNAME manual validation is actually a post method

Documentation said that it should be a patch but actual method mirrors the others and is a post.

Thanks for the catch, @JoshuaJackson-jobvite!


text:
Ruleset_managed_challenge_1436

For https://github.com/cloudflare/terraform-provider-cloudflare/issues/1436
Following change in go https://github.com/cloudflare/terraform-provider-cloudflare/pull/1442 I added acceptance test and also updated the documentation for the action managed_challenge in rulesets

thank you @vences, this is perfect!

authors: jacobbednarz vences


text:
KVStore corrupts binary data

I'm trying to store binary data in the KV, but the data appears to getting corrupted.  Consider this code:
.get_async("/test", |req, ctx| async move {
            let kv = ctx.kv("UPLOAD")?;
            let bytes = [0xd2, 0xda, 0x1e, 0x77, 0x3c, 0x50, 0xd9, 0x35, 0x9a, 0x89, 0xd5, 0x53, 0xc3, 0x81, 0x6d, 0x20];
            kv.put_bytes("test", &bytes)?.execute().await?;

            let bytes_from_kv = kv.get("test").await?.unwrap();
            assert_eq!(&bytes, bytes_from_kv.as_bytes());

            Response::empty()
        })

Trying to run this yields this assertion failure:
panicked at 'assertion failed: `(left == right)`
  left: `[210, 218, 30, 119, 60, 80, 217, 53, 154, 137, 213, 83, 195, 129, 109, 32]`,
 right: `[239, 191, 189, 239, 191, 189, 30, 119, 60, 80, 239, 191, 189, 53, 239, 191, 189, 239, 191, 189, 239, 191, 189, 83, 195, 129, 109, 32]`', src/lib.rs:56:13

However when I download this value via the https://dash.cloudflare.com/, the data is correct (so it seems like the corruption is happening during the fetch on the rust side)
Version info

worker-rs version v0.0.7
wrangler version 1.19.6
rustc version 1.59.0-nightly (7abab1efb 2021-12-17)

Thanks for filing this - @zebp, any chance you can take a look?

Hey @eminence! This was fixed already in a currently unreleased version of my worker-kv crate. Sadly this version contains necessary breaking changes to reach parity with the JS KV api. Simply specify worker-kv manually with:
[dependencies]
worker-kv = { git = "https://github.com/zebp/worker-kv" }

and this is the route handler I used:
let kv = worker_kv::KvStore::from_this(&ctx.get_env(), "UPLOAD").unwrap();
let bytes = [
    0xd2, 0xda, 0x1e, 0x77, 0x3c, 0x50, 0xd9, 0x35, 0x9a, 0x89, 0xd5, 0x53, 0xc3, 0x81,
    0x6d, 0x20,
];
kv.put_bytes("test", &bytes)
    .unwrap()
    .execute()
    .await
    .unwrap();

let bytes_from_kv = kv.get("test").bytes().await.unwrap().unwrap();
assert_eq!(&bytes, bytes_from_kv.as_slice());

Response::empty()

Thanks, @zebp! Do you think you'll want to cut a release for your crate soon? I think we're about ready to get a 0.0.8 workers-rs released, but could also definitely consider a breaking change and bump the minor version.

Thanks @zebp, I'll give this a try.  It would be nice to have this as part of a release at some point.
IMHO, a breaking API change is justified to fix a silent data corruption bug, especially in a 0.0.x project (where breaking changes are allowed at any time)

I confirm your latest version works @zebp , thanks.  Unfortunately mixing in your version is not ideal since it's not possible to use the question-mark operator to propagate errors:
error[E0277]: the trait bound `worker::Error: From<worker_kv::KvError>` is not satisfied
the trait `From<worker_kv::KvError>` is not implemented for `worker::Error`

So I'm definitely looking forward to seeing this fully integrated into the next workers-rs version 👍

Thanks, @zebp! Do you think you'll want to cut a release for your crate soon? I think we're about ready to get a 0.0.8 workers-rs released, but could also definitely consider a breaking change and bump the minor version.

I was going to wait for the inline-js issues to be fully resolved so I implement streaming, but I guess this issue seems important enough for worker-kv to get a release. I'll do that tonight or tomorrow.

Ok cool, thank you! Re: inline-js, I need to re-visit where it was left off. Maybe we can get both in for the next release...

I just released worker-kv 0.5.0 and created #103 updating the dependency.

authors: eminence nilslice zebp

repo: cloudflare/workers-rs


text:
[BUG] Missing port in Request.url

Is there an existing issue for this?

[X] I have searched the existing issues

What version of workers-rs are you using?
0.0.16
Describe the bug
On localhost req.url().unwrap().port() returns always None instead of ie. 8787.
Steps To Reproduce
No response

The URL in a remote development Worker isn't the localhost:8787 that you access it on locally, that localhost:8787 is just the proxy that Wrangler runs locally to connect you to the remote preview on the edge.
Remote development is the default in Wrangler 2, if you use local development (--(experimental)-local in Wrangler 2 or default in Wrangler 3) then it would have a URL of http://127.0.0.1:8787/.

authors: KianNH jancespivo


text:
Wrangler and C3: Move the cli folder of C3 into @cloudflare/cli and make Wrangler and C3 depend on it

What this PR solves:
We want to start using @clack/core and C3-like components in Wrangler, so the logical next step is to make these components available to wrangler and C3.

./helpers/cli.ts in C3 is now index.ts in @cloudflare/cli, didn't include openInBrowser,C3_DEFAULTS and WRANGLER_DEFAULTS.
./helpers/colors in C3 is now colors.ts in @cloudflare/cli.
./helpers/interactive.ts in C3 is now interactive.ts in @cloudflare/cli.

[x] Tests (a basic test)
[x] Changeset (Changeset guidelines)

Reviewer is to perform the following, as applicable:

Checked for inclusion of relevant tests
Checked for inclusion of a relevant changeset
Checked for creation of associated docs updates
Manually pulled down the changes and spot-tested

Note for PR author:
We want to celebrate and highlight awesome PR review! If you think this PR received a particularly high-caliber review, please assign it the label highlight pr review so future reviewers can take inspiration and learn from it.

Changes LGTM. I've been intending to revisit some of the naming in interactive.ts for a while, which I'd still like to do at some point. I'm curious to hear more about your use case, perhaps we can chat some time.

Sure! We also want to introduce some changes we did where we added a multiselect component, perhaps we can discuss those naming changes in that future PR and talk more about our interactivity use-cases.

Rebased

Rebased again

repo: cloudflare/workers-sdk


text:
Only make requests to create namespace API if absolutely necessary, because it is flaky

When testing sites live preview frequently, I ran into
wrangler preview --watch
 Creating namespace for Workers Site "__blog-daniel-workers_sites_assets_preview"
Error: [ApiError { code: 10000, message: "{\"request_id\":\"51c032b3f1fa6da6-SJC\",\"message\":\"An error occurred processing your request. You may contact support with the given request_id to troubleshoot.\"}", other: {} }]

I think we need to find another way to persist namespace bucket knowledge in memory or in wrangler.toml to prevent making excessive calls to the KV namespace API.

let's test and see if we still need this!

I'm not aware of any flakiness here; can you file an internal ticket about this?

my history is fuzzy, but I believe this ticket was created before we increased wrangler's request timeout.

@ashleymichal do you think we can close this?

nah the timeout increase should  have fixed this.

authors: EverlastingBugstopper ashleygwilliams ashleymichal gabbifish steveklabnik

repo: cloudflare/wrangler


text:
🐛 BUG: kv-binding goes wrong?

What version of Wrangler are you using?
2.0.3
What operating system are you using?
Windows 10
Describe the Bug
kv-binding goes wrong?
i've create a kv and bind it in wrangler.toml:
kv_namespaces = [
  { binding = "COMMENTS", id = <my key> }
]

(i can confirm it on Cloudflare dashboard.)
but when i call COMMENTS, throw a error ReferenceError: COMMENTS is not defined:
index.js:
export default {
  async fetch(request) {

    let value = await COMMENTS.get('aaa');

    // Return the value, as is, for the Response
    return new Response(value)
  },
};

wrangler cli error:
下午1:43:19 GET / 500
✘ [ERROR] Uncaught (in promise) ReferenceError: COMMENTS is not defined

      at Object.fetch (index.js:4:17)

✘ [ERROR] Uncaught (in response) ReferenceError: COMMENTS is not defined

✘ [ERROR] Uncaught (in promise) ReferenceError: COMMENTS is not defined

      at Object.fetch (index.js:4:17)

✘ [ERROR] Uncaught (in response) ReferenceError: COMMENTS is not defined

but if i use the env.COMMENTS everything works fine:
export default {
  async fetch(request, env) {

    let value = await env.COMMENTS.get('aaa');

    // Return the value, as is, for the Response
    return new Response(value)
  },
};

This is as designed. When you write a worker as a service-worker format (i.e. with addEventListener('fetch', () => ), then bindings are available as globals in the Worker. When using the modules format worker (i.e with export default { fetch(request, env) {...) then bindings are available on env. Sorry for the confusion! We should have easily referrable docs on this, and I'll follow up on this internally.
Closing this issue since it works as expected.

https://developers.cloudflare.com/workers/runtime-apis/kv/#referencing-kv-from-durable-objects-and-workers-using-modules-syntax

authors: koeninger s0urcelab threepointone


text:
No Docker-Daemon authentication

Currently we are not authenticating traffik from Windows to the Docker-Host. This enables every process that can access and use port 2375 on localhost to access the File System and Network in the User-Context without further restrictions.
We should think about adding TLS authentication to the Endpoint (https://docs.docker.com/engine/security/protect-access/#use-tls-https-to-protect-the-docker-daemon-socket) as soon as possible.

This project has been discontinued.

repo: cloudflightio/dockerinwsl


text:
Fail to list plugin of webhooks

There are 3 types of repository.

remote (old legacy for v1.x)
manged
local

Customer uses "managed" or "local" by installation guide.
It failed to fetch plugin list of webhook.

https://github.com/cloudforet-io/console/blob/f33d74a7d0e9961ff3982d452a89b835b55e3e51/apps/web/src/services/project/components/ProjectAlertWebhookAddModal.vue#L78
/* api */
const getRepositoryID = async () => {
const res = await SpaceConnector.clientV2.repository.repository.list<RepositoryListParameters, ListResponse>({
repository_type: 'remote',
});
const repositoryId = res.results ? res.results[0].repository_id : '';
return repositoryId;
};

Thank you for reporting this issue!
We’re aware of this and are actively working on an improvement to the Alert feature that will address this problem. The update is planned for an upcoming release, and it should resolve the issue you're experiencing.

authors: WANZARGEN choonho

repo: cloudforet-io/console


text:
Eirini component emits app container metrics too frequently

Description
Based on cf tail output (from the log-cache CLI plugin), the Eirini component emits app container metrics every 200ms:
$ cf tail dora
   2019-06-17T07:20:46.19-0700 [dora/0] GAUGE cpu:1.000000 percentage disk:42000000.000000 bytes disk_quota:10.000000 bytes memory:25088000.000000 bytes memory_quota:10.000000 bytes
   2019-06-17T07:20:46.39-0700 [dora/0] GAUGE cpu:1.000000 percentage disk:42000000.000000 bytes disk_quota:10.000000 bytes memory:25088000.000000 bytes memory_quota:10.000000 bytes
   2019-06-17T07:20:46.59-0700 [dora/0] GAUGE cpu:1.000000 percentage disk:42000000.000000 bytes disk_quota:10.000000 bytes memory:25088000.000000 bytes memory_quota:10.000000 bytes
   2019-06-17T07:20:46.79-0700 [dora/0] GAUGE cpu:1.000000 percentage disk:42000000.000000 bytes disk_quota:10.000000 bytes memory:25088000.000000 bytes memory_quota:10.000000 bytes
   2019-06-17T07:20:46.99-0700 [dora/0] GAUGE cpu:1.000000 percentage disk:42000000.000000 bytes disk_quota:10.000000 bytes memory:25088000.000000 bytes memory_quota:10.000000 bytes

This frequency of metric emission is inefficient, as the metric readings do not change that fast, and at scale may put significant unnecessary load on the Loggregator system. App developers consuming the unified stream of loggregator messages for the app will also observe that these messages overwhelm other types of loggregator messages (such as app stdout and stderr).
For comparison, the Diego cell rep defaults to emitting these metrics every 15s (https://github.com/cloudfoundry/diego-release/blob/v2.32.0/jobs/rep/spec#L136-L138), although platform operators may configure it.
Steps to reproduce
cf push <APP>
cf install-plugin -r CF-Community "log-cache"
cf tail <APP> -f

What was expected to happen
App developer should see container metrics about every 15s per app instance.
What actually happened
App developer see container metrics about every 200ms per app instance.
/cc @jenspinney

Hey @emalm,
we are not able to reproduce it using cf tail. Did you test this on a Bosh-Deployed CF?
Anyhow, we were able to reproduce it with nozzle. Just curious how you made cf tail work?

Hey, @JulzDiverse, yes, I observed this on a BOSH-deployed, cf-deployment-based CF. I believe cf tail relies on the log-cache component instead of the older loggregator traffic-controller, so maybe the log-cache isn't deployed in your environments?
Thanks,
Eric

Hey @emalm, yes that explains it. Thanks! Log-Cache isn't part of the SCF deployment. At least not in the SCF version we use.

Hey @emalm,
We added a parameter to Eirini, which allows you to override the frequency of metric emissions: https://github.com/cloudfoundry-incubator/eirini/blob/8e2628588ea522bed3def8c46f68ff3742ea1586/models.go#L67
If not set, it defaults to 15 seconds as in Diego.
Best regards,
Georgi && @JulzDiverse

Thanks, @gdankov ! FYI, the loggregator traffic-controller and firehose are now officially deprecated, and the Loggregator team working with clients to migrate them to log-cache and the reverse-log-proxy as their equivalents, so you may want to make sure you're starting to bundle those in. Or maybe this is an upstream SCF problem for you.
Best,
Eric

authors: JulzDiverse emalm gdankov

repo: cloudfoundry-incubator/eirini


text:
Consider adding BOSH login reference

In the Destroying the Cluster section, it may be helpful to point out that this command requires bosh-cli -e kubo login with a note about finding the credentials.

The issue has been fixed by #164

authors: alex-slynko kdolor-pivotal

repo: cloudfoundry-incubator/kubo-deployment


text:
Fix gem bundle call during packaging

We've seen some failures because it was simply using "bundle install" which does not properly install the gems into BOSH_INSTALL_TARGET.
After upgrading to ruby 3.2 it seems that the gems instead are installed into /usr/local/bundle which does not transfer from the compilation VM into the package so when the CPI is later invoked the gems are missing.
Copied the pattern from the bosh director packaging script

For future reference, the script being sourced probably comes from here: https://github.com/cloudfoundry/bosh-package-ruby-release/blob/bcd9ebc14d6d42819968884033ee1028d8d5f4c5/ci/templates/src/compile.env.erb

authors: jpalermo klakin-pivotal

repo: cloudfoundry/bosh-aws-cpi-release


text:
[Feature]: Developer can switch to a different CF API via cf api

Blockers/Dependencies

cloudfoundry/cf-k8s-api#10
cloudfoundry/cf-k8s-api#56

Background
When switching from an API to the other, the CLI deletes the user authentication tokens so that the user is effectively logged out. When working against cf-k8s, the equivalent of those tokens is the reference to the auth-info in use, which in turn allows us to retrive the tokens/certs. When switching to a different API, this information should be deleted too.
Acceptance Criteria
GIVEN I am logged into a cf-k8s API (e.g. I have selected an auth-info)
WHEN I run cf api against a different cf-k8s API
THEN I see something like this in $CF_HOME/config.json:
"CFOnK8s": {
    "Enabled": true,
    "AuthInfo": ""
}

GIVEN I am logged into a cf-k8s API
WHEN I run cf api against a traditional CF API
THEN I see something like this in $CF_HOME/config.json:
"CFOnK8s": {
    "Enabled": false,
    "AuthInfo": ""
}

Dev Notes
There's a couple of ways we can achieve this.

One way is to add the Kubernetes specific behaviour to the existing one, as we know they're not going to interfere. This is what we do in the spike (although the functionality discussed in this story would be achieved by changing the SetTarget method, not ClearTarget).
Alternatively, we could consider adding a ClearTokenInformation method to the Config interface and have separate implementations. This would be more elegant but we would end up with an asymmetry: ClearTokenInformation would be used to undo SetTokenInformation in one case and to undo SetKubernetesUser in the other.

PR


text:
Request for improvments for cups command responce

on MAC

cf cups service-name -p "{"username": "XXXX","password": "****","host": "XXXX.cloudant.com","port": 443,"url": "https://XXXx@XXXX.cloudant.com"}"
cf bind-service app-name service-name
cf restage app-name
cf env app-name

According to command line responses:

service was created successfully
bind was successful.

In reality: double quotes prevent setting proper access to service, application might not start after restate, environmental variables are not set:
"...No user-defined env variables have been set
No running env variables have been set
No staging env variables have been set"
What i expect: if command wrapped in double quotes instead of single quotes as expected by CF  and command was not processed as expected - inform user by returning an error or warning message.

We'll take a look.
The double/single quotes are expected by your terminal (hence being different depending on the platform you're on) - what the CLI gets passed in does not have the outside quotes anymore.
We can explore with what we do get passed in whether we can detect issues like this and act on them.

Hi @alenakl we are closing this issue because what is being returned is not a CF CLI issue, it is a bash issue. When the command runs, it has no way of validation the user input as it receives these values as arbitrary strings from bash. To get around this we've added explicit documentation to the cups command stating that the user should use single quotes to force bash to treat that whole value as a single string.
Let us know if you have any feedback by reopening this issue or creating a new one. Thanks.

authors: XenoPhex alenakl dkoper

repo: cloudfoundry/cli


text:
CLI version 6.16 ssh-code command returns oauth-token instead of ssh code

Expected behavior:
> cf ssh-code
vVVMWl

Observed behavior:
> cf ssh-code
bearer [REDACTED]

This behavior appears to be related to this refactoring: https://github.com/cloudfoundry/cli/commit/eaae429bdbec277d4aa0edbadb03b8301d1c91af
It looks like ssh-code command returns the direct result of the authRepo.RefreshAuthToken() call instead of doing further processing to exchange that for a one-time ssh code.
I have verified that this works as expected with version 6.15.
Command
cf ssh-code
CLI Version
6.16.0+ee4d916-2016-02-24
Error
Returns oauth-token instead of ssh code
Stack Trace
CC API Version
v228
CF Trace
For bugs related to HTTP requests, please run the command with env var CF_TRACE=true
Platform Details
Mac OS X 10.11.3
Shell
Terminal (bash)

Yikes.
Thanks for the catch! We'll be releasing a patch release soon.

This has been fixed in cf CLI v6.16.1.
https://github.com/cloudfoundry/cli#downloads
Thanks again!
Dies Koper
CF CLI PM

authors: dkoper krishicks selzoc


text:
Add bobbygeeze as doc approver

I am a new docs writer from VMware and am requesting to become an approver for all docs areas.
This affects 3 working groups and requires approval from the tech leads in each working group:

App Runtime Platform - @ameowlia
App Runtime Interfaces - @gerg
Foundational Infrastructure - @beyhan and/or @rkoster

@bobbygeeze please resolve the Easy CLA check first

New hire: please tell me how to resolve the CLA check?  Thanks
From: Ruben Koster @.>
Date: Monday, February 13, 2023 at 7:47 AM
To: cloudfoundry/community @.>
Cc: Bob Graczyk (c) @.>, Mention @.>
Subject: Re: [cloudfoundry/community] Add bobbygeeze as doc approver (PR #533)
!! External Email
@bobbygeezehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbobbygeeze&data=05|01|bobg%40vmware.com|29ab964c5bbe4ba49cf008db0dc06eab|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638118892378738501|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=EDggVJFxEpro4LzRwL7JPxl1DJpVDjTIbXe2whke74I%3D&reserved=0 please resolve the Easy CLA check first
—
Reply to this email directly, view it on GitHubhttps://github.com/cloudfoundry/community/pull/533#issuecomment-1427883288, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA5UL3K42UUE236SQXISDPQLWXIUNDANCNFSM6AAAAAAUWYTJAY&data=05|01|bobg%40vmware.com|29ab964c5bbe4ba49cf008db0dc06eab|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638118892378738501|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=%2FlEpHjqN6Ptxbw1EX9jwfU2rJnUDUbyVY3vL1ThnXrA%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>
!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.

Please follow the instructions in: https://github.com/cloudfoundry/community/pull/533#issuecomment-1424476984

CLA is now signed.
I approve for the ARP group. Waiting on:

App Runtime Interfaces - @Gerg
Foundational Infrastructure - @beyhan and/or @rkoster

authors: ameowlia bobbygeeze rkoster

repo: cloudfoundry/community


text:
Pushing docker image based applications to Windows diego-cells

Summary
Hi, is docker-image based applications deployment supported for Windows diego-cells?
Could not find any explicit mention in the docs regarding this specific behavior.
Thanks

docker images for windows are not currently supported

authors: omergdn winkingturtle-vmw

repo: cloudfoundry/diego-release


text:
Bump haproxy version to 2.6.7

Automatic bump from version 2.6.6 to version 2.6.7, downloaded from https://www.haproxy.org/download/2.6/src/haproxy-2.6.7.tar.gz.
After merge, consider releasing a new version of haproxy-boshrelease.

retest this please

authors: CFN-CI plowin

repo: cloudfoundry/haproxy-boshrelease


text:
ci: add golang-release autobumper

Add a golang-release autobumper for pcap-release.
Autobumper code based on @a18e's code.

Sample PR: https://github.com/cloudfoundry/pcap-release/pull/67
Successful run (created the above PR): https://concourse.cfi.sapcloud.io/teams/main/pipelines/pcap-release/jobs/autobump-dependencies/builds/31
Successful run when a bump branch already exists (no-op): https://concourse.cfi.sapcloud.io/teams/main/pipelines/pcap-release/jobs/autobump-dependencies/builds/30

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

v1.1.0
GitHub release

Your semantic-release bot :package::rocket:

authors: CFN-CI peanball

repo: cloudfoundry/pcap-release


text:
Hide Role Collections From Subscriber Subaccount

Dear Colleagues,

I am part of the SAP ETD-Cloud team which is working on a multi-tenant application that is deployed  in Cloud Foundry.

For authentication, authorization, application routing and role generation we use XSUAA. The roles are created from the mta.yaml which resides in the SAP Application Router. The application is part of the Cloud Foundry subscription and it is multi-tenant based.

Is there any possibility to hide custom roles for the subscribed accounts? We would like to have certain roles visible only to our central tenant while others should be available for all the subscribers, including us.

Regards,
Borcea Cristian.

In the meantime I received an answer from https://jam4.sapjam.com/questions/CeuzmBDvGSvFoOVwvsKMMo.
Thank you very much :)

repo: cloudfoundry/uaa


text:
Missing index on table groups for postgres

What version of UAA are you running?
4.7 from master branch
How are you deploying the UAA?
I am deploying the UAA

using cf-deployment
into SAP Cloud Platform

What did you do?
SQL monitor to check performance there
What did you see ?
SQL with bad performance in postgres
select count(*) from groups where ((LOWER(displayName) = LOWER(?) and LOWER(identity_zone_id) = LOWER(?)))
Workaround
We created a temp index
CREATE INDEX TMP_group__perf_group_lower_idx ON groups(LOWER(displayname),LOWER(identity_zone_id));
This increased performance significantly esp. if many groups entries exist.
Maybe you need to create 2 idx one with lower one without, however we see in case of postgres only slow queries like:
select count(*) from groups where ((LOWER(displayName) = LOWER(?) and LOWER(identity_zone_id) = LOWER(?)))

similar to #647

hi, @fhanik , should I create a PR for this ? because this hurt us very much on postgres

Hi @strehle and @tack-sap , out of curiosity, when does select count(*) from groups where ((LOWER(displayName) = LOWER(?) and LOWER(identity_zone_id) = LOWER(?))) run on startup, and why is the lowercase index necessary? Also, have you guys run into problems with some group displaynames being only differentiated by upper/lowercase?

the lower index is something that is postgres specific. we found several of such issues with dynatrace monitoring and if we add the lower index (often in addition to existing index) then it speeds up.


text:
Fix recursion error on empty page files

This fixes an error @erikjung ran into if a page source file is empty. With fixture and tests.

:+1:

authors: erikjung lyzadanger

repo: cloudfour/drizzle-builder


text:
Is there an existing issue already for this bug?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new bug.

I have read the troubleshooting guide

[X] I have read the troubleshooting guide and I think this is a new bug.

I am running a supported version of CloudNativePG

[X] I have read the troubleshooting guide and I think this is a new bug.

Contact Details
hossein.dehghanpoor@gmail.com
Version
1.21.1
What version of Kubernetes are you using?
1.28
What is your Kubernetes environment?
Self-managed: kind (evaluation)
How did you install the operator?
YAML manifest
What happened?
I have bunch of SQL queries which I need to use to initialize the database.
if I copy and paste all scripts to the database cluster, no error happens, but if i use postInitApplicationSQLRefs, i get bunch of errors (either secretRefs or configMapRefs).
is there any other ways that i can run my scripts after database cluster is initiated?
Cluster resource
No response
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

The provided information is not enough to understand the issue. There are some limitations with post init scripts, as they are run as part of the init job (so the cluster is not fully up yet).

thank you for reply
my init script files are about 4000 lines and i would be a mess here to upload.
i just want to find another way to run these init files after cluster creation. if you have any solution it would be appreciated.

I have another problem - it doesn't get executed at all.
  bootstrap:
    initdb:
      database: mydb
      owner: mydb
      secret:
        name: mydb-db-creds
      localeCollate: "en_US.UTF-8"
      localeCType: "en_US.UTF-8"
      import:
        type: microservice
        databases:
          - mydb_legacy
        source:
          externalCluster: legacy-mydb-db
      postInitApplicationSQLRefs:
        secretRefs:
          - name: post-init-sql
            key: post-init.sql

hi @hosseindehghanpoor @kamikaze
This still happening? Remember that the secrets has a limit as mentioned here https://kubernetes.io/docs/concepts/configuration/secret/#restriction-data-size and you could be hitting this limit
Are you still having this issue? Closing for new, please reopen this same issue if still happening
Regards,

authors: gbartolini hosseindehghanpoor kamikaze sxd

repo: cloudnative-pg/cloudnative-pg


text:
セッション削除依頼（講演ご辞退）

下記のセッションについて、講演ご辞退のため
掲載の取り下げをお願いできますか。
削除後、Dreamkastのお知らせ、Twitter等でご案内したいと思います。
morimorikunさん
熱いぜ、CloudNative！
https://event.cloudnativedays.jp/cndo2021/talks/381
DAEBUM LEEさん
仮）Flask application dockerization best practices for production
https://event.cloudnativedays.jp/cndo2021/talks/271
泉 将之さん
gRPC でつくる GraphQL - CloudNative とモダンフロントエンドをつなぐ Schema Driven Development
https://event.cloudnativedays.jp/cndo2021/talks/871

こちら対処済みです @nabemasat

authors: jacopen nabemasat

repo: cloudnativedaysjp/dreamkast


text:
Adding CloudFormation on ECS example

Closes #24

I need to test this in AWS before merge

I would argue that we commit the documentation as is letting people know that this is just an example and we don't maintain these examples. I don't think it would be any more out of date than https://github.com/cloudposse/bastion/tree/main/examples/compose. The docker-compose file in there is still set to 2! I would be surprised if https://github.com/cloudposse/bastion/tree/main/examples/compose still works straight out of the box.

Validated with https://www.cloudformation-validator.com/ ✅

I would argue that we commit the documentation as is letting people know that this is just an example and we don't maintain these examples. I don't think it would be any more out of date than https://github.com/cloudposse/bastion/tree/main/examples/compose. The docker-compose file in there is still set to version 2! I would be surprised if https://github.com/cloudposse/bastion/tree/main/examples/compose still works straight out of the box.

Agreed

authors: jonmchan max-lobur

repo: cloudposse/bastion


text:
Remove unused "region" variable from docs

what
Removes usage of the region variable from documentation.
why
Historically region may have been a variable that could be passed to the terraform-aws-dynamic-subnets module, but currently it looks like it no longer is.
Relates to: https://github.com/cloudposse/terraform-aws-vpc/pull/44

/codefresh run test

/codefresh run test

/codefresh run test

Thanks @aknysh! I've rebuilt the README now using make.

/codefresh run test

authors: aknysh alexpls


text:
feat : add support to send sns topic name

what
It is not possible to assign a custom name to the SNS.
why
When creating an SNS, giving a descriptive name to the task it is intended for is important to understand the purpose at a glance.
Important
To merge this feature, it is necessary to first review and merge this, and change source of module sns_topic
Support SNS name on module sns

/terratest

authors: hans-d paintoxic

repo: cloudposse/terraform-aws-health-events


text:
Adding new default GIT_PUSH policy for admin stacks

what

Adding new default GIT_PUSH policy to trigger admin stacks

why

Spacelift changed the default stack trigger behavior (see ref for details)

references

https://spacelift.io/changelog/en/changing-the-default-stack-push-behavior

/test test/bats

authors: danjbh nitrocode


text:
Add setup for building and publishing docs

Since there are no docs yet, I thought the python support could be a good place to start with separate module docs for https://github.com/cloudstateio/cloudstate/issues/104.
This adds an sbt project for building and publishing docs, which uses the same setup and theme as for the main cloudstate documentation.
The docs are basically empty — just showing the properties for version (automatically based on git tags) and an extref created for cloudstate core docs.
This is set up for publishing docs automatically from travis — updated for multi-language build (which should work, let's confirm). The snapshot version should be published automatically once this PR is merged (and the PR should check the regular tests are still configured correctly).
Docs will be published to https://cloudstate.io/docs/python/current/ (and /{version}) on releases, and to https://cloudstate.io/docs/python/snapshot/ on doc changes to master.
Once actual docs are added, we can retro-publish for the current version (0.1.0), and we can add to the doc home page here: https://cloudstate.io/docs/

@pvlugter thanks for putting this together and give a starting point for language support docs. 👍

authors: marcellanz pvlugter

repo: cloudstateio/python-support


text:
enha: optimize importer offline

PR Type
Enhancement, Performance

Description

Refactored the importer offline process for improved parallelism:

Separated tasks for fetching, executing, and saving blocks
Implemented batch processing for block execution and saving
Introduced new constants for channel capacities and batch sizes

Added batch saving capability to PermanentStorage, RocksPermanentStorage, and StratusStorage
Implemented a sync write option for RocksDB, controlled by a new configuration parameter
Optimized inmemory temporary storage operations
Removed the rocks_batch_writer module and its implementation
Updated RocksStorageState to include the sync write option and batch saving functionality
Added new configuration options for RocksDB in the importer-offline environment file
Improved error handling and logging throughout the affected modules

Changes walkthrough 📝
Relevant filesConfiguration changes2 files

historic_events_processor.rsUpdate RocksStorageState initialization

src/bin/historic_events_processor.rs
Updated RocksStorageState::new() call to include a new boolean parameter

+1/-1

importer-offline.env.localUpdate importer-offline configuration

config/importer-offline.env.local
Added ROCKS_DISABLE_SYNC_WRITE=true  Added ROCKS_CACHE_SIZE_MULTIPLIER=0.1

+3/-0

Enhancement8 files

importer_offline.rsRefactor importer offline for improved parallelism

src/bin/importer_offline.rs
Refactored the importer offline process to use separate tasks for fetching, executing, and saving blocks  Introduced new constants for channel capacities and batch sizes  Implemented batch processing for block execution and saving  Updated error handling and logging

+167/-114

inmemory_temporary.rsOptimize inmemory temporary storage

src/eth/storage/inmemory/inmemory_temporary.rs
Removed unused imports  Optimized do_read_slot function using find_map  Minor code cleanup and refactoring

+18/-38

permanent_storage.rsAdd batch saving capability to PermanentStorage

src/eth/storage/permanent_storage.rs
Added save_block_batch method to PermanentStorage trait  Introduced rocks_disable_sync_write configuration option

+11/-1

mod.rsRemove rocks_batch_writer module

src/eth/storage/rocks/mod.rs

Removed rocks_batch_writer module

+0/-3

rocks_batch_writer.rsRemove rocks_batch_writer implementation

src/eth/storage/rocks/rocks_batch_writer.rs

Entire file removed

+0/-78

rocks_permanent.rsAdd batch saving to RocksPermanentStorage

src/eth/storage/rocks/rocks_permanent.rs
Updated RocksPermanentStorage::new() to include enable_sync_write parameter  Implemented save_block_batch method

+13/-2

rocks_state.rsImplement batch saving and sync write option

src/eth/storage/rocks/rocks_state.rs
Added enable_sync_write field to RocksStorageState  Implemented save_block_batch method  Refactored save_block to use prepare_block_insertion  Updated write_in_batch_for_multiple_cfs to use enable_sync_write option

+43/-14

stratus_storage.rsAdd batch saving capability to StratusStorage

src/eth/storage/stratus_storage.rs
Implemented save_block_batch method for StratusStorage  Added checks for block number consistency in batch saving

+49/-0

Formatting1 files

mod.rsReorder storage module imports

src/eth/storage/mod.rs

Reordered module imports

+1/-2

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Final benchmark:
RPS Stats: Max: 1003.00, Min: 492.00, Avg: 900.00, StdDev: 52.59
TPS Stats: Max: 991.00, Min: 801.00, Avg: 806.90, StdDev: 103.29
Plot: View Plot

repo: cloudwalk/stratus


text:
test: add more tests for pkg/app/middlewares/server/basic_auth

What type of PR is this?
test: Adding missing tests or correcting existing tests

Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[x] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.
为pkg/app/middlewares/server/basic_auth添加更多单元测试
(Optional) More detail description for this PR(en: English/zh: Chinese).
en: Add BasicAuth related tests in pkg/app/middlewares/server/basic_auth, basic_auth.go reaches 100% coverage
zh(optional): 添加pkg/app/middlewares/server/basic_auth中BasicAuth相关的测试，basic_auth.go达到100%覆盖

Which issue(s) this PR fixes:
#257

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please resign the CLA🙏

@welkeyever CLA signed

authors: CLAassistant mamil welkeyever

repo: cloudwego/hertz


text:
修复批量插入段错误，支持从库查询

修复批量插入时报段错误
支持从库查询

此仓库已经没有维护，建议以 skynet 中的版本为准： https://github.com/cloudwu/skynet/blob/master/lualib-src/lua-mongo.c

authors: cloudwu pnegyafu

repo: cloudwu/lua-mongo


text:
3rd/lua/lstring.c 中引入了 uintptr_t 需 #include <stdint.h>

3rd/lua/lstring.c
中引入了 uintptr_t 需 #include <stdint.h>

前两天在 ssm 分支改了忘记合并过来。

authors: cloudwu dpull

repo: cloudwu/skynet


text:
feat: Show graph for file:// + optional isolation

@izelnakri (context) was looking for a Deno graphviz tool which can be used locally, and noticed that I treat all local files as just one file:// 'module':

So I decided to use a better default of treating each directory as a module: (Note that deps.ts is special cased to prevent false-positive of cycles when subdirectories used the parent's deps.ts)

And also added an --isolate-files CLI flag to make one node per file:

Hoping this proves useful now.

@danopia this is awesome, thanks for working on it! I particularly liked the different coloring between std, dep, and project files. Also having a list of directories in each dependency addition looks great! However I was thinking of a more general/customizable CLI interface. What do you think of this?
$ deno-graph test/some_module.ts # shows all files in the graph(including files in external dependencies)
$ deno-graph test/some_module.ts group-by=module # external dependency origin summary + file graph for internal files
$ deno-graph test/some_module.ts group-by=origin  # current behavior: group by external dependencies, and a single node for the internal.

This previous suggestion I made seems lacking some consideration as well. Idea I had is for the cli to operate on input files rather than on a project just like any test runner.
It seems there are 4 spheres and 2 toggles. 4 spheres. 4 spheres:

std,
external dependency,
input targets
input folder (could be ignored and ${CWD}/deps.ts chosen if no target file provided)
project folder(could be ignored and . could refer to the input folder reference).

2 toggles:

Display the graph by folder summary(single node)
Display the graph by file summary(many nodes connected)

Based on these we can group the spheres by toggles and if no flag provided, make it as detailed as possible:
$ deno-graph test/some_module.ts  test/some_module_2.ts # equivalent to --group-std-by=file --group-deps-by=file --group-project-files-by=file
$ deno-graph test/some_module.ts test/some_module_2.ts  --group-std-by=folder --group-deps-by=folder --group-project-files-by=file

So I suggest 3 optional flags and 2 toggles for these flags: file or folder. What do you think?

Thanks for the feedback. I think it's also important - for this project's goal at least - to consider how new functionality would be useful on the web interface as well. I have #3 open to track this, with the example of clicking one box -> a view where that specific package is exploded into its constituent files.
Grouping into file / std / third-party makes some sense, but unfortunately there's a longer list of known registries and also when you are looking at a published module via the web interface, even the main module would be considered a third-party dependency by current logic.
So given these concerns I think bucketing might not be so simple, but I agree that explicit grouping settings would be a useful feature and something worth designing. Currently I only have an 'isolated std' flag which would be easily expressed using your example flags.
Not super related, but I've also considering Graphviz subgraphs for any module which is 'exploded' but haven't had a proper chance to see how applicable the feature would be for me.

but unfortunately there's a longer list of known registries and also when you are looking at a published module via the web interface, even the main module would be considered a third-party dependency by current logic.

Yes, coloring logic could be separated as a different feature/flag and could perhaps even accept a optional config file/reference(package.json) for this if needed.
Also subgraphs could be additional nice step in the future/release bump indeed.
Let me know if I could help in anyway, thanks again for working on this!

authors: danopia izelnakri

repo: cloudydeno/module-visualizer


text:
why ignore_idx==255? in lovasz loss?

Hi @HYOJINPARK @yjyoo3312
Thank you for the wonderful library! Wow!
why set ignore_idx==255 in lovasz loss? which is the person label in target .
Can you guide me please.....
============
main.py
if train_config["loss"] == "Lovasz":
from etc.lovasz_losses import lovasz_hinge
criteria = lovasz_hinge(ignore=data_config["ignore_idx"])
else:
from etc.Criteria import CrossEntropyLoss2d
criteria = CrossEntropyLoss2d(weight,ignore=data_config["ignore_idx"])  # weight

SINet.json
"data_config": {
"cash" : "./pickle_file/portrait.p",
"dataset_name": "CVportrait",
"data_dir": "../../Link512DATA/",
"classes" : 2,
"ignore_idx" : 255,
"num_work" : 4,

Hello @Minotaur-CN Thanks for your asking
0 is a background 1 is a foreground.
Most of the segmentation dataset denotes 255 for not labeled pixel (It means hard to classify into any class)or ignoring in training.
Therefore, I just follow this custom by my habit.
However, that does not need in here, cuz the dataset is 0 or 255 and 255 convert to 1

Yes, Thanks for your quick response!
I saw normal CVDataset processing class, which using 255 as the portrait label.
which label should be used for training?
Regards!
================
class CVDataset(torch.utils.data.Dataset):
'''
Class to load the dataset
'''
def init(self, imList, labelList, Enc=True, transform=None, edge=False):
'''
:param imList: image list (Note that these lists have been processed and pickled using the loadData.py)
:param labelList: label list (Note that these lists have been processed and pickled using the loadData.py)
:param transform: Type of transformation. SEe CVTransforms.py for supported transformations
'''
self.imList = imList
self.labelList = labelList
self.transform = transform
print("This num of data is " +str(len(imList)))
self.edge = edge
if Enc :
self.kernel_size = 5
else:
self.kernel_size = 15
def __len__(self):
    return len(self.imList)

def __getitem__(self, idx):
    '''
    :param idx: Index of the image file
    :return: returns the image and corresponding label file.
    '''
    image_name = self.imList[idx]
    label_name = self.labelList[idx]
    image = cv2.imread(image_name)
    label = cv2.imread(label_name, 0)
    label_bool = 255 * ((label > 200).astype(np.uint8))

    if self.transform:
        [image, label] = self.transform(image, label_bool)
    if self.edge:
        np_label = 255 * label.data.numpy().astype(np.uint8)
        kernel = np.ones((self.kernel_size , self.kernel_size ), np.uint8)

authors: HYOJINPARK Minotaur-CN


text:
NewCorner label

As we are displying only that contributers whose atleast one PR is accepted.. so "contributer" label assign to them.. so no need more "Newcorner" label..

/assign

done, removed now!

repo: clubgamma/club-gamma-frontend